get_maintainer.pl wrong and undeterministic? (was: Re: [PATCH v2 2/2] usb: gadget: udc: Handle gadget_connect failure during bind operation)
From: Geert Uytterhoeven
Date: Thu Apr 27 2023 - 05:09:32 EST
Hi Krishna,
CC Joe and lkml
On Thu, Apr 27, 2023 at 10:44 AM Krishna Kurapati PSSNV
<quic_kriskura@xxxxxxxxxxx> wrote:
> On 4/27/2023 1:23 PM, Geert Uytterhoeven wrote:
> > On Thu, Apr 27, 2023 at 5:49 AM Krishna Kurapati PSSNV
> > <quic_kriskura@xxxxxxxxxxx> wrote:
> >> On 4/26/2023 2:48 PM, Geert Uytterhoeven wrote:
> >>> On Wed, Apr 26, 2023 at 3:17 AM Krishna Kurapati PSSNV
> >>> <quic_kriskura@xxxxxxxxxxx> wrote:
> >>>> Hi Alan, Geert,
> >>>>
> >>>> Can you help review and provide comments/approval on the following patch.
> >>>
> >>> I don't know why you are addressing me, as I never touched the affected
> >>> file, am not listed as its maintainer, and don't know much about USB UDC.
> >
> >> Apologies. I must have caused some confusion because of same name. I
> >> must have specified clearly whom I was referring to.
> >>
> >> I CC'd and was referring to Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> >> for comments.
> >
> > That's actually me, too ;-)
> >
> >> As per the output of get_maintainer.pl
> >>
> >> ./scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c
> >>
> >> Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> (supporter:USB
> >> SUBSYSTEM,commit_signer:6/6=100%,authored:1/6=17%,removed_lines:2/26=8%)
> >> Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> >> (commit_signer:4/6=67%,authored:3/6=50%,added_lines:36/45=80%,removed_lines:17/26=65%)
> >> TaoXue <xuetao09@xxxxxxxxxx> (commit_signer:1/6=17%)
> >> "Rafael J. Wysocki" <rafael@xxxxxxxxxx> (commit_signer:1/6=17%)
> >> Geert Uytterhoeven <geert+renesas@xxxxxxxxx> (commit_signer:1/6=17%)
> >> Colin Ian King <colin.i.king@xxxxxxxxx> (authored:1/6=17%)
> >> Jiantao Zhang <water.zhangjiantao@xxxxxxxxxx>
> >> (authored:1/6=17%,added_lines:6/45=13%,removed_lines:6/26=23%)
> >
> > Interesting, I don't see me listed when running that command (on v6.3 and
> > next-20230425), and I never authored any change to that file.
> > What is the tree (commit sha1) you are running ./scripts/get_maintainer.pl on?
>
> I checked it on linux-next a couple of weeks back and it showed me this.
> But when I synced latest linux kernel, it didn't show it today 😅
> Not sure, what is the diff here.
Interesting:
$ git checkout next-20230425
Updating files: 100% (7386/7386), done.
Previous HEAD position was 198925fae644b009 Add linux-next
specific files for 20230329
HEAD is now at f600e0bbde8562a0 Add linux-next specific files for 20230425
$ scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c
Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> (supporter:USB
SUBSYSTEM,commit_signer:11/11=100%,authored:3/11=27%,removed_lines:5/73=7%)
Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
(commit_signer:4/11=36%,authored:3/11=27%,added_lines:36/182=20%,removed_lines:17/73=23%)
Badhri Jagan Sridharan <badhri@xxxxxxxxxx>
(commit_signer:2/11=18%,authored:2/11=18%,added_lines:107/182=59%,removed_lines:44/73=60%)
Elson Roy Serrao <quic_eserrao@xxxxxxxxxxx>
(commit_signer:1/11=9%,added_lines:27/182=15%)
Sebastian Reichel <sre@xxxxxxxxxx> (commit_signer:1/11=9%)
Colin Ian King <colin.i.king@xxxxxxxxx> (authored:1/11=9%)
Jiantao Zhang <water.zhangjiantao@xxxxxxxxxx>
(authored:1/11=9%,removed_lines:6/73=8%)
linux-usb@xxxxxxxxxxxxxxx (open list:USB SUBSYSTEM)
linux-kernel@xxxxxxxxxxxxxxx (open list)
$ git checkout next-20230329
Updating files: 100% (7386/7386), done.
Previous HEAD position was f600e0bbde8562a0 Add linux-next
specific files for 20230425
HEAD is now at 198925fae644b009 Add linux-next specific files for 20230329
$ scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c
Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> (supporter:USB
SUBSYSTEM,commit_signer:8/8=100%,authored:3/8=38%,added_lines:5/48=10%,removed_lines:5/29=17%)
Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
(commit_signer:3/8=50%,authored:3/8=38%,added_lines:36/48=75%,removed_lines:17/29=59%)
Geert Uytterhoeven <geert+renesas@xxxxxxxxx> (commit_signer:1/8=12%)
Sebastian Reichel <sre@xxxxxxxxxx> (commit_signer:1/8=12%)
Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx> (commit_signer:1/8=12%)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
I, Sebastian, and Heikki never touched this file...
Jiantao Zhang <water.zhangjiantao@xxxxxxxxxx>
(authored:1/8=12%,added_lines:6/48=12%,removed_lines:6/29=21%)
Colin Ian King <colin.i.king@xxxxxxxxx> (authored:1/8=12%)
linux-usb@xxxxxxxxxxxxxxx (open list:USB SUBSYSTEM)
linux-kernel@xxxxxxxxxxxxxxx (open list)
$ scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c
Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> (supporter:USB
SUBSYSTEM,commit_signer:8/8=100%,authored:3/8=38%,added_lines:5/48=10%,removed_lines:5/29=17%)
Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
(commit_signer:4/8=50%,authored:3/8=38%,added_lines:36/48=75%,removed_lines:17/29=59%)
"Rafael J. Wysocki" <rafael@xxxxxxxxxx> (commit_signer:1/8=12%)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Heikki and I are gone, but Rafael is new and also hasn't touched this file?
Colin Ian King <colin.i.king@xxxxxxxxx>
(commit_signer:1/8=12%,authored:1/8=12%)
Sebastian Reichel <sre@xxxxxxxxxx> (commit_signer:1/8=12%)
Jiantao Zhang <water.zhangjiantao@xxxxxxxxxx>
(authored:1/8=12%,added_lines:6/48=12%,removed_lines:6/29=21%)
linux-usb@xxxxxxxxxxxxxxx (open list:USB SUBSYSTEM)
linux-kernel@xxxxxxxxxxxxxxx (open list)
You can see the differences when running the following multiple times:
$ diff <(scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c)
<(scripts/get_maintainer.pl drivers/usb/gadget/udc/core.c)
Looks like scripts/get_maintainer.pl (a) shows wrong committers (they
did provide other e.g. Reviewed-by tags), and (b) is not deterministic?
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds