[Linux-kernel-mentees] [PATCH] checkpatch: Documentation: include sign off for reverts

Lukas Bulwahn lukas.bulwahn at gmail.com
Tue Nov 10 16:45:03 UTC 2020


On Tue, Nov 10, 2020 at 4:34 PM Lukas Bulwahn <lukas.bulwahn at gmail.com> wrote:
>
> On Tue, Nov 10, 2020 at 4:27 PM Aditya <yashsri421 at gmail.com> wrote:
> >
> > On 10/11/20 8:19 pm, Lukas Bulwahn wrote:
> > > On Tue, Nov 10, 2020 at 2:42 PM Aditya Srivastava <yashsri421 at gmail.com> wrote:
> > >>
> > >> Currently, we do not have any documentation on commit reverts regarding
> > >> the requirement of Signed-off-by tag for it. This may be misleading to
> > >> the users.
> > >>
> > >> On evaluating MISSING_SIGN_OFF over v4.13..v5.8, it was found that
> > > Just some minor nitpicking on wording:
> > >
> > > Evaluating MISSING_SIGN_OFF checkpatch warnings on v4.13..v5.8 showed that
> > >> 4 out of 11 cases for this error is because of missing sign offs in
> > >
> > > 4 out of 11 cases missing a sign-off are revert commits.
> > >
> > >> revert commits.
> > >>
> > >> Add documentation regarding the same.
> > >>
> > >
> > > ...to document the community consensus and let readers know.
> > >
> > > Send the revised patch out to the appropriate developers and mailing
> > > lists and please CC me and Joe (because it is also on checkpatch
> > > rules).
> > >
> > Thanks for reviewing. I have updated the message.
> > Actually when I ran ./scripts/get_maintainer.pl over this patch I am
> > getting a long list of maintainers, so I wanted to confirm once if I
> > should send to everyone in this list.
> >

Maybe you misunderstand my previous email:

you should send it to those two people:

> > Jonathan Corbet <corbet at lwn.net>
> > (maintainer:DOCUMENTATION,commit_signer:9/11=82%)
> > Drew DeVault <sir at cmpwn.com>
> > (commit_signer:4/11=36%,authored:4/11=36%,added_lines:53/80=66%,removed_lines:212/228=93%)
>


> It could be that Drew was contracted long ago and is not in the
> community anymore.
>
> You can drop these five people:
>

And "drop from the list" means: You should not send it to those five
people below.

> > Mauro Carvalho Chehab <mchehab+huawei at kernel.org>
> > (commit_signer:2/11=18%,authored:2/11=18%,added_lines:9/80=11%,removed_lines:13/228=6%)
> > Krzysztof Kozlowski <krzk at kernel.org>
> > (commit_signer:2/11=18%,authored:2/11=18%,added_lines:9/80=11%)
> > Greg Kroah-Hartman <gregkh at linuxfoundation.org> (commit_signer:1/11=9%)
> > "Alexander A. Klimov" <grandmaster at al2klimov.de> (authored:1/11=9%)
> > Tom Rix <trix at redhat.com> (added_lines:5/80=6%)
>
> These are the right mailing lists:
>
> > linux-doc at vger.kernel.org (open list:DOCUMENTATION)
> > linux-kernel at vger.kernel.org (open list)
> >
>
>
> Lukas


More information about the Linux-kernel-mentees mailing list