* [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).