[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2 1/2] x86/VPMU: return correct fixed PMC count
On 11/25/2015 10:31 AM, Jan Beulich wrote: However, I just noticed that various control and status registers are not available for v1. I wonder whether we should even support version 1 since we'd need to add whole lot of 'if (supported)' throughout the code plus there are some assumptions about existence of IA32_PERF_GLOBAL_CTRL so we'll need to add additional logic to handle that too. And it's not clear to me if it's all worth it.Indeed, let's not support v1 then for now and leave the exercise to add all the if()s to whoever cares for such support. And, in fact, I think we should drop model check in core2_vpmu_init() and only test for PMU version. Especially in light of XSA-163. We could limit support to versions 2 and 3 only if we want to be on the safe side. If people agree I'll send a patch (on Monday). -boris _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |