2020-05-07 05:05:38

by Wei Hu

[permalink] [raw]
Subject: [PATCH v3 0/2] Fix PCI HyperV device error handling

This series better handles some PCI HyperV error cases in general
and for kdump case. Some of review comments from previous individual
patch reviews, including splitting into separate patches, have already
been incorporated. Thanks Lorenzo Pieralisi for the review and
suggestions, as well as Michael Kelley's contribution to the commit
log.

Thanks,
Wei


Wei Hu (2):
PCI: hv: Fix the PCI HyperV probe failure path to release resource
properly
PCI: hv: Retry PCI bus D0 entry when the first attempt failed with
invalid device state

drivers/pci/controller/pci-hyperv.c | 60 ++++++++++++++++++++++++++---
1 file changed, 54 insertions(+), 6 deletions(-)

--
2.20.1


2020-05-11 11:24:49

by Lorenzo Pieralisi

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix PCI HyperV device error handling

On Thu, May 07, 2020 at 01:01:26PM +0800, Wei Hu wrote:
> This series better handles some PCI HyperV error cases in general
> and for kdump case. Some of review comments from previous individual
> patch reviews, including splitting into separate patches, have already
> been incorporated. Thanks Lorenzo Pieralisi for the review and
> suggestions, as well as Michael Kelley's contribution to the commit
> log.
>
> Thanks,
> Wei
>
>
> Wei Hu (2):
> PCI: hv: Fix the PCI HyperV probe failure path to release resource
> properly
> PCI: hv: Retry PCI bus D0 entry when the first attempt failed with
> invalid device state
>
> drivers/pci/controller/pci-hyperv.c | 60 ++++++++++++++++++++++++++---
> 1 file changed, 54 insertions(+), 6 deletions(-)

Applied to pci/hv, thanks.

Lorenzo

2020-05-21 02:42:20

by Michael Kelley (LINUX)

[permalink] [raw]
Subject: RE: [PATCH v3 0/2] Fix PCI HyperV device error handling

From: Lorenzo Pieralisi <[email protected]> Sent: Monday, May 11, 2020 4:22 AM
>
> On Thu, May 07, 2020 at 01:01:26PM +0800, Wei Hu wrote:
> > This series better handles some PCI HyperV error cases in general
> > and for kdump case. Some of review comments from previous individual
> > patch reviews, including splitting into separate patches, have already
> > been incorporated. Thanks Lorenzo Pieralisi for the review and
> > suggestions, as well as Michael Kelley's contribution to the commit
> > log.
> >
> > Thanks,
> > Wei
> >
> >
> > Wei Hu (2):
> > PCI: hv: Fix the PCI HyperV probe failure path to release resource
> > properly
> > PCI: hv: Retry PCI bus D0 entry when the first attempt failed with
> > invalid device state
> >
> > drivers/pci/controller/pci-hyperv.c | 60 ++++++++++++++++++++++++++---
> > 1 file changed, 54 insertions(+), 6 deletions(-)
>
> Applied to pci/hv, thanks.
>

Lorenzo --

Will you be bringing these fixes into 5.7? The main fix is the 2nd patch, but
there wasn't a clear "Fixes:" tag to add because the problem is due more to
how Hyper-V operates than a bug in a previous Linux commit. We have a
customer experiencing the problem, so getting the fix into the main tree
sooner rather than later is helpful.

Thx,

Michael

2020-05-21 17:44:25

by Lorenzo Pieralisi

[permalink] [raw]
Subject: Re: [PATCH v3 0/2] Fix PCI HyperV device error handling

On Thu, May 21, 2020 at 02:39:58AM +0000, Michael Kelley wrote:
> From: Lorenzo Pieralisi <[email protected]> Sent: Monday, May 11, 2020 4:22 AM
> >
> > On Thu, May 07, 2020 at 01:01:26PM +0800, Wei Hu wrote:
> > > This series better handles some PCI HyperV error cases in general
> > > and for kdump case. Some of review comments from previous individual
> > > patch reviews, including splitting into separate patches, have already
> > > been incorporated. Thanks Lorenzo Pieralisi for the review and
> > > suggestions, as well as Michael Kelley's contribution to the commit
> > > log.
> > >
> > > Thanks,
> > > Wei
> > >
> > >
> > > Wei Hu (2):
> > > PCI: hv: Fix the PCI HyperV probe failure path to release resource
> > > properly
> > > PCI: hv: Retry PCI bus D0 entry when the first attempt failed with
> > > invalid device state
> > >
> > > drivers/pci/controller/pci-hyperv.c | 60 ++++++++++++++++++++++++++---
> > > 1 file changed, 54 insertions(+), 6 deletions(-)
> >
> > Applied to pci/hv, thanks.
> >
>
> Lorenzo --
>
> Will you be bringing these fixes into 5.7? The main fix is the 2nd patch, but
> there wasn't a clear "Fixes:" tag to add because the problem is due more to
> how Hyper-V operates than a bug in a previous Linux commit. We have a
> customer experiencing the problem, so getting the fix into the main tree
> sooner rather than later is helpful.

We usually send fixes at -rc* if the bug was introduced in the previous
release, at the moment this is v5.8 material not planned for such a late
-rc* (ie -rc7). We can send patches to stable and/or apply a Fixes: tag
if that can help when the commits hit mainline.

Lorenzo