All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Cody Zuschlag <cody.zuschlag@gmail.com>
Cc: Jan Beulich <jbeulich@suse.com>,
	andrew.cooper3@citrix.com, anthony.perard@vates.tech,
	michal.orzel@amd.com, julien@xen.org, sstabellini@kernel.org,
	xen-devel@lists.xenproject.org
Subject: Re: [PATCH v2] docs/process: document AI-assisted patch tags
Date: Tue, 26 May 2026 18:09:05 +0200	[thread overview]
Message-ID: <ahXFodLdshZozUCD@macbook.local> (raw)
In-Reply-To: <3e2ad5b9-ef65-41fd-87bf-823b98fa0c3d@suse.com>

On Thu, May 21, 2026 at 05:46:18PM +0200, Jan Beulich wrote:
> On 19.05.2026 16:20, Cody Zuschlag wrote:
> > --- a/docs/process/sending-patches.pandoc
> > +++ b/docs/process/sending-patches.pandoc
> > @@ -195,6 +195,25 @@ E.g.:
> >  
> >      Requested-by: Jane Doe <jane.doe@example.org>
> >  
> > +### Assisted-by:
> > +
> > +This optional tag can be used when AI-assisted tooling contributed to the
> > +preparation of a patch.
> 
> Noticed only while reading your other email, plus checking the Linux
> wording: s/can/should/ at the very least? (Personally I'd make it a
> requirement.)
> 
> > It is in the format:
> > +
> > +    Assisted-by: <tool-name>[:<model-version>] [tool...]
> > +
> > +E.g.:
> > +
> > +    Assisted-by: Claude:claude-3-opus coccinelle sparse
> > +    Assisted-by: ChatGPT:gpt-5.5
> > +
> > +Basic development tools, such as git, compilers, make, and editors, should
> > +not be listed.
> > +
> > +The person submitting the patch remains responsible for reviewing any
> > +AI-generated content and ensuring the contribution is correct, appropriate,
> > +and compliant with Xen's contribution and licensing requirements.
> > +
> >  ### Signed-off-by:
> >  
> >  This mandatory tag specifies the author(s) of a patch (for each author a
> > @@ -209,6 +228,9 @@ E.g.:
> >  The author must be a natural person (not a team or just a company) and the
> >  `Signed-off-by:` tag must include the real name of the author (no pseudonym).
> >  
> > +AI tools must not add `Signed-off-by:` tags, as only natural
> > +persons can certify the `Developer's Certificate of Origin`.
> 
> Linux emphasizes "MUST NOT". I think that's advisable for us as well. I
> also wonder whether some of their other wording might be worth taking
> verbatim, rather than making our own, slightly different variant.
> 
> Nit: Wrap differently to make better use of line length?

With the proposed changes:

Acked-by: Roger Pau Monné <roger.pau@citrix.com>

Thanks, Roger.


      parent reply	other threads:[~2026-05-26 16:09 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-19 14:20 [PATCH v2] docs/process: document AI-assisted patch tags Cody Zuschlag
2026-05-21 15:46 ` Jan Beulich
2026-05-21 19:30   ` Stefano Stabellini
2026-05-26 16:09   ` Roger Pau Monné [this message]

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=ahXFodLdshZozUCD@macbook.local \
    --to=roger.pau@citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=anthony.perard@vates.tech \
    --cc=cody.zuschlag@gmail.com \
    --cc=jbeulich@suse.com \
    --cc=julien@xen.org \
    --cc=michal.orzel@amd.com \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.