* [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, leoli 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: Zhao Chenhui, linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list, Li Yang 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? - 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 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: Zhao Chenhui, linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list, Li Yang 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: Zhao Chenhui, linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list, Li Yang 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: >> >>> 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. 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 ? > > 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 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: Kumar Gala, linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list, Li Yang 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: Zhao Chenhui, linuxppc-dev@lists.ozlabs.org list, linux-kernel@vger.kernel.org list, Li Yang 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: >>> >>>> 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 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: > > 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: >>>> >>>>> 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 > > 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, leoli 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(-) 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).