>>> On 03.07.18 at 22:55, <andrew.coop...@citrix.com> wrote: > From: Roger Pau Monné <roger....@citrix.com> > > As with CPUID, the an architectural form is used for representing the MSR
Stray "the". Apart from that same remark(s) as for the respective CPUID change, plus ... > @@ -325,6 +325,13 @@ typedef struct xen_cpuid_leaf { > } xen_cpuid_leaf_t; > DEFINE_XEN_GUEST_HANDLE(xen_cpuid_leaf_t); > > +typedef struct xen_msr_entry { > + uint32_t idx; > + uint32_t flags; /* Reserved MBZ. */ ... it remains unclear in this patch whose responsibility it is to clear this field. I wonder whether copy_msr_to_buffer() should make sure of this. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel