Re: [Xen-devel] [PATCH v6 00/31] Add ITS support

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Wed, 2015-09-02 at 16:38 +0100, Ian Campbell wrote:
> > 
> > Changes in v6:
> > 
> >  - Rebased to latest staging branch.
> >  - Compiled all the patches individually for both arm32 and arm64
> >  - Split the patch "xen/arm: ITS: Allocate irq descriptors for LPIs" 
> > into 
> > two.
> >    One for allocating LPI irq_desc and other patch for allocating 
> > pending_irq desc
> >    for LPIs
> >  - Following new patches are introduced
> >      1) xen/arm: Rename NR_IRQs and vgic_num_irqs helper function
> >      2) xen/arm: ITS: Introduce msi_desc for LPIs
> >      3) xen/arm: Move vgic locking inside get_irq_priority callback
> >      4) xen/arm: ITS: Store LPIs allocated and IRQ ID bits per domain
> >      5) xen/arm: ITS: Introduce helper to get number of event IDs
> >      6) xen/arm: ITS: Add virtual ITS availability check helper
> >      7) xen/arm: ITS: Add 32-bit access to GICR_TYPER) 
> >      8) xen/arm: ITS: Allocate pending_lpi descriptors for LPIs
> > 
> >  - Based on below patch set
> >      http://lists.xen.org/archives/html/xen-devel/2015-08/msg00168.html
> > 
> >  Some Major TODOs:
> >    1) Avoid making vits_process_cmd() static in later point of time
> >    2) How to handle LPI that does not have LPI config table entry.
> >    3) Enable/disable ITS to Dom0
> 
> http://article.gmane.org/gmane.comp.emulators.xen.devel/258198 doesn't 
> seem
> to be reflected here as either a change/new-patch or a Major TODO, and it
> is pretty Major I think.

I was reminded by Julien's response to patch #29 that this is only really a
problem once we are exposing the ITS to domU, and I concur with his request
for a big fat warning comment (the existing ASSERT I think is sufficient to
avoid runtime problems).

Ian.

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