[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v25 00/15] x86/PMU: Xen PMU PV(H) support



>>> On 19.06.15 at 20:44, <boris.ostrovsky@xxxxxxxxxx> wrote:

While making another scan through this series now that some more
reviews from Dietmar are trickling in, I notice:

> Boris Ostrovsky (15):
>   common/symbols: Export hypervisor symbols to privileged guest
>   x86/VPMU: Add public xenpmu.h
>   x86/VPMU: Make vpmu not HVM-specific
>   x86/VPMU: Interface for setting PMU mode and flags

still missing a VMX maintainer's ack

>   x86/VPMU: Initialize VPMUs with __initcall

same here plus no review (albeit I wouldn't make the latter a
requirement)

>   x86/VPMU: Initialize PMU for PV(H) guests

same regarding review state

>   x86/VPMU: Save VPMU state for PV guests during context switch
>   x86/VPMU: When handling MSR accesses, leave fault injection to callers

again same regarding review state

>   x86/VPMU: Add support for PMU register handling on PV guests
>   x86/VPMU: Use pre-computed masks when checking validity of MSRs
>   VPMU/AMD: Check MSR values before writing to hardware

no review yet (and here I'd really like to have one)

>   x86/VPMU: Handle PMU interrupts for PV(H) guests

same here

>   x86/VPMU: Merge vpmu_rdmsr and vpmu_wrmsr
>   x86/VPMU: Add privileged PMU mode

here a review would again be nice, but I'd again not make it a
requirement

>   x86/VPMU: Move VPMU files up from hvm/ directory

Jan


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.