Re: [PATCH] Documentation: remove the requirement for LLM attribution

[Lorenzo Stoakes <ljs@kernel.org>]

On Tue, Jul 07, 2026 at 11:49:31AM +0200, David Hildenbrand (Arm) wrote:
> On 7/7/26 11:20, Lorenzo Stoakes wrote:
> > On Fri, Jul 03, 2026 at 08:22:20PM +0200, David Hildenbrand (Arm) wrote:
> >> On 7/3/26 18:32, Laurent Pinchart wrote:
> >>>
> >>> I'd support a patch that replaces
> >>>
> >>> Assisted-by: Claude:claude-3-opus coccinelle sparse
> >>>
> >>> with
> >>>
> >>> Assisted-by: LLM # generate some test cases
> >>>
> >>> and rewrites the Attribution section of
> >>> Documentation/process/coding-assistants.rst accordingly.
> >>>
> >>> I think most people in this mail thread have expressed that how
> >>> generative AI was used is the most important information, and several
> >>> people (including myself) have expressed a desire to stop the free
> >>> advertising. Unless I missed something, I don't think anyone has
> >>> expressed an interest in keeping the agent name and model.
> >>
> >> Yes, that's something I would enjoy seeing. Who reading along has a problem with
> >> that? (assuming that the tag must stay)
> >
> > I think the silence suggests nobody :)
> >
> > What we could do is send a patch to simply add a section on adding a comment
> > explaining _what_ used an LLM and strongly suggesting it would be helpful to do
> > so.
> >
> > That should hopefully be uncontroversial, then later (or alongside that?) we
> > could submit an RFC for making the language move to 'you MUST do this', as that
> > will probably be the more controversial bit.
>
> Maybe we should just spell out about which scenario we care about where we really
> expect the disclosure.
>
> Sure, it might be more controversial and we could have that in a separate patch.
>
> I was thinking of the following end result:
>
> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> index 899f4459c52d2..4c2ab4dfc6da7 100644
> --- a/Documentation/process/coding-assistants.rst
> +++ b/Documentation/process/coding-assistants.rst
> @@ -40,20 +40,37 @@ Attribution
>  ===========
>
>  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::
> +helps track the evolving role of AI in the development process. Further,
> +for reviewers and maintainers it is also crucially important to know how
> +AI tools were used.
>
> -  Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]
> +Contributions that used AI to generate significant portions of code,
> +comments, or patch descriptions must include an Assisted-by tag in the
> +following format::
>
> -Where (preferred):
> +  Assisted-by: LLM # brief description of usage
> +
> +Or alternatively::
> +
> +  Assisted-by: AGENT_NAME:MODEL_VERSION # brief description of usage
> +
> +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)
> +
> +If other tools were used, they should be specified through a dedicated
> +Assisted-by tag in the following format::
> +
> +  Assisted-by: [TOOL1] [TOOL2]
> +
> +Where ``[TOOL1] [TOOL2]`` are specialized analysis tools used
> +(e.g., coccinelle, sparse, smatch, clang-tidy)
>
>  Basic development tools (git, gcc, make, editors) should not be listed.
>
> -Example::
> +Examples::
>
> -  Assisted-by: Claude:claude-3-opus coccinelle sparse
> +  Assisted-by: LLM # translate patch description
> +  Assisted-by: Claude:claude-3-opus # generate most of the code
> +  Assisted-by: coccinelle sparse

LGTM! Do you want to send that then? People can comment on the actual path then
(probably worth cc'ing everybody here on that also).

>
>
> But I am not really sure why we should specify other tools here, really.
> People usually do that as part of the patch description, including sharing
> coccinelle scripts etc.
>
> Because looking at the history:
>

(for succinctness :P)

$ git log | grep "Assisted-by" | grep -Ei "(coccinelle|sparse|smatch|clang)" | wc -l
30

Yeah, doesn't seem much used, but I guess for the non-controversial version
worth keeping that stuff around in the doc.

> --
> Cheers,
>
> David

Cheers, Lorenzo