mbox series

[v3,0/2] Fix PCI HyperV device error handling

Message ID 20200507050126.10871-1-weh@microsoft.com
Headers show
Series Fix PCI HyperV device error handling | expand

Message

Wei Hu May 7, 2020, 5:01 a.m. UTC
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(-)

Comments

Lorenzo Pieralisi May 11, 2020, 11:21 a.m. UTC | #1
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
Michael Kelley (LINUX) May 21, 2020, 2:39 a.m. UTC | #2
From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> 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
Lorenzo Pieralisi May 21, 2020, 5:42 p.m. UTC | #3
On Thu, May 21, 2020 at 02:39:58AM +0000, Michael Kelley wrote:
> From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> 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