From: Srivatsa S. Bhat <hidden> Date: 2021-11-08 20:45:10
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)".
[1]. https://lore.kernel.org/r/YPfp0Ff6KuyPlyrc@kroah.com
[2]. https://lore.kernel.org/r/1626861766-11115-1-git-send-email-jhansen@vmware.com
Signed-off-by: Srivatsa S. Bhat (VMware) <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
Acked-by: Zack Rusin <redacted>
---
MAINTAINERS | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
From: Nadav Amit <hidden> Date: 2021-11-08 20:45:14
On Nov 8, 2021, at 12:30 PM, Srivatsa S. Bhat [off-list ref] wrote:
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)”.
From: Joe Perches <joe@perches.com> Date: 2021-11-08 23:38:07
On Mon, 2021-11-08 at 12:30 -0800, Srivatsa S. Bhat wrote:
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)".
This MAINTAINERS file is for _public_ use, marking something
non-public isn't useful.
private makes no sense and likely these L: entries shouldn't exist.
This MAINTAINERS file is for _public_ use, marking something
non-public isn't useful.
But Greg has a point. Corporations like to send us code with a list
as the maintainer and MODULE_AUTHOR set to corp's name. We deal with
humans, not legal entities.
I've been trying to get them to use "M: email" without the name,
but "L: list (private)" also works.
Either way I feel like we need _some_ way to tell humans from corporate
"please CC this address" entries.
private makes no sense and likely these L: entries shouldn't exist.
From: Srivatsa S. Bhat <hidden> Date: 2021-11-09 00:22:24
+Greg, Thomas
Hi Joe,
On 11/8/21 3:37 PM, Joe Perches wrote:
On Mon, 2021-11-08 at 12:30 -0800, Srivatsa S. Bhat wrote:
quoted
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)".
This MAINTAINERS file is for _public_ use, marking something
non-public isn't useful.
private makes no sense and likely these L: entries shouldn't exist.
Well, the public can send messages to this list, but membership is
restricted.
In many ways, I believe this is similar to x86@kernel.org, which is an
email alias that anyone can post to in order to reach the x86
maintainer community for patch review. I see x86@kernel.org listed as
both L: and M: in the MAINTAINERS file, among different entries.
Although the @vmware list ids refer to VMware-internal mailing lists
as opposed to email aliases, they serve a very similar purpose -- to
inform VMware folks about patches to the relevant subsystems.
Is there a consensus on how such lists should be specified? One
suggestion (from Greg in the email thread referenced above) was to
mark it as private, which is what this patch does. Maybe we can find a
better alternative?
How about specifying such lists using M: (indicating that this address
can be used to reach maintainers), as long as that is not the only M:
entry for a given subsystem (i.e., it includes real people's email id
as well)? I think that would address Greg's primary objection too from
that other thread (related to personal responsibility as maintainers).
Regards,
Srivatsa
This MAINTAINERS file is for _public_ use, marking something
non-public isn't useful.
But Greg has a point. Corporations like to send us code with a list
as the maintainer and MODULE_AUTHOR set to corp's name. We deal with
humans, not legal entities.
MAINTAINERS is used not for corporations private use but
to find out _who_ to send and cc patches and defect reports.
A "private" email address used only for corporate internal review
cannot receive patches.
I've been trying to get them to use "M: email" without the name,
but "L: list (private)" also works.
Either way I feel like we need _some_ way to tell humans from corporate
"please CC this address" entries.
This is not the way AFAIKT.
quoted
private makes no sense and likely these L: entries shouldn't exist.
From: Joe Perches <joe@perches.com> Date: 2021-11-09 00:37:58
On Mon, 2021-11-08 at 16:22 -0800, Srivatsa S. Bhat wrote:
+Greg, Thomas
Hi Joe,
On 11/8/21 3:37 PM, Joe Perches wrote:
quoted
On Mon, 2021-11-08 at 12:30 -0800, Srivatsa S. Bhat wrote:
quoted
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)".
This MAINTAINERS file is for _public_ use, marking something
non-public isn't useful.
private makes no sense and likely these L: entries shouldn't exist.
Well, the public can send messages to this list, but membership is
restricted.
Ah, new information.
That's not quite what the commit message describes.
In many ways, I believe this is similar to x86@kernel.org, which is an
email alias that anyone can post to in order to reach the x86
maintainer community for patch review. I see x86@kernel.org listed as
both L: and M: in the MAINTAINERS file, among different entries.
Although the @vmware list ids refer to VMware-internal mailing lists
as opposed to email aliases, they serve a very similar purpose -- to
inform VMware folks about patches to the relevant subsystems.
Is there a consensus on how such lists should be specified?
Not so far as I know.
One
suggestion (from Greg in the email thread referenced above) was to
mark it as private, which is what this patch does. Maybe we can find a
better alternative?
How about specifying such lists using M: (indicating that this address
can be used to reach maintainers), as long as that is not the only M:
entry for a given subsystem (i.e., it includes real people's email id
as well)? I think that would address Greg's primary objection too from
that other thread (related to personal responsibility as maintainers).
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 something like a comment mechanism should be added to the
MAINTAINERS file.
Maybe #
so this entry could be something like:
M: VMware <foo> maintainers <linux-<foo>-maintainers@vmlinux.com> # VMware's ever changing internal maintainers list
From: Nadav Amit <hidden> Date: 2021-11-09 00:58:57
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 something like a comment mechanism should be added to the
MAINTAINERS file.
Maybe #
so this entry could be something like:
M: VMware <foo> maintainers <linux-<foo>-maintainers@vmlinux.com> # VMware's ever changing internal maintainers list
Admittedly, I do not care much about how it turns to be.
But if it is modified, it should be very clear who the maintainer
is, and not to entangle the mailing list and the maintainer.
I am personally not subscribed to the internal pv-drivers mailing
list, which is not just for memory ballooning, and is also listed
as a maintainer for vmmouse, pvscsi, vmxnet3 and others.
As I am the only maintainer of VMware balloon, if someone is
mistaken and sends an email only to the mailing list and not me,
he might be disappointed.
From: Srivatsa S. Bhat (VMware) <redacted>
VMware mailing lists in the MAINTAINERS file are private lists meant
for VMware-internal review/notification for patches to the respective
subsystems. So, in an earlier discussion [1][2], it was recommended to
mark them as such. Update all the remaining VMware mailing list
references to use that format -- "L: list@address (private)".
[1]. https://lore.kernel.org/r/YPfp0Ff6KuyPlyrc@kroah.com
[2]. https://lore.kernel.org/r/1626861766-11115-1-git-send-email-jhansen@vmware.com
Signed-off-by: Srivatsa S. Bhat (VMware) <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
Acked-by: Zack Rusin <redacted>
---
MAINTAINERS | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
Maybe replace "(private)" with "(mail alias)"?
This makes it rather clear that it is a valid address to send patches
to, but there isn't public read access like to a ML.
Juergen
From: Joe Perches <joe@perches.com> Date: 2021-11-09 21:57:42
On Tue, 2021-11-09 at 00:58 +0000, Nadav Amit wrote:
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.
From: Srivatsa S. Bhat <hidden> Date: 2021-11-10 17:16:59
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:
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(-)
From: Nadav Amit <hidden> Date: 2021-11-10 17:40:18
quoted 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(-)
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.
From: Srivatsa S. Bhat <hidden> Date: 2021-11-10 19:10:01
On Wed, Nov 10, 2021 at 05:40:09PM +0000, Nadav Amit wrote:
quoted
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(-)
This patch that you just sent seems to go on top of the previous patches
(as it removes "L: pv-drivers@vmware.com (private)”).
Actually, that's a bit misleading, since I had corrected that entry in
the first patch itself, while adding myself as the maintainer. So
there are still only 2 patches in this series right now.
Thanks for pointing this out! I'll move the VMware list modifications
out of the first patch, to avoid confusion.
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.
I'll send out a v3, and also add Zack Rusin as the maintainer for the
vmmouse sub-driver, since it does not have a named maintainer at the
moment (Zack indicated that he will be taking up the maintainership).
Thank you!
Regards,
Srivatsa