Thread (13 messages) 13 messages, 4 authors, 2021-11-10

Re: [PATCH 2/2] MAINTAINERS: Mark VMware mailing list entries as private

From: Nadav Amit <hidden>
Date: 2021-11-10 17:40:18
Also in: dri-devel, linux-input, linux-rdma, linux-scsi, lkml, netdev

quoted hunk ↗ jump to hunk
On Nov 10, 2021, at 9:20 AM, Srivatsa S. Bhat [off-list ref] wrote:

On Tue, Nov 09, 2021 at 01:57:31PM -0800, Joe Perches wrote:
quoted
On Tue, 2021-11-09 at 00:58 +0000, Nadav Amit wrote:
quoted
quoted
On Nov 8, 2021, at 4:37 PM, Joe Perches [off-list ref] 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 [off-list ref]
M:	VMware <foo> maintainers <linux-<foo>-maintainers@vmlinux.com>
Maybe adding entries like

M:	Named maintainer [off-list ref]
R:	VMware <foo> reviewers <linux-<foo>-maintainers@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)" <redacted>
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-alias@vmware.com".

Signed-off-by: Srivatsa S. Bhat (VMware) <redacted>
Cc: Zack Rusin <redacted>
Cc: Nadav Amit <redacted>
Cc: Vivek Thampi <redacted>
Cc: Vishal Bhakta <redacted>
Cc: Ronak Doshi <redacted>
Cc: pv-drivers@vmware.com
Cc: linux-graphics-maintainer@vmware.com
Cc: dri-devel@lists.freedesktop.org
Cc: linux-rdma@vger.kernel.org
Cc: linux-scsi@vger.kernel.org
Cc: netdev@vger.kernel.org
Cc: linux-input@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" [off-list ref]
M:	Zack Rusin [off-list ref]
+R:	VMware Graphics Reviewers [off-list ref]
L:	dri-devel@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 [off-list ref]
M:	Srivatsa S. Bhat (VMware) [off-list ref]
-L:	pv-drivers@vmware.com (private)
+R:	VMware PV-Drivers Reviewers [off-list ref]
This patch that you just sent seems to go on top of the previous patches
(as it removes "L: pv-drivers@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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help