linux-kernel - Re: [PATCH 6.1] softirq: Add trace points for tasklet entry/exit

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: <CANDhNCoao7P8RWbugxY70K-+HRPSy=NypgNXM3DjheHRmznmjw@mail.gmail.com>
Date: Thu, 21 Aug 2025 15:05:16 -0700
From: John Stultz <jstultz@...gle.com>
To: Sumanth Gavini <sumanth.gavini@...oo.com>
Cc: boqun.feng@...il.com, clingutla@...eaurora.org, elavila@...gle.com, 
	gregkh@...uxfoundation.org, kprateek.nayak@....com, 
	linux-kernel@...r.kernel.org, mhiramat@...nel.org, mingo@...nel.org, 
	rostedt@...dmis.org, ryotkkr98@...il.com, sashal@...nel.org, 
	stable@...r.kernel.org, tglx@...utronix.de
Subject: Re: [PATCH 6.1] softirq: Add trace points for tasklet entry/exit

On Thu, Aug 21, 2025 at 2:02 PM Sumanth Gavini <sumanth.gavini@...oo.com> wrote:
> >>
> >> Just following up on my patch submitted with subject "Subject: [PATCH 6.1] softirq: Add trace points for tasklet entry/exit".
> >>
> >> Original message: https://lore.kernel.org/all/20250812161755.609600-1-sumanth.gavini@yahoo.com/
> >>
> >> Would you have any feedback on this change? I'd be happy to address any comments or concerns.
> >>
> >> This patch fixes this three bugs
> >> 1. https://syzkaller.appspot.com/bug?extid=5284a86a0b0a31ab266a
> >> 2. https://syzkaller.appspot.com/bug?extid=296695c8ae3c7da3d511
> >> 3. https://syzkaller.appspot.com/bug?extid=97f2ac670e5e7a3b48e4
>
> > How does a patch adding a tracepoint fix the bugs highlighted here?
> > It seems maybe it would help in debugging those issues, but I'm not
> > sure I see how it would fix them.
>
> This patch is related to linux 6.1/backports, the backports(https://syzkaller.appspot.com/linux-6.1/backports)
> I see this patch would fix these bugs. Let me know if my understand is wrong.

But that doesn't explain why or how it fixes the bugs.  I'm not
opposed to stable taking this, but the reasoning should be clear, if
that is the motivation for including this change.
I fret there is something incidental in this patch that avoids the
problem, and that those issues may need a deeper fix rather then to
hide them with this change.

thanks
-john