linux-kernel - [BUG] get_maintainers: Inconsistent behavior on multiple runs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAKohpoknWcxnrgTQUBFR5eEpbcs_=BbsjKUuzKJVqx3QFXv7AQ@mail.gmail.com>
Date:   Thu, 5 Oct 2017 12:21:24 +0530
From:   Viresh Kumar <viresh.kumar@...aro.org>
To:     Joe Perches <joe@...ches.com>
Cc:     Manu Gautam <mgautam@...eaurora.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Vincent Guittot <vincent.guittot@...aro.org>
Subject: [BUG] get_maintainers: Inconsistent behavior on multiple runs

Hi Joe,

Recently I was Cc'd for some of the patches from Manu, as
get_maintainers incorrectly mentioned me.

Apart from the fact that I never contributed to the mentioned
file (drivers/phy/qualcomm/phy-qcom-qmp.c), the behavior (output)
of get_maintainers changes on multiple runs. I tried following tests on
pm/linux-next branch today (should be same on any of the latest -rc
releases). There was no difference in the environment between the
two runs.

$ scripts/get_maintainer.pl drivers/phy/qualcomm/phy-qcom-qmp.c
Kishon Vijay Abraham I <kishon@...com> (supporter:GENERIC PHY
FRAMEWORK,commit_signer:7/7=100%)
Vivek Gautam <vivek.gautam@...eaurora.org>
(commit_signer:5/7=71%,authored:3/7=43%)
Varadarajan Narayanan <varada@...eaurora.org>
(commit_signer:2/7=29%,authored:2/7=29%)
Fengguang Wu <fengguang.wu@...el.com> (commit_signer:1/7=14%,authored:1/7=14%)
Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com> (commit_signer:1/7=14%)
Wei Yongjun <weiyongjun1@...wei.com> (authored:1/7=14%)
linux-kernel@...r.kernel.org (open list:GENERIC PHY FRAMEWORK)

$ scripts/get_maintainer.pl drivers/phy/qualcomm/phy-qcom-qmp.c
Kishon Vijay Abraham I <kishon@...com> (supporter:GENERIC PHY
FRAMEWORK,commit_signer:7/7=100%)
Vivek Gautam <vivek.gautam@...eaurora.org>
(commit_signer:5/7=71%,authored:3/7=43%)
Varadarajan Narayanan <varada@...eaurora.org>
(commit_signer:2/7=29%,authored:2/7=29%)
Krzysztof Kozlowski <krzk@...nel.org> (commit_signer:1/7=14%)
Viresh Kumar <viresh.kumar@...aro.org> (commit_signer:1/7=14%)
Wei Yongjun <weiyongjun1@...wei.com> (authored:1/7=14%)
Fengguang Wu <fengguang.wu@...el.com> (authored:1/7=14%)
linux-kernel@...r.kernel.org (open list:GENERIC PHY FRAMEWORK)

The second one incorrectly adds me to the list. Also the order of entries
is different, not sure if it should be.

The same inconsistency is seen while running on the real patch (attached).

$ scripts/get_maintainer.pl ~/Downloads/original_msg.txt
Kishon Vijay Abraham I <kishon@...com> (supporter:GENERIC PHY
FRAMEWORK,commit_signer:7/7=100%)
Vivek Gautam <vivek.gautam@...eaurora.org>
(commit_signer:5/7=71%,authored:3/7=43%)
Varadarajan Narayanan <varada@...eaurora.org>
(commit_signer:2/7=29%,authored:2/7=29%)
Heiko Stuebner <heiko@...ech.de> (commit_signer:1/7=14%)
Jaehoon Chung <jh80.chung@...sung.com> (commit_signer:1/7=14%)
Wei Yongjun <weiyongjun1@...wei.com> (authored:1/7=14%)
Fengguang Wu <fengguang.wu@...el.com> (authored:1/7=14%)
linux-kernel@...r.kernel.org (open list:GENERIC PHY FRAMEWORK)

$ scripts/get_maintainer.pl ~/Downloads/original_msg.txt
Kishon Vijay Abraham I <kishon@...com> (supporter:GENERIC PHY
FRAMEWORK,commit_signer:7/7=100%)
Vivek Gautam <vivek.gautam@...eaurora.org>
(commit_signer:5/7=71%,authored:3/7=43%)
Varadarajan Narayanan <varada@...eaurora.org>
(commit_signer:2/7=29%,authored:2/7=29%)
Viresh Kumar <viresh.kumar@...aro.org> (commit_signer:1/7=14%)
Fengguang Wu <fengguang.wu@...el.com> (commit_signer:1/7=14%,authored:1/7=14%)
Wei Yongjun <weiyongjun1@...wei.com> (authored:1/7=14%)
linux-kernel@...r.kernel.org (open list:GENERIC PHY FRAMEWORK)

I have no clue on why this happened and so including you in the chain :)

--
viresh

View attachment "original_msg.txt" of type "text/plain" (9818 bytes)