* [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend @ 2012-07-20 12:47 Zhao Chenhui 2012-07-27 21:58 ` Kumar Gala 2012-09-19 14:52 ` Kumar Gala 0 siblings, 2 replies; 9+ messages in thread From: Zhao Chenhui @ 2012-07-20 12:47 UTC (permalink / raw) To: linuxppc-dev, galak; +Cc: linux-kernel During suspend, all interrupts including IPI will be disabled. In this case, the suspend process will hang in SMP. To prevent this, pass the flag IRQF_NO_SUSPEND when requesting IPI irq. Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> Signed-off-by: Li Yang <leoli@freescale.com> --- arch/powerpc/kernel/smp.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c index fecb038..d26bbf8 100644 --- a/arch/powerpc/kernel/smp.c +++ b/arch/powerpc/kernel/smp.c @@ -171,7 +171,7 @@ int smp_request_message_ipi(int virq, int msg) } #endif err = request_irq(virq, smp_ipi_action[msg], - IRQF_PERCPU | IRQF_NO_THREAD, + IRQF_PERCPU | IRQF_NO_THREAD | IRQF_NO_SUSPEND, smp_ipi_name[msg], 0); WARN(err < 0, "unable to request_irq %d for %s (rc %d)\n", virq, smp_ipi_name[msg], err); -- 1.6.4.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-07-20 12:47 [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend Zhao Chenhui @ 2012-07-27 21:58 ` Kumar Gala 2012-07-27 22:20 ` Benjamin Herrenschmidt 2012-09-19 14:52 ` Kumar Gala 1 sibling, 1 reply; 9+ messages in thread From: Kumar Gala @ 2012-07-27 21:58 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: > During suspend, all interrupts including IPI will be disabled. In this = case, > the suspend process will hang in SMP. To prevent this, pass the flag > IRQF_NO_SUSPEND when requesting IPI irq. >=20 > Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> > Signed-off-by: Li Yang <leoli@freescale.com> > --- > arch/powerpc/kernel/smp.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) BenH, Can you ack? - k >=20 > diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c > index fecb038..d26bbf8 100644 > --- a/arch/powerpc/kernel/smp.c > +++ b/arch/powerpc/kernel/smp.c > @@ -171,7 +171,7 @@ int smp_request_message_ipi(int virq, int msg) > } > #endif > err =3D request_irq(virq, smp_ipi_action[msg], > - IRQF_PERCPU | IRQF_NO_THREAD, > + IRQF_PERCPU | IRQF_NO_THREAD | = IRQF_NO_SUSPEND, > smp_ipi_name[msg], 0); > WARN(err < 0, "unable to request_irq %d for %s (rc %d)\n", > virq, smp_ipi_name[msg], err); > --=20 > 1.6.4.1 >=20 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-07-27 21:58 ` Kumar Gala @ 2012-07-27 22:20 ` Benjamin Herrenschmidt 2012-07-28 14:03 ` Kumar Gala 2012-08-02 10:04 ` Zhao Chenhui 0 siblings, 2 replies; 9+ messages in thread From: Benjamin Herrenschmidt @ 2012-07-27 22:20 UTC (permalink / raw) To: Kumar Gala Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Fri, 2012-07-27 at 16:58 -0500, Kumar Gala wrote: > On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: > > > During suspend, all interrupts including IPI will be disabled. In this case, > > the suspend process will hang in SMP. To prevent this, pass the flag > > IRQF_NO_SUSPEND when requesting IPI irq. > > > > Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> > > Signed-off-by: Li Yang <leoli@freescale.com> > > --- > > arch/powerpc/kernel/smp.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > BenH, > > Can you ack? No I'll merge it but not until it's been in next for a bit unless you have some strong emergency there, it's on my mental list of things to shovel into next after rc1. Curiosity: didn't we use to disable all non-boot CPUs on suspend ? Cheers, Ben. > - k > > > > > diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c > > index fecb038..d26bbf8 100644 > > --- a/arch/powerpc/kernel/smp.c > > +++ b/arch/powerpc/kernel/smp.c > > @@ -171,7 +171,7 @@ int smp_request_message_ipi(int virq, int msg) > > } > > #endif > > err = request_irq(virq, smp_ipi_action[msg], > > - IRQF_PERCPU | IRQF_NO_THREAD, > > + IRQF_PERCPU | IRQF_NO_THREAD | IRQF_NO_SUSPEND, > > smp_ipi_name[msg], 0); > > WARN(err < 0, "unable to request_irq %d for %s (rc %d)\n", > > virq, smp_ipi_name[msg], err); > > -- > > 1.6.4.1 > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-07-27 22:20 ` Benjamin Herrenschmidt @ 2012-07-28 14:03 ` Kumar Gala 2012-08-02 10:04 ` Zhao Chenhui 1 sibling, 0 replies; 9+ messages in thread From: Kumar Gala @ 2012-07-28 14:03 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Jul 27, 2012, at 5:20 PM, Benjamin Herrenschmidt wrote: > On Fri, 2012-07-27 at 16:58 -0500, Kumar Gala wrote: >> On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: >>=20 >>> During suspend, all interrupts including IPI will be disabled. In = this case, >>> the suspend process will hang in SMP. To prevent this, pass the flag >>> IRQF_NO_SUSPEND when requesting IPI irq. >>>=20 >>> Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> >>> Signed-off-by: Li Yang <leoli@freescale.com> >>> --- >>> arch/powerpc/kernel/smp.c | 2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>=20 >> BenH, >>=20 >> Can you ack? >=20 > No I'll merge it but not until it's been in next for a bit unless you > have some strong emergency there, it's on my mental list of things to > shovel into next after rc1. I wasn't clear, I was meaning for my 'next' tree, not 3.6. Trying to = get these FSL PM patches into my 'next' for 3.7. - k > Curiosity: didn't we use to disable all non-boot CPUs on suspend ? >=20 > Cheers, > Ben. >=20 >> - k >>=20 >>>=20 >>> diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c >>> index fecb038..d26bbf8 100644 >>> --- a/arch/powerpc/kernel/smp.c >>> +++ b/arch/powerpc/kernel/smp.c >>> @@ -171,7 +171,7 @@ int smp_request_message_ipi(int virq, int msg) >>> } >>> #endif >>> err =3D request_irq(virq, smp_ipi_action[msg], >>> - IRQF_PERCPU | IRQF_NO_THREAD, >>> + IRQF_PERCPU | IRQF_NO_THREAD | = IRQF_NO_SUSPEND, >>> smp_ipi_name[msg], 0); >>> WARN(err < 0, "unable to request_irq %d for %s (rc %d)\n", >>> virq, smp_ipi_name[msg], err); >>> --=20 >>> 1.6.4.1 >>>=20 >=20 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-07-27 22:20 ` Benjamin Herrenschmidt 2012-07-28 14:03 ` Kumar Gala @ 2012-08-02 10:04 ` Zhao Chenhui 2012-08-10 12:40 ` Kumar Gala 1 sibling, 1 reply; 9+ messages in thread From: Zhao Chenhui @ 2012-08-02 10:04 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list On Sat, Jul 28, 2012 at 08:20:31AM +1000, Benjamin Herrenschmidt wrote: > On Fri, 2012-07-27 at 16:58 -0500, Kumar Gala wrote: > > On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: > > > > > During suspend, all interrupts including IPI will be disabled. In this case, > > > the suspend process will hang in SMP. To prevent this, pass the flag > > > IRQF_NO_SUSPEND when requesting IPI irq. > > > > > > Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> > > > Signed-off-by: Li Yang <leoli@freescale.com> > > > --- > > > arch/powerpc/kernel/smp.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > BenH, > > > > Can you ack? > > No I'll merge it but not until it's been in next for a bit unless you > have some strong emergency there, it's on my mental list of things to > shovel into next after rc1. > > Curiosity: didn't we use to disable all non-boot CPUs on suspend ? > > Cheers, > Ben. Yes, we disabled all non-boot CPUs on suspend by calling disable_nonboot_cpus(). The disable_nonboot_cpus() needs IPIs to work. But prior to calling disable_nonboot_cpus(), the IPIs are disabled in dpm_suspend_noirq(). -Chenhui ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-08-02 10:04 ` Zhao Chenhui @ 2012-08-10 12:40 ` Kumar Gala 2012-09-13 18:16 ` Kumar Gala 0 siblings, 1 reply; 9+ messages in thread From: Kumar Gala @ 2012-08-10 12:40 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Aug 2, 2012, at 5:04 AM, Zhao Chenhui wrote: > On Sat, Jul 28, 2012 at 08:20:31AM +1000, Benjamin Herrenschmidt = wrote: >> On Fri, 2012-07-27 at 16:58 -0500, Kumar Gala wrote: >>> On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: >>>=20 >>>> During suspend, all interrupts including IPI will be disabled. In = this case, >>>> the suspend process will hang in SMP. To prevent this, pass the = flag >>>> IRQF_NO_SUSPEND when requesting IPI irq. >>>>=20 >>>> Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> >>>> Signed-off-by: Li Yang <leoli@freescale.com> >>>> --- >>>> arch/powerpc/kernel/smp.c | 2 +- >>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>=20 >>> BenH, >>>=20 >>> Can you ack? >>=20 >> No I'll merge it but not until it's been in next for a bit unless you >> have some strong emergency there, it's on my mental list of things to >> shovel into next after rc1. >>=20 >> Curiosity: didn't we use to disable all non-boot CPUs on suspend ? >>=20 >> Cheers, >> Ben. >=20 > Yes, we disabled all non-boot CPUs on suspend by calling = disable_nonboot_cpus(). > The disable_nonboot_cpus() needs IPIs to work. But prior to > calling disable_nonboot_cpus(), the IPIs are disabled in = dpm_suspend_noirq(). >=20 > -Chenhui Benh, Ack? - k= ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-08-10 12:40 ` Kumar Gala @ 2012-09-13 18:16 ` Kumar Gala 2012-09-13 22:11 ` Benjamin Herrenschmidt 0 siblings, 1 reply; 9+ messages in thread From: Kumar Gala @ 2012-09-13 18:16 UTC (permalink / raw) To: Benjamin Herrenschmidt Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Aug 10, 2012, at 7:40 AM, Kumar Gala wrote: >=20 > On Aug 2, 2012, at 5:04 AM, Zhao Chenhui wrote: >=20 >> On Sat, Jul 28, 2012 at 08:20:31AM +1000, Benjamin Herrenschmidt = wrote: >>> On Fri, 2012-07-27 at 16:58 -0500, Kumar Gala wrote: >>>> On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: >>>>=20 >>>>> During suspend, all interrupts including IPI will be disabled. In = this case, >>>>> the suspend process will hang in SMP. To prevent this, pass the = flag >>>>> IRQF_NO_SUSPEND when requesting IPI irq. >>>>>=20 >>>>> Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> >>>>> Signed-off-by: Li Yang <leoli@freescale.com> >>>>> --- >>>>> arch/powerpc/kernel/smp.c | 2 +- >>>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>>=20 >>>> BenH, >>>>=20 >>>> Can you ack? >>>=20 >>> No I'll merge it but not until it's been in next for a bit unless = you >>> have some strong emergency there, it's on my mental list of things = to >>> shovel into next after rc1. >>>=20 >>> Curiosity: didn't we use to disable all non-boot CPUs on suspend ? >>>=20 >>> Cheers, >>> Ben. >>=20 >> Yes, we disabled all non-boot CPUs on suspend by calling = disable_nonboot_cpus(). >> The disable_nonboot_cpus() needs IPIs to work. But prior to >> calling disable_nonboot_cpus(), the IPIs are disabled in = dpm_suspend_noirq(). >>=20 >> -Chenhui >=20 > Benh, Ack? Ben? - k= ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-09-13 18:16 ` Kumar Gala @ 2012-09-13 22:11 ` Benjamin Herrenschmidt 0 siblings, 0 replies; 9+ messages in thread From: Benjamin Herrenschmidt @ 2012-09-13 22:11 UTC (permalink / raw) To: Kumar Gala Cc: linuxppc-dev@lists.ozlabs.org list, Zhao Chenhui, linux-kernel@vger.kernel.org list On Thu, 2012-09-13 at 13:16 -0500, Kumar Gala wrote: > >> Yes, we disabled all non-boot CPUs on suspend by calling > disable_nonboot_cpus(). > >> The disable_nonboot_cpus() needs IPIs to work. But prior to > >> calling disable_nonboot_cpus(), the IPIs are disabled in > dpm_suspend_noirq(). Sure, no biggie on this one, forgot to ack it, just stick my ack in and put it in your tree. Cheers, Ben. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend 2012-07-20 12:47 [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend Zhao Chenhui 2012-07-27 21:58 ` Kumar Gala @ 2012-09-19 14:52 ` Kumar Gala 1 sibling, 0 replies; 9+ messages in thread From: Kumar Gala @ 2012-09-19 14:52 UTC (permalink / raw) To: Zhao Chenhui; +Cc: linuxppc-dev, linux-kernel On Jul 20, 2012, at 7:47 AM, Zhao Chenhui wrote: > During suspend, all interrupts including IPI will be disabled. In this = case, > the suspend process will hang in SMP. To prevent this, pass the flag > IRQF_NO_SUSPEND when requesting IPI irq. >=20 > Signed-off-by: Zhao Chenhui <chenhui.zhao@freescale.com> > Signed-off-by: Li Yang <leoli@freescale.com> > --- > arch/powerpc/kernel/smp.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) applied to next - k= ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2012-09-19 14:52 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-07-20 12:47 [PATCH] powerpc/smp: Do not disable IPI interrupts during suspend Zhao Chenhui 2012-07-27 21:58 ` Kumar Gala 2012-07-27 22:20 ` Benjamin Herrenschmidt 2012-07-28 14:03 ` Kumar Gala 2012-08-02 10:04 ` Zhao Chenhui 2012-08-10 12:40 ` Kumar Gala 2012-09-13 18:16 ` Kumar Gala 2012-09-13 22:11 ` Benjamin Herrenschmidt 2012-09-19 14:52 ` Kumar Gala
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).