[PATCH 1/4] PCI: Add pci_bus_set_ops

[PATCH 1/4] PCI: Add pci_bus_set_ops

[Huang Ying]

[-- Attachment #1: Type: text/plain, Size: 1533 bytes --]

pci_bus_set_ops changes pci_ops associated with a pci_bus. This can be
used by some debug tool such as PCIE AER error injection to fake some
PCI configuration registers.

Signed-off-by: Huang Ying <ying.huang@intel.com>

---
 drivers/pci/access.c |   19 +++++++++++++++++++
 include/linux/pci.h  |    1 +
 2 files changed, 20 insertions(+)

--- a/drivers/pci/access.c
+++ b/drivers/pci/access.c
@@ -66,6 +66,25 @@ EXPORT_SYMBOL(pci_bus_write_config_byte)
 EXPORT_SYMBOL(pci_bus_write_config_word);
 EXPORT_SYMBOL(pci_bus_write_config_dword);
 
+/**
+ * pci_bus_set_ops - Set raw operations of pci bus
+ * @bus:	pci bus struct
+ * @ops:	new raw operations
+ *
+ * Return previous raw operations
+ */
+struct pci_ops *pci_bus_set_ops(struct pci_bus *bus, struct pci_ops *ops)
+{
+	struct pci_ops *old_ops;
+	unsigned long flags;
+
+	spin_lock_irqsave(&pci_lock, flags);
+	old_ops = bus->ops;
+	bus->ops = ops;
+	spin_unlock_irqrestore(&pci_lock, flags);
+	return old_ops;
+}
+EXPORT_SYMBOL(pci_bus_set_ops);
 
 /**
  * pci_read_vpd - Read one entry from Vital Product Data
--- a/include/linux/pci.h
+++ b/include/linux/pci.h
@@ -639,6 +639,7 @@ int pci_bus_write_config_word(struct pci
 			      int where, u16 val);
 int pci_bus_write_config_dword(struct pci_bus *bus, unsigned int devfn,
 			       int where, u32 val);
+struct pci_ops *pci_bus_set_ops(struct pci_bus *bus, struct pci_ops *ops);
 
 static inline int pci_read_config_byte(struct pci_dev *dev, int where, u8 *val)
 {


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [PATCH 1/4] PCI: Add pci_bus_set_ops

[Jesse Barnes]

On Fri, 24 Apr 2009 10:45:17 +0800
Huang Ying <ying.huang@intel.com> wrote:

> pci_bus_set_ops changes pci_ops associated with a pci_bus. This can be
> used by some debug tool such as PCIE AER error injection to fake some
> PCI configuration registers.
> 
> Signed-off-by: Huang Ying <ying.huang@intel.com>

Applied to linux-next, thanks.

-- 
Jesse Barnes, Intel Open Source Technology Center