Re: [Enable Designware XGMAC VLAN Stripping Feature 1/2] dt-bindings: net: snps,dwmac: Add description for rx-vlan-offload

[Joe Perches]

On Fri, 2023-07-21 at 18:55 -0700, Jakub Kicinski wrote:
> On Fri, 21 Jul 2023 18:21:32 +0200 Krzysztof Kozlowski wrote:
> > > $ ./scripts/get_maintainer.pl  --scm  -f drivers/net/ethernet/stmicro/stmmac/dwxgmac2_core.c  
> > 
> > That's not how you run it. get_maintainers.pl should be run on patches
> > or on all files, not just some selection.
> 
> Adding Joe for visibility (I proposed to print a warning when people 
> do this and IIRC he wasn't on board).

What's the issue here?  Other than _how_ the script was used,
I don't see an actual problem with the script itself.

https://lore.kernel.org/lkml/20230721062617.9810-1-boon.khai.ng@xxxxxxxxx/

As far as I can tell, the patch series address list was identical
for the 0/2, 1/2, and 2/2 submissions:

--------------------------------

0/2:

   From: Boon@xxxxxxxxxxxxxxxxxxxx, Khai@xxxxxxxxxxxxxxxxxxxx, "Ng <boon.khai.ng"@intel.com
   To: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>,
   	Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>,
   	Jose Abreu <joabreu@xxxxxxxxxxxx>,
   	"David S . Miller" <davem@xxxxxxxxxxxxx>,
   	Eric Dumazet <edumazet@xxxxxxxxxx>,
   	Jakub Kicinski <kuba@xxxxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>,
   	Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>,
   	netdev@xxxxxxxxxxxxxxx, linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx,
   	linux-arm-kernel@xxxxxxxxxxxxxxxxxxx,
   	linux-kernel@xxxxxxxxxxxxxxx
   Cc: Boon Khai Ng <boon.khai.ng@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxx>,
   	Mun Yew Tham <mun.yew.tham@xxxxxxxxx>,
   	Leong Ching Swee <leong.ching.swee@xxxxxxxxx>,
   	G Thomas Rohan <rohan.g.thomas@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxxxxxxxx>
   Subject: [Enable Designware XGMAC VLAN Stripping Feature 0/2]
   
1/2:

   From: Boon@xxxxxxxxxxxxxxxxxxxx, Khai@xxxxxxxxxxxxxxxxxxxx, "Ng <boon.khai.ng"@intel.com
   To: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>,
   	Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>,
   	Jose Abreu <joabreu@xxxxxxxxxxxx>,
   	"David S . Miller" <davem@xxxxxxxxxxxxx>,
   	Eric Dumazet <edumazet@xxxxxxxxxx>,
   	Jakub Kicinski <kuba@xxxxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>,
   	Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>,
   	netdev@xxxxxxxxxxxxxxx, linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx,
   	linux-arm-kernel@xxxxxxxxxxxxxxxxxxx,
   	linux-kernel@xxxxxxxxxxxxxxx
   Cc: Boon Khai Ng <boon.khai.ng@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxx>,
   	Mun Yew Tham <mun.yew.tham@xxxxxxxxx>,
   	Leong Ching Swee <leong.ching.swee@xxxxxxxxx>,
   	G Thomas Rohan <rohan.g.thomas@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxxxxxxxx>
   Subject: [Enable Designware XGMAC VLAN Stripping Feature 1/2] dt-bindings: net: snps,dwmac: Add description for rx-vlan-offload
   
2/2:

   From: Boon@xxxxxxxxxxxxxxxxxxxx, Khai@xxxxxxxxxxxxxxxxxxxx, "Ng <boon.khai.ng"@intel.com
   To: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>,
   	Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx>,
   	Jose Abreu <joabreu@xxxxxxxxxxxx>,
   	"David S . Miller" <davem@xxxxxxxxxxxxx>,
   	Eric Dumazet <edumazet@xxxxxxxxxx>,
   	Jakub Kicinski <kuba@xxxxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>,
   	Maxime Coquelin <mcoquelin.stm32@xxxxxxxxx>,
   	netdev@xxxxxxxxxxxxxxx, linux-stm32@xxxxxxxxxxxxxxxxxxxxxxxxxxxx,
   	linux-arm-kernel@xxxxxxxxxxxxxxxxxxx,
   	linux-kernel@xxxxxxxxxxxxxxx
   Cc: Boon Khai Ng <boon.khai.ng@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxx>,
   	Mun Yew Tham <mun.yew.tham@xxxxxxxxx>,
   	Leong Ching Swee <leong.ching.swee@xxxxxxxxx>,
   	G Thomas Rohan <rohan.g.thomas@xxxxxxxxx>,
   	Shevchenko Andriy <andriy.shevchenko@xxxxxxxxxxxxxxx>
   Subject: [Enable Designware XGMAC VLAN Stripping Feature 2/2] net: stmmac: dwxgmac2: Add support for HW-accelerated VLAN Stripping
   
--------------------------------

vger has a limit on the number of recipients for a single email
so patch series that cc all possible recipients can be bounced
and not forwarded by vger.

So I think when submitting a patch series, it's necessary to send
just the cover letter to all mailing lists for all files/paths
modified by any file in the patch series and specific patches
are sent to maintainers, reviewers and the specific mailing lists
modified by the specific patch.

I use the scripts below to send patch series where a patch series
are the only files in individual directories.

(Well I used to use, I'm not actively reading or creating kernel patches right now)

$ cat ~/bin/to.sh
#!/bin/bash

opts="--nogit --nogit-fallback --norolestats --pattern-depth=1"

if [[ $(basename $1) =~ ^0000- ]] ; then
    ./scripts/get_maintainer.pl --nom $opts  $(dirname $1)/*
else
    maint=$(./scripts/get_maintainer.pl --nol $opts $1)
    if [ "$maint" == "" ] ; then
	echo "linux-kernel@xxxxxxxxxxxxxxx"
    else
	echo "$maint"
    fi
fi


$ cat ~/bin/cc.sh
#!/bin/bash

opts="--nogit --nogit-fallback --norolestats"
maint_file=$(mktemp -t XXXXXXXX.cc)

if [[ $(basename $1) =~ ^0000- ]] ; then
    ./scripts/get_maintainer.pl $opts $(dirname $1)/* |  \
	~/bin/remove_undesirable_emails.sh > $maint_file
    count=$(wc -c $maint_file | cut -f1 -d" ")
    if [[ $count -lt 512 ]] ; then
	cat $maint_file
    else
	./scripts/get_maintainer.pl -nom -nor $opts $(dirname $1)/* | \
	    ~/bin/remove_undesirable_emails.sh
    fi
else
    ./scripts/get_maintainer.pl $opts $1 | \
	~/bin/remove_undesirable_emails.sh > $maint_file
    count=$(wc -l $maint_file | cut -f1 -d" ")
    if [[ $count -gt 0 ]] ; then
	cat $maint_file
    else
	./scripts/get_maintainer.pl --git --git-fallback --norolestats $1 | \
	    ~/bin/remove_undesirable_emails.sh
    fi
fi
rm -f $maint_file


$ cat ~/bin/remove_undesirable_emails.sh
grep -vPi "(?:\bIngo\s+Molnar\b)"

$

(nb: Ingo asked not to receive any emails from me)

And these scripts are used with git send-email with a .gitconfig block

[sendemail]
	chainreplyto = false
	thread = false
	suppresscc = self
	cccmd = ~/bin/cc.sh
	tocmd = ~/bin/to.sh

These scripts would have added 2 mailing lists to patch 0/n:

devicetree@xxxxxxxxxxxxxxx (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
linux-renesas-soc@xxxxxxxxxxxxxxx (open list:ARM/RISC-V/RENESAS ARCHITECTURE)

and it would also have had a different recipient list for 1/2 as well

$ ./scripts/get_maintainer.pl -f Documentation/devicetree/bindings/net/snps,dwmac.yaml
"David S. Miller" <davem@xxxxxxxxxxxxx> (maintainer:NETWORKING DRIVERS)
Eric Dumazet <edumazet@xxxxxxxxxx> (maintainer:NETWORKING DRIVERS)
Jakub Kicinski <kuba@xxxxxxxxxx> (maintainer:NETWORKING DRIVERS)
Paolo Abeni <pabeni@xxxxxxxxxx> (maintainer:NETWORKING DRIVERS)
Rob Herring <robh+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
Conor Dooley <conor+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
Richard Cochran <richardcochran@xxxxxxxxx> (maintainer:PTP HARDWARE CLOCK SUPPORT)
Geert Uytterhoeven <geert+renesas@xxxxxxxxx> (supporter:ARM/RISC-V/RENESAS ARCHITECTURE)
Magnus Damm <magnus.damm@xxxxxxxxx> (supporter:ARM/RISC-V/RENESAS ARCHITECTURE)
Alexandre Torgue <alexandre.torgue@xxxxxxxxxxx> (in file)
Giuseppe Cavallaro <peppe.cavallaro@xxxxxx> (in file)
Jose Abreu <joabreu@xxxxxxxxxxxx> (in file)
netdev@xxxxxxxxxxxxxxx (open list:NETWORKING DRIVERS)
devicetree@xxxxxxxxxxxxxxx (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
linux-kernel@xxxxxxxxxxxxxxx (open list)
linux-renesas-soc@xxxxxxxxxxxxxxx (open list:ARM/RISC-V/RENESAS ARCHITECTURE)