Re: [RFC PATCH v2 12/22] xen/arm: its: Update irq descriptor for LPIs support

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

Hello Vijay,

On 19/03/2015 14:37, vijay.kilari@gmail.com wrote:
> diff --git a/xen/include/asm-arm/irq.h b/xen/include/asm-arm/irq.h
> index 435dfcd..f091739 100644
> --- a/xen/include/asm-arm/irq.h
> +++ b/xen/include/asm-arm/irq.h
> @@ -17,6 +17,8 @@ struct arch_pirq
>   struct arch_irq_desc {
>       int eoi_cpu;
>       unsigned int type;
> +    unsigned int virq;
> +    struct its_device *dev;
>   };

It seems you again miss my comment... As said on v1 this is not the 
solution. You add data for any IRQ (around 16K in Xen) just for handling 
LPIs.

I provided a patch to handle virq != irq [1] and we should use it in 
order to diverge handling between LPIs and SPIs.

If you are not happy with it, please see why.

Regards,

[1] https://patches.linaro.org/43012/

-- 
Julien Grall