Re: [PATCH v2] hung_task: Add detection count to hung task warning message

[Lance Yang]

On 2025/7/25 06:10, Andrew Morton wrote:
> On Thu, 24 Jul 2025 20:06:52 +0800 Lance Yang <lance.yang@xxxxxxxxx> wrote:
>
> > On 2025/7/24 15:23, Ye Liu wrote:
> > > From: Ye Liu <liuye@xxxxxxxxxx>
> > >
> > > Add [#N] to hung task warnings to show occurrence count.
> > > This helps quickly identify warning order when multiple
> > > messages appear in logs.
> >
> > Hmm... once we have this number, what do we do with it?
>
> Yes, adding to the changelog a more comprehensive description of the
> use-cases would help get this patch into Linux.

Yep, exactly. Please sell this change to us! A good changelog should
explain not just what the patch does, but why it's necessary ;p

> > While I totally get the desire for a quick reference number, my thinking
> > is that the hung_task_detect_count counter is the intended and more
> > reliable way for users to check.
>
> But that's what this patch does?  Confused.

I'm just not sure if this quick reference number is something users
actually need, as there doesn't seem to be a strong use-case for it yet.

My point was that actively checking the sysctl is the proper way to get the
total count. But of course, if users genuinely need this inline number, I
have no objection ;)

> > But like I said before, let's see what Andrew thinks ;)
>
> Andrew thinks it's a pain when the title of a patch is changed!
> Fortunately the changelog had a "Link to v1".

https://lore.kernel.org/lkml/20250721031755.1418556-1-ye.liu@xxxxxxxxx/

Thanks,
Lance