subject:"Re\: \[PATCH\] VMW_PVSCSI\: Change to update maintainer details \(name, email\)"

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-20 Thread Martin K. Petersen

> "Jim" == Jim Gill  writes:

Updated MAINTAINERS accordingly and applied to 4.8/scsi-queue.

-- 
Martin K. Petersen  Oracle Linux Engineering

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-20 Thread Martin K. Petersen

> "Jim" == Jim Gill  writes:

Updated MAINTAINERS accordingly and applied to 4.8/scsi-queue.

-- 
Martin K. Petersen  Oracle Linux Engineering

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Jim Gill

I don’t yet know enough to comment on the relocation of vmw_pvscsi into a new 
vmware folder.
I was considering offering a new patch that removed the “maintained by” from 
the two source
files and updated MAINTAINERS, but I’m happy to defer to people with more 
experience. I’m
fine either way. 

On 6/17/16, 11:06 AM, "Joe Perches"  wrote:



>
>Perhaps a patch like this:
>
>From e727c6549e3be466ec3c79e919502cb0b9909b03 Mon Sep 17 00:00:00 2001
>Message-Id: 
>
>From: Joe Perches 
>Date: Fri, 17 Jun 2016 10:56:49 -0700
>Subject: [PATCH] vmw_pvscsi: Move into separate directory, Jim Gill is 
>MAINTAINER
>
>Separate directories for drivers are generally better.
>
>Miscellanea:
>
>o Update the MAINTAINER entry
>o Remove maintainer and FSF addresses from driver files
>---
> MAINTAINERS| 5 ++---
> drivers/scsi/Kconfig   | 8 +---
> drivers/scsi/Makefile  | 2 +-
> drivers/scsi/vmware/Kconfig| 7 +++
> drivers/scsi/vmware/Makefile   | 1 +
> drivers/scsi/{ => vmware}/vmw_pvscsi.c | 7 ---
> drivers/scsi/{ => vmware}/vmw_pvscsi.h | 7 ---
> 7 files changed, 12 insertions(+), 25 deletions(-)
> create mode 100644 drivers/scsi/vmware/Kconfig
> create mode 100644 drivers/scsi/vmware/Makefile
> rename drivers/scsi/{ => vmware}/vmw_pvscsi.c (99%)
> rename drivers/scsi/{ => vmware}/vmw_pvscsi.h (98%)
>
>diff --git a/MAINTAINERS b/MAINTAINERS
>index d174e34..2763582 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -12421,12 +12421,11 @@ S:   Maintained
> F:drivers/net/vmxnet3/
> 
> VMware PVSCSI driver
>-M:Arvind Kumar 
>+M:Jim Gill 
> M:VMware PV-Drivers 
> L:linux-s...@vger.kernel.org
> S:Maintained
>-F:drivers/scsi/vmw_pvscsi.c
>-F:drivers/scsi/vmw_pvscsi.h
>+F:drivers/scsi/vmware/
> 
> VOLTAGE AND CURRENT REGULATOR FRAMEWORK
> M:Liam Girdwood 
>diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
>index 1918f54..339c230 100644
>--- a/drivers/scsi/Kconfig
>+++ b/drivers/scsi/Kconfig
>@@ -576,13 +576,7 @@ config SCSI_FLASHPOINT
> substantial, so users of MultiMaster Host Adapters may not
> wish to include it.
> 
>-config VMWARE_PVSCSI
>-  tristate "VMware PVSCSI driver support"
>-  depends on PCI && SCSI && X86
>-  help
>-This driver supports VMware's para virtualized SCSI HBA.
>-To compile this driver as a module, choose M here: the
>-module will be called vmw_pvscsi.
>+source "drivers/scsi/vmware/Kconfig"
> 
> config XEN_SCSI_FRONTEND
>   tristate "XEN SCSI frontend driver"
>diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile
>index 862ab4e..6cfefaa 100644
>--- a/drivers/scsi/Makefile
>+++ b/drivers/scsi/Makefile
>@@ -141,7 +141,7 @@ obj-$(CONFIG_BE2ISCSI) += libiscsi.o be2iscsi/
> obj-$(CONFIG_SCSI_ESAS2R) += esas2r/
> obj-$(CONFIG_SCSI_PMCRAID)+= pmcraid.o
> obj-$(CONFIG_SCSI_VIRTIO) += virtio_scsi.o
>-obj-$(CONFIG_VMWARE_PVSCSI)   += vmw_pvscsi.o
>+obj-$(CONFIG_VMWARE_PVSCSI)   += vmware/
> obj-$(CONFIG_XEN_SCSI_FRONTEND)   += xen-scsifront.o
> obj-$(CONFIG_HYPERV_STORAGE)  += hv_storvsc.o
> obj-$(CONFIG_SCSI_WD719X) += wd719x.o
>diff --git a/drivers/scsi/vmware/Kconfig b/drivers/scsi/vmware/Kconfig
>new file mode 100644
>index 000..3c0c53b
>--- /dev/null
>+++ b/drivers/scsi/vmware/Kconfig
>@@ -0,0 +1,7 @@
>+config VMWARE_PVSCSI
>+  tristate "VMware PVSCSI driver support"
>+  depends on PCI && SCSI && X86
>+  help
>+This driver supports VMware's para virtualized SCSI HBA.
>+To compile this driver as a module, choose M here: the
>+module will be called vmw_pvscsi.
>diff --git a/drivers/scsi/vmware/Makefile b/drivers/scsi/vmware/Makefile
>new file mode 100644
>index 000..ae8d278
>--- /dev/null
>+++ b/drivers/scsi/vmware/Makefile
>@@ -0,0 +1 @@
>+obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
>diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmware/vmw_pvscsi.c
>similarity index 99%
>rename from drivers/scsi/vmw_pvscsi.c
>rename to drivers/scsi/vmware/vmw_pvscsi.c
>index 6164634..eb1229e 100644
>--- a/drivers/scsi/vmw_pvscsi.c
>+++ b/drivers/scsi/vmware/vmw_pvscsi.c
>@@ -12,13 +12,6 @@
>  * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
>  * NON INFRINGEMENT.  See the GNU General Public License for more
>  * details.
>- *
>- * You should have received a copy of the GNU General Public License
>- * along with this program; if not, write to the Free Software
>- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
>- *
>- * Maintained by: Arvind Kumar 
>- *
>  */
> 
> #include 
>diff --git a/drivers/scsi/vmw_pvscsi.h b/drivers/scsi/vmware/vmw_pvscsi.h
>similarity index 98%
>rename from

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Jim Gill

I don’t yet know enough to comment on the relocation of vmw_pvscsi into a new 
vmware folder.
I was considering offering a new patch that removed the “maintained by” from 
the two source
files and updated MAINTAINERS, but I’m happy to defer to people with more 
experience. I’m
fine either way. 

On 6/17/16, 11:06 AM, "Joe Perches"  wrote:



>
>Perhaps a patch like this:
>
>From e727c6549e3be466ec3c79e919502cb0b9909b03 Mon Sep 17 00:00:00 2001
>Message-Id: 
>
>From: Joe Perches 
>Date: Fri, 17 Jun 2016 10:56:49 -0700
>Subject: [PATCH] vmw_pvscsi: Move into separate directory, Jim Gill is 
>MAINTAINER
>
>Separate directories for drivers are generally better.
>
>Miscellanea:
>
>o Update the MAINTAINER entry
>o Remove maintainer and FSF addresses from driver files
>---
> MAINTAINERS| 5 ++---
> drivers/scsi/Kconfig   | 8 +---
> drivers/scsi/Makefile  | 2 +-
> drivers/scsi/vmware/Kconfig| 7 +++
> drivers/scsi/vmware/Makefile   | 1 +
> drivers/scsi/{ => vmware}/vmw_pvscsi.c | 7 ---
> drivers/scsi/{ => vmware}/vmw_pvscsi.h | 7 ---
> 7 files changed, 12 insertions(+), 25 deletions(-)
> create mode 100644 drivers/scsi/vmware/Kconfig
> create mode 100644 drivers/scsi/vmware/Makefile
> rename drivers/scsi/{ => vmware}/vmw_pvscsi.c (99%)
> rename drivers/scsi/{ => vmware}/vmw_pvscsi.h (98%)
>
>diff --git a/MAINTAINERS b/MAINTAINERS
>index d174e34..2763582 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -12421,12 +12421,11 @@ S:   Maintained
> F:drivers/net/vmxnet3/
> 
> VMware PVSCSI driver
>-M:Arvind Kumar 
>+M:Jim Gill 
> M:VMware PV-Drivers 
> L:linux-s...@vger.kernel.org
> S:Maintained
>-F:drivers/scsi/vmw_pvscsi.c
>-F:drivers/scsi/vmw_pvscsi.h
>+F:drivers/scsi/vmware/
> 
> VOLTAGE AND CURRENT REGULATOR FRAMEWORK
> M:Liam Girdwood 
>diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
>index 1918f54..339c230 100644
>--- a/drivers/scsi/Kconfig
>+++ b/drivers/scsi/Kconfig
>@@ -576,13 +576,7 @@ config SCSI_FLASHPOINT
> substantial, so users of MultiMaster Host Adapters may not
> wish to include it.
> 
>-config VMWARE_PVSCSI
>-  tristate "VMware PVSCSI driver support"
>-  depends on PCI && SCSI && X86
>-  help
>-This driver supports VMware's para virtualized SCSI HBA.
>-To compile this driver as a module, choose M here: the
>-module will be called vmw_pvscsi.
>+source "drivers/scsi/vmware/Kconfig"
> 
> config XEN_SCSI_FRONTEND
>   tristate "XEN SCSI frontend driver"
>diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile
>index 862ab4e..6cfefaa 100644
>--- a/drivers/scsi/Makefile
>+++ b/drivers/scsi/Makefile
>@@ -141,7 +141,7 @@ obj-$(CONFIG_BE2ISCSI) += libiscsi.o be2iscsi/
> obj-$(CONFIG_SCSI_ESAS2R) += esas2r/
> obj-$(CONFIG_SCSI_PMCRAID)+= pmcraid.o
> obj-$(CONFIG_SCSI_VIRTIO) += virtio_scsi.o
>-obj-$(CONFIG_VMWARE_PVSCSI)   += vmw_pvscsi.o
>+obj-$(CONFIG_VMWARE_PVSCSI)   += vmware/
> obj-$(CONFIG_XEN_SCSI_FRONTEND)   += xen-scsifront.o
> obj-$(CONFIG_HYPERV_STORAGE)  += hv_storvsc.o
> obj-$(CONFIG_SCSI_WD719X) += wd719x.o
>diff --git a/drivers/scsi/vmware/Kconfig b/drivers/scsi/vmware/Kconfig
>new file mode 100644
>index 000..3c0c53b
>--- /dev/null
>+++ b/drivers/scsi/vmware/Kconfig
>@@ -0,0 +1,7 @@
>+config VMWARE_PVSCSI
>+  tristate "VMware PVSCSI driver support"
>+  depends on PCI && SCSI && X86
>+  help
>+This driver supports VMware's para virtualized SCSI HBA.
>+To compile this driver as a module, choose M here: the
>+module will be called vmw_pvscsi.
>diff --git a/drivers/scsi/vmware/Makefile b/drivers/scsi/vmware/Makefile
>new file mode 100644
>index 000..ae8d278
>--- /dev/null
>+++ b/drivers/scsi/vmware/Makefile
>@@ -0,0 +1 @@
>+obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
>diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmware/vmw_pvscsi.c
>similarity index 99%
>rename from drivers/scsi/vmw_pvscsi.c
>rename to drivers/scsi/vmware/vmw_pvscsi.c
>index 6164634..eb1229e 100644
>--- a/drivers/scsi/vmw_pvscsi.c
>+++ b/drivers/scsi/vmware/vmw_pvscsi.c
>@@ -12,13 +12,6 @@
>  * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
>  * NON INFRINGEMENT.  See the GNU General Public License for more
>  * details.
>- *
>- * You should have received a copy of the GNU General Public License
>- * along with this program; if not, write to the Free Software
>- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
>- *
>- * Maintained by: Arvind Kumar 
>- *
>  */
> 
> #include 
>diff --git a/drivers/scsi/vmw_pvscsi.h b/drivers/scsi/vmware/vmw_pvscsi.h
>similarity index 98%
>rename from drivers/scsi/vmw_pvscsi.h
>rename to drivers/scsi/vmware/vmw_pvscsi.h
>index 12712c9..5f675b9 100644
>--- a/drivers/scsi/vmw_pvscsi.h
>+++ b/drivers/scsi/vmware/vmw_pvscsi.h
>@@ -12,13 +12,6 @@
>  * MERCHANTABILITY OR FITNESS

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Joe Perches

On Fri, 2016-06-17 at 17:44 +, Jim Gill wrote:
> On 6/16/16, 8:11 PM, "Julian Calaby"  wrote:
> 
> 
> 
> > 
> > []
> > 
> > > 
> > > > 
> > > > On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> > > []
> > > The question to me is whether or not Jim Gill is
> > > taking over the maintainership of the entire
> > > VMware PVSCSI driver or just a few files of it.
> > As I see it, he's taking over maintainership of all of it: it's only
> > files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.
> This is correct, I am taking over maintainership of the entire vmw_pvscsi 
> driver.

Perhaps a patch like this:

>From e727c6549e3be466ec3c79e919502cb0b9909b03 Mon Sep 17 00:00:00 2001
Message-Id: 

From: Joe Perches 
Date: Fri, 17 Jun 2016 10:56:49 -0700
Subject: [PATCH] vmw_pvscsi: Move into separate directory, Jim Gill is 
MAINTAINER

Separate directories for drivers are generally better.

Miscellanea:

o Update the MAINTAINER entry
o Remove maintainer and FSF addresses from driver files
---
 MAINTAINERS| 5 ++---
 drivers/scsi/Kconfig   | 8 +---
 drivers/scsi/Makefile  | 2 +-
 drivers/scsi/vmware/Kconfig| 7 +++
 drivers/scsi/vmware/Makefile   | 1 +
 drivers/scsi/{ => vmware}/vmw_pvscsi.c | 7 ---
 drivers/scsi/{ => vmware}/vmw_pvscsi.h | 7 ---
 7 files changed, 12 insertions(+), 25 deletions(-)
 create mode 100644 drivers/scsi/vmware/Kconfig
 create mode 100644 drivers/scsi/vmware/Makefile
 rename drivers/scsi/{ => vmware}/vmw_pvscsi.c (99%)
 rename drivers/scsi/{ => vmware}/vmw_pvscsi.h (98%)

diff --git a/MAINTAINERS b/MAINTAINERS
index d174e34..2763582 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12421,12 +12421,11 @@ S:Maintained
 F: drivers/net/vmxnet3/
 
 VMware PVSCSI driver
-M: Arvind Kumar 
+M: Jim Gill 
 M: VMware PV-Drivers 
 L: linux-s...@vger.kernel.org
 S: Maintained
-F: drivers/scsi/vmw_pvscsi.c
-F: drivers/scsi/vmw_pvscsi.h
+F: drivers/scsi/vmware/
 
 VOLTAGE AND CURRENT REGULATOR FRAMEWORK
 M: Liam Girdwood 
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 1918f54..339c230 100644
--- a/drivers/scsi/Kconfig
+++ b/drivers/scsi/Kconfig
@@ -576,13 +576,7 @@ config SCSI_FLASHPOINT
      substantial, so users of MultiMaster Host Adapters may not
      wish to include it.
 
-config VMWARE_PVSCSI
-   tristate "VMware PVSCSI driver support"
-   depends on PCI && SCSI && X86
-   help
-     This driver supports VMware's para virtualized SCSI HBA.
-     To compile this driver as a module, choose M here: the
-     module will be called vmw_pvscsi.
+source "drivers/scsi/vmware/Kconfig"
 
 config XEN_SCSI_FRONTEND
    tristate "XEN SCSI frontend driver"
diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile
index 862ab4e..6cfefaa 100644
--- a/drivers/scsi/Makefile
+++ b/drivers/scsi/Makefile
@@ -141,7 +141,7 @@ obj-$(CONFIG_BE2ISCSI)  += libiscsi.o be2iscsi/
 obj-$(CONFIG_SCSI_ESAS2R)  += esas2r/
 obj-$(CONFIG_SCSI_PMCRAID) += pmcraid.o
 obj-$(CONFIG_SCSI_VIRTIO)  += virtio_scsi.o
-obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
+obj-$(CONFIG_VMWARE_PVSCSI)+= vmware/
 obj-$(CONFIG_XEN_SCSI_FRONTEND)+= xen-scsifront.o
 obj-$(CONFIG_HYPERV_STORAGE)   += hv_storvsc.o
 obj-$(CONFIG_SCSI_WD719X)  += wd719x.o
diff --git a/drivers/scsi/vmware/Kconfig b/drivers/scsi/vmware/Kconfig
new file mode 100644
index 000..3c0c53b
--- /dev/null
+++ b/drivers/scsi/vmware/Kconfig
@@ -0,0 +1,7 @@
+config VMWARE_PVSCSI
+   tristate "VMware PVSCSI driver support"
+   depends on PCI && SCSI && X86
+   help
+     This driver supports VMware's para virtualized SCSI HBA.
+     To compile this driver as a module, choose M here: the
+     module will be called vmw_pvscsi.
diff --git a/drivers/scsi/vmware/Makefile b/drivers/scsi/vmware/Makefile
new file mode 100644
index 000..ae8d278
--- /dev/null
+++ b/drivers/scsi/vmware/Makefile
@@ -0,0 +1 @@
+obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmware/vmw_pvscsi.c
similarity index 99%
rename from drivers/scsi/vmw_pvscsi.c
rename to drivers/scsi/vmware/vmw_pvscsi.c
index 6164634..eb1229e 100644
--- a/drivers/scsi/vmw_pvscsi.c
+++ b/drivers/scsi/vmware/vmw_pvscsi.c
@@ -12,13 +12,6 @@
  * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
  * NON INFRINGEMENT.  See the GNU General Public License for more
  * details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * Maintained by:

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Joe Perches

On Fri, 2016-06-17 at 17:44 +, Jim Gill wrote:
> On 6/16/16, 8:11 PM, "Julian Calaby"  wrote:
> 
> 
> 
> > 
> > []
> > 
> > > 
> > > > 
> > > > On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> > > []
> > > The question to me is whether or not Jim Gill is
> > > taking over the maintainership of the entire
> > > VMware PVSCSI driver or just a few files of it.
> > As I see it, he's taking over maintainership of all of it: it's only
> > files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.
> This is correct, I am taking over maintainership of the entire vmw_pvscsi 
> driver.

Perhaps a patch like this:

>From e727c6549e3be466ec3c79e919502cb0b9909b03 Mon Sep 17 00:00:00 2001
Message-Id: 

From: Joe Perches 
Date: Fri, 17 Jun 2016 10:56:49 -0700
Subject: [PATCH] vmw_pvscsi: Move into separate directory, Jim Gill is 
MAINTAINER

Separate directories for drivers are generally better.

Miscellanea:

o Update the MAINTAINER entry
o Remove maintainer and FSF addresses from driver files
---
 MAINTAINERS| 5 ++---
 drivers/scsi/Kconfig   | 8 +---
 drivers/scsi/Makefile  | 2 +-
 drivers/scsi/vmware/Kconfig| 7 +++
 drivers/scsi/vmware/Makefile   | 1 +
 drivers/scsi/{ => vmware}/vmw_pvscsi.c | 7 ---
 drivers/scsi/{ => vmware}/vmw_pvscsi.h | 7 ---
 7 files changed, 12 insertions(+), 25 deletions(-)
 create mode 100644 drivers/scsi/vmware/Kconfig
 create mode 100644 drivers/scsi/vmware/Makefile
 rename drivers/scsi/{ => vmware}/vmw_pvscsi.c (99%)
 rename drivers/scsi/{ => vmware}/vmw_pvscsi.h (98%)

diff --git a/MAINTAINERS b/MAINTAINERS
index d174e34..2763582 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -12421,12 +12421,11 @@ S:Maintained
 F: drivers/net/vmxnet3/
 
 VMware PVSCSI driver
-M: Arvind Kumar 
+M: Jim Gill 
 M: VMware PV-Drivers 
 L: linux-s...@vger.kernel.org
 S: Maintained
-F: drivers/scsi/vmw_pvscsi.c
-F: drivers/scsi/vmw_pvscsi.h
+F: drivers/scsi/vmware/
 
 VOLTAGE AND CURRENT REGULATOR FRAMEWORK
 M: Liam Girdwood 
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
index 1918f54..339c230 100644
--- a/drivers/scsi/Kconfig
+++ b/drivers/scsi/Kconfig
@@ -576,13 +576,7 @@ config SCSI_FLASHPOINT
      substantial, so users of MultiMaster Host Adapters may not
      wish to include it.
 
-config VMWARE_PVSCSI
-   tristate "VMware PVSCSI driver support"
-   depends on PCI && SCSI && X86
-   help
-     This driver supports VMware's para virtualized SCSI HBA.
-     To compile this driver as a module, choose M here: the
-     module will be called vmw_pvscsi.
+source "drivers/scsi/vmware/Kconfig"
 
 config XEN_SCSI_FRONTEND
    tristate "XEN SCSI frontend driver"
diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile
index 862ab4e..6cfefaa 100644
--- a/drivers/scsi/Makefile
+++ b/drivers/scsi/Makefile
@@ -141,7 +141,7 @@ obj-$(CONFIG_BE2ISCSI)  += libiscsi.o be2iscsi/
 obj-$(CONFIG_SCSI_ESAS2R)  += esas2r/
 obj-$(CONFIG_SCSI_PMCRAID) += pmcraid.o
 obj-$(CONFIG_SCSI_VIRTIO)  += virtio_scsi.o
-obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
+obj-$(CONFIG_VMWARE_PVSCSI)+= vmware/
 obj-$(CONFIG_XEN_SCSI_FRONTEND)+= xen-scsifront.o
 obj-$(CONFIG_HYPERV_STORAGE)   += hv_storvsc.o
 obj-$(CONFIG_SCSI_WD719X)  += wd719x.o
diff --git a/drivers/scsi/vmware/Kconfig b/drivers/scsi/vmware/Kconfig
new file mode 100644
index 000..3c0c53b
--- /dev/null
+++ b/drivers/scsi/vmware/Kconfig
@@ -0,0 +1,7 @@
+config VMWARE_PVSCSI
+   tristate "VMware PVSCSI driver support"
+   depends on PCI && SCSI && X86
+   help
+     This driver supports VMware's para virtualized SCSI HBA.
+     To compile this driver as a module, choose M here: the
+     module will be called vmw_pvscsi.
diff --git a/drivers/scsi/vmware/Makefile b/drivers/scsi/vmware/Makefile
new file mode 100644
index 000..ae8d278
--- /dev/null
+++ b/drivers/scsi/vmware/Makefile
@@ -0,0 +1 @@
+obj-$(CONFIG_VMWARE_PVSCSI)+= vmw_pvscsi.o
diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmware/vmw_pvscsi.c
similarity index 99%
rename from drivers/scsi/vmw_pvscsi.c
rename to drivers/scsi/vmware/vmw_pvscsi.c
index 6164634..eb1229e 100644
--- a/drivers/scsi/vmw_pvscsi.c
+++ b/drivers/scsi/vmware/vmw_pvscsi.c
@@ -12,13 +12,6 @@
  * MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE, GOOD TITLE or
  * NON INFRINGEMENT.  See the GNU General Public License for more
  * details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
- *
- * Maintained by: Arvind Kumar 
- *
  */
 
 #include 
diff --git a/drivers/scsi/vmw_pvscsi.h b/drivers/scsi/vmware/vmw_pvscsi.h
similarity index 98%
rename from drivers/scsi/vmw_pvscsi.h
rename to drivers/scsi/vmware/vmw_pvscsi.h
index

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Jim Gill

On 6/16/16, 8:11 PM, "Julian Calaby"  wrote:



>[]
>
>>>On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
>> []
>>The question to me is whether or not Jim Gill is
>> taking over the maintainership of the entire
>> VMware PVSCSI driver or just a few files of it.
>
>As I see it, he's taking over maintainership of all of it: it's only
>files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.

This is correct, I am taking over maintainership of the entire vmw_pvscsi 
driver.

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-17 Thread Jim Gill

On 6/16/16, 8:11 PM, "Julian Calaby"  wrote:



>[]
>
>>>On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
>> []
>>The question to me is whether or not Jim Gill is
>> taking over the maintainership of the entire
>> VMware PVSCSI driver or just a few files of it.
>
>As I see it, he's taking over maintainership of all of it: it's only
>files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.

This is correct, I am taking over maintainership of the entire vmw_pvscsi 
driver.

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Joe,

On Fri, Jun 17, 2016 at 1:04 PM, Joe Perches  wrote:
> On Fri, 2016-06-17 at 12:44 +1000, Julian Calaby wrote:
>> Hi Joe,
>
> rehi Julian.

(I always put a salutation on my emails and always finish them with
"Thanks," =) )

>> On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> []
>> > get_maintainer.pl also has a rarely used "--file-emails" option to
>> > scan for what appears to be email addresses in specific files.
>> >
>> > $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
>> > Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
>> > file)
>> > VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
>> > "James E.J. Bottomley"  (maintainer:SCSI 
>> > SUBSYSTEM)
>> > "Martin K. Petersen"  (maintainer:SCSI 
>> > SUBSYSTEM)
>> > linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
>> > linux-kernel@vger.kernel.org (open list)
>> >
>> > note the "in file" after Arvind's name
>> Didn't know this, however my point stands: the maintainer line in the
>> file is redundant if we find maintainers through MAINTAINERS or the
>> get_maintainer.pl script.
>
> Yes, I'm not suggesting anything else.  Jim's name
> should appear in the MAINTAINERS file somewhere.
>
> The question to me is whether or not Jim Gill is
> taking over the maintainership of the entire
> VMware PVSCSI driver or just a few files of it.

As I see it, he's taking over maintainership of all of it: it's only
files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Joe,

On Fri, Jun 17, 2016 at 1:04 PM, Joe Perches  wrote:
> On Fri, 2016-06-17 at 12:44 +1000, Julian Calaby wrote:
>> Hi Joe,
>
> rehi Julian.

(I always put a salutation on my emails and always finish them with
"Thanks," =) )

>> On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> []
>> > get_maintainer.pl also has a rarely used "--file-emails" option to
>> > scan for what appears to be email addresses in specific files.
>> >
>> > $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
>> > Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
>> > file)
>> > VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
>> > "James E.J. Bottomley"  (maintainer:SCSI 
>> > SUBSYSTEM)
>> > "Martin K. Petersen"  (maintainer:SCSI 
>> > SUBSYSTEM)
>> > linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
>> > linux-kernel@vger.kernel.org (open list)
>> >
>> > note the "in file" after Arvind's name
>> Didn't know this, however my point stands: the maintainer line in the
>> file is redundant if we find maintainers through MAINTAINERS or the
>> get_maintainer.pl script.
>
> Yes, I'm not suggesting anything else.  Jim's name
> should appear in the MAINTAINERS file somewhere.
>
> The question to me is whether or not Jim Gill is
> taking over the maintainership of the entire
> VMware PVSCSI driver or just a few files of it.

As I see it, he's taking over maintainership of all of it: it's only
files are drivers/scsi/vmw_pvscsi.[ch] AFAIK.

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Joe Perches

On Fri, 2016-06-17 at 12:44 +1000, Julian Calaby wrote:
> Hi Joe,

rehi Julian.

> On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
[]
> > get_maintainer.pl also has a rarely used "--file-emails" option to
> > scan for what appears to be email addresses in specific files.
> > 
> > $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
> > Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
> > file)
> > VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> > "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> > "Martin K. Petersen"  (maintainer:SCSI 
> > SUBSYSTEM)
> > linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> > linux-kernel@vger.kernel.org (open list)
> > 
> > note the "in file" after Arvind's name
> Didn't know this, however my point stands: the maintainer line in the
> file is redundant if we find maintainers through MAINTAINERS or the
> get_maintainer.pl script.

Yes, I'm not suggesting anything else.  Jim's name
should appear in the MAINTAINERS file somewhere.

The question to me is whether or not Jim Gill is
taking over the maintainership of the entire
VMware PVSCSI driver or just a few files of it.

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Joe Perches

On Fri, 2016-06-17 at 12:44 +1000, Julian Calaby wrote:
> Hi Joe,

rehi Julian.

> On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
[]
> > get_maintainer.pl also has a rarely used "--file-emails" option to
> > scan for what appears to be email addresses in specific files.
> > 
> > $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
> > Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
> > file)
> > VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> > "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> > "Martin K. Petersen"  (maintainer:SCSI 
> > SUBSYSTEM)
> > linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> > linux-kernel@vger.kernel.org (open list)
> > 
> > note the "in file" after Arvind's name
> Didn't know this, however my point stands: the maintainer line in the
> file is redundant if we find maintainers through MAINTAINERS or the
> get_maintainer.pl script.

Yes, I'm not suggesting anything else.  Jim's name
should appear in the MAINTAINERS file somewhere.

The question to me is whether or not Jim Gill is
taking over the maintainership of the entire
VMware PVSCSI driver or just a few files of it.

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Joe,

On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> On Fri, 2016-06-17 at 12:18 +1000, Julian Calaby wrote:
>> ./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c
>
> just fyi:  the script name is not plural

Thanks, must have typo'd it between running it and typing it into the email.

> $ ./scripts/get_maintainer.pl  -f drivers/scsi/vmw_pvscsi.c
> Arvind Kumar  (maintainer:VMware PVSCSI driver)
> VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> "Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
> linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> linux-kernel@vger.kernel.org (open list)
>
> and another fyi:
>
> get_maintainer.pl also has a rarely used "--file-emails" option to
> scan for what appears to be email addresses in specific files.
>
> $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
> Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
> file)
> VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> "Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
> linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> linux-kernel@vger.kernel.org (open list)
>
> note the "in file" after Arvind's name

Didn't know this, however my point stands: the maintainer line in the
file is redundant if we find maintainers through MAINTAINERS or the
get_maintainer.pl script.

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Joe,

On Fri, Jun 17, 2016 at 12:33 PM, Joe Perches  wrote:
> On Fri, 2016-06-17 at 12:18 +1000, Julian Calaby wrote:
>> ./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c
>
> just fyi:  the script name is not plural

Thanks, must have typo'd it between running it and typing it into the email.

> $ ./scripts/get_maintainer.pl  -f drivers/scsi/vmw_pvscsi.c
> Arvind Kumar  (maintainer:VMware PVSCSI driver)
> VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> "Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
> linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> linux-kernel@vger.kernel.org (open list)
>
> and another fyi:
>
> get_maintainer.pl also has a rarely used "--file-emails" option to
> scan for what appears to be email addresses in specific files.
>
> $ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
> Arvind Kumar  (maintainer:VMware PVSCSI driver,in 
> file)
> VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
> "James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
> "Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
> linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
> linux-kernel@vger.kernel.org (open list)
>
> note the "in file" after Arvind's name

Didn't know this, however my point stands: the maintainer line in the
file is redundant if we find maintainers through MAINTAINERS or the
get_maintainer.pl script.

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Joe Perches

On Fri, 2016-06-17 at 12:18 +1000, Julian Calaby wrote:
> ./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c

just fyi:  the script name is not plural

$ ./scripts/get_maintainer.pl  -f drivers/scsi/vmw_pvscsi.c
Arvind Kumar  (maintainer:VMware PVSCSI driver)
VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
"James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
"Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
linux-kernel@vger.kernel.org (open list)

and another fyi:

get_maintainer.pl also has a rarely used "--file-emails" option to
scan for what appears to be email addresses in specific files.

$ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
Arvind Kumar  (maintainer:VMware PVSCSI driver,in file)
VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
"James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
"Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
linux-kernel@vger.kernel.org (open list)

note the "in file" after Arvind's name

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Joe Perches

On Fri, 2016-06-17 at 12:18 +1000, Julian Calaby wrote:
> ./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c

just fyi:  the script name is not plural

$ ./scripts/get_maintainer.pl  -f drivers/scsi/vmw_pvscsi.c
Arvind Kumar  (maintainer:VMware PVSCSI driver)
VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
"James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
"Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
linux-kernel@vger.kernel.org (open list)

and another fyi:

get_maintainer.pl also has a rarely used "--file-emails" option to
scan for what appears to be email addresses in specific files.

$ ./scripts/get_maintainer.pl  -f --file-emails drivers/scsi/vmw_pvscsi.c
Arvind Kumar  (maintainer:VMware PVSCSI driver,in file)
VMware PV-Drivers  (maintainer:VMware PVSCSI driver)
"James E.J. Bottomley"  (maintainer:SCSI SUBSYSTEM)
"Martin K. Petersen"  (maintainer:SCSI SUBSYSTEM)
linux-s...@vger.kernel.org (open list:VMware PVSCSI driver)
linux-kernel@vger.kernel.org (open list)

note the "in file" after Arvind's name

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Arvind,

On Fri, Jun 17, 2016 at 12:03 PM, Arvind Kumar <arvindku...@vmware.com> wrote:
> Hi Julian,
>
> Thanks for spotting that. We will update that too.
>
> I don't really see it as redundant rather as a quick reference instead of 
> digging out the MAINTAINERS file and then search for vmw_pvscsi.c file. 
> Specially for those readers who might not even know about the MAINTAINERS 
> file.

You can use the get_maintainers script in the scripts directory to
quickly get information from MAINTAINERS, i.e.:

./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c

or

./scripts/get_maintainers.pl patchfile.patch

Thanks,

Julian Calaby


>
> Thanks!
> Arvind
> 
> From: Julian Calaby <julian.cal...@gmail.com>
> Sent: Thursday, June 16, 2016 6:48 PM
> To: Jim Gill
> Cc: j...@linux.vnet.ibm.com; Martin K. Petersen; Arvind Kumar; 
> pv-driv...@vmware.com; linux-scsi; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, 
> email)
>
> Hi Jim,
>
> On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill <jg...@vmware.com> wrote:
>> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
>> From: Jim Gill <jg...@vmware.com>
>> Date: Thu, 16 Jun 2016 14:10:43 -0700
>> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
>> email)
>>
>> Signed-off-by: Jim Gill <jg...@vmware.com>
>> ---
>>  drivers/scsi/vmw_pvscsi.c | 2 +-
>>  drivers/scsi/vmw_pvscsi.h | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
>> index 6164634..4a0d3cd 100644
>> --- a/drivers/scsi/vmw_pvscsi.c
>> +++ b/drivers/scsi/vmw_pvscsi.c
>> @@ -17,7 +17,7 @@
>>   * along with this program; if not, write to the Free Software
>>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
>> USA.
>>   *
>> - * Maintained by: Arvind Kumar <arvindku...@vmware.com>
>> + * Maintained by: Jim Gill <jg...@vmware.com>
>
> Shouldn't you update MAINTAINERs too? And isn't having this
> information in these files redundant?
>
> Thanks,
>
> --
> Julian Calaby
>
> Email: julian.cal...@gmail.com
> Profile: 
> https://urldefense.proofpoint.com/v2/url?u=http-3A__www.google.com_profiles_julian.calaby_=CwIBaQ=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs=bUMaNc7nC9xbXtaMJrOvIIPNpPH0chY2kdRsskQn6GY=JibRIMgSBQLffjc5PofQnpJ0k_gawmvafvIKpsEaMzk=CeVpY1FwvY9Qlb0VDeQUXeeL3kuVkcIKTZIC7exS6SM=



-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Arvind,

On Fri, Jun 17, 2016 at 12:03 PM, Arvind Kumar  wrote:
> Hi Julian,
>
> Thanks for spotting that. We will update that too.
>
> I don't really see it as redundant rather as a quick reference instead of 
> digging out the MAINTAINERS file and then search for vmw_pvscsi.c file. 
> Specially for those readers who might not even know about the MAINTAINERS 
> file.

You can use the get_maintainers script in the scripts directory to
quickly get information from MAINTAINERS, i.e.:

./scripts/get_maintainers.pl -f drivers/scsi/vmw_pvscsi.c

or

./scripts/get_maintainers.pl patchfile.patch

Thanks,

Julian Calaby


>
> Thanks!
> Arvind
> 
> From: Julian Calaby 
> Sent: Thursday, June 16, 2016 6:48 PM
> To: Jim Gill
> Cc: j...@linux.vnet.ibm.com; Martin K. Petersen; Arvind Kumar; 
> pv-driv...@vmware.com; linux-scsi; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, 
> email)
>
> Hi Jim,
>
> On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill  wrote:
>> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
>> From: Jim Gill 
>> Date: Thu, 16 Jun 2016 14:10:43 -0700
>> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
>> email)
>>
>> Signed-off-by: Jim Gill 
>> ---
>>  drivers/scsi/vmw_pvscsi.c | 2 +-
>>  drivers/scsi/vmw_pvscsi.h | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
>> index 6164634..4a0d3cd 100644
>> --- a/drivers/scsi/vmw_pvscsi.c
>> +++ b/drivers/scsi/vmw_pvscsi.c
>> @@ -17,7 +17,7 @@
>>   * along with this program; if not, write to the Free Software
>>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
>> USA.
>>   *
>> - * Maintained by: Arvind Kumar 
>> + * Maintained by: Jim Gill 
>
> Shouldn't you update MAINTAINERs too? And isn't having this
> information in these files redundant?
>
> Thanks,
>
> --
> Julian Calaby
>
> Email: julian.cal...@gmail.com
> Profile: 
> https://urldefense.proofpoint.com/v2/url?u=http-3A__www.google.com_profiles_julian.calaby_=CwIBaQ=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs=bUMaNc7nC9xbXtaMJrOvIIPNpPH0chY2kdRsskQn6GY=JibRIMgSBQLffjc5PofQnpJ0k_gawmvafvIKpsEaMzk=CeVpY1FwvY9Qlb0VDeQUXeeL3kuVkcIKTZIC7exS6SM=



-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Arvind Kumar

Hi Julian,

Thanks for spotting that. We will update that too.

I don't really see it as redundant rather as a quick reference instead of 
digging out the MAINTAINERS file and then search for vmw_pvscsi.c file. 
Specially for those readers who might not even know about the MAINTAINERS file.

Thanks!
Arvind

From: Julian Calaby <julian.cal...@gmail.com>
Sent: Thursday, June 16, 2016 6:48 PM
To: Jim Gill
Cc: j...@linux.vnet.ibm.com; Martin K. Petersen; Arvind Kumar; 
pv-driv...@vmware.com; linux-scsi; linux-kernel@vger.kernel.org
Subject: Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, 
email)

Hi Jim,

On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill <jg...@vmware.com> wrote:
> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
> From: Jim Gill <jg...@vmware.com>
> Date: Thu, 16 Jun 2016 14:10:43 -0700
> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
> email)
>
> Signed-off-by: Jim Gill <jg...@vmware.com>
> ---
>  drivers/scsi/vmw_pvscsi.c | 2 +-
>  drivers/scsi/vmw_pvscsi.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
> index 6164634..4a0d3cd 100644
> --- a/drivers/scsi/vmw_pvscsi.c
> +++ b/drivers/scsi/vmw_pvscsi.c
> @@ -17,7 +17,7 @@
>   * along with this program; if not, write to the Free Software
>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
> USA.
>   *
> - * Maintained by: Arvind Kumar <arvindku...@vmware.com>
> + * Maintained by: Jim Gill <jg...@vmware.com>

Shouldn't you update MAINTAINERs too? And isn't having this
information in these files redundant?

Thanks,

--
Julian Calaby

Email: julian.cal...@gmail.com
Profile: 
https://urldefense.proofpoint.com/v2/url?u=http-3A__www.google.com_profiles_julian.calaby_=CwIBaQ=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs=bUMaNc7nC9xbXtaMJrOvIIPNpPH0chY2kdRsskQn6GY=JibRIMgSBQLffjc5PofQnpJ0k_gawmvafvIKpsEaMzk=CeVpY1FwvY9Qlb0VDeQUXeeL3kuVkcIKTZIC7exS6SM=

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Arvind Kumar

Hi Julian,

Thanks for spotting that. We will update that too.

I don't really see it as redundant rather as a quick reference instead of 
digging out the MAINTAINERS file and then search for vmw_pvscsi.c file. 
Specially for those readers who might not even know about the MAINTAINERS file.

Thanks!
Arvind

From: Julian Calaby 
Sent: Thursday, June 16, 2016 6:48 PM
To: Jim Gill
Cc: j...@linux.vnet.ibm.com; Martin K. Petersen; Arvind Kumar; 
pv-driv...@vmware.com; linux-scsi; linux-kernel@vger.kernel.org
Subject: Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, 
email)

Hi Jim,

On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill  wrote:
> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
> From: Jim Gill 
> Date: Thu, 16 Jun 2016 14:10:43 -0700
> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
> email)
>
> Signed-off-by: Jim Gill 
> ---
>  drivers/scsi/vmw_pvscsi.c | 2 +-
>  drivers/scsi/vmw_pvscsi.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
> index 6164634..4a0d3cd 100644
> --- a/drivers/scsi/vmw_pvscsi.c
> +++ b/drivers/scsi/vmw_pvscsi.c
> @@ -17,7 +17,7 @@
>   * along with this program; if not, write to the Free Software
>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
> USA.
>   *
> - * Maintained by: Arvind Kumar 
> + * Maintained by: Jim Gill 

Shouldn't you update MAINTAINERs too? And isn't having this
information in these files redundant?

Thanks,

--
Julian Calaby

Email: julian.cal...@gmail.com
Profile: 
https://urldefense.proofpoint.com/v2/url?u=http-3A__www.google.com_profiles_julian.calaby_=CwIBaQ=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs=bUMaNc7nC9xbXtaMJrOvIIPNpPH0chY2kdRsskQn6GY=JibRIMgSBQLffjc5PofQnpJ0k_gawmvafvIKpsEaMzk=CeVpY1FwvY9Qlb0VDeQUXeeL3kuVkcIKTZIC7exS6SM=

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Jim,

On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill  wrote:
> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
> From: Jim Gill 
> Date: Thu, 16 Jun 2016 14:10:43 -0700
> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
> email)
>
> Signed-off-by: Jim Gill 
> ---
>  drivers/scsi/vmw_pvscsi.c | 2 +-
>  drivers/scsi/vmw_pvscsi.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
> index 6164634..4a0d3cd 100644
> --- a/drivers/scsi/vmw_pvscsi.c
> +++ b/drivers/scsi/vmw_pvscsi.c
> @@ -17,7 +17,7 @@
>   * along with this program; if not, write to the Free Software
>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
> USA.
>   *
> - * Maintained by: Arvind Kumar 
> + * Maintained by: Jim Gill 

Shouldn't you update MAINTAINERs too? And isn't having this
information in these files redundant?

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Julian Calaby

Hi Jim,

On Fri, Jun 17, 2016 at 11:05 AM, Jim Gill  wrote:
> From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
> From: Jim Gill 
> Date: Thu, 16 Jun 2016 14:10:43 -0700
> Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name,
> email)
>
> Signed-off-by: Jim Gill 
> ---
>  drivers/scsi/vmw_pvscsi.c | 2 +-
>  drivers/scsi/vmw_pvscsi.h | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
> index 6164634..4a0d3cd 100644
> --- a/drivers/scsi/vmw_pvscsi.c
> +++ b/drivers/scsi/vmw_pvscsi.c
> @@ -17,7 +17,7 @@
>   * along with this program; if not, write to the Free Software
>   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301
> USA.
>   *
> - * Maintained by: Arvind Kumar 
> + * Maintained by: Jim Gill 

Shouldn't you update MAINTAINERs too? And isn't having this
information in these files redundant?

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Arvind Kumar

Looks fine to me.

Acked-by: Arvind Kumar (arvindku...@vmware.com)

Thanks!
Arvind

From: Jim Gill 
Sent: Thursday, June 16, 2016 6:05 PM
To: j...@linux.vnet.ibm.com; martin.peter...@oracle.com
Cc: Arvind Kumar; pv-driv...@vmware.com; linux-s...@vger.kernel.org; 
linux-kernel@vger.kernel.org
Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

 From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
From: Jim Gill 
Date: Thu, 16 Jun 2016 14:10:43 -0700
Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Signed-off-by: Jim Gill 
---
  drivers/scsi/vmw_pvscsi.c | 2 +-
  drivers/scsi/vmw_pvscsi.h | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
index 6164634..4a0d3cd 100644
--- a/drivers/scsi/vmw_pvscsi.c
+++ b/drivers/scsi/vmw_pvscsi.c
@@ -17,7 +17,7 @@
   * along with this program; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
   *
- * Maintained by: Arvind Kumar 
+ * Maintained by: Jim Gill 
   *
   */

diff --git a/drivers/scsi/vmw_pvscsi.h b/drivers/scsi/vmw_pvscsi.h
index 12712c9..c097d2c 100644
--- a/drivers/scsi/vmw_pvscsi.h
+++ b/drivers/scsi/vmw_pvscsi.h
@@ -17,7 +17,7 @@
   * along with this program; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
   *
- * Maintained by: Arvind Kumar 
+ * Maintained by: Jim Gill 
   *
   */

--
2.5.0

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

2016-06-16 Thread Arvind Kumar

Looks fine to me.

Acked-by: Arvind Kumar (arvindku...@vmware.com)

Thanks!
Arvind

From: Jim Gill 
Sent: Thursday, June 16, 2016 6:05 PM
To: j...@linux.vnet.ibm.com; martin.peter...@oracle.com
Cc: Arvind Kumar; pv-driv...@vmware.com; linux-s...@vger.kernel.org; 
linux-kernel@vger.kernel.org
Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

 From 6a076cc00ec12c6f9cba58ee7e4c3dec49e1e7e5 Mon Sep 17 00:00:00 2001
From: Jim Gill 
Date: Thu, 16 Jun 2016 14:10:43 -0700
Subject: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Signed-off-by: Jim Gill 
---
  drivers/scsi/vmw_pvscsi.c | 2 +-
  drivers/scsi/vmw_pvscsi.h | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
index 6164634..4a0d3cd 100644
--- a/drivers/scsi/vmw_pvscsi.c
+++ b/drivers/scsi/vmw_pvscsi.c
@@ -17,7 +17,7 @@
   * along with this program; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
   *
- * Maintained by: Arvind Kumar 
+ * Maintained by: Jim Gill 
   *
   */

diff --git a/drivers/scsi/vmw_pvscsi.h b/drivers/scsi/vmw_pvscsi.h
index 12712c9..c097d2c 100644
--- a/drivers/scsi/vmw_pvscsi.h
+++ b/drivers/scsi/vmw_pvscsi.h
@@ -17,7 +17,7 @@
   * along with this program; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
   *
- * Maintained by: Arvind Kumar 
+ * Maintained by: Jim Gill 
   *
   */

--
2.5.0

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

Re: [PATCH] VMW_PVSCSI: Change to update maintainer details (name, email)

24 matches

Site Navigation

Mail list logo

Footer information