On Mon, Jun 03, 2024 at 09:49:39PM +0200, Hans de Goede wrote: > Hi, > > On 5/31/24 3:04 AM, Dan Williams wrote: > > Changes since v1 [1]: > > - split out the new warning into its own patch (Bjorn) > > - include the provisional fix to the pci_reset_bus() path > > > > [1]: http://lore.kernel.org/r/171709637423.1568751.11773767969980847536.stgit@xxxxxxxxxxxxxxxxxxxxxxxxx > > > > Hi Bjorn, > > > > Here is the targeted revert of the cfg_access_lock lockdep > > infrastructure, but with the new proposed warning for catching "unlocked > > pci_bridge_secondary_bus_reset()" events broken out into its own change. > > I also included the proposed fix for at least one known case where > > pci_bridge_secondary_bus_reset() is being called without > > cfg_access_lock. > > > > Given there may be more cases to unwind, and the fact that I am not > > convinced patch3 will be problem free I would, as you suggested, > > consider patch2 and patch3 v6.11 material. Patch1 is urgent for v6.10-rc > > to put out these lockdep false positive reports. > > I can confirm that this series fixes the lockdep errors which > I was seeing: > > Tested-by: Hans de Goede <hdegoede@xxxxxxxxxx> Added to the three patches in this series, thanks, Hans! > > --- > > > > Dan Williams (3): > > PCI: Revert the cfg_access_lock lockdep mechanism > > PCI: Warn on missing cfg_access_lock during secondary bus reset > > PCI: Add missing bridge lock to pci_bus_lock() > > > > > > drivers/pci/access.c | 4 ---- > > drivers/pci/pci.c | 8 +++++++- > > drivers/pci/probe.c | 3 --- > > include/linux/lockdep.h | 5 ----- > > include/linux/pci.h | 2 -- > > 5 files changed, 7 insertions(+), 15 deletions(-) > > > > base-commit: 56fb6f92854f29dcb6c3dc3ba92eeda1b615e88c > > >
