Re: [PATCH v2 24/33] spapr, xics, xive: Move set_irq from SpaprIrq to SpaprInterruptController

2019-09-30 Thread David Gibson
On Mon, Sep 30, 2019 at 09:22:16AM +0200, Greg Kurz wrote: > On Mon, 30 Sep 2019 12:41:39 +1000 > David Gibson wrote: > > > On Fri, Sep 27, 2019 at 04:27:12PM +0200, Greg Kurz wrote: > > > On Fri, 27 Sep 2019 15:50:19 +1000 > > > David Gibson wrote: > > > > > > > This method depends only on the

Re: [PATCH v2 24/33] spapr, xics, xive: Move set_irq from SpaprIrq to SpaprInterruptController

2019-09-30 Thread Greg Kurz
On Mon, 30 Sep 2019 12:41:39 +1000 David Gibson wrote: > On Fri, Sep 27, 2019 at 04:27:12PM +0200, Greg Kurz wrote: > > On Fri, 27 Sep 2019 15:50:19 +1000 > > David Gibson wrote: > > > > > This method depends only on the active irq controller. Now that we've > > > formalized the notion of acti

Re: [PATCH v2 24/33] spapr, xics, xive: Move set_irq from SpaprIrq to SpaprInterruptController

2019-09-29 Thread Cédric Le Goater
On 27/09/2019 07:50, David Gibson wrote: > This method depends only on the active irq controller. Now that we've > formalized the notion of active controller we can dispatch directly through > that, rather than dispatching via SpaprIrq with the dual version having > to do a second conditional disp

Re: [PATCH v2 24/33] spapr, xics, xive: Move set_irq from SpaprIrq to SpaprInterruptController

2019-09-29 Thread David Gibson
On Fri, Sep 27, 2019 at 04:27:12PM +0200, Greg Kurz wrote: > On Fri, 27 Sep 2019 15:50:19 +1000 > David Gibson wrote: > > > This method depends only on the active irq controller. Now that we've > > formalized the notion of active controller we can dispatch directly through > > that, rather than

Re: [PATCH v2 24/33] spapr, xics, xive: Move set_irq from SpaprIrq to SpaprInterruptController

2019-09-27 Thread Greg Kurz
On Fri, 27 Sep 2019 15:50:19 +1000 David Gibson wrote: > This method depends only on the active irq controller. Now that we've > formalized the notion of active controller we can dispatch directly through > that, rather than dispatching via SpaprIrq with the dual version having > to do a second