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(-)
--
-Denny
