From: Bart Van Assche <bvanassche@acm.org>
To: James Bottomley <James.Bottomley@HansenPartnership.com>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
linux-fsdevel@vger.kernel.org
Subject: Re: [LSF/MM/BPF TOPIC] Documenting the correct pushback on AI inspired (and other) fixes in older drivers
Date: Thu, 5 Feb 2026 08:30:53 -0800 [thread overview]
Message-ID: <8075dab2-49db-408e-bff6-5de6b0b372cd@acm.org> (raw)
In-Reply-To: <32e620691c0ecf76f469a21bffaba396f207ccb9.camel@HansenPartnership.com>
On 2/5/26 1:51 AM, James Bottomley wrote:
> I'm not even sure they're all AI found [...]
Some people use the checkpatch output or scripts from
scripts/coccinelle/ to create patches (I don't do this).
> I'd like to see us formulate a document we can put into the kernel and
> point to when they come along. Probably formulated along the lines of
> "first do no harm" and pointing out that every "fix" carries risk and
> we have to set that risk against what we actually get in terms of
> benefits. So require the submitter to specify:
>
> * What are the user visible effects (memory leak = none), transient
> bad stats data, or actual data corruption or kernel crash (latter
> being most serious)
> * how likely (or often) will this be seen? If about once a kernel boot(
> or less), at this point if you have anything less than corruption or
> a crash, don't bother fixing it because the effect is too minor
> * For bad stats data, is there an existing tool that uses the data, if
> not don't bother and even if so show it leads to issues
> * How was the fix tested (to reduce risk) i.e. do you have the
> hardware or an acceptable emulation? If not, report the issue, but
> don't bother sending the fix.
>
> I think this is just a starting point, and, obviously, it's a bit
> driver centric, but we can probably add generalizations for filesystems
> (and even mm and bpf).
We don't want to forbid tree-wide API changes, isn't it? See also
Documentation/process/stable-api-nonsense.rst.
Sometimes I post patches myself that only have been compile-tested.
These patches could help with deciding where to draw the line. A few
examples:
* [PATCH] scsi: mpt3sas: Simplify the workqueue allocation code,
January 2026
(https://lore.kernel.org/linux-scsi/20260106185655.2526800-1-bvanassche@acm.org/).
* [PATCH 0/2] Fix locking bugs in error paths, February 2025
(https://lore.kernel.org/linux-scsi/20250210203936.2946494-1-bvanassche@acm.org/)
Thanks,
Bart.
next prev parent reply other threads:[~2026-02-05 16:30 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-05 9:51 [LSF/MM/BPF TOPIC] Documenting the correct pushback on AI inspired (and other) fixes in older drivers James Bottomley
2026-02-05 16:30 ` Bart Van Assche [this message]
2026-02-05 20:54 ` Matthew Wilcox
2026-02-05 22:38 ` James Bottomley
2026-02-05 16:40 ` Haris Iqbal
2026-02-05 22:40 ` James Bottomley
2026-02-05 23:37 ` Chuck Lever
2026-02-05 22:57 ` Finn Thain
2026-02-06 5:18 ` Darrick J. Wong
2026-02-06 22:38 ` Finn Thain
2026-02-08 17:58 ` James Bottomley
2026-02-08 23:41 ` Finn Thain
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=8075dab2-49db-408e-bff6-5de6b0b372cd@acm.org \
--to=bvanassche@acm.org \
--cc=James.Bottomley@HansenPartnership.com \
--cc=linux-block@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
/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