Hi Ian,

On 15/07/2015 10:32, Ian Campbell wrote:
and save 2 byte if not more with the alignment per irq_desc.

If this is a concern then I would say we would either want a separate
array of per-pLPI information which we do not want in irq_desc because
it is irq specific, or do add a pointer to its_desc which points to an
array of per-event information.

I noticed that we have a field msi_desc in the structure irq_desc. On ARM, the structure msi_desc is not defined at all but still waste a 8 byte for the pointer in the irq_desc.

Given that LPI is an MSI, I'm wondering if we could move all LPI related data in this msi_desc. This would avoid to introduce new field in the irq_desc structure.

The msi_desc would look like:

struct msi_desc
{
        its_device *dev;
        unsigned int eventID;
};

Regards,

--
Julien Grall

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

Reply via email to