2013-09-04 09:00:25

by Alok Kataria

[permalink] [raw]
Subject: [PATCH] Add an entry in MAINTAINERS for VMware's hypervisor interface

Hey,

This change adds an entry to the maintainers file to explicitly state
that any changes to vmware.c should be sent to the authors of the file
as well.

If you are thinking why do this now, off late we have seen quite a few
changes that touched the "arch/x86/kernel/cpu/vmware.c" file but were
not directed to me, someone else made me aware of those changes. It
could be that my use of different SOB lines "Alok N Kataria" vs "Alok
Kataria" confuses get_maintainer.pl, but adding this entry in
MAINTAINERS should solve the problem at hand. For one of the changes
[1], I did verify that get_maintainer.pl, now expectedly includes me in
the cc list.

Thanks,
Alok

[1] http://marc.info/?l=linux-kernel&m=137571112610984&w=2

>From : Alok Kataria <[email protected]>

Add an entry in MAINTAINERS file to reflect the authors of a file.

Signed-off-by: Alok Kataria <[email protected]>

Index: linux-2.6/MAINTAINERS
===================================================================
--- linux-2.6.orig/MAINTAINERS 2013-09-04 14:07:34.404027068 +0530
+++ linux-2.6/MAINTAINERS 2013-09-04 14:20:39.714026943 +0530
@@ -9038,6 +9038,12 @@
F: drivers/vme/
F: include/linux/vme*

+VMWARE HYPERVISOR INTERFACE
+M: Alok Kataria <[email protected]>
+L: [email protected]
+S: Supported
+F: arch/x86/kernel/cpu/vmware.c
+
VMWARE VMXNET3 ETHERNET DRIVER
M: Shreyas Bhatewara <[email protected]>
M: "VMware, Inc." <[email protected]>



2013-09-04 11:46:27

by H. Peter Anvin

[permalink] [raw]
Subject: Re: [PATCH] Add an entry in MAINTAINERS for VMware's hypervisor interface

On 09/04/2013 01:53 AM, Alok Kataria wrote:
> Hey,
>
> This change adds an entry to the maintainers file to explicitly state
> that any changes to vmware.c should be sent to the authors of the file
> as well.
>
> If you are thinking why do this now, off late we have seen quite a few
> changes that touched the "arch/x86/kernel/cpu/vmware.c" file but were
> not directed to me, someone else made me aware of those changes. It
> could be that my use of different SOB lines "Alok N Kataria" vs "Alok
> Kataria" confuses get_maintainer.pl, but adding this entry in
> MAINTAINERS should solve the problem at hand. For one of the changes
> [1], I did verify that get_maintainer.pl, now expectedly includes me in
> the cc list.
>
> Thanks,
> Alok
>

That is indeed what the maintainers file is for.

> Add an entry in MAINTAINERS file to reflect the authors of a file.

No, it reflects the *maintainer(s)* of a file.

-hpa

Subject: [tip:x86/urgent] x86, doc: Add an entry in MAINTAINERS for arch/ x86/kernel/cpu/vmware.c

Commit-ID: db14e78a61691888ded5896da5f171d662a4d2bd
Gitweb: http://git.kernel.org/tip/db14e78a61691888ded5896da5f171d662a4d2bd
Author: Alok Kataria <[email protected]>
AuthorDate: Wed, 4 Sep 2013 14:23:41 +0530
Committer: H. Peter Anvin <[email protected]>
CommitDate: Wed, 4 Sep 2013 04:49:07 -0700

x86, doc: Add an entry in MAINTAINERS for arch/x86/kernel/cpu/vmware.c

Add an entry in MAINTAINERS file to reflect the maintainers of this
file.

Signed-off-by: Alok Kataria <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: H. Peter Anvin <[email protected]>
---
MAINTAINERS | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index b140c81..3d48faa 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9026,6 +9026,12 @@ F: drivers/staging/vme/
F: drivers/vme/
F: include/linux/vme*

+VMWARE HYPERVISOR INTERFACE
+M: Alok Kataria <[email protected]>
+L: [email protected]
+S: Supported
+F: arch/x86/kernel/cpu/vmware.c
+
VMWARE VMXNET3 ETHERNET DRIVER
M: Shreyas Bhatewara <[email protected]>
M: "VMware, Inc." <[email protected]>

2013-09-04 17:45:11

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] Add an entry in MAINTAINERS for VMware's hypervisor interface

On Wed, 2013-09-04 at 14:23 +0530, Alok Kataria wrote:
> Hey,
>
> This change adds an entry to the maintainers file to explicitly state
> that any changes to vmware.c should be sent to the authors of the file
> as well.

It never hurts to add MAINTAINER entries.

> If you are thinking why do this now, off late we have seen quite a few
> changes that touched the "arch/x86/kernel/cpu/vmware.c" file but were
> not directed to me, someone else made me aware of those changes.

Many people don't use get_maintainer

> It
> could be that my use of different SOB lines "Alok N Kataria" vs "Alok
> Kataria" confuses get_maintainer.pl,

That shouldn't be a problem unless the email addresses are
also different.

> +++ linux-2.6/MAINTAINERS 2013-09-04 14:20:39.714026943 +0530
[]
> +VMWARE HYPERVISOR INTERFACE
> +M: Alok Kataria <[email protected]>
> +L: [email protected]
> +S: Supported
> +F: arch/x86/kernel/cpu/vmware.c
> +

It'd be nicer to use tabs after the colon

2013-09-05 04:46:59

by Alok Kataria

[permalink] [raw]
Subject: Re: [PATCH] Add an entry in MAINTAINERS for VMware's hypervisor interface

On Wed, 2013-09-04 at 10:16 -0700, Joe Perches wrote:
> On Wed, 2013-09-04 at 14:23 +0530, Alok Kataria wrote:
> > Hey,
> >
> > This change adds an entry to the maintainers file to explicitly state
> > that any changes to vmware.c should be sent to the authors of the file
> > as well.
>
> It never hurts to add MAINTAINER entries.
>
> > If you are thinking why do this now, off late we have seen quite a few
> > changes that touched the "arch/x86/kernel/cpu/vmware.c" file but were
> > not directed to me, someone else made me aware of those changes.
>
> Many people don't use get_maintainer
>
> > It
> > could be that my use of different SOB lines "Alok N Kataria" vs "Alok
> > Kataria" confuses get_maintainer.pl,
>
> That shouldn't be a problem unless the email addresses are
> also different.
>
> > +++ linux-2.6/MAINTAINERS 2013-09-04 14:20:39.714026943 +0530
> []
> > +VMWARE HYPERVISOR INTERFACE
> > +M: Alok Kataria <[email protected]>
> > +L: [email protected]
> > +S: Supported
> > +F: arch/x86/kernel/cpu/vmware.c
> > +
>
> It'd be nicer to use tabs after the colon

Oops, my current workspace had a different tab setting, thanks for
noticing. Regenerated the diffs on latest tip, tree. Peter can you
please apply this one as well.
---

Fix whitespace issues.

Signed-off-by: Alok Kataria <[email protected]>

Index: linux.trees.git/MAINTAINERS
===================================================================
--- linux.trees.git.orig/MAINTAINERS 2013-09-05 10:11:31.784339571 +0530
+++ linux.trees.git/MAINTAINERS 2013-09-05 10:11:47.364339672 +0530
@@ -9041,10 +9041,10 @@
F: include/linux/vme*

VMWARE HYPERVISOR INTERFACE
-M: Alok Kataria <[email protected]>
-L: [email protected]
-S: Supported
-F: arch/x86/kernel/cpu/vmware.c
+M: Alok Kataria <[email protected]>
+L: [email protected]
+S: Supported
+F: arch/x86/kernel/cpu/vmware.c

VMWARE VMXNET3 ETHERNET DRIVER
M: Shreyas Bhatewara <[email protected]>

Subject: [tip:x86/urgent] x86, doc: Add an entry in MAINTAINERS for arch/ x86/kernel/cpu/vmware.c

Commit-ID: 4488e09b4582c3d9cae1601351e26584e208d877
Gitweb: http://git.kernel.org/tip/4488e09b4582c3d9cae1601351e26584e208d877
Author: Alok Kataria <[email protected]>
AuthorDate: Wed, 4 Sep 2013 14:23:41 +0530
Committer: H. Peter Anvin <[email protected]>
CommitDate: Wed, 4 Sep 2013 22:00:04 -0700

x86, doc: Add an entry in MAINTAINERS for arch/x86/kernel/cpu/vmware.c

Add an entry in MAINTAINERS file to reflect the maintainers of this
file.

Signed-off-by: Alok Kataria <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: H. Peter Anvin <[email protected]>
[ hpa: fixed tabs vs spaces ]
---
MAINTAINERS | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index defc053..fba8134 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9016,6 +9016,12 @@ F: drivers/staging/vme/
F: drivers/vme/
F: include/linux/vme*

+VMWARE HYPERVISOR INTERFACE
+M: Alok Kataria <[email protected]>
+L: [email protected]
+S: Supported
+F: arch/x86/kernel/cpu/vmware.c
+
VMWARE VMXNET3 ETHERNET DRIVER
M: Shreyas Bhatewara <[email protected]>
M: "VMware, Inc." <[email protected]>