[PATCH] PCI: Block config access during BIST

Some PCI adapters (eg.  ipr scsi adapters) have an exposure today in that they
issue BIST to the adapter to reset the card.  If, during the time it takes to
complete BIST, userspace attempts to access PCI config space, the host bus
bridge will master abort the access since the ipr adapter does not respond on
the PCI bus for a brief period of time when running BIST.  On PPC64 hardware,
this master abort results in the host PCI bridge isolating that PCI device
from the rest of the system, making the device unusable until Linux is
rebooted.  This patch is an attempt to close that exposure by introducing some
blocking code in the PCI code.  When blocked, writes will be humored and reads
will return the cached value.  Ben Herrenschmidt has also mentioned that he
plans to use this in PPC power management.

Signed-off-by: Brian King <brking@us.ibm.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

 drivers/pci/access.c    |   89 ++++++++++++++++++++++++++++++++++++++++++++++++
 drivers/pci/pci-sysfs.c |   20 +++++-----
 drivers/pci/pci.h       |    7 +++
 drivers/pci/proc.c      |   28 +++++++--------
 drivers/pci/syscall.c   |   14 +++----
 include/linux/pci.h     |    7 +++
 6 files changed, 134 insertions(+), 31 deletions(-)

1 files changed, 10 insertions, 10 deletions

diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c
index 2898830c496f..965a5934623a 100644
--- a/drivers/pci/pci-sysfs.c
+++ b/drivers/pci/pci-sysfs.c
@@ -130,7 +130,7 @@ pci_read_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
 
         if ((off & 1) && size) {
                 u8 val;
-                pci_read_config_byte(dev, off, &val);
+                pci_user_read_config_byte(dev, off, &val);
                 data[off - init_off] = val;
                 off++;
                 size--;
@@ -138,7 +138,7 @@ pci_read_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
 
         if ((off & 3) && size > 2) {
                 u16 val;
-                pci_read_config_word(dev, off, &val);
+                pci_user_read_config_word(dev, off, &val);
                 data[off - init_off] = val & 0xff;
                 data[off - init_off + 1] = (val >> 8) & 0xff;
                 off += 2;
@@ -147,7 +147,7 @@ pci_read_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
 
         while (size > 3) {
                 u32 val;
-                pci_read_config_dword(dev, off, &val);
+                pci_user_read_config_dword(dev, off, &val);
                 data[off - init_off] = val & 0xff;
                 data[off - init_off + 1] = (val >> 8) & 0xff;
                 data[off - init_off + 2] = (val >> 16) & 0xff;
@@ -158,7 +158,7 @@ pci_read_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
 
         if (size >= 2) {
                 u16 val;
-                pci_read_config_word(dev, off, &val);
+                pci_user_read_config_word(dev, off, &val);
                 data[off - init_off] = val & 0xff;
                 data[off - init_off + 1] = (val >> 8) & 0xff;
                 off += 2;
@@ -167,7 +167,7 @@ pci_read_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
 
         if (size > 0) {
                 u8 val;
-                pci_read_config_byte(dev, off, &val);
+                pci_user_read_config_byte(dev, off, &val);
                 data[off - init_off] = val;
                 off++;
                 --size;
@@ -192,7 +192,7 @@ pci_write_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
         }
         
         if ((off & 1) && size) {
-                pci_write_config_byte(dev, off, data[off - init_off]);
+                pci_user_write_config_byte(dev, off, data[off - init_off]);
                 off++;
                 size--;
         }
@@ -200,7 +200,7 @@ pci_write_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
         if ((off & 3) && size > 2) {
                 u16 val = data[off - init_off];
                 val |= (u16) data[off - init_off + 1] << 8;
-                pci_write_config_word(dev, off, val);
+                pci_user_write_config_word(dev, off, val);
                 off += 2;
                 size -= 2;
         }
@@ -210,7 +210,7 @@ pci_write_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
                 val |= (u32) data[off - init_off + 1] << 8;
                 val |= (u32) data[off - init_off + 2] << 16;
                 val |= (u32) data[off - init_off + 3] << 24;
-                pci_write_config_dword(dev, off, val);
+                pci_user_write_config_dword(dev, off, val);
                 off += 4;
                 size -= 4;
         }
@@ -218,13 +218,13 @@ pci_write_config(struct kobject *kobj, char *buf, loff_t off, size_t count)
         if (size >= 2) {
                 u16 val = data[off - init_off];
                 val |= (u16) data[off - init_off + 1] << 8;
-                pci_write_config_word(dev, off, val);
+                pci_user_write_config_word(dev, off, val);
                 off += 2;
                 size -= 2;
         }
 
         if (size) {
-                pci_write_config_byte(dev, off, data[off - init_off]);
+                pci_user_write_config_byte(dev, off, data[off - init_off]);
                 off++;
                 --size;
         }