[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 25.11.15 at 20:32, <boris.ostrovsky@xxxxxxxxxx> wrote: > 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). Yes please. Jan _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |