> -----Original Message----- > From: Jan Beulich [mailto:[email protected]] > Sent: 05 November 2018 13:00 > To: Paul Durrant <[email protected]> > Cc: Andrew Cooper <[email protected]>; Wei Liu > <[email protected]>; xen-devel <[email protected]> > Subject: Re: [PATCH v2 7/9] viridian: define type for the 'virtual VP > assist page' > > >>> On 31.10.18 at 13:43, <[email protected]> wrote: > > --- a/xen/include/asm-x86/hvm/viridian.h > > +++ b/xen/include/asm-x86/hvm/viridian.h > > @@ -92,7 +92,7 @@ struct viridian_vcpu > > { > > struct { > > union viridian_page_msr msr; > > - void *va; > > + void *ptr; > > Why void rather than the actual type? You can't use the typedef > (for it being local to the .c file), but you can use the union tag > here in a forward declaring manner. >
Ok. I'll have a look at that. Paul > Jan > _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
