Re: [Xen-devel] [PATCH v2 7/9] viridian: define type for the 'virtual VP assist page'

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 31.10.18 at 13:43, <paul.durrant@xxxxxxxxxx> 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.

Jan



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