> On Nov 10, 2021, at 9:20 AM, Srivatsa S. Bhat <sriva...@csail.mit.edu> wrote: > > On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote: >> On Tue, 2021-11-09 at 00:58 +0000, Nadav Amit wrote: >>>> On Nov 8, 2021, at 4:37 PM, Joe Perches <j...@perches.com> wrote: >>>> On Mon, 2021-11-08 at 16:22 -0800, Srivatsa S. Bhat wrote: >>>> >>>> So it's an exploder not an actual maintainer and it likely isn't >>>> publically archived with any normal list mechanism. >>>> >>>> So IMO "private" isn't appropriate. Neither is "L:" >>>> Perhaps just mark it as what it is as an "exploder". >>>> >>>> Or maybe these blocks should be similar to: >>>> >>>> M: Name of Lead Developer <someb...@vmware.com> >>>> M: VMware <foo> maintainers <linux-<foo>-maintain...@vmlinux.com> >> >> Maybe adding entries like >> >> M: Named maintainer <whoe...@vmware.com> >> R: VMware <foo> reviewers <linux-<foo>-maintain...@vmware.com> >> >> would be best/simplest. >> > > Sure, that sounds good to me. I also considered adding "(email alias)" > like Juergen suggested, but I think the R: entry is clear enough. > Please find the updated patch below. > > --- > > From f66faa238facf504cfc66325912ce7af8cbf79ec Mon Sep 17 00:00:00 2001 > From: "Srivatsa S. Bhat (VMware)" <sriva...@csail.mit.edu> > Date: Mon, 8 Nov 2021 11:46:57 -0800 > Subject: [PATCH v2 2/2] MAINTAINERS: Mark VMware mailing list entries as email > aliases > > VMware mailing lists in the MAINTAINERS file are private lists meant > for VMware-internal review/notification for patches to the respective > subsystems. Anyone can post to these addresses, but there is no public > read access like open mailing lists, which makes them more like email > aliases instead (to reach out to reviewers). > > So update all the VMware mailing list references in the MAINTAINERS > file to mark them as such, using "R: email-al...@vmware.com". > > Signed-off-by: Srivatsa S. Bhat (VMware) <sriva...@csail.mit.edu> > Cc: Zack Rusin <za...@vmware.com> > Cc: Nadav Amit <na...@vmware.com> > Cc: Vivek Thampi <vitha...@vmware.com> > Cc: Vishal Bhakta <vbha...@vmware.com> > Cc: Ronak Doshi <dos...@vmware.com> > Cc: pv-driv...@vmware.com > Cc: linux-graphics-maintai...@vmware.com > Cc: dri-de...@lists.freedesktop.org > Cc: linux-r...@vger.kernel.org > Cc: linux-s...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > --- > MAINTAINERS | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 118cf8170d02..4372d79027e9 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -6134,8 +6134,8 @@ T: git git://anongit.freedesktop.org/drm/drm-misc > F: drivers/gpu/drm/vboxvideo/ > > DRM DRIVER FOR VMWARE VIRTUAL GPU > -M: "VMware Graphics" <linux-graphics-maintai...@vmware.com> > M: Zack Rusin <za...@vmware.com> > +R: VMware Graphics Reviewers <linux-graphics-maintai...@vmware.com> > L: dri-de...@lists.freedesktop.org > S: Supported > T: git git://anongit.freedesktop.org/drm/drm-misc > @@ -14189,7 +14189,7 @@ F: include/uapi/linux/ppdev.h > PARAVIRT_OPS INTERFACE > M: Juergen Gross <jgr...@suse.com> > M: Srivatsa S. Bhat (VMware) <sriva...@csail.mit.edu> > -L: pv-driv...@vmware.com (private) > +R: VMware PV-Drivers Reviewers <pv-driv...@vmware.com>
This patch that you just sent seems to go on top of the previous patches (as it removes "L: pv-driv...@vmware.com (private)”). Since the patches were still not merged, I would presume you should squash the old 2/2 with this new patch and send v3 of these patches. _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
