The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH] coding-assistants: simplify attribution
@ 2026-07-22 20:34 Jonathan Corbet
  2026-07-22 20:48 ` Konstantin Ryabitsev
                   ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Jonathan Corbet @ 2026-07-22 20:34 UTC (permalink / raw)
  To: linux-kernel, Christian Brauner
  Cc: Lorenzo Stoakes, Jeff Layton, Jori Koolstra, Sasha Levin,
	tech-board, Jakub Kicinski, workflows

As suggested by Christian, I have applied his patch to a branch and
simplified the changelog, see below.

I'm nervous about applying it still because I don't think it's my role
to set this sort of policy; I really want to feel that there is a
consensus behind it.  So there's some additional CCs here because I
would like to hear from some folks:

- This policy document originally came out of the TAB; is the TAB (on
  CC) OK with this change?

- At least one maintainer (on CC) have said that they currently strip
  Assisted-by tags out of patches before applying them, a practice that
  would seem to undermine the whole policy.  Would this change be
  sufficient to end that practice?

Thanks,

jon

From c31aa376a355e3b84b9e87ae1dbe19b394d4c82c Mon Sep 17 00:00:00 2001
From: Christian Brauner <brauner@kernel.org>
Date: Wed, 1 Jul 2026 17:54:48 +0200
Subject: [PATCH] coding-assistants: simplify attribution

The requirement to identify specific models used in the Assisted-by tag
provides free advertising to proprietary software companies while adding
little or no useful information.  Change the requirement to simply:

  Assisted-by: LLM

to capture the fact that an LLM was used without tracking which one.

Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>
Reviewed-by: Lorenzo Stoakes (ARM) <ljs@kernel.org>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Reviewed-by: Jori Koolstra <jkoolstra@xs4all.nl>
Message-ID: <20260701-work-coding-assistants-v1-1-a20a94d1d606@kernel.org>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
---
 Documentation/process/coding-assistants.rst | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
index 899f4459c52d..fe34f3e7e828 100644
--- a/Documentation/process/coding-assistants.rst
+++ b/Documentation/process/coding-assistants.rst
@@ -43,12 +43,8 @@ When AI tools contribute to kernel development, proper attribution
 helps track the evolving role of AI in the development process.
 Contributions should include an Assisted-by tag in the following format::
 
-  Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]
+  Assisted-by: LLM [TOOL1] [TOOL2]
 
-Where:
-
-* ``AGENT_NAME`` is the name of the AI tool or framework
-* ``MODEL_VERSION`` is the specific model version used
 * ``[TOOL1] [TOOL2]`` are optional specialized analysis tools used
   (e.g., coccinelle, sparse, smatch, clang-tidy)
 
@@ -56,4 +52,4 @@ Basic development tools (git, gcc, make, editors) should not be listed.
 
 Example::
 
-  Assisted-by: Claude:claude-3-opus coccinelle sparse
+  Assisted-by: LLM coccinelle sparse
-- 
2.55.0


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:34 [PATCH] coding-assistants: simplify attribution Jonathan Corbet
@ 2026-07-22 20:48 ` Konstantin Ryabitsev
  2026-07-24 18:17   ` Mauro Carvalho Chehab
  2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
  2026-07-24 19:15 ` [tech-board] " David Hildenbrand (Arm)
                   ` (2 subsequent siblings)
  3 siblings, 2 replies; 12+ messages in thread
From: Konstantin Ryabitsev @ 2026-07-22 20:48 UTC (permalink / raw)
  To: Jonathan Corbet
  Cc: linux-kernel, Christian Brauner, Lorenzo Stoakes, Jeff Layton,
	Jori Koolstra, Sasha Levin, tech-board, Jakub Kicinski, workflows

On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
> -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> +  Assisted-by: LLM coccinelle sparse

I've been playing with this a bit, and it's pretty easy to teach the agent to
add a few tag-like notes as to how exactly it was used, e.g.:

    https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf

This has:

| Assisted-by: LLM [design, codegen, tests]

It tells us a bit more about *how* the LLM was used, without introducing the
advertisement or pinning exact models.

This format should allow for multiple tool usage:

| Assisted-by: LLM [triage, tests], coccinelle

I don't suggest this is included in this version, but it *is* fairly
straightforward to add to AGENTS.md and tells reviewers just how exactly the
LLM was used.

-K

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:48 ` Konstantin Ryabitsev
@ 2026-07-24 18:17   ` Mauro Carvalho Chehab
  2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
  1 sibling, 0 replies; 12+ messages in thread
From: Mauro Carvalho Chehab @ 2026-07-24 18:17 UTC (permalink / raw)
  To: Konstantin Ryabitsev
  Cc: Jonathan Corbet, linux-kernel, Christian Brauner, Lorenzo Stoakes,
	Jeff Layton, Jori Koolstra, Sasha Levin, tech-board,
	Jakub Kicinski, workflows

On Wed, 22 Jul 2026 16:48:03 -0400
Konstantin Ryabitsev <mricon@kernel.org> wrote:

> On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
> > -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> > +  Assisted-by: LLM coccinelle sparse  
> 
> I've been playing with this a bit, and it's pretty easy to teach the agent to
> add a few tag-like notes as to how exactly it was used, e.g.:
> 
>     https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf
> 
> This has:
> 
> | Assisted-by: LLM [design, codegen, tests]
> 
> It tells us a bit more about *how* the LLM was used, without introducing the
> advertisement or pinning exact models.
> 
> This format should allow for multiple tool usage:
> 
> | Assisted-by: LLM [triage, tests], coccinelle
> 
> I don't suggest this is included in this version, but it *is* fairly
> straightforward to add to AGENTS.md and tells reviewers just how exactly the
> LLM was used.

I'm fine with either way.

Reviewed-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

Regards,
Mauro

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [tech-board] [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:34 [PATCH] coding-assistants: simplify attribution Jonathan Corbet
  2026-07-22 20:48 ` Konstantin Ryabitsev
@ 2026-07-24 19:15 ` David Hildenbrand (Arm)
  2026-08-03 18:56 ` Jonathan Corbet
  2026-08-04 15:28 ` Jeff Layton
  3 siblings, 0 replies; 12+ messages in thread
From: David Hildenbrand (Arm) @ 2026-07-24 19:15 UTC (permalink / raw)
  To: corbet, linux-kernel, Christian Brauner
  Cc: Lorenzo Stoakes, Jeff Layton, Jori Koolstra, Sasha Levin,
	tech-board, Jakub Kicinski, workflows

On 7/22/26 22:34, Jonathan Corbet via groups.linuxfoundation.org wrote:
> As suggested by Christian, I have applied his patch to a branch and
> simplified the changelog, see below.
> 
> I'm nervous about applying it still because I don't think it's my role
> to set this sort of policy; I really want to feel that there is a
> consensus behind it.  So there's some additional CCs here because I
> would like to hear from some folks:
> 
> - This policy document originally came out of the TAB; is the TAB (on
>   CC) OK with this change?
> 
> - At least one maintainer (on CC) have said that they currently strip
>   Assisted-by tags out of patches before applying them, a practice that
>   would seem to undermine the whole policy.  Would this change be
>   sufficient to end that practice?
> 
> Thanks,
> 
> jon
> 
> From c31aa376a355e3b84b9e87ae1dbe19b394d4c82c Mon Sep 17 00:00:00 2001
> From: Christian Brauner <brauner@kernel.org>
> Date: Wed, 1 Jul 2026 17:54:48 +0200
> Subject: [PATCH] coding-assistants: simplify attribution
> 
> The requirement to identify specific models used in the Assisted-by tag
> provides free advertising to proprietary software companies while adding
> little or no useful information.  Change the requirement to simply:
> 
>   Assisted-by: LLM
> 
> to capture the fact that an LLM was used without tracking which one.
> 
> Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>
> Reviewed-by: Lorenzo Stoakes (ARM) <ljs@kernel.org>
> Reviewed-by: Jeff Layton <jlayton@kernel.org>
> Reviewed-by: Jori Koolstra <jkoolstra@xs4all.nl>
> Message-ID: <20260701-work-coding-assistants-v1-1-a20a94d1d606@kernel.org>
> Signed-off-by: Jonathan Corbet <corbet@lwn.net>
> ---
>  Documentation/process/coding-assistants.rst | 8 ++------
>  1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> index 899f4459c52d..fe34f3e7e828 100644
> --- a/Documentation/process/coding-assistants.rst
> +++ b/Documentation/process/coding-assistants.rst
> @@ -43,12 +43,8 @@ When AI tools contribute to kernel development, proper attribution
>  helps track the evolving role of AI in the development process.
>  Contributions should include an Assisted-by tag in the following format::
>  
> -  Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]
> +  Assisted-by: LLM [TOOL1] [TOOL2]
>  
> -Where:
> -
> -* ``AGENT_NAME`` is the name of the AI tool or framework
> -* ``MODEL_VERSION`` is the specific model version used
>  * ``[TOOL1] [TOOL2]`` are optional specialized analysis tools used
>    (e.g., coccinelle, sparse, smatch, clang-tidy)
>  
> @@ -56,4 +52,4 @@ Basic development tools (git, gcc, make, editors) should not be listed.
>  
>  Example::
>  
> -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> +  Assisted-by: LLM coccinelle sparse

LGTM

Reviewed-by: David Hildenbrand (arm) <david@kernel.org>

-- 
Cheers,

David

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:34 [PATCH] coding-assistants: simplify attribution Jonathan Corbet
  2026-07-22 20:48 ` Konstantin Ryabitsev
  2026-07-24 19:15 ` [tech-board] " David Hildenbrand (Arm)
@ 2026-08-03 18:56 ` Jonathan Corbet
  2026-08-03 19:50   ` [tech-board] " Miguel Ojeda
  2026-08-04 15:28 ` Jeff Layton
  3 siblings, 1 reply; 12+ messages in thread
From: Jonathan Corbet @ 2026-08-03 18:56 UTC (permalink / raw)
  To: linux-kernel, Christian Brauner
  Cc: Lorenzo Stoakes, Jeff Layton, Jori Koolstra, Sasha Levin,
	tech-board, Jakub Kicinski, workflows

Jonathan Corbet <corbet@lwn.net> writes:

> As suggested by Christian, I have applied his patch to a branch and
> simplified the changelog, see below.
>
> I'm nervous about applying it still because I don't think it's my role
> to set this sort of policy; I really want to feel that there is a
> consensus behind it.  So there's some additional CCs here because I
> would like to hear from some folks:
>
> - This policy document originally came out of the TAB; is the TAB (on
>   CC) OK with this change?
>
> - At least one maintainer (on CC) have said that they currently strip
>   Assisted-by tags out of patches before applying them, a practice that
>   would seem to undermine the whole policy.  Would this change be
>   sufficient to end that practice?
>

So I was not overwhelmed by responses :)  I'm going to take silence for
assent in this case and send the patch through.

Thanks,

jon

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [tech-board] [PATCH] coding-assistants: simplify attribution
  2026-08-03 18:56 ` Jonathan Corbet
@ 2026-08-03 19:50   ` Miguel Ojeda
  0 siblings, 0 replies; 12+ messages in thread
From: Miguel Ojeda @ 2026-08-03 19:50 UTC (permalink / raw)
  To: corbet
  Cc: linux-kernel, Christian Brauner, Lorenzo Stoakes, Jeff Layton,
	Jori Koolstra, Sasha Levin, tech-board, Jakub Kicinski, workflows

On Mon, Aug 3, 2026 at 9:42 PM Jonathan Corbet via
groups.linuxfoundation.org <corbet=lwn.net@groups.linuxfoundation.org>
wrote:
>
> So I was not overwhelmed by responses :)  I'm going to take silence for
> assent in this case and send the patch through.

On the TAB side, we didn't meet since it was sent, but yeah, it
doesn't look like there are strong opinions against this.

Acked-by: Miguel Ojeda <ojeda@kernel.org>

Personally, I think it is fine either way (e.g. someone could argue
the advertising could help open models and harnesses etc.).

The patch removes the "Where:", not sure if that is intentional.

Thanks!

Cheers,
Miguel

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:48 ` Konstantin Ryabitsev
  2026-07-24 18:17   ` Mauro Carvalho Chehab
@ 2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
  2026-08-04  9:23     ` David Hildenbrand (Arm)
  2026-08-04 11:09     ` Jori Koolstra
  1 sibling, 2 replies; 12+ messages in thread
From: Lorenzo Stoakes (ARM) @ 2026-08-04  8:10 UTC (permalink / raw)
  To: Konstantin Ryabitsev
  Cc: Jonathan Corbet, linux-kernel, Christian Brauner, Jeff Layton,
	Jori Koolstra, Sasha Levin, tech-board, Jakub Kicinski, workflows,
	David Hildenbrand (Arm)

On Wed, Jul 22, 2026 at 04:48:03PM -0400, Konstantin Ryabitsev wrote:
> On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
> > -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> > +  Assisted-by: LLM coccinelle sparse
>
> I've been playing with this a bit, and it's pretty easy to teach the agent to
> add a few tag-like notes as to how exactly it was used, e.g.:
>
>     https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf
>
> This has:
>
> | Assisted-by: LLM [design, codegen, tests]
>
> It tells us a bit more about *how* the LLM was used, without introducing the
> advertisement or pinning exact models.
>
> This format should allow for multiple tool usage:
>
> | Assisted-by: LLM [triage, tests], coccinelle
>
> I don't suggest this is included in this version, but it *is* fairly
> straightforward to add to AGENTS.md and tells reviewers just how exactly the
> LLM was used.

I think perhaps this should be a follow-up (this stuff is so fraught that I am
super-glad to see Christian's suggestion fly through and want to take gains
iteratively on this :), but completely agree.

David suggested something like:

Assisted-by: LLM # < what it actually did in the patch >

So I think we're kinda aligned on that?

Probably worth keeping the 'what it did' bit fuzzy.

>
> -K

--
Cheers, Lorenzo

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
@ 2026-08-04  9:23     ` David Hildenbrand (Arm)
  2026-08-04 11:09     ` Jori Koolstra
  1 sibling, 0 replies; 12+ messages in thread
From: David Hildenbrand (Arm) @ 2026-08-04  9:23 UTC (permalink / raw)
  To: Lorenzo Stoakes (ARM), Konstantin Ryabitsev
  Cc: Jonathan Corbet, linux-kernel, Christian Brauner, Jeff Layton,
	Jori Koolstra, Sasha Levin, tech-board, Jakub Kicinski, workflows

On 8/4/26 10:10, Lorenzo Stoakes (ARM) wrote:
> On Wed, Jul 22, 2026 at 04:48:03PM -0400, Konstantin Ryabitsev wrote:
>> On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
>>> -  Assisted-by: Claude:claude-3-opus coccinelle sparse
>>> +  Assisted-by: LLM coccinelle sparse
>>
>> I've been playing with this a bit, and it's pretty easy to teach the agent to
>> add a few tag-like notes as to how exactly it was used, e.g.:
>>
>>     https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf
>>
>> This has:
>>
>> | Assisted-by: LLM [design, codegen, tests]
>>
>> It tells us a bit more about *how* the LLM was used, without introducing the
>> advertisement or pinning exact models.
>>
>> This format should allow for multiple tool usage:
>>
>> | Assisted-by: LLM [triage, tests], coccinelle
>>
>> I don't suggest this is included in this version, but it *is* fairly
>> straightforward to add to AGENTS.md and tells reviewers just how exactly the
>> LLM was used.
> 
> I think perhaps this should be a follow-up (this stuff is so fraught that I am
> super-glad to see Christian's suggestion fly through and want to take gains
> iteratively on this :), but completely agree.
> 
> David suggested something like:
> 
> Assisted-by: LLM # < what it actually did in the patch >
> 
> So I think we're kinda aligned on that?

Once the patch from Christian is upstream I can follow up with some
changes. I was holding that back for now until there is agreement on
the LLM part.

-- 
Cheers,

David

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
  2026-08-04  9:23     ` David Hildenbrand (Arm)
@ 2026-08-04 11:09     ` Jori Koolstra
  2026-08-04 11:49       ` Lorenzo Stoakes (ARM)
  1 sibling, 1 reply; 12+ messages in thread
From: Jori Koolstra @ 2026-08-04 11:09 UTC (permalink / raw)
  To: Lorenzo Stoakes (ARM), Konstantin Ryabitsev
  Cc: Jonathan Corbet, linux-kernel, Christian Brauner, Jeff Layton,
	Sasha Levin, tech-board, Jakub Kicinski, workflows,
	David Hildenbrand (Arm)


> Op 04-08-2026 10:10 CEST schreef Lorenzo Stoakes (ARM) <ljs@kernel.org>:
> 
>  
> On Wed, Jul 22, 2026 at 04:48:03PM -0400, Konstantin Ryabitsev wrote:
> > On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
> > > -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> > > +  Assisted-by: LLM coccinelle sparse
> >
> > I've been playing with this a bit, and it's pretty easy to teach the agent to
> > add a few tag-like notes as to how exactly it was used, e.g.:
> >
> >     https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf
> >
> > This has:
> >
> > | Assisted-by: LLM [design, codegen, tests]
> >
> > It tells us a bit more about *how* the LLM was used, without introducing the
> > advertisement or pinning exact models.
> >
> > This format should allow for multiple tool usage:
> >
> > | Assisted-by: LLM [triage, tests], coccinelle
> >
> > I don't suggest this is included in this version, but it *is* fairly
> > straightforward to add to AGENTS.md and tells reviewers just how exactly the
> > LLM was used.
> 
> I think perhaps this should be a follow-up (this stuff is so fraught that I am
> super-glad to see Christian's suggestion fly through and want to take gains
> iteratively on this :), but completely agree.
> 
> David suggested something like:
> 
> Assisted-by: LLM # < what it actually did in the patch >
> 

We should have some format to explain what the assistant was used for,
but not sure if a single line after the tag is going to cut it. I agree
it is the most simple, but I have personally used an LLM for a patch where
the explanation of what exactly it was put to do would have been very
hard to fit in a format like this without significant loss of information.

So, maybe below the ---? I guess that most maintainers don't want this kind
of explanation to be part of the commit message (or maybe they do?)

Best,
Jori.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-08-04 11:09     ` Jori Koolstra
@ 2026-08-04 11:49       ` Lorenzo Stoakes (ARM)
  0 siblings, 0 replies; 12+ messages in thread
From: Lorenzo Stoakes (ARM) @ 2026-08-04 11:49 UTC (permalink / raw)
  To: Jori Koolstra
  Cc: Konstantin Ryabitsev, Jonathan Corbet, linux-kernel,
	Christian Brauner, Jeff Layton, Sasha Levin, tech-board,
	Jakub Kicinski, workflows, David Hildenbrand (Arm)

On Tue, Aug 04, 2026 at 01:09:23PM +0200, Jori Koolstra wrote:
>
> > Op 04-08-2026 10:10 CEST schreef Lorenzo Stoakes (ARM) <ljs@kernel.org>:
> >
> >
> > On Wed, Jul 22, 2026 at 04:48:03PM -0400, Konstantin Ryabitsev wrote:
> > > On Wed, Jul 22, 2026 at 02:34:40PM -0600, Jonathan Corbet wrote:
> > > > -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> > > > +  Assisted-by: LLM coccinelle sparse
> > >
> > > I've been playing with this a bit, and it's pretty easy to teach the agent to
> > > add a few tag-like notes as to how exactly it was used, e.g.:
> > >
> > >     https://git.kernel.org/pub/scm/utils/b4/b4.git/commit/?id=018b87444d32eb2ffee7cb25f0785e54aa35f7bf
> > >
> > > This has:
> > >
> > > | Assisted-by: LLM [design, codegen, tests]
> > >
> > > It tells us a bit more about *how* the LLM was used, without introducing the
> > > advertisement or pinning exact models.
> > >
> > > This format should allow for multiple tool usage:
> > >
> > > | Assisted-by: LLM [triage, tests], coccinelle
> > >
> > > I don't suggest this is included in this version, but it *is* fairly
> > > straightforward to add to AGENTS.md and tells reviewers just how exactly the
> > > LLM was used.
> >
> > I think perhaps this should be a follow-up (this stuff is so fraught that I am
> > super-glad to see Christian's suggestion fly through and want to take gains
> > iteratively on this :), but completely agree.
> >
> > David suggested something like:
> >
> > Assisted-by: LLM # < what it actually did in the patch >
> >
>
> We should have some format to explain what the assistant was used for,
> but not sure if a single line after the tag is going to cut it. I agree
> it is the most simple, but I have personally used an LLM for a patch where
> the explanation of what exactly it was put to do would have been very
> hard to fit in a format like this without significant loss of information.
>
> So, maybe below the ---? I guess that most maintainers don't want this kind
> of explanation to be part of the commit message (or maybe they do?)

I think best to keep discussion of this to the follow up patch to avoid the
usual combinatorial thread explosion on this topic :)

>
> Best,
> Jori.

--
Cheers, Lorenzo

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-07-22 20:34 [PATCH] coding-assistants: simplify attribution Jonathan Corbet
                   ` (2 preceding siblings ...)
  2026-08-03 18:56 ` Jonathan Corbet
@ 2026-08-04 15:28 ` Jeff Layton
  2026-08-04 15:35   ` Jonathan Corbet
  3 siblings, 1 reply; 12+ messages in thread
From: Jeff Layton @ 2026-08-04 15:28 UTC (permalink / raw)
  To: Jonathan Corbet, linux-kernel, Christian Brauner
  Cc: Lorenzo Stoakes, Jori Koolstra, Sasha Levin, tech-board,
	Jakub Kicinski, workflows

On Wed, 2026-07-22 at 14:34 -0600, Jonathan Corbet wrote:
> As suggested by Christian, I have applied his patch to a branch and
> simplified the changelog, see below.
> 
> I'm nervous about applying it still because I don't think it's my role
> to set this sort of policy; I really want to feel that there is a
> consensus behind it.  So there's some additional CCs here because I
> would like to hear from some folks:
> 
> - This policy document originally came out of the TAB; is the TAB (on
>   CC) OK with this change?
> 
> - At least one maintainer (on CC) have said that they currently strip
>   Assisted-by tags out of patches before applying them, a practice that
>   would seem to undermine the whole policy.  Would this change be
>   sufficient to end that practice?
> 
> Thanks,
> 
> jon
> 
> From c31aa376a355e3b84b9e87ae1dbe19b394d4c82c Mon Sep 17 00:00:00 2001
> From: Christian Brauner <brauner@kernel.org>
> Date: Wed, 1 Jul 2026 17:54:48 +0200
> Subject: [PATCH] coding-assistants: simplify attribution
> 
> The requirement to identify specific models used in the Assisted-by tag
> provides free advertising to proprietary software companies while adding
> little or no useful information.  Change the requirement to simply:
> 
>   Assisted-by: LLM
> 
> to capture the fact that an LLM was used without tracking which one.
> 
> Signed-off-by: Christian Brauner (Amutable) <brauner@kernel.org>
> Reviewed-by: Lorenzo Stoakes (ARM) <ljs@kernel.org>
> Reviewed-by: Jeff Layton <jlayton@kernel.org>
> Reviewed-by: Jori Koolstra <jkoolstra@xs4all.nl>
> Message-ID: <20260701-work-coding-assistants-v1-1-a20a94d1d606@kernel.org>
> Signed-off-by: Jonathan Corbet <corbet@lwn.net>
> ---
>  Documentation/process/coding-assistants.rst | 8 ++------
>  1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> index 899f4459c52d..fe34f3e7e828 100644
> --- a/Documentation/process/coding-assistants.rst
> +++ b/Documentation/process/coding-assistants.rst
> @@ -43,12 +43,8 @@ When AI tools contribute to kernel development, proper attribution
>  helps track the evolving role of AI in the development process.
>  Contributions should include an Assisted-by tag in the following format::
>  
> -  Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]
> +  Assisted-by: LLM [TOOL1] [TOOL2]
>  
> -Where:
> -
> -* ``AGENT_NAME`` is the name of the AI tool or framework
> -* ``MODEL_VERSION`` is the specific model version used
>  * ``[TOOL1] [TOOL2]`` are optional specialized analysis tools used
>    (e.g., coccinelle, sparse, smatch, clang-tidy)
>  
> @@ -56,4 +52,4 @@ Basic development tools (git, gcc, make, editors) should not be listed.
>  
>  Example::
>  
> -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> +  Assisted-by: LLM coccinelle sparse

One thing that needs to be changed in conjunction with this is
checkpatch.pl. When I try to use the new format, I get:

● a9b9b7e55379: btrfs: pre-allocate delayed dir index for non-overwrite rename
  ● checkpatch.pl: :2036: WARNING: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format

This patch fixes it for me, but I'm not sure it's restrictive enough.
Jon, feel free to fold this into the Documentation patch, or I can send
it separately if this seems right.

-----------------------8<-------------------------

[PATCH] checkpatch.pl: adapt to new Assisted-by: format

Signed-off-by: Jeff Layton <jlayton@kernel.org>
---
 scripts/checkpatch.pl | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 2b7a42bbdd94..7961d9584bc5 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3124,11 +3124,12 @@ sub process {
 				}
 			}
 
-			# Assisted-by uses AGENT_NAME:MODEL_VERSION format, not email
+			# Assisted-by uses a free-form value (e.g. "LLM"), not an
+			# email address, so skip the email format checks below.
 			if ($sign_off =~ /^Assisted-by:/i) {
-				if ($email !~ /^\S+:\S+/) {
+				if ($email =~ /^\s*$/) {
 					WARN("BAD_SIGN_OFF",
-					     "Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format\n" . $herecurr);
+					     "Assisted-by requires a value\n" . $herecurr);
 				}
 				next;
 			}
-- 
2.55.0



^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] coding-assistants: simplify attribution
  2026-08-04 15:28 ` Jeff Layton
@ 2026-08-04 15:35   ` Jonathan Corbet
  0 siblings, 0 replies; 12+ messages in thread
From: Jonathan Corbet @ 2026-08-04 15:35 UTC (permalink / raw)
  To: Jeff Layton, linux-kernel, Christian Brauner
  Cc: Lorenzo Stoakes, Jori Koolstra, Sasha Levin, tech-board,
	Jakub Kicinski, workflows

Jeff Layton <jlayton@kernel.org> writes:

> One thing that needs to be changed in conjunction with this is
> checkpatch.pl. When I try to use the new format, I get:
>
> ● a9b9b7e55379: btrfs: pre-allocate delayed dir index for non-overwrite rename
>   ● checkpatch.pl: :2036: WARNING: Assisted-by expects 'AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]' format
>
> This patch fixes it for me, but I'm not sure it's restrictive enough.
> Jon, feel free to fold this into the Documentation patch, or I can send
> it separately if this seems right.

I'd say send it separately, probably with a copy to Joe Perches...

Thanks,

jon


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2026-08-04 15:35 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-22 20:34 [PATCH] coding-assistants: simplify attribution Jonathan Corbet
2026-07-22 20:48 ` Konstantin Ryabitsev
2026-07-24 18:17   ` Mauro Carvalho Chehab
2026-08-04  8:10   ` Lorenzo Stoakes (ARM)
2026-08-04  9:23     ` David Hildenbrand (Arm)
2026-08-04 11:09     ` Jori Koolstra
2026-08-04 11:49       ` Lorenzo Stoakes (ARM)
2026-07-24 19:15 ` [tech-board] " David Hildenbrand (Arm)
2026-08-03 18:56 ` Jonathan Corbet
2026-08-03 19:50   ` [tech-board] " Miguel Ojeda
2026-08-04 15:28 ` Jeff Layton
2026-08-04 15:35   ` Jonathan Corbet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox