* [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
@ 2016-08-23 4:42 ` Dexuan Cui
0 siblings, 0 replies; 10+ messages in thread
From: Dexuan Cui @ 2016-08-23 4:42 UTC (permalink / raw)
To: Bjorn Helgaas, linux-pci, gregkh, KY Srinivasan, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, helgaas, Stephen Hemminger
1. use zero-length array to make the code more readable.
2. remove an unused struct member.
3. small error handling improvement to some error cases.
Dexuan Cui (5):
PCI: hv: use zero-length message in struct pci_packet
PCI: hv: use pci_function_description[0] in struct definitions
PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
PCI: hv: hv_compose_msi_msg: handle the 'ret' value
PCI: hv: hv_pci_generic_compl(): handle the error case
drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++-----------------=
----
1 file changed, 29 insertions(+), 27 deletions(-)
--=20
2.7.4
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
2016-08-23 4:42 ` Dexuan Cui
(?)
@ 2016-09-06 15:40 ` Bjorn Helgaas
2016-09-06 17:06 ` KY Srinivasan
-1 siblings, 1 reply; 10+ messages in thread
From: Bjorn Helgaas @ 2016-09-06 15:40 UTC (permalink / raw)
To: Dexuan Cui
Cc: Bjorn Helgaas, linux-pci, gregkh, KY Srinivasan, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, Stephen Hemminger
On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
>
> 1. use zero-length array to make the code more readable.
> 2. remove an unused struct member.
> 3. small error handling improvement to some error cases.
>
> Dexuan Cui (5):
> PCI: hv: use zero-length message in struct pci_packet
> PCI: hv: use pci_function_description[0] in struct definitions
> PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> PCI: hv: hv_pci_generic_compl(): handle the error case
>
> drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++---------------------
> 1 file changed, 29 insertions(+), 27 deletions(-)
I'm waiting for an ack from the Hyper-V maintainers:
Hyper-V CORE AND DRIVERS
M: "K. Y. Srinivasan" <kys@microsoft.com>
M: Haiyang Zhang <haiyangz@microsoft.com>
L: devel@linuxdriverproject.org
S: Maintained
...
F: drivers/pci/host/pci-hyperv.c
...
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
2016-09-06 15:40 ` Bjorn Helgaas
@ 2016-09-06 17:06 ` KY Srinivasan
0 siblings, 0 replies; 10+ messages in thread
From: KY Srinivasan @ 2016-09-06 17:06 UTC (permalink / raw)
To: Bjorn Helgaas, Dexuan Cui
Cc: Bjorn Helgaas, linux-pci, gregkh, linux-kernel, devel, olaf, apw,
jasowang, vkuznets, Haiyang Zhang, Hadden Hoppert,
Stephen Hemminger
> -----Original Message-----
> From: Bjorn Helgaas [mailto:helgaas@kernel.org]
> Sent: Tuesday, September 6, 2016 9:11 PM
> To: Dexuan Cui <decui@microsoft.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; Stephen Hemminger <sthemmin@microsoft.com>
> Subject: Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>
> On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
> >
> > 1. use zero-length array to make the code more readable.
> > 2. remove an unused struct member.
> > 3. small error handling improvement to some error cases.
> >
> > Dexuan Cui (5):
> > PCI: hv: use zero-length message in struct pci_packet
> > PCI: hv: use pci_function_description[0] in struct definitions
> > PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> > PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> > PCI: hv: hv_pci_generic_compl(): handle the error case
> >
> > drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++--------------------
> -
> > 1 file changed, 29 insertions(+), 27 deletions(-)
>
> I'm waiting for an ack from the Hyper-V maintainers:
I have Acked these patches.
K. Y
>
> Hyper-V CORE AND DRIVERS
> M: "K. Y. Srinivasan" <kys@microsoft.com>
> M: Haiyang Zhang <haiyangz@microsoft.com>
> L: devel@linuxdriverproject.org
> S: Maintained
> ...
> F: drivers/pci/host/pci-hyperv.c
> ...
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
@ 2016-09-06 17:06 ` KY Srinivasan
0 siblings, 0 replies; 10+ messages in thread
From: KY Srinivasan @ 2016-09-06 17:06 UTC (permalink / raw)
To: Bjorn Helgaas, Dexuan Cui
Cc: Bjorn Helgaas, linux-pci, gregkh, linux-kernel, devel, olaf, apw,
jasowang, vkuznets, Haiyang Zhang, Hadden Hoppert,
Stephen Hemminger
> -----Original Message-----
> From: Bjorn Helgaas [mailto:helgaas@kernel.org]
> Sent: Tuesday, September 6, 2016 9:11 PM
> To: Dexuan Cui <decui@microsoft.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; Stephen Hemminger <sthemmin@microsoft.com>
> Subject: Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>=20
> On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
> >
> > 1. use zero-length array to make the code more readable.
> > 2. remove an unused struct member.
> > 3. small error handling improvement to some error cases.
> >
> > Dexuan Cui (5):
> > PCI: hv: use zero-length message in struct pci_packet
> > PCI: hv: use pci_function_description[0] in struct definitions
> > PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> > PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> > PCI: hv: hv_pci_generic_compl(): handle the error case
> >
> > drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++-------------=
-------
> -
> > 1 file changed, 29 insertions(+), 27 deletions(-)
>=20
> I'm waiting for an ack from the Hyper-V maintainers:
I have Acked these patches.
K. Y
>=20
> Hyper-V CORE AND DRIVERS
> M: "K. Y. Srinivasan" <kys@microsoft.com>
> M: Haiyang Zhang <haiyangz@microsoft.com>
> L: devel@linuxdriverproject.org
> S: Maintained
> ...
> F: drivers/pci/host/pci-hyperv.c
> ...
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
2016-08-23 4:42 ` Dexuan Cui
@ 2016-09-06 17:05 ` KY Srinivasan
-1 siblings, 0 replies; 10+ messages in thread
From: KY Srinivasan @ 2016-09-06 17:05 UTC (permalink / raw)
To: Dexuan Cui, Bjorn Helgaas, linux-pci, gregkh, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, helgaas, Stephen Hemminger
> -----Original Message-----
> From: Dexuan Cui
> Sent: Tuesday, August 23, 2016 10:13 AM
> To: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; helgaas@kernel.org; Stephen Hemminger
> <sthemmin@microsoft.com>
> Subject: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>
>
> 1. use zero-length array to make the code more readable.
> 2. remove an unused struct member.
> 3. small error handling improvement to some error cases.
>
> Dexuan Cui (5):
> PCI: hv: use zero-length message in struct pci_packet
> PCI: hv: use pci_function_description[0] in struct definitions
> PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> PCI: hv: hv_pci_generic_compl(): handle the error case
Acked-by: KY Srinivasan <kys@microsoft.com>
K. Y
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
@ 2016-09-06 17:05 ` KY Srinivasan
0 siblings, 0 replies; 10+ messages in thread
From: KY Srinivasan @ 2016-09-06 17:05 UTC (permalink / raw)
To: Dexuan Cui, Bjorn Helgaas, linux-pci, gregkh, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, helgaas, Stephen Hemminger
> -----Original Message-----
> From: Dexuan Cui
> Sent: Tuesday, August 23, 2016 10:13 AM
> To: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; helgaas@kernel.org; Stephen Hemminger
> <sthemmin@microsoft.com>
> Subject: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>=20
>=20
> 1. use zero-length array to make the code more readable.
> 2. remove an unused struct member.
> 3. small error handling improvement to some error cases.
>=20
> Dexuan Cui (5):
> PCI: hv: use zero-length message in struct pci_packet
> PCI: hv: use pci_function_description[0] in struct definitions
> PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> PCI: hv: hv_pci_generic_compl(): handle the error case
Acked-by: KY Srinivasan <kys@microsoft.com>
K. Y
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
2016-08-23 4:42 ` Dexuan Cui
` (2 preceding siblings ...)
(?)
@ 2016-09-06 17:24 ` Bjorn Helgaas
2016-09-07 1:56 ` Dexuan Cui
-1 siblings, 1 reply; 10+ messages in thread
From: Bjorn Helgaas @ 2016-09-06 17:24 UTC (permalink / raw)
To: Dexuan Cui
Cc: Bjorn Helgaas, linux-pci, gregkh, KY Srinivasan, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, Stephen Hemminger
On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
>
> 1. use zero-length array to make the code more readable.
> 2. remove an unused struct member.
> 3. small error handling improvement to some error cases.
>
> Dexuan Cui (5):
> PCI: hv: use zero-length message in struct pci_packet
> PCI: hv: use pci_function_description[0] in struct definitions
> PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> PCI: hv: hv_pci_generic_compl(): handle the error case
>
> drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++---------------------
> 1 file changed, 29 insertions(+), 27 deletions(-)
Applied with KY's ack to pci/host-hv for v4.9, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
2016-09-06 17:24 ` Bjorn Helgaas
@ 2016-09-07 1:56 ` Dexuan Cui
0 siblings, 0 replies; 10+ messages in thread
From: Dexuan Cui @ 2016-09-07 1:56 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Bjorn Helgaas, linux-pci, gregkh, KY Srinivasan, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, Stephen Hemminger, Jake Oshins
> From: Bjorn Helgaas [mailto:helgaas@kernel.org]
> Sent: Wednesday, September 7, 2016 1:25
> To: Dexuan Cui <decui@microsoft.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; Stephen Hemminger <sthemmin@microsoft.com>
> Subject: Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>
> On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
> >
> > 1. use zero-length array to make the code more readable.
> > 2. remove an unused struct member.
> > 3. small error handling improvement to some error cases.
> >
> > Dexuan Cui (5):
> > PCI: hv: use zero-length message in struct pci_packet
> > PCI: hv: use pci_function_description[0] in struct definitions
> > PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> > PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> > PCI: hv: hv_pci_generic_compl(): handle the error case
> >
> > drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++--------------------
> -
> > 1 file changed, 29 insertions(+), 27 deletions(-)
>
> Applied with KY's ack to pci/host-hv for v4.9, thanks!
Thanks!
BTW, it looks I forgot to CC Jake Oshins somehow... sorry for that.
@Jake: these are just small cleanups. :-)
Thanks,
-- Dexuan
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
@ 2016-09-07 1:56 ` Dexuan Cui
0 siblings, 0 replies; 10+ messages in thread
From: Dexuan Cui @ 2016-09-07 1:56 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Bjorn Helgaas, linux-pci, gregkh, KY Srinivasan, linux-kernel,
devel, olaf, apw, jasowang, vkuznets, Haiyang Zhang,
Hadden Hoppert, Stephen Hemminger, Jake Oshins
> From: Bjorn Helgaas [mailto:helgaas@kernel.org]
> Sent: Wednesday, September 7, 2016 1:25
> To: Dexuan Cui <decui@microsoft.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>; linux-pci@vger.kernel.org;
> gregkh@linuxfoundation.org; KY Srinivasan <kys@microsoft.com>; linux-
> kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de;
> apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; Haiyang
> Zhang <haiyangz@microsoft.com>; Hadden Hoppert
> <haddenh@microsoft.com>; Stephen Hemminger <sthemmin@microsoft.com>
> Subject: Re: [PATCH 0/5] PCI: hv: some minor bug fixes and cleanups
>=20
> On Tue, Aug 23, 2016 at 04:42:41AM +0000, Dexuan Cui wrote:
> >
> > 1. use zero-length array to make the code more readable.
> > 2. remove an unused struct member.
> > 3. small error handling improvement to some error cases.
> >
> > Dexuan Cui (5):
> > PCI: hv: use zero-length message in struct pci_packet
> > PCI: hv: use pci_function_description[0] in struct definitions
> > PCI: hv: remove the unused 'wrk' in struct hv_pcibus_device
> > PCI: hv: hv_compose_msi_msg: handle the 'ret' value
> > PCI: hv: hv_pci_generic_compl(): handle the error case
> >
> > drivers/pci/host/pci-hyperv.c | 56 ++++++++++++++++++++++-------------=
-------
> -
> > 1 file changed, 29 insertions(+), 27 deletions(-)
>=20
> Applied with KY's ack to pci/host-hv for v4.9, thanks!
Thanks!
BTW, it looks I forgot to CC Jake Oshins somehow... sorry for that.
@Jake: these are just small cleanups. :-)
Thanks,
-- Dexuan
^ permalink raw reply [flat|nested] 10+ messages in thread