Re: [PATCH 3/8] x86/AMD: Read VPMU MSRs from context when it is not loaded into HW

[Boris Ostrovsky <boris.ostrovsky@oracle.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2318 bytes --]

On 06/19/2013 07:32 PM, Boris Ostrovsky wrote:
> On 06/19/2013 06:56 PM, Suravee Suthikulanit wrote:
>>> >
>>> > http://lists.xen.org/archives/html/xen-devel/2013-04/msg01028.html:
>>> >
>>> > diff --git a/tools/libxc/xc_cpuid_x86.c b/tools/libxc/xc_cpuid_x86.c
>>> > index 17efc0f..c269468 100644
>>> > --- a/tools/libxc/xc_cpuid_x86.c
>>> > +++ b/tools/libxc/xc_cpuid_x86.c
>>> > @@ -112,6 +112,7 @@ static void amd_xc_cpuid_policy(
>>> >                       bitmaskof(X86_FEATURE_XOP) |
>>> >                       bitmaskof(X86_FEATURE_FMA4) |
>>> >                       bitmaskof(X86_FEATURE_TBM) |
>>> > +                    bitmaskof(X86_FEATURE_PERFCTR_CORE) |
>>> >                       bitmaskof(X86_FEATURE_LWP));
>>> > regs[3] &= (0x0183f3ff | /* features shared with 0x00000001:EDX */
>>> >                       (is_pae ? bitmaskof(X86_FEATURE_NX) : 0) |
>>> >
>>> > Or maybe not since vpmu is not on by default .. ?
>>>
>>> I would say not yet. As the vpmu=1 (at least on Intel) has some issues.
>>> Until that is fixed and vpmu=1 is by default lets leave it as so.
>>>
>>> >
>>>
>> Konrad, Boris:
>> I would like to ask you to reconsider accepting this patch for 4.3.
>
> I think it missed 4.3 anyway.


OK, I think I misunderstood your question --- I thought you were asking 
to back out this patch because you found that something is broken with 
it. Instead you do want to have it in 4.3. Sorry.

This is probably request for George then (but I suspect it's too late now).

-boris


>
>> This bit and vpmu=1 are independent of each other.  Without vpmu=1 option, PERF in HVM guest
>> will not work regardless of this bit. So, it should be safe to always setting this bit.
>> However, if user set vpmu=1 and not _manually_ setting this bit, the PERF logic will
>> break and users will be getting incorrect result.
>
>
> Why would you need to set this bit by hand in addition to having the 
> patch? With this patch I see ecx=0x00a18bf3. I.e. bit 23 is set, as 
> expected.
>
> -boris
>
>
>> The bit is currently used in the Linux PERF logic for all family15h to
>> tell that there are 6 counters instead of 4 counters (when bit the is not set).
>> Also, it will be using a different set of event constrain. The current Linux PERF
>> core PMU logic assume that this bit will always be available.


[-- Attachment #1.2: Type: text/html, Size: 4785 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel