Re: [PATCH] cpuidle: undelaying cpuidle in dpm_{suspend|resume}()

From: Greg Kroah-Hartman
Date: Tue Oct 29 2019 - 04:32:27 EST


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?

:(