linux-kernel - Re: [PATCH v2] PM / core: Allow configuring the DPM watchdog to warn earlier than panic

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAAFQd5B30=9TT+ipyWaHdB1XMVagvEAJAw8grdU0Vva8+6JqLA@mail.gmail.com>
Date: Fri, 10 Jan 2025 11:50:07 +0900
From: Tomasz Figa <tfiga@...omium.org>
To: Douglas Anderson <dianders@...omium.org>
Cc: "Rafael J . Wysocki" <rafael@...nel.org>, Pavel Machek <pavel@....cz>, Len Brown <len.brown@...el.com>, 
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-kernel@...r.kernel.org, 
	linux-pm@...r.kernel.org
Subject: Re: [PATCH v2] PM / core: Allow configuring the DPM watchdog to warn
 earlier than panic

On Fri, Jan 10, 2025 at 6:01 AM Douglas Anderson <dianders@...omium.org> wrote:
>
> Allow configuring the DPM watchdog to warn about slow suspend/resume
> functions without causing a system panic(). This allows you to set the
> DPM_WATCHDOG_WARNING_TIMEOUT to something like 5 or 10 seconds to get
> warnings about slow suspend/resume functions that eventually succeed.
>
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
> ---
>
> Changes in v2:
> - Print the warning at warn level, not emergency level.
> - Add help text to DPM_WATCHDOG_WARNING_TIMEOUT.
>
>  drivers/base/power/main.c | 24 +++++++++++++++++++-----
>  kernel/power/Kconfig      | 21 ++++++++++++++++++++-
>  2 files changed, 39 insertions(+), 6 deletions(-)
>

Thanks for addressing my comments.

Reviewed-by: Tomasz Figa <tfiga@...omium.org>

Best regards,
Tomasz