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] [thread-next>] [day] [month] [year] [list]
Message-ID: <3e672e64-c03d-4129-ac13-13a45135b0f1@linux.dev>
Date: Sun, 30 Nov 2025 13:43:51 +0800
From: Lance Yang <lance.yang@...ux.dev>
To: Aaron Tomlin <atomlin@...mlin.com>
Cc: linux-kernel@...r.kernel.org, mhiramat@...nel.org,
 akpm@...ux-foundation.org, gregkh@...uxfoundation.org,
 Petr Mladek <pmladek@...e.com>
Subject: Re: [PATCH] hung_task: Migrate hung_task_detect_count to sysfs

Hi Aaron,

Thanks for the patch.

On 2025/11/30 07:51, Aaron Tomlin wrote:
> The hung_task_detect_count metric, which tracks the cumulative number of
> tasks detected as hung since boot, is currently exposed via the legacy
> /proc/sys/kernel/hung_task_detect_count sysctl file.
> 
> Migrate this metric to a read-only file in the /sys/kernel/ hierarchy:
> /sys/kernel/hung_task_detect_count. The sysctl file is removed, and the
> metric is now controlled by the CONFIG_SYSFS Kconfig option.

Emm, I don't think we should do this :)

Removing the sysctl file is going to break userspace tools that
currently read it ...

And, yeah, while moving read-only stats to sysfs makes sense on
paper, is it really worth the churn? Since we'd likely have to
keep the old sysctl file anyway for backward compatibility, we'd
just end up maintaining two interfaces fo the exact same counter :)

[...]

Thanks anyway,
Lance

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ