Linux SCSI subsystem development
 help / color / mirror / Atom feed
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.

  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