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

[Julien Grall <julien.grall@linaro.org>]

On 03/20/2014 05:14 PM, Stefano Stabellini wrote:
> On Wed, 19 Mar 2014, vijay.kilari@gmail.com 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.
> 

Most of the cast can be removed without this trick :).

IHMO, void * might better to avoid expanding the domain/VCPU structure.
We are already close to the 1 page size.

Regards,

-- 
Julien Grall