Re: [PATCH v2] checkpatch: add "Reported-and-tested-by:" tag

From: Dmitry Vyukov
Date: Wed Jul 17 2024 - 06:25:15 EST


On Wed, 17 Jul 2024 at 00:44, Carlos Llamas <cmllamas@xxxxxxxxxx> wrote:
>
> On Tue, Jul 16, 2024 at 09:11:08AM +0200, Dmitry Vyukov wrote:
> > On Tue, 16 Jul 2024 at 07:24, Carlos Llamas <cmllamas@xxxxxxxxxx> wrote:
> > >
> > > If combined tags are discouraged then syzbot should perhaps stop
> > > suggesting the `Reported-and-tested-by:` tag? I would imagine this not
> > > only applies to the tip tree and that other maintainers agree with not
> > > using combined tags.
> > >
> > > FWIW, this tag in particular though is quite popular:
> > > $ git log --grep '^Reported-and-tested-by: ' origin/master |wc -l
> > > 82056
>
> The number of patches with this tag is actually 2689. Sorry, I forgot to
> add the --oneline option to my initial query.
>
> >
> > +syzkaller mailing list
> >
> > We can add whatever text you will say. What is the official position?
>
> I don't know if there is an official position but IMO individual tags
> are the safest option. This would even allow tools such as b4 to collect
> the tags automatically. So splitting out 'Reported-and-tested-by:' seems
> trivial enough?
>
> Reported-by: syzbot+<context>
> Tested-by: syzbot+<context>

I've sent https://github.com/google/syzkaller/pull/5060/files
to split the tag into 2 in syzbot emails.