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

2019-10-01 Thread David Gibson
On Tue, Oct 01, 2019 at 01:43:12PM +0200, Cédric Le Goater wrote: > On 01/10/2019 10:11, David Gibson wrote: > > On Tue, Oct 01, 2019 at 09:41:27AM +0200, Cédric Le Goater wrote: > >> On 01/10/2019 08:47, David Gibson wrote: > >>> On Tue, Oct 01, 2019 at 07:43:51AM +0200, Cédric Le Goater wrote: >

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

2019-10-01 Thread Cédric Le Goater
On 01/10/2019 10:11, David Gibson wrote: > On Tue, Oct 01, 2019 at 09:41:27AM +0200, Cédric Le Goater wrote: >> On 01/10/2019 08:47, David Gibson wrote: >>> On Tue, Oct 01, 2019 at 07:43:51AM +0200, Cédric Le Goater wrote: On 01/10/2019 04:31, David Gibson wrote: > On Mon, Sep 30, 2019 at

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

2019-10-01 Thread David Gibson
On Tue, Oct 01, 2019 at 09:41:27AM +0200, Cédric Le Goater wrote: > On 01/10/2019 08:47, David Gibson wrote: > > On Tue, Oct 01, 2019 at 07:43:51AM +0200, Cédric Le Goater wrote: > >> On 01/10/2019 04:31, David Gibson wrote: > >>> On Mon, Sep 30, 2019 at 12:13:14PM +0200, Cédric Le Goater wrote: >

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

2019-10-01 Thread Cédric Le Goater
On 01/10/2019 08:47, David Gibson wrote: > On Tue, Oct 01, 2019 at 07:43:51AM +0200, Cédric Le Goater wrote: >> On 01/10/2019 04:31, David Gibson wrote: >>> On Mon, Sep 30, 2019 at 12:13:14PM +0200, Cédric Le Goater wrote: On 30/09/2019 08:14, David Gibson wrote: > On Mon, Sep 30, 2019 at

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

2019-10-01 Thread David Gibson
On Tue, Oct 01, 2019 at 07:43:51AM +0200, Cédric Le Goater wrote: > On 01/10/2019 04:31, David Gibson wrote: > > On Mon, Sep 30, 2019 at 12:13:14PM +0200, Cédric Le Goater wrote: > >> On 30/09/2019 08:14, David Gibson wrote: > >>> On Mon, Sep 30, 2019 at 07:28:45AM +0200, Cédric Le Goater wrote: >

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

2019-09-30 Thread Cédric Le Goater
On 01/10/2019 04:31, David Gibson wrote: > On Mon, Sep 30, 2019 at 12:13:14PM +0200, Cédric Le Goater wrote: >> On 30/09/2019 08:14, David Gibson wrote: >>> On Mon, Sep 30, 2019 at 07:28:45AM +0200, Cédric Le Goater wrote: On 30/09/2019 03:49, David Gibson wrote: > On Fri, Sep 27, 2019 at

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

2019-09-30 Thread David Gibson
On Mon, Sep 30, 2019 at 12:13:14PM +0200, Cédric Le Goater wrote: > On 30/09/2019 08:14, David Gibson wrote: > > On Mon, Sep 30, 2019 at 07:28:45AM +0200, Cédric Le Goater wrote: > >> On 30/09/2019 03:49, David Gibson wrote: > >>> On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: >

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

2019-09-30 Thread Cédric Le Goater
On 30/09/2019 08:14, David Gibson wrote: > On Mon, Sep 30, 2019 at 07:28:45AM +0200, Cédric Le Goater wrote: >> On 30/09/2019 03:49, David Gibson wrote: >>> On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: On Fri, 27 Sep 2019 15:50:16 +1000 David Gibson wrote: > This

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

2019-09-30 Thread David Gibson
On Mon, Sep 30, 2019 at 07:28:45AM +0200, Cédric Le Goater wrote: > On 30/09/2019 03:49, David Gibson wrote: > > On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: > >> On Fri, 27 Sep 2019 15:50:16 +1000 > >> David Gibson wrote: > >> > >>> This method essentially represents code which

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

2019-09-29 Thread Cédric Le Goater
On 27/09/2019 07:50, David Gibson wrote: > This method essentially represents code which belongs to the interrupt > controller, but needs to be called on all possible intcs, rather than > just the currently active one. The "dual" version therefore calls > into the xics and xive versions

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

2019-09-29 Thread Cédric Le Goater
On 30/09/2019 03:49, David Gibson wrote: > On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: >> On Fri, 27 Sep 2019 15:50:16 +1000 >> David Gibson wrote: >> >>> This method essentially represents code which belongs to the interrupt >>> controller, but needs to be called on all possible

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

2019-09-29 Thread David Gibson
On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: > On Fri, 27 Sep 2019 15:50:16 +1000 > David Gibson wrote: > > > This method essentially represents code which belongs to the interrupt > > controller, but needs to be called on all possible intcs, rather than > > just the currently

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

2019-09-29 Thread David Gibson
On Fri, Sep 27, 2019 at 12:16:49PM +0200, Greg Kurz wrote: > On Fri, 27 Sep 2019 15:50:16 +1000 > David Gibson wrote: > > > This method essentially represents code which belongs to the interrupt > > controller, but needs to be called on all possible intcs, rather than > > just the currently

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

2019-09-27 Thread Greg Kurz
On Fri, 27 Sep 2019 15:50:16 +1000 David Gibson wrote: > This method essentially represents code which belongs to the interrupt > controller, but needs to be called on all possible intcs, rather than > just the currently active one. The "dual" version therefore calls > into the xics and xive

[PATCH v2 21/33] spapr, xics, xive: Move cpu_intc_create from SpaprIrq to SpaprInterruptController

2019-09-27 Thread David Gibson
This method essentially represents code which belongs to the interrupt controller, but needs to be called on all possible intcs, rather than just the currently active one. The "dual" version therefore calls into the xics and xive versions confusingly. Handle this more directly, by making it