]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
vfio/pci: Lock upstream bridge for vfio_pci_core_disable()
authorAnthony Pighin (Nokia) <anthony.pighin@nokia.com>
Fri, 16 Jan 2026 15:31:26 +0000 (15:31 +0000)
committerAlex Williamson <alex@shazbot.org>
Mon, 19 Jan 2026 17:52:09 +0000 (10:52 -0700)
commit962ae6892d8bd208b2d1e2b358f07551ddc8d32f
tree356fcda4380f0e220fa2f4782e1b2806299f144a
parentfab06e956fce42f07ad35ca53073504bad2a080f
vfio/pci: Lock upstream bridge for vfio_pci_core_disable()

The commit 7e89efc6e9e4 ("Lock upstream bridge for pci_reset_function()")
added locking of the upstream bridge to the reset function. To catch
paths that are not properly locked, the commit 920f6468924f ("Warn on
missing cfg_access_lock during secondary bus reset") added a warning
if the PCI configuration space was not locked during a secondary bus reset
request.

When a VFIO PCI device is released from userspace ownership, an attempt
to reset the PCI device function may be made. If so, and the upstream bridge
is not locked, the release request results in a warning:

   pcieport 0000:00:00.0: unlocked secondary bus reset via:
   pci_reset_bus_function+0x188/0x1b8

Add missing upstream bridge locking to vfio_pci_core_disable().

Fixes: 7e89efc6e9e4 ("PCI: Lock upstream bridge for pci_reset_function()")
Signed-off-by: Anthony Pighin <anthony.pighin@nokia.com>
Link: https://lore.kernel.org/r/BN0PR08MB695171D3AB759C65B6438B5D838DA@BN0PR08MB6951.namprd08.prod.outlook.com
Signed-off-by: Alex Williamson <alex@shazbot.org>
drivers/vfio/pci/vfio_pci_core.c