public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Aaron Tomlin <atomlin@atomlin.com>
To: akpm@linux-foundation.org, lance.yang@linux.dev,
	mhiramat@kernel.org, gregkh@linuxfoundation.org,
	pmladek@suse.com, joel.granados@kernel.org
Cc: neelx@suse.com, sean@ashe.io, mproche@gmail.com,
	chjohnst@gmail.com, nick.lange@gmail.com,
	linux-kernel@vger.kernel.org
Subject: [PATCH v8 0/3] hung_task: Provide runtime reset interface for hung task detector
Date: Sat,  7 Feb 2026 19:21:13 -0500	[thread overview]
Message-ID: <20260208002116.3790497-1-atomlin@atomlin.com> (raw)

Hi Lance, Greg, Petr, Joel, Masami, Andrew,

This series introduces the ability to reset /proc/sys/kernel/hung_task_detect_count.

Writing a "0" value to this file atomically resets the counter of detected
hung tasks. This functionality provides system administrators with the
means to clear the cumulative diagnostic history following incident
resolution, thereby simplifying subsequent monitoring without necessitating
a system restart.

Please let me know your thoughts.


Changes since v7 [1]:
 - Immediately update the global counter as soon as a hung task is
   detected (Petr Mladek)
 - Added Reviewed-by: tag from Masami Hiramatsu and Aaron Tomlin

Changes since v6 [2]:
 - Decoupled the detection logic from the reporting logic.
   check_hung_task() was renamed to hung_task_info() and the task_is_hung()
   check was hoisted into the primary check_hung_uninterruptible_tasks()
   loop (Petr Mladek)

 - Changed the global statistic update mechanism from incremental atomic
   increments (per task) to a single batched atomic_long_cmpxchg_relaxed()
   at the end of the scan (Petr Mladek)

 - Strengthened the memory ordering using atomic_long_read_acquire() and
   atomic_long_cmpxchg_release(). This ensures strict synchronisation
   between the scan and concurrent userspace resets (Petr Mladek)

 - Updated the inline reporting comment to remove the hardcoded "2 minutes"
   reference

Changes since v5 [3]:
 - Introduced a preparatory patch (Joel Granados)
 - Extended custom proc_handler to handle SYSCTL_USER_TO_KERN writes,
   strictly validating that only a value of "0" is permitted for resets
   (Joel Granados)
 - Transitioned from atomic_long_inc_return_relaxed() to a more robust
   read_acquire/cmpxchg_release pattern to ensure "All-or-Nothing" scan
   updates (Petr Mladek)
 - Re-introduce hung_task_diagnostics(). For better readability and
   consistent metadata publication

Changes since v4 [4]:
 - Added missing underflow check (Lance Yang) 

Changes since v3 [5]:
 - Use atomic operations to ensure cross-CPU visibility and prevent an integer underflow
 - Use acquire/release semantics for memory ordering (Petr Mladek)
 - Move quoted string to a single line (Petr Mladek)
 - Remove variables coredump_msg and disable_msg to simplify code (Petr Mladek)
 - Add trailing "\n" to all strings to ensure immediate console flushing (Petr Mladek)
 - Improve the hung task counter documentation (Joel Granados)
 - Reject non-zero writes with -EINVAL (Joel Granados)
 - Translate to the new sysctl API (Petr Mladek)

Changes since v2 [6]:
 - Avoided a needless double update to hung_task_detect_count (Lance Yang)
 - Restored previous use of pr_err() for each message (Greg KH)
 - Provided a complete descriptive comment for the helper

Changes since v1 [7]:
 - Removed write-only sysfs attribute (Lance Yang)
 - Modified procfs hung_task_detect_count instead (Lance Yang)
 - Introduced a custom proc_handler
 - Updated documentation (Lance Yang)
 - Added 'static inline' as a hint to eliminate any function call overhead
 - Removed clutter through encapsulation

[1]: https://lore.kernel.org/lkml/20260125135848.3356585-1-atomlin@atomlin.com/
[2]: https://lore.kernel.org/lkml/20260115023229.3028462-1-atomlin@atomlin.com/
[3]: https://lore.kernel.org/lkml/20251231004125.2380105-1-atomlin@atomlin.com/
[4]: https://lore.kernel.org/lkml/20251222014210.2032214-1-atomlin@atomlin.com/
[5]: https://lore.kernel.org/all/20251216030036.1822217-1-atomlin@atomlin.com/
[6]: https://lore.kernel.org/lkml/20251211033004.1628875-1-atomlin@atomlin.com/
[7]: https://lore.kernel.org/lkml/20251209041218.1583600-1-atomlin@atomlin.com/

Aaron Tomlin (2):
  hung_task: Refactor detection logic and atomicise detection count
  hung_task: Enable runtime reset of hung_task_detect_count

Petr Mladek (1):
  hung_task: Increment the global counter immediately

 Documentation/admin-guide/sysctl/kernel.rst |   3 +-
 kernel/hung_task.c                          | 101 ++++++++++++++------
 2 files changed, 75 insertions(+), 29 deletions(-)

-- 
2.51.0


             reply	other threads:[~2026-02-08  0:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-08  0:21 Aaron Tomlin [this message]
2026-02-08  0:21 ` [PATCH v8 1/3] hung_task: Refactor detection logic and atomicise detection count Aaron Tomlin
2026-02-08  0:21 ` [PATCH v8 2/3] hung_task: Enable runtime reset of hung_task_detect_count Aaron Tomlin
2026-02-20  7:06   ` Joel Granados
2026-03-02 22:30     ` Aaron Tomlin
2026-02-08  0:21 ` [PATCH v8 3/3] hung_task: Increment the global counter immediately Aaron Tomlin
2026-02-08  4:13   ` Lance Yang
2026-02-08  4:11 ` [PATCH v8 0/3] hung_task: Provide runtime reset interface for hung task detector Lance Yang
2026-02-08  4:19   ` Lance Yang
2026-02-08 19:49   ` Aaron Tomlin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260208002116.3790497-1-atomlin@atomlin.com \
    --to=atomlin@atomlin.com \
    --cc=akpm@linux-foundation.org \
    --cc=chjohnst@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=joel.granados@kernel.org \
    --cc=lance.yang@linux.dev \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mhiramat@kernel.org \
    --cc=mproche@gmail.com \
    --cc=neelx@suse.com \
    --cc=nick.lange@gmail.com \
    --cc=pmladek@suse.com \
    --cc=sean@ashe.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox