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

Re: [Xen-devel] [PATCH v22 02/14] x86/VPMU: Add public xenpmu.h



>>> On 27.05.15 at 15:44, <boris.ostrovsky@xxxxxxxxxx> wrote:
> Sorry, I meant amd/intel members of the union below (I forgot we were 
> already in the arch header file):
> 
> +    /*
> +     * Vendor-specific PMU registers.
> +     * RW for both hypervisor and guest.
> +     * Guest's updates to this field are verified and then loaded by the
> +     * hypervisor into hardware during XENPMU_flush
> +     */
> +    union {
> +        struct xen_pmu_amd_ctxt amd;
> +        struct xen_pmu_intel_ctxt intel;
> +
> +        /*
> +         * Padding for contexts (fixed parts only, does not include MSR banks
> +         * that are specified by offsets)
> +         */
> +#define XENPMU_CTXT_PAD_SZ  128
> +        uint8_t pad[XENPMU_CTXT_PAD_SZ];
> +    } c;
> +};
> 
> I think they are first used in patch 11 so I assume you also want me to 
> just keep the pad here (with a comment explaining why it is here) until 
> that patch.

Ah, those ones I simply recalled having checked in the previous
version already.

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®.