Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-06-02 Thread Stefano Stabellini
On Fri, 2 Jun 2017, Julien Grall wrote: > Hi, > > On 05/31/2017 07:49 PM, Julien Grall wrote: > > > I don't want to introduce any more hardcoding, I just would like the > > > existing vgic_migrate_irq to be called. > > > > It is going to need a bit of rework to get it working with LPI as the

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-06-02 Thread Julien Grall
Hi, On 05/31/2017 07:49 PM, Julien Grall wrote: I don't want to introduce any more hardcoding, I just would like the existing vgic_migrate_irq to be called. It is going to need a bit of rework to get it working with LPI as the code is currently gated with (p->desc). For what benefits? Not

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-31 Thread Julien Grall
Hi Stefano, On 31/05/2017 18:53, Stefano Stabellini wrote: On Wed, 31 May 2017, Julien Grall wrote: Hi Stefano, On 30/05/17 23:35, Stefano Stabellini wrote: +/* + * TODO: lookup currently-in-guest virtual IRQs and migrate them, + * as the locking may be fragile otherwise. + *

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-31 Thread Stefano Stabellini
On Wed, 31 May 2017, Julien Grall wrote: > Hi Stefano, > > On 30/05/17 23:35, Stefano Stabellini wrote: > > > +/* > > > + * TODO: lookup currently-in-guest virtual IRQs and migrate them, > > > + * as the locking may be fragile otherwise. > > > + * This is not easy to do at the

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-31 Thread Julien Grall
Hi Stefano, On 30/05/17 23:35, Stefano Stabellini wrote: +/* + * TODO: lookup currently-in-guest virtual IRQs and migrate them, + * as the locking may be fragile otherwise. + * This is not easy to do at the moment, but should become easier + * with the introduction of a

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-30 Thread Stefano Stabellini
On Fri, 26 May 2017, Andre Przywara wrote: > The MOVI command moves the interrupt affinity from one redistributor > (read: VCPU) to another. > For now migration of "live" LPIs is not yet implemented, but we store > the changed affinity in our virtual ITTE and the pending_irq. > > Signed-off-by:

[Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-26 Thread Andre Przywara
The MOVI command moves the interrupt affinity from one redistributor (read: VCPU) to another. For now migration of "live" LPIs is not yet implemented, but we store the changed affinity in our virtual ITTE and the pending_irq. Signed-off-by: Andre Przywara ---