Re: [Xen-devel] [RFC PATCH v1 03/10] xen/arm: move vgic data to vgic driver

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Wed, 19 Mar 2014, vijay.kilari@xxxxxxxxx wrote:
> @@ -142,7 +133,7 @@ struct arch_domain
>          spinlock_t lock;
>          int ctlr;
>          int nr_lines; /* Number of SPIs */
> -        struct vgic_irq_rank *shared_irqs;
> +        void *shared_irqs;

I would prefer to avoid void* fields. Either we introduce a
vgic_irq_rank union that can be v2 or v3, or we could have a single
opaque void* pointer that each vgic implementation can use for whatever
they like. I think that the union is probably better because it would
let us avoid quite a few casts in the code.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel