]> git.karo-electronics.de Git - karo-tx-linux.git/commit
PCI: Rework config space blocking services
authorJan Kiszka <jan.kiszka@siemens.com>
Fri, 4 Nov 2011 08:45:59 +0000 (09:45 +0100)
committerJesse Barnes <jbarnes@virtuousgeek.org>
Tue, 6 Dec 2011 17:26:09 +0000 (09:26 -0800)
commit6c03c7b748b605b0c1de57eb454ddc8dad83e4ef
tree070fe83512c0944e4c26ac0e5c16b798711bedb3
parent35d5bc0b1b9f185e9966a48bd7c4b57c96af3c8b
PCI: Rework config space blocking services

pci_block_user_cfg_access was designed for the use case that a single
context, the IPR driver, temporarily delays user space accesses to the
config space via sysfs. This assumption became invalid by the time
pci_dev_reset was added as locking instance. Today, if you run two loops
in parallel that reset the same device via sysfs, you end up with a
kernel BUG as pci_block_user_cfg_access detect the broken assumption.

This reworks the pci_block_user_cfg_access to a sleeping service
pci_cfg_access_lock and an atomic-compatible variant called
pci_cfg_access_trylock. The former not only blocks user space access as
before but also waits if access was already locked. The latter service
just returns false in this case, allowing the caller to resolve the
conflict instead of raising a BUG.

Adaptions of the ipr driver were originally written by Brian King.

Acked-by: Brian King <brking@linux.vnet.ibm.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
drivers/pci/access.c
drivers/pci/iov.c
drivers/pci/pci.c
drivers/scsi/ipr.c
drivers/scsi/ipr.h
drivers/uio/uio_pci_generic.c
include/linux/pci.h