lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 29 Jun 2015 18:04:57 +0530 From: Nitish Ambastha <nitish.a@...sung.com> To: rjw@...ysocki.net, pavel@....cz, len.brown@...el.com, linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org Cc: cpgs@...sung.com, nitish.a@...sung.com, nits.ambastha@...il.com Subject: [PATCH 1/1] kernel/power/autosleep.c: check for pm_suspend() return before queueing suspend again Prevent tight loop for suspend-resume when some devices failed to suspend If some devices failed to suspend, we monitor this error in try_to_suspend(). pm_suspend() is already an 'int' returning function, how about checking return from pm_suspend() before queueing suspend again? For devices which do not register for pending events, this will prevent tight loop for suspend-resume in suspend abort scenarios due to device suspend failures Signed-off-by: Nitish Ambastha <nitish.a@...sung.com> --- kernel/power/autosleep.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/kernel/power/autosleep.c b/kernel/power/autosleep.c index 9012ecf..3923148 100644 --- a/kernel/power/autosleep.c +++ b/kernel/power/autosleep.c @@ -26,6 +26,7 @@ static struct wakeup_source *autosleep_ws; static void try_to_suspend(struct work_struct *work) { unsigned int initial_count, final_count; + int error = 0; if (!pm_get_wakeup_count(&initial_count, true)) goto out; @@ -45,10 +46,20 @@ static void try_to_suspend(struct work_struct *work) if (autosleep_state >= PM_SUSPEND_MAX) hibernate(); else - pm_suspend(autosleep_state); + error = pm_suspend(autosleep_state); mutex_unlock(&autosleep_lock); + if (error) { + /* + * If some devices failed to suspend, its better to wait + * to prevent system from trying to suspend in tight loop + */ + pr_info("PM: suspend returned (%d)\n", error); + schedule_timeout_uninterruptible(HZ / 2); + goto out; + } + if (!pm_get_wakeup_count(&final_count, false)) goto out; -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists