* [PATCH] cpuidle: undelaying cpuidle in dpm_{suspend|resume}()
@ 2019-10-29 7:31 Ikjoon Jang
2019-10-29 8:32 ` Greg Kroah-Hartman
0 siblings, 1 reply; 2+ messages in thread
From: Ikjoon Jang @ 2019-10-29 7:31 UTC (permalink / raw)
To: linux-pm
Cc: Rafael J . Wysocki, Pavel Machek ),
Len Brown, Greg Kroah-Hartman, linux-kernel, Ikjoon Jang
cpuidle is paused only during dpm_suspend_noirq() ~ dpm_resume_noirq().
But some device drivers need random sized IOs in dpm_{suspend|resume}()
stage (e.g. re-downloading firmware in resume).
And with such a device, cpuidle's latencies could be critical to
response time of system suspend/resume.
To minimize those latencies, we could apply pm_qos to such device drivers,
but simply undelaying cpuidle from dpm_suspend_noirq() to dpm suspend()
seems no harm.
---
drivers/base/power/main.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index 134a8af51511..5928dd2139e8 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -773,8 +773,6 @@ void dpm_resume_noirq(pm_message_t state)
resume_device_irqs();
device_wakeup_disarm_wake_irqs();
-
- cpuidle_resume();
}
static pm_callback_t dpm_subsys_resume_early_cb(struct device *dev,
@@ -1069,6 +1067,7 @@ void dpm_resume(pm_message_t state)
cpufreq_resume();
devfreq_resume();
+ cpuidle_resume();
trace_suspend_resume(TPS("dpm_resume"), state.event, false);
}
@@ -1411,8 +1410,6 @@ int dpm_suspend_noirq(pm_message_t state)
{
int ret;
- cpuidle_pause();
-
device_wakeup_arm_wake_irqs();
suspend_device_irqs();
@@ -1830,6 +1827,7 @@ int dpm_suspend(pm_message_t state)
trace_suspend_resume(TPS("dpm_suspend"), state.event, true);
might_sleep();
+ cpuidle_pause();
devfreq_suspend();
cpufreq_suspend();
--
2.24.0.rc0.303.g954a862665-goog
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] cpuidle: undelaying cpuidle in dpm_{suspend|resume}()
2019-10-29 7:31 [PATCH] cpuidle: undelaying cpuidle in dpm_{suspend|resume}() Ikjoon Jang
@ 2019-10-29 8:32 ` Greg Kroah-Hartman
0 siblings, 0 replies; 2+ messages in thread
From: Greg Kroah-Hartman @ 2019-10-29 8:32 UTC (permalink / raw)
To: Ikjoon Jang
Cc: linux-pm, Rafael J . Wysocki, Pavel Machek ), Len Brown, linux-kernel
On Tue, Oct 29, 2019 at 03:31:45PM +0800, Ikjoon Jang wrote:
> cpuidle is paused only during dpm_suspend_noirq() ~ dpm_resume_noirq().
> But some device drivers need random sized IOs in dpm_{suspend|resume}()
> stage (e.g. re-downloading firmware in resume).
> And with such a device, cpuidle's latencies could be critical to
> response time of system suspend/resume.
>
> To minimize those latencies, we could apply pm_qos to such device drivers,
> but simply undelaying cpuidle from dpm_suspend_noirq() to dpm suspend()
> seems no harm.
> ---
> drivers/base/power/main.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
no signed-off-by?
:(
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-29 8:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-29 7:31 [PATCH] cpuidle: undelaying cpuidle in dpm_{suspend|resume}() Ikjoon Jang
2019-10-29 8:32 ` Greg Kroah-Hartman
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).