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

Re: [Xen-devel] [PATCH v3 3/5] viridian: define type for the 'virtual VP assist page'


  • To: 'Jan Beulich' <JBeulich@xxxxxxxx>
  • From: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
  • Date: Fri, 9 Nov 2018 10:31:03 +0000
  • Accept-language: en-GB, en-US
  • Cc: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Fri, 09 Nov 2018 10:31:37 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHUdof7YvLsr1BE/0ixrHJAziTYZ6VHLt4AgAAShIA=
  • Thread-topic: [PATCH v3 3/5] viridian: define type for the 'virtual VP assist page'

> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> Sent: 09 November 2018 10:24
> To: Paul Durrant <Paul.Durrant@xxxxxxxxxx>
> Cc: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>; xen-devel <xen-
> devel@xxxxxxxxxxxxxxxxxxxx>
> Subject: Re: [PATCH v3 3/5] viridian: define type for the 'virtual VP
> assist page'
> 
> >>> On 07.11.18 at 11:52, <paul.durrant@xxxxxxxxxx> wrote:
> > --- a/xen/include/asm-x86/hvm/viridian.h
> > +++ b/xen/include/asm-x86/hvm/viridian.h
> > @@ -20,11 +20,13 @@ union viridian_page_msr
> >      } fields;
> >  };
> >
> > +typedef union _HV_VP_ASSIST_PAGE HV_VP_ASSIST_PAGE;
> > +
> >  struct viridian_vcpu
> >  {
> >      struct {
> >          union viridian_page_msr msr;
> > -        void *va;
> > +        HV_VP_ASSIST_PAGE *ptr;
> >          bool pending;
> >      } vp_assist;
> >      uint64_t crash_param[5];
> 
> I'll commit this as is, but even better would have been to avoid
> having the typedef here (visible to everyone).
> 

It goes away in the next patch anyway, when the generalized guest_page is 
introduced.

  Paul

> Jan
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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