Re: [MAINTAINER SUMMIT] Rethinking the acceptance policy for "trivial" patches

[Greg KH <greg@kroah.com>]

On Fri, Apr 23, 2021 at 09:58:30AM +0200, Mauro Carvalho Chehab wrote:
> Em Thu, 22 Apr 2021 11:52:35 -0400
> Steven Rostedt <rostedt@goodmis.org> escreveu:
> 
> > On Thu, 22 Apr 2021 08:48:21 -0700
> > James Bottomley <James.Bottomley@HansenPartnership.com> wrote:
> > 
> > > On Thu, 2021-04-22 at 08:42 -0700, James Bottomley wrote:
> > > [...]  
> > > >    2. Improving the requirement for bug fixes and large series, like
> > > > cover letters to everyone, adding fixes: tag and clear explanation.    
> > > 
> > > Just on this one, can we get the mailing list to help now we're moving
> > > to a new infrastructure?  I was already thinking of asking if it could
> > > reject email with html parts rather than simply losing it, but perhaps
> > > it could reject threaded submissions where the cover letter isn't
> > > correctly cc'd?  I know that's a big ask because there has to be an
> > > easy way to recognize them (heuristics on the PATCH tag?) and a way to
> > > recognize missing cc's (perhaps simply that someone cc'd on the
> > > threaded [PATCH x/y] reply isn't cc'd on [PATCH 0/y]?)  
> > 
> > Unfortunately, this breaks all quilt users, as quilt does not support this.
> 
> This will also break patch series that touch several subsystems.
> 
> Out of curiosity, I ran my script letting it to place at the cover letter
> maintainers, reviewers and mailing lists, for this patch series:
> 
> 	[PATCH 000/190] Revertion of all of the umn.edu commits
> 	https://lore.kernel.org/lkml/YIJyzkgglMrAzIwh@kroah.com/T/#m087445f69f5dd590b9ad5f4cdd62c2a812956435
> 
> The number of e-mails to be C/c is 221 e-mails! (see enclosed)
> 
> An e-mail like that will almost for sure be ignored by  all mail
> servers[1], as the e-mail will be considered as SPAM.
> 
> [1] Except if the servers would have explicit rules to allow such
>     really big c/c list to be accepted from maintainers, which is
>     risky.
> 
> Looking at the actual e-mail from Greg at lore, the CC list was a lot
> smaller than that:
> 
> Cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
>         Linus Torvalds <torvalds@linux-foundation.org>,
>         Aditya Pakki <pakki001@umn.edu>, Kangjie Lu <kjlu@umn.edu>,
>         Qiushi Wu <wu000273@umn.edu>, x86@kernel.org,
>         Bjorn Helgaas <bhelgaas@google.com>,
>         "Rafael J. Wysocki" <rjw@rjwysocki.net>,
>         Arnd Bergmann <arnd@arndb.de>, David Airlie <airlied@linux.ie>,
>         Michael Turquette <mturquette@baylibre.com>,
>         Bjorn Andersson <bjorn.andersson@linaro.org>,
>         Linus Walleij <linus.walleij@linaro.org>,
>         Bartosz Golaszewski <bgolaszewski@baylibre.com>,
>         Daniel Vetter <daniel@ffwll.ch>,
>         Jean Delvare <jdelvare@suse.com>,
>         Guenter Roeck <linux@roeck-us.net>,
>         Jiri Kosina <jikos@kernel.org>, Will Deacon <will@kernel.org>,
>         Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
>         Jakub Kicinski <kuba@kernel.org>,
>         "David S. Miller" <davem@davemloft.net>,
>         Johan Hovold <johan@kernel.org>,
>         Jiri Slaby <jirislaby@kernel.org>,
>         Pablo Neira Ayuso <pablo@netfilter.org>,
>         Johannes Berg <johannes@sipsolutions.net>,
>         Takashi Iwai <tiwai@suse.com>
> 
> (Not sure what criteria Greg used to shorten the C/c list)

I looked at the actual maintainers for the whole list of patches and
made a judgement call to slim it down to something "manageable"

In other words, I was forced to do it "by hand" :(

thanks,

greg k-h