On Fri, Aug 31, 2018 at 10:33:54AM -0700, Dennis Dalessandro wrote:
> Hi Bjorn, Doug and Jason,
>
> As discussed on the pci list [1] our driver was broken with the following three
> changes in 4.19 merge window:
>
> c6a44ba950d1 ("PCI: Rename pci_try_reset_bus() to pci_reset_bus()")
> 409888e0966e ("IB/hfi1: Use pci_try_reset_bus() for initiating PCI Secondary Bus Reset")
> 811c5cb37df4 ("PCI: Unify try slot and bus reset API")
>
> Ideally I'd like to see those patches reverted but we could also go with
> something like the following series. I know there is a desire to have a more
> clean API but since this is already the rc phase I think that is best delayed.
> Clearly the first attempt at an API clean up did not work out so well.
>
> I'm not sure how you maintainers want to handle this since the fix straddles
> both subsystems so I'm sending it to both linux-rdma and linux-pci.
>
> At the request of discussion on the linux-pci list I have also filed a bug [2].
>
> [1] https://marc.info/?l=linux-pci&m=153539692917785&w=2
> [2] https://bugzilla.kernel.org/show_bug.cgi?id=200985
>
> Thanks
>
>
> ---
>
> Dennis Dalessandro (2):
> PCI: Fix faulty logic in pci_reset_bus()
> IB/hfi1,PCI: Allow bus reset while probing
>
>
> drivers/infiniband/hw/hfi1/pcie.c | 11 ++++-------
> drivers/pci/pci.c | 3 ++-
> include/linux/pci.h | 3 +++
> 3 files changed, 9 insertions(+), 8 deletions(-)
I applied both of these to for-linus for v4.19, thanks!
