diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-21 21:58:52 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-21 21:58:52 -0400 |
commit | 6109e2ce2600e2db26cd0424bb9c6ed019723288 (patch) | |
tree | 54b5d347bf12e0a987edfb52f287399f748a9a38 /drivers | |
parent | 0961d6581c870850342ad6ea25263763433d666f (diff) | |
parent | ac81860ea073daed50246af54db706c6e491f240 (diff) |
Merge branch 'linux-next' of git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/pci-2.6
* 'linux-next' of git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/pci-2.6: (36 commits)
PCI: hotplug: pciehp: Removed check for hotplug of display devices
PCI: read memory ranges out of Broadcom CNB20LE host bridge
PCI: Allow manual resource allocation for PCI hotplug bridges
x86/PCI: make ACPI MCFG reserved error messages ACPI specific
PCI hotplug: Use kmemdup
PM/PCI: Update PCI power management documentation
PCI: output FW warning in pci_read/write_vpd
PCI: fix typos pci_device_dis/enable to pci_dis/enable_device in comments
PCI quirks: disable msi on AMD rs4xx internal gfx bridges
PCI: Disable MSI for MCP55 on P5N32-E SLI
x86/PCI: irq and pci_ids patch for additional Intel Cougar Point DeviceIDs
PCI: aerdrv: trivial cleanup for aerdrv_core.c
PCI: aerdrv: trivial cleanup for aerdrv.c
PCI: aerdrv: introduce default_downstream_reset_link
PCI: aerdrv: rework find_aer_service
PCI: aerdrv: remove is_downstream
PCI: aerdrv: remove magical ROOT_ERR_STATUS_MASKS
PCI: aerdrv: redefine PCI_ERR_ROOT_*_SRC
PCI: aerdrv: rework do_recovery
PCI: aerdrv: rework get_e_source()
...
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/edac/amd76x_edac.c | 2 | ||||
-rw-r--r-- | drivers/edac/i82443bxgx_edac.c | 2 | ||||
-rw-r--r-- | drivers/edac/r82600_edac.c | 2 | ||||
-rw-r--r-- | drivers/pci/Kconfig | 2 | ||||
-rw-r--r-- | drivers/pci/access.c | 41 | ||||
-rw-r--r-- | drivers/pci/hotplug/cpqphp_core.c | 3 | ||||
-rw-r--r-- | drivers/pci/hotplug/pciehp_pci.c | 17 | ||||
-rw-r--r-- | drivers/pci/pci-sysfs.c | 44 | ||||
-rw-r--r-- | drivers/pci/pci.c | 4 | ||||
-rw-r--r-- | drivers/pci/pcie/aer/aer_inject.c | 2 | ||||
-rw-r--r-- | drivers/pci/pcie/aer/aerdrv.c | 179 | ||||
-rw-r--r-- | drivers/pci/pcie/aer/aerdrv.h | 6 | ||||
-rw-r--r-- | drivers/pci/pcie/aer/aerdrv_core.c | 558 | ||||
-rw-r--r-- | drivers/pci/quirks.c | 21 | ||||
-rw-r--r-- | drivers/pci/slot.c | 48 |
15 files changed, 511 insertions, 420 deletions
diff --git a/drivers/edac/amd76x_edac.c b/drivers/edac/amd76x_edac.c index f2330f81cb5e..cace0a7b707a 100644 --- a/drivers/edac/amd76x_edac.c +++ b/drivers/edac/amd76x_edac.c | |||
@@ -294,7 +294,7 @@ static int __devinit amd76x_init_one(struct pci_dev *pdev, | |||
294 | { | 294 | { |
295 | debugf0("%s()\n", __func__); | 295 | debugf0("%s()\n", __func__); |
296 | 296 | ||
297 | /* don't need to call pci_device_enable() */ | 297 | /* don't need to call pci_enable_device() */ |
298 | return amd76x_probe1(pdev, ent->driver_data); | 298 | return amd76x_probe1(pdev, ent->driver_data); |
299 | } | 299 | } |
300 | 300 | ||
diff --git a/drivers/edac/i82443bxgx_edac.c b/drivers/edac/i82443bxgx_edac.c index 7f3884fcbd46..2bf2c5051bfe 100644 --- a/drivers/edac/i82443bxgx_edac.c +++ b/drivers/edac/i82443bxgx_edac.c | |||
@@ -354,7 +354,7 @@ static int __devinit i82443bxgx_edacmc_init_one(struct pci_dev *pdev, | |||
354 | 354 | ||
355 | debugf0("MC: " __FILE__ ": %s()\n", __func__); | 355 | debugf0("MC: " __FILE__ ": %s()\n", __func__); |
356 | 356 | ||
357 | /* don't need to call pci_device_enable() */ | 357 | /* don't need to call pci_enable_device() */ |
358 | rc = i82443bxgx_edacmc_probe1(pdev, ent->driver_data); | 358 | rc = i82443bxgx_edacmc_probe1(pdev, ent->driver_data); |
359 | 359 | ||
360 | if (mci_pdev == NULL) | 360 | if (mci_pdev == NULL) |
diff --git a/drivers/edac/r82600_edac.c b/drivers/edac/r82600_edac.c index d55f8e9de788..6a822c631ef5 100644 --- a/drivers/edac/r82600_edac.c +++ b/drivers/edac/r82600_edac.c | |||
@@ -354,7 +354,7 @@ static int __devinit r82600_init_one(struct pci_dev *pdev, | |||
354 | { | 354 | { |
355 | debugf0("%s()\n", __func__); | 355 | debugf0("%s()\n", __func__); |
356 | 356 | ||
357 | /* don't need to call pci_device_enable() */ | 357 | /* don't need to call pci_enable_device() */ |
358 | return r82600_probe1(pdev, ent->driver_data); | 358 | return r82600_probe1(pdev, ent->driver_data); |
359 | } | 359 | } |
360 | 360 | ||
diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig index 7858a117e80b..34ef70d562b2 100644 --- a/drivers/pci/Kconfig +++ b/drivers/pci/Kconfig | |||
@@ -19,7 +19,7 @@ config PCI_MSI | |||
19 | by using the 'pci=nomsi' option. This disables MSI for the | 19 | by using the 'pci=nomsi' option. This disables MSI for the |
20 | entire system. | 20 | entire system. |
21 | 21 | ||
22 | If you don't know what to do here, say N. | 22 | If you don't know what to do here, say Y. |
23 | 23 | ||
24 | config PCI_DEBUG | 24 | config PCI_DEBUG |
25 | bool "PCI Debugging" | 25 | bool "PCI Debugging" |
diff --git a/drivers/pci/access.c b/drivers/pci/access.c index 2f646fe1260f..531bc697d800 100644 --- a/drivers/pci/access.c +++ b/drivers/pci/access.c | |||
@@ -13,7 +13,7 @@ | |||
13 | * configuration space. | 13 | * configuration space. |
14 | */ | 14 | */ |
15 | 15 | ||
16 | static DEFINE_SPINLOCK(pci_lock); | 16 | static DEFINE_RAW_SPINLOCK(pci_lock); |
17 | 17 | ||
18 | /* | 18 | /* |
19 | * Wrappers for all PCI configuration access functions. They just check | 19 | * Wrappers for all PCI configuration access functions. They just check |
@@ -33,10 +33,10 @@ int pci_bus_read_config_##size \ | |||
33 | unsigned long flags; \ | 33 | unsigned long flags; \ |
34 | u32 data = 0; \ | 34 | u32 data = 0; \ |
35 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ | 35 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ |
36 | spin_lock_irqsave(&pci_lock, flags); \ | 36 | raw_spin_lock_irqsave(&pci_lock, flags); \ |
37 | res = bus->ops->read(bus, devfn, pos, len, &data); \ | 37 | res = bus->ops->read(bus, devfn, pos, len, &data); \ |
38 | *value = (type)data; \ | 38 | *value = (type)data; \ |
39 | spin_unlock_irqrestore(&pci_lock, flags); \ | 39 | raw_spin_unlock_irqrestore(&pci_lock, flags); \ |
40 | return res; \ | 40 | return res; \ |
41 | } | 41 | } |
42 | 42 | ||
@@ -47,9 +47,9 @@ int pci_bus_write_config_##size \ | |||
47 | int res; \ | 47 | int res; \ |
48 | unsigned long flags; \ | 48 | unsigned long flags; \ |
49 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ | 49 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ |
50 | spin_lock_irqsave(&pci_lock, flags); \ | 50 | raw_spin_lock_irqsave(&pci_lock, flags); \ |
51 | res = bus->ops->write(bus, devfn, pos, len, value); \ | 51 | res = bus->ops->write(bus, devfn, pos, len, value); \ |
52 | spin_unlock_irqrestore(&pci_lock, flags); \ | 52 | raw_spin_unlock_irqrestore(&pci_lock, flags); \ |
53 | return res; \ | 53 | return res; \ |
54 | } | 54 | } |
55 | 55 | ||
@@ -79,10 +79,10 @@ struct pci_ops *pci_bus_set_ops(struct pci_bus *bus, struct pci_ops *ops) | |||
79 | struct pci_ops *old_ops; | 79 | struct pci_ops *old_ops; |
80 | unsigned long flags; | 80 | unsigned long flags; |
81 | 81 | ||
82 | spin_lock_irqsave(&pci_lock, flags); | 82 | raw_spin_lock_irqsave(&pci_lock, flags); |
83 | old_ops = bus->ops; | 83 | old_ops = bus->ops; |
84 | bus->ops = ops; | 84 | bus->ops = ops; |
85 | spin_unlock_irqrestore(&pci_lock, flags); | 85 | raw_spin_unlock_irqrestore(&pci_lock, flags); |
86 | return old_ops; | 86 | return old_ops; |
87 | } | 87 | } |
88 | EXPORT_SYMBOL(pci_bus_set_ops); | 88 | EXPORT_SYMBOL(pci_bus_set_ops); |
@@ -136,9 +136,9 @@ static noinline void pci_wait_ucfg(struct pci_dev *dev) | |||
136 | __add_wait_queue(&pci_ucfg_wait, &wait); | 136 | __add_wait_queue(&pci_ucfg_wait, &wait); |
137 | do { | 137 | do { |
138 | set_current_state(TASK_UNINTERRUPTIBLE); | 138 | set_current_state(TASK_UNINTERRUPTIBLE); |
139 | spin_unlock_irq(&pci_lock); | 139 | raw_spin_unlock_irq(&pci_lock); |
140 | schedule(); | 140 | schedule(); |
141 | spin_lock_irq(&pci_lock); | 141 | raw_spin_lock_irq(&pci_lock); |
142 | } while (dev->block_ucfg_access); | 142 | } while (dev->block_ucfg_access); |
143 | __remove_wait_queue(&pci_ucfg_wait, &wait); | 143 | __remove_wait_queue(&pci_ucfg_wait, &wait); |
144 | } | 144 | } |
@@ -150,11 +150,11 @@ int pci_user_read_config_##size \ | |||
150 | int ret = 0; \ | 150 | int ret = 0; \ |
151 | u32 data = -1; \ | 151 | u32 data = -1; \ |
152 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ | 152 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ |
153 | spin_lock_irq(&pci_lock); \ | 153 | raw_spin_lock_irq(&pci_lock); \ |
154 | if (unlikely(dev->block_ucfg_access)) pci_wait_ucfg(dev); \ | 154 | if (unlikely(dev->block_ucfg_access)) pci_wait_ucfg(dev); \ |
155 | ret = dev->bus->ops->read(dev->bus, dev->devfn, \ | 155 | ret = dev->bus->ops->read(dev->bus, dev->devfn, \ |
156 | pos, sizeof(type), &data); \ | 156 | pos, sizeof(type), &data); \ |
157 | spin_unlock_irq(&pci_lock); \ | 157 | raw_spin_unlock_irq(&pci_lock); \ |
158 | *val = (type)data; \ | 158 | *val = (type)data; \ |
159 | return ret; \ | 159 | return ret; \ |
160 | } | 160 | } |
@@ -165,11 +165,11 @@ int pci_user_write_config_##size \ | |||
165 | { \ | 165 | { \ |
166 | int ret = -EIO; \ | 166 | int ret = -EIO; \ |
167 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ | 167 | if (PCI_##size##_BAD) return PCIBIOS_BAD_REGISTER_NUMBER; \ |
168 | spin_lock_irq(&pci_lock); \ | 168 | raw_spin_lock_irq(&pci_lock); \ |
169 | if (unlikely(dev->block_ucfg_access)) pci_wait_ucfg(dev); \ | 169 | if (unlikely(dev->block_ucfg_access)) pci_wait_ucfg(dev); \ |
170 | ret = dev->bus->ops->write(dev->bus, dev->devfn, \ | 170 | ret = dev->bus->ops->write(dev->bus, dev->devfn, \ |
171 | pos, sizeof(type), val); \ | 171 | pos, sizeof(type), val); \ |
172 | spin_unlock_irq(&pci_lock); \ | 172 | raw_spin_unlock_irq(&pci_lock); \ |
173 | return ret; \ | 173 | return ret; \ |
174 | } | 174 | } |
175 | 175 | ||
@@ -220,8 +220,13 @@ static int pci_vpd_pci22_wait(struct pci_dev *dev) | |||
220 | return 0; | 220 | return 0; |
221 | } | 221 | } |
222 | 222 | ||
223 | if (time_after(jiffies, timeout)) | 223 | if (time_after(jiffies, timeout)) { |
224 | dev_printk(KERN_DEBUG, &dev->dev, | ||
225 | "vpd r/w failed. This is likely a firmware " | ||
226 | "bug on this device. Contact the card " | ||
227 | "vendor for a firmware update."); | ||
224 | return -ETIMEDOUT; | 228 | return -ETIMEDOUT; |
229 | } | ||
225 | if (fatal_signal_pending(current)) | 230 | if (fatal_signal_pending(current)) |
226 | return -EINTR; | 231 | return -EINTR; |
227 | if (!cond_resched()) | 232 | if (!cond_resched()) |
@@ -396,10 +401,10 @@ void pci_block_user_cfg_access(struct pci_dev *dev) | |||
396 | unsigned long flags; | 401 | unsigned long flags; |
397 | int was_blocked; | 402 | int was_blocked; |
398 | 403 | ||
399 | spin_lock_irqsave(&pci_lock, flags); | 404 | raw_spin_lock_irqsave(&pci_lock, flags); |
400 | was_blocked = dev->block_ucfg_access; | 405 | was_blocked = dev->block_ucfg_access; |
401 | dev->block_ucfg_access = 1; | 406 | dev->block_ucfg_access = 1; |
402 | spin_unlock_irqrestore(&pci_lock, flags); | 407 | raw_spin_unlock_irqrestore(&pci_lock, flags); |
403 | 408 | ||
404 | /* If we BUG() inside the pci_lock, we're guaranteed to hose | 409 | /* If we BUG() inside the pci_lock, we're guaranteed to hose |
405 | * the machine */ | 410 | * the machine */ |
@@ -417,7 +422,7 @@ void pci_unblock_user_cfg_access(struct pci_dev *dev) | |||
417 | { | 422 | { |
418 | unsigned long flags; | 423 | unsigned long flags; |
419 | 424 | ||
420 | spin_lock_irqsave(&pci_lock, flags); | 425 | raw_spin_lock_irqsave(&pci_lock, flags); |
421 | 426 | ||
422 | /* This indicates a problem in the caller, but we don't need | 427 | /* This indicates a problem in the caller, but we don't need |
423 | * to kill them, unlike a double-block above. */ | 428 | * to kill them, unlike a double-block above. */ |
@@ -425,6 +430,6 @@ void pci_unblock_user_cfg_access(struct pci_dev *dev) | |||
425 | 430 | ||
426 | dev->block_ucfg_access = 0; | 431 | dev->block_ucfg_access = 0; |
427 | wake_up_all(&pci_ucfg_wait); | 432 | wake_up_all(&pci_ucfg_wait); |
428 | spin_unlock_irqrestore(&pci_lock, flags); | 433 | raw_spin_unlock_irqrestore(&pci_lock, flags); |
429 | } | 434 | } |
430 | EXPORT_SYMBOL_GPL(pci_unblock_user_cfg_access); | 435 | EXPORT_SYMBOL_GPL(pci_unblock_user_cfg_access); |
diff --git a/drivers/pci/hotplug/cpqphp_core.c b/drivers/pci/hotplug/cpqphp_core.c index 6644337d63d6..b3e5580c837b 100644 --- a/drivers/pci/hotplug/cpqphp_core.c +++ b/drivers/pci/hotplug/cpqphp_core.c | |||
@@ -1075,13 +1075,12 @@ static int cpqhpc_probe(struct pci_dev *pdev, const struct pci_device_id *ent) | |||
1075 | 1075 | ||
1076 | /* make our own copy of the pci bus structure, | 1076 | /* make our own copy of the pci bus structure, |
1077 | * as we like tweaking it a lot */ | 1077 | * as we like tweaking it a lot */ |
1078 | ctrl->pci_bus = kmalloc(sizeof(*ctrl->pci_bus), GFP_KERNEL); | 1078 | ctrl->pci_bus = kmemdup(pdev->bus, sizeof(*ctrl->pci_bus), GFP_KERNEL); |
1079 | if (!ctrl->pci_bus) { | 1079 | if (!ctrl->pci_bus) { |
1080 | err("out of memory\n"); | 1080 | err("out of memory\n"); |
1081 | rc = -ENOMEM; | 1081 | rc = -ENOMEM; |
1082 | goto err_free_ctrl; | 1082 | goto err_free_ctrl; |
1083 | } | 1083 | } |
1084 | memcpy(ctrl->pci_bus, pdev->bus, sizeof(*ctrl->pci_bus)); | ||
1085 | 1084 | ||
1086 | ctrl->bus = pdev->bus->number; | 1085 | ctrl->bus = pdev->bus->number; |
1087 | ctrl->rev = pdev->revision; | 1086 | ctrl->rev = pdev->revision; |
diff --git a/drivers/pci/hotplug/pciehp_pci.c b/drivers/pci/hotplug/pciehp_pci.c index 0a16444c14c9..2fce726758d2 100644 --- a/drivers/pci/hotplug/pciehp_pci.c +++ b/drivers/pci/hotplug/pciehp_pci.c | |||
@@ -84,12 +84,6 @@ int pciehp_configure_device(struct slot *p_slot) | |||
84 | dev = pci_get_slot(parent, PCI_DEVFN(0, fn)); | 84 | dev = pci_get_slot(parent, PCI_DEVFN(0, fn)); |
85 | if (!dev) | 85 | if (!dev) |
86 | continue; | 86 | continue; |
87 | if ((dev->class >> 16) == PCI_BASE_CLASS_DISPLAY) { | ||
88 | ctrl_err(ctrl, "Cannot hot-add display device %s\n", | ||
89 | pci_name(dev)); | ||
90 | pci_dev_put(dev); | ||
91 | continue; | ||
92 | } | ||
93 | if ((dev->hdr_type == PCI_HEADER_TYPE_BRIDGE) || | 87 | if ((dev->hdr_type == PCI_HEADER_TYPE_BRIDGE) || |
94 | (dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)) { | 88 | (dev->hdr_type == PCI_HEADER_TYPE_CARDBUS)) { |
95 | pciehp_add_bridge(dev); | 89 | pciehp_add_bridge(dev); |
@@ -133,15 +127,9 @@ int pciehp_unconfigure_device(struct slot *p_slot) | |||
133 | presence = 0; | 127 | presence = 0; |
134 | 128 | ||
135 | for (j = 0; j < 8; j++) { | 129 | for (j = 0; j < 8; j++) { |
136 | struct pci_dev* temp = pci_get_slot(parent, PCI_DEVFN(0, j)); | 130 | struct pci_dev *temp = pci_get_slot(parent, PCI_DEVFN(0, j)); |
137 | if (!temp) | 131 | if (!temp) |
138 | continue; | 132 | continue; |
139 | if ((temp->class >> 16) == PCI_BASE_CLASS_DISPLAY) { | ||
140 | ctrl_err(ctrl, "Cannot remove display device %s\n", | ||
141 | pci_name(temp)); | ||
142 | pci_dev_put(temp); | ||
143 | continue; | ||
144 | } | ||
145 | if (temp->hdr_type == PCI_HEADER_TYPE_BRIDGE && presence) { | 133 | if (temp->hdr_type == PCI_HEADER_TYPE_BRIDGE && presence) { |
146 | pci_read_config_byte(temp, PCI_BRIDGE_CONTROL, &bctl); | 134 | pci_read_config_byte(temp, PCI_BRIDGE_CONTROL, &bctl); |
147 | if (bctl & PCI_BRIDGE_CTL_VGA) { | 135 | if (bctl & PCI_BRIDGE_CTL_VGA) { |
@@ -149,7 +137,8 @@ int pciehp_unconfigure_device(struct slot *p_slot) | |||
149 | "Cannot remove display device %s\n", | 137 | "Cannot remove display device %s\n", |
150 | pci_name(temp)); | 138 | pci_name(temp)); |
151 | pci_dev_put(temp); | 139 | pci_dev_put(temp); |
152 | continue; | 140 | rc = EINVAL; |
141 | break; | ||
153 | } | 142 | } |
154 | } | 143 | } |
155 | pci_remove_bus_device(temp); | 144 | pci_remove_bus_device(temp); |
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c index 6309c5a2528f..afd2fbf7d797 100644 --- a/drivers/pci/pci-sysfs.c +++ b/drivers/pci/pci-sysfs.c | |||
@@ -979,7 +979,12 @@ static ssize_t reset_store(struct device *dev, | |||
979 | 979 | ||
980 | if (val != 1) | 980 | if (val != 1) |
981 | return -EINVAL; | 981 | return -EINVAL; |
982 | return pci_reset_function(pdev); | 982 | |
983 | result = pci_reset_function(pdev); | ||
984 | if (result < 0) | ||
985 | return result; | ||
986 | |||
987 | return count; | ||
983 | } | 988 | } |
984 | 989 | ||
985 | static struct device_attribute reset_attr = __ATTR(reset, 0200, NULL, reset_store); | 990 | static struct device_attribute reset_attr = __ATTR(reset, 0200, NULL, reset_store); |
@@ -1030,6 +1035,39 @@ error: | |||
1030 | return retval; | 1035 | return retval; |
1031 | } | 1036 | } |
1032 | 1037 | ||
1038 | static void pci_remove_slot_links(struct pci_dev *dev) | ||
1039 | { | ||
1040 | char func[10]; | ||
1041 | struct pci_slot *slot; | ||
1042 | |||
1043 | sysfs_remove_link(&dev->dev.kobj, "slot"); | ||
1044 | list_for_each_entry(slot, &dev->bus->slots, list) { | ||
1045 | if (slot->number != PCI_SLOT(dev->devfn)) | ||
1046 | continue; | ||
1047 | snprintf(func, 10, "function%d", PCI_FUNC(dev->devfn)); | ||
1048 | sysfs_remove_link(&slot->kobj, func); | ||
1049 | } | ||
1050 | } | ||
1051 | |||
1052 | static int pci_create_slot_links(struct pci_dev *dev) | ||
1053 | { | ||
1054 | int result = 0; | ||
1055 | char func[10]; | ||
1056 | struct pci_slot *slot; | ||
1057 | |||
1058 | list_for_each_entry(slot, &dev->bus->slots, list) { | ||
1059 | if (slot->number != PCI_SLOT(dev->devfn)) | ||
1060 | continue; | ||
1061 | result = sysfs_create_link(&dev->dev.kobj, &slot->kobj, "slot"); | ||
1062 | if (result) | ||
1063 | goto out; | ||
1064 | snprintf(func, 10, "function%d", PCI_FUNC(dev->devfn)); | ||
1065 | result = sysfs_create_link(&slot->kobj, &dev->dev.kobj, func); | ||
1066 | } | ||
1067 | out: | ||
1068 | return result; | ||
1069 | } | ||
1070 | |||
1033 | int __must_check pci_create_sysfs_dev_files (struct pci_dev *pdev) | 1071 | int __must_check pci_create_sysfs_dev_files (struct pci_dev *pdev) |
1034 | { | 1072 | { |
1035 | int retval; | 1073 | int retval; |
@@ -1092,6 +1130,8 @@ int __must_check pci_create_sysfs_dev_files (struct pci_dev *pdev) | |||
1092 | if (retval) | 1130 | if (retval) |
1093 | goto err_vga_file; | 1131 | goto err_vga_file; |
1094 | 1132 | ||
1133 | pci_create_slot_links(pdev); | ||
1134 | |||
1095 | return 0; | 1135 | return 0; |
1096 | 1136 | ||
1097 | err_vga_file: | 1137 | err_vga_file: |
@@ -1141,6 +1181,8 @@ void pci_remove_sysfs_dev_files(struct pci_dev *pdev) | |||
1141 | if (!sysfs_initialized) | 1181 | if (!sysfs_initialized) |
1142 | return; | 1182 | return; |
1143 | 1183 | ||
1184 | pci_remove_slot_links(pdev); | ||
1185 | |||
1144 | pci_remove_capabilities_sysfs(pdev); | 1186 | pci_remove_capabilities_sysfs(pdev); |
1145 | 1187 | ||
1146 | if (pdev->cfg_size < PCI_CFG_SPACE_EXP_SIZE) | 1188 | if (pdev->cfg_size < PCI_CFG_SPACE_EXP_SIZE) |
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index 1df7c508814e..60f30e7f1c8c 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c | |||
@@ -1193,7 +1193,7 @@ void pci_disable_enabled_device(struct pci_dev *dev) | |||
1193 | * anymore. This only involves disabling PCI bus-mastering, if active. | 1193 | * anymore. This only involves disabling PCI bus-mastering, if active. |
1194 | * | 1194 | * |
1195 | * Note we don't actually disable the device until all callers of | 1195 | * Note we don't actually disable the device until all callers of |
1196 | * pci_device_enable() have called pci_device_disable(). | 1196 | * pci_enable_device() have called pci_disable_device(). |
1197 | */ | 1197 | */ |
1198 | void | 1198 | void |
1199 | pci_disable_device(struct pci_dev *dev) | 1199 | pci_disable_device(struct pci_dev *dev) |
@@ -1631,7 +1631,6 @@ void pci_pm_init(struct pci_dev *dev) | |||
1631 | * let the user space enable it to wake up the system as needed. | 1631 | * let the user space enable it to wake up the system as needed. |
1632 | */ | 1632 | */ |
1633 | device_set_wakeup_capable(&dev->dev, true); | 1633 | device_set_wakeup_capable(&dev->dev, true); |
1634 | device_set_wakeup_enable(&dev->dev, false); | ||
1635 | /* Disable the PME# generation functionality */ | 1634 | /* Disable the PME# generation functionality */ |
1636 | pci_pme_active(dev, false); | 1635 | pci_pme_active(dev, false); |
1637 | } else { | 1636 | } else { |
@@ -1655,7 +1654,6 @@ void platform_pci_wakeup_init(struct pci_dev *dev) | |||
1655 | return; | 1654 | return; |
1656 | 1655 | ||
1657 | device_set_wakeup_capable(&dev->dev, true); | 1656 | device_set_wakeup_capable(&dev->dev, true); |
1658 | device_set_wakeup_enable(&dev->dev, false); | ||
1659 | platform_pci_sleep_wake(dev, false); | 1657 | platform_pci_sleep_wake(dev, false); |
1660 | } | 1658 | } |
1661 | 1659 | ||
diff --git a/drivers/pci/pcie/aer/aer_inject.c b/drivers/pci/pcie/aer/aer_inject.c index f8f425b8731d..909924692b8a 100644 --- a/drivers/pci/pcie/aer/aer_inject.c +++ b/drivers/pci/pcie/aer/aer_inject.c | |||
@@ -168,7 +168,7 @@ static u32 *find_pci_config_dword(struct aer_error *err, int where, | |||
168 | target = &err->root_status; | 168 | target = &err->root_status; |
169 | rw1cs = 1; | 169 | rw1cs = 1; |
170 | break; | 170 | break; |
171 | case PCI_ERR_ROOT_COR_SRC: | 171 | case PCI_ERR_ROOT_ERR_SRC: |
172 | target = &err->source_id; | 172 | target = &err->source_id; |
173 | break; | 173 | break; |
174 | } | 174 | } |
diff --git a/drivers/pci/pcie/aer/aerdrv.c b/drivers/pci/pcie/aer/aerdrv.c index 7a711ee314b7..484cc55194b8 100644 --- a/drivers/pci/pcie/aer/aerdrv.c +++ b/drivers/pci/pcie/aer/aerdrv.c | |||
@@ -72,13 +72,120 @@ void pci_no_aer(void) | |||
72 | pcie_aer_disable = 1; /* has priority over 'forceload' */ | 72 | pcie_aer_disable = 1; /* has priority over 'forceload' */ |
73 | } | 73 | } |
74 | 74 | ||
75 | static int set_device_error_reporting(struct pci_dev *dev, void *data) | ||
76 | { | ||
77 | bool enable = *((bool *)data); | ||
78 | |||
79 | if ((dev->pcie_type == PCI_EXP_TYPE_ROOT_PORT) || | ||
80 | (dev->pcie_type == PCI_EXP_TYPE_UPSTREAM) || | ||
81 | (dev->pcie_type == PCI_EXP_TYPE_DOWNSTREAM)) { | ||
82 | if (enable) | ||
83 | pci_enable_pcie_error_reporting(dev); | ||
84 | else | ||
85 | pci_disable_pcie_error_reporting(dev); | ||
86 | } | ||
87 | |||
88 | if (enable) | ||
89 | pcie_set_ecrc_checking(dev); | ||
90 | |||
91 | return 0; | ||
92 | } | ||
93 | |||
94 | /** | ||
95 | * set_downstream_devices_error_reporting - enable/disable the error reporting bits on the root port and its downstream ports. | ||
96 | * @dev: pointer to root port's pci_dev data structure | ||
97 | * @enable: true = enable error reporting, false = disable error reporting. | ||
98 | */ | ||
99 | static void set_downstream_devices_error_reporting(struct pci_dev *dev, | ||
100 | bool enable) | ||
101 | { | ||
102 | set_device_error_reporting(dev, &enable); | ||
103 | |||
104 | if (!dev->subordinate) | ||
105 | return; | ||
106 | pci_walk_bus(dev->subordinate, set_device_error_reporting, &enable); | ||
107 | } | ||
108 | |||
109 | /** | ||
110 | * aer_enable_rootport - enable Root Port's interrupts when receiving messages | ||
111 | * @rpc: pointer to a Root Port data structure | ||
112 | * | ||
113 | * Invoked when PCIe bus loads AER service driver. | ||
114 | */ | ||
115 | static void aer_enable_rootport(struct aer_rpc *rpc) | ||
116 | { | ||
117 | struct pci_dev *pdev = rpc->rpd->port; | ||
118 | int pos, aer_pos; | ||
119 | u16 reg16; | ||
120 | u32 reg32; | ||
121 | |||
122 | pos = pci_pcie_cap(pdev); | ||
123 | /* Clear PCIe Capability's Device Status */ | ||
124 | pci_read_config_word(pdev, pos+PCI_EXP_DEVSTA, ®16); | ||
125 | pci_write_config_word(pdev, pos+PCI_EXP_DEVSTA, reg16); | ||
126 | |||
127 | /* Disable system error generation in response to error messages */ | ||
128 | pci_read_config_word(pdev, pos + PCI_EXP_RTCTL, ®16); | ||
129 | reg16 &= ~(SYSTEM_ERROR_INTR_ON_MESG_MASK); | ||
130 | pci_write_config_word(pdev, pos + PCI_EXP_RTCTL, reg16); | ||
131 | |||
132 | aer_pos = pci_find_ext_capability(pdev, PCI_EXT_CAP_ID_ERR); | ||
133 | /* Clear error status */ | ||
134 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_ROOT_STATUS, ®32); | ||
135 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_ROOT_STATUS, reg32); | ||
136 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_COR_STATUS, ®32); | ||
137 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_COR_STATUS, reg32); | ||
138 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_UNCOR_STATUS, ®32); | ||
139 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_UNCOR_STATUS, reg32); | ||
140 | |||
141 | /* | ||
142 | * Enable error reporting for the root port device and downstream port | ||
143 | * devices. | ||
144 | */ | ||
145 | set_downstream_devices_error_reporting(pdev, true); | ||
146 | |||
147 | /* Enable Root Port's interrupt in response to error messages */ | ||
148 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_ROOT_COMMAND, ®32); | ||
149 | reg32 |= ROOT_PORT_INTR_ON_MESG_MASK; | ||
150 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_ROOT_COMMAND, reg32); | ||
151 | } | ||
152 | |||
153 | /** | ||
154 | * aer_disable_rootport - disable Root Port's interrupts when receiving messages | ||
155 | * @rpc: pointer to a Root Port data structure | ||
156 | * | ||
157 | * Invoked when PCIe bus unloads AER service driver. | ||
158 | */ | ||
159 | static void aer_disable_rootport(struct aer_rpc *rpc) | ||
160 | { | ||
161 | struct pci_dev *pdev = rpc->rpd->port; | ||
162 | u32 reg32; | ||
163 | int pos; | ||
164 | |||
165 | /* | ||
166 | * Disable error reporting for the root port device and downstream port | ||
167 | * devices. | ||
168 | */ | ||
169 | set_downstream_devices_error_reporting(pdev, false); | ||
170 | |||
171 | pos = pci_find_ext_capability(pdev, PCI_EXT_CAP_ID_ERR); | ||
172 | /* Disable Root's interrupt in response to error messages */ | ||
173 | pci_read_config_dword(pdev, pos + PCI_ERR_ROOT_COMMAND, ®32); | ||
174 | reg32 &= ~ROOT_PORT_INTR_ON_MESG_MASK; | ||
175 | pci_write_config_dword(pdev, pos + PCI_ERR_ROOT_COMMAND, reg32); | ||
176 | |||
177 | /* Clear Root's error status reg */ | ||
178 | pci_read_config_dword(pdev, pos + PCI_ERR_ROOT_STATUS, ®32); | ||
179 | pci_write_config_dword(pdev, pos + PCI_ERR_ROOT_STATUS, reg32); | ||
180 | } | ||
181 | |||
75 | /** | 182 | /** |
76 | * aer_irq - Root Port's ISR | 183 | * aer_irq - Root Port's ISR |
77 | * @irq: IRQ assigned to Root Port | 184 | * @irq: IRQ assigned to Root Port |
78 | * @context: pointer to Root Port data structure | 185 | * @context: pointer to Root Port data structure |
79 | * | 186 | * |
80 | * Invoked when Root Port detects AER messages. | 187 | * Invoked when Root Port detects AER messages. |
81 | **/ | 188 | */ |
82 | irqreturn_t aer_irq(int irq, void *context) | 189 | irqreturn_t aer_irq(int irq, void *context) |
83 | { | 190 | { |
84 | unsigned int status, id; | 191 | unsigned int status, id; |
@@ -97,13 +204,13 @@ irqreturn_t aer_irq(int irq, void *context) | |||
97 | 204 | ||
98 | /* Read error status */ | 205 | /* Read error status */ |
99 | pci_read_config_dword(pdev->port, pos + PCI_ERR_ROOT_STATUS, &status); | 206 | pci_read_config_dword(pdev->port, pos + PCI_ERR_ROOT_STATUS, &status); |
100 | if (!(status & ROOT_ERR_STATUS_MASKS)) { | 207 | if (!(status & (PCI_ERR_ROOT_UNCOR_RCV|PCI_ERR_ROOT_COR_RCV))) { |
101 | spin_unlock_irqrestore(&rpc->e_lock, flags); | 208 | spin_unlock_irqrestore(&rpc->e_lock, flags); |
102 | return IRQ_NONE; | 209 | return IRQ_NONE; |
103 | } | 210 | } |
104 | 211 | ||
105 | /* Read error source and clear error status */ | 212 | /* Read error source and clear error status */ |
106 | pci_read_config_dword(pdev->port, pos + PCI_ERR_ROOT_COR_SRC, &id); | 213 | pci_read_config_dword(pdev->port, pos + PCI_ERR_ROOT_ERR_SRC, &id); |
107 | pci_write_config_dword(pdev->port, pos + PCI_ERR_ROOT_STATUS, status); | 214 | pci_write_config_dword(pdev->port, pos + PCI_ERR_ROOT_STATUS, status); |
108 | 215 | ||
109 | /* Store error source for later DPC handler */ | 216 | /* Store error source for later DPC handler */ |
@@ -135,7 +242,7 @@ EXPORT_SYMBOL_GPL(aer_irq); | |||
135 | * @dev: pointer to the pcie_dev data structure | 242 | * @dev: pointer to the pcie_dev data structure |
136 | * | 243 | * |
137 | * Invoked when Root Port's AER service is loaded. | 244 | * Invoked when Root Port's AER service is loaded. |
138 | **/ | 245 | */ |
139 | static struct aer_rpc *aer_alloc_rpc(struct pcie_device *dev) | 246 | static struct aer_rpc *aer_alloc_rpc(struct pcie_device *dev) |
140 | { | 247 | { |
141 | struct aer_rpc *rpc; | 248 | struct aer_rpc *rpc; |
@@ -144,15 +251,11 @@ static struct aer_rpc *aer_alloc_rpc(struct pcie_device *dev) | |||
144 | if (!rpc) | 251 | if (!rpc) |
145 | return NULL; | 252 | return NULL; |
146 | 253 | ||
147 | /* | 254 | /* Initialize Root lock access, e_lock, to Root Error Status Reg */ |
148 | * Initialize Root lock access, e_lock, to Root Error Status Reg, | ||
149 | * Root Error ID Reg, and Root error producer/consumer index. | ||
150 | */ | ||
151 | spin_lock_init(&rpc->e_lock); | 255 | spin_lock_init(&rpc->e_lock); |
152 | 256 | ||
153 | rpc->rpd = dev; | 257 | rpc->rpd = dev; |
154 | INIT_WORK(&rpc->dpc_handler, aer_isr); | 258 | INIT_WORK(&rpc->dpc_handler, aer_isr); |
155 | rpc->prod_idx = rpc->cons_idx = 0; | ||
156 | mutex_init(&rpc->rpc_mutex); | 259 | mutex_init(&rpc->rpc_mutex); |
157 | init_waitqueue_head(&rpc->wait_release); | 260 | init_waitqueue_head(&rpc->wait_release); |
158 | 261 | ||
@@ -167,7 +270,7 @@ static struct aer_rpc *aer_alloc_rpc(struct pcie_device *dev) | |||
167 | * @dev: pointer to the pcie_dev data structure | 270 | * @dev: pointer to the pcie_dev data structure |
168 | * | 271 | * |
169 | * Invoked when PCI Express bus unloads or AER probe fails. | 272 | * Invoked when PCI Express bus unloads or AER probe fails. |
170 | **/ | 273 | */ |
171 | static void aer_remove(struct pcie_device *dev) | 274 | static void aer_remove(struct pcie_device *dev) |
172 | { | 275 | { |
173 | struct aer_rpc *rpc = get_service_data(dev); | 276 | struct aer_rpc *rpc = get_service_data(dev); |
@@ -179,7 +282,8 @@ static void aer_remove(struct pcie_device *dev) | |||
179 | 282 | ||
180 | wait_event(rpc->wait_release, rpc->prod_idx == rpc->cons_idx); | 283 | wait_event(rpc->wait_release, rpc->prod_idx == rpc->cons_idx); |
181 | 284 | ||
182 | aer_delete_rootport(rpc); | 285 | aer_disable_rootport(rpc); |
286 | kfree(rpc); | ||
183 | set_service_data(dev, NULL); | 287 | set_service_data(dev, NULL); |
184 | } | 288 | } |
185 | } | 289 | } |
@@ -190,7 +294,7 @@ static void aer_remove(struct pcie_device *dev) | |||
190 | * @id: pointer to the service id data structure | 294 | * @id: pointer to the service id data structure |
191 | * | 295 | * |
192 | * Invoked when PCI Express bus loads AER service driver. | 296 | * Invoked when PCI Express bus loads AER service driver. |
193 | **/ | 297 | */ |
194 | static int __devinit aer_probe(struct pcie_device *dev) | 298 | static int __devinit aer_probe(struct pcie_device *dev) |
195 | { | 299 | { |
196 | int status; | 300 | int status; |
@@ -230,47 +334,30 @@ static int __devinit aer_probe(struct pcie_device *dev) | |||
230 | * @dev: pointer to Root Port's pci_dev data structure | 334 | * @dev: pointer to Root Port's pci_dev data structure |
231 | * | 335 | * |
232 | * Invoked by Port Bus driver when performing link reset at Root Port. | 336 | * Invoked by Port Bus driver when performing link reset at Root Port. |
233 | **/ | 337 | */ |
234 | static pci_ers_result_t aer_root_reset(struct pci_dev *dev) | 338 | static pci_ers_result_t aer_root_reset(struct pci_dev *dev) |
235 | { | 339 | { |
236 | u16 p2p_ctrl; | 340 | u32 reg32; |
237 | u32 status; | ||
238 | int pos; | 341 | int pos; |
239 | 342 | ||
240 | pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); | 343 | pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); |
241 | 344 | ||
242 | /* Disable Root's interrupt in response to error messages */ | 345 | /* Disable Root's interrupt in response to error messages */ |
243 | pci_write_config_dword(dev, pos + PCI_ERR_ROOT_COMMAND, 0); | 346 | pci_read_config_dword(dev, pos + PCI_ERR_ROOT_COMMAND, ®32); |
244 | 347 | reg32 &= ~ROOT_PORT_INTR_ON_MESG_MASK; | |
245 | /* Assert Secondary Bus Reset */ | 348 | pci_write_config_dword(dev, pos + PCI_ERR_ROOT_COMMAND, reg32); |
246 | pci_read_config_word(dev, PCI_BRIDGE_CONTROL, &p2p_ctrl); | ||
247 | p2p_ctrl |= PCI_BRIDGE_CTL_BUS_RESET; | ||
248 | pci_write_config_word(dev, PCI_BRIDGE_CONTROL, p2p_ctrl); | ||
249 | |||
250 | /* | ||
251 | * we should send hot reset message for 2ms to allow it time to | ||
252 | * propogate to all downstream ports | ||
253 | */ | ||
254 | msleep(2); | ||
255 | |||
256 | /* De-assert Secondary Bus Reset */ | ||
257 | p2p_ctrl &= ~PCI_BRIDGE_CTL_BUS_RESET; | ||
258 | pci_write_config_word(dev, PCI_BRIDGE_CONTROL, p2p_ctrl); | ||
259 | 349 | ||
260 | /* | 350 | aer_do_secondary_bus_reset(dev); |
261 | * System software must wait for at least 100ms from the end | ||
262 | * of a reset of one or more device before it is permitted | ||
263 | * to issue Configuration Requests to those devices. | ||
264 | */ | ||
265 | msleep(200); | ||
266 | dev_printk(KERN_DEBUG, &dev->dev, "Root Port link has been reset\n"); | 351 | dev_printk(KERN_DEBUG, &dev->dev, "Root Port link has been reset\n"); |
267 | 352 | ||
353 | /* Clear Root Error Status */ | ||
354 | pci_read_config_dword(dev, pos + PCI_ERR_ROOT_STATUS, ®32); | ||
355 | pci_write_config_dword(dev, pos + PCI_ERR_ROOT_STATUS, reg32); | ||
356 | |||
268 | /* Enable Root Port's interrupt in response to error messages */ | 357 | /* Enable Root Port's interrupt in response to error messages */ |
269 | pci_read_config_dword(dev, pos + PCI_ERR_ROOT_STATUS, &status); | 358 | pci_read_config_dword(dev, pos + PCI_ERR_ROOT_COMMAND, ®32); |
270 | pci_write_config_dword(dev, pos + PCI_ERR_ROOT_STATUS, status); | 359 | reg32 |= ROOT_PORT_INTR_ON_MESG_MASK; |
271 | pci_write_config_dword(dev, | 360 | pci_write_config_dword(dev, pos + PCI_ERR_ROOT_COMMAND, reg32); |
272 | pos + PCI_ERR_ROOT_COMMAND, | ||
273 | ROOT_PORT_INTR_ON_MESG_MASK); | ||
274 | 361 | ||
275 | return PCI_ERS_RESULT_RECOVERED; | 362 | return PCI_ERS_RESULT_RECOVERED; |
276 | } | 363 | } |
@@ -281,7 +368,7 @@ static pci_ers_result_t aer_root_reset(struct pci_dev *dev) | |||
281 | * @error: error severity being notified by port bus | 368 | * @error: error severity being notified by port bus |
282 | * | 369 | * |
283 | * Invoked by Port Bus driver during error recovery. | 370 | * Invoked by Port Bus driver during error recovery. |
284 | **/ | 371 | */ |
285 | static pci_ers_result_t aer_error_detected(struct pci_dev *dev, | 372 | static pci_ers_result_t aer_error_detected(struct pci_dev *dev, |
286 | enum pci_channel_state error) | 373 | enum pci_channel_state error) |
287 | { | 374 | { |
@@ -294,7 +381,7 @@ static pci_ers_result_t aer_error_detected(struct pci_dev *dev, | |||
294 | * @dev: pointer to Root Port's pci_dev data structure | 381 | * @dev: pointer to Root Port's pci_dev data structure |
295 | * | 382 | * |
296 | * Invoked by Port Bus driver during nonfatal recovery. | 383 | * Invoked by Port Bus driver during nonfatal recovery. |
297 | **/ | 384 | */ |
298 | static void aer_error_resume(struct pci_dev *dev) | 385 | static void aer_error_resume(struct pci_dev *dev) |
299 | { | 386 | { |
300 | int pos; | 387 | int pos; |
@@ -321,7 +408,7 @@ static void aer_error_resume(struct pci_dev *dev) | |||
321 | * aer_service_init - register AER root service driver | 408 | * aer_service_init - register AER root service driver |
322 | * | 409 | * |
323 | * Invoked when AER root service driver is loaded. | 410 | * Invoked when AER root service driver is loaded. |
324 | **/ | 411 | */ |
325 | static int __init aer_service_init(void) | 412 | static int __init aer_service_init(void) |
326 | { | 413 | { |
327 | if (pcie_aer_disable) | 414 | if (pcie_aer_disable) |
@@ -335,7 +422,7 @@ static int __init aer_service_init(void) | |||
335 | * aer_service_exit - unregister AER root service driver | 422 | * aer_service_exit - unregister AER root service driver |
336 | * | 423 | * |
337 | * Invoked when AER root service driver is unloaded. | 424 | * Invoked when AER root service driver is unloaded. |
338 | **/ | 425 | */ |
339 | static void __exit aer_service_exit(void) | 426 | static void __exit aer_service_exit(void) |
340 | { | 427 | { |
341 | pcie_port_service_unregister(&aerdriver); | 428 | pcie_port_service_unregister(&aerdriver); |
diff --git a/drivers/pci/pcie/aer/aerdrv.h b/drivers/pci/pcie/aer/aerdrv.h index bd833ea3ba49..7aaae2d2bd67 100644 --- a/drivers/pci/pcie/aer/aerdrv.h +++ b/drivers/pci/pcie/aer/aerdrv.h | |||
@@ -17,9 +17,6 @@ | |||
17 | #define AER_FATAL 1 | 17 | #define AER_FATAL 1 |
18 | #define AER_CORRECTABLE 2 | 18 | #define AER_CORRECTABLE 2 |
19 | 19 | ||
20 | /* Root Error Status Register Bits */ | ||
21 | #define ROOT_ERR_STATUS_MASKS 0x0f | ||
22 | |||
23 | #define SYSTEM_ERROR_INTR_ON_MESG_MASK (PCI_EXP_RTCTL_SECEE| \ | 20 | #define SYSTEM_ERROR_INTR_ON_MESG_MASK (PCI_EXP_RTCTL_SECEE| \ |
24 | PCI_EXP_RTCTL_SENFEE| \ | 21 | PCI_EXP_RTCTL_SENFEE| \ |
25 | PCI_EXP_RTCTL_SEFEE) | 22 | PCI_EXP_RTCTL_SEFEE) |
@@ -117,8 +114,7 @@ static inline pci_ers_result_t merge_result(enum pci_ers_result orig, | |||
117 | } | 114 | } |
118 | 115 | ||
119 | extern struct bus_type pcie_port_bus_type; | 116 | extern struct bus_type pcie_port_bus_type; |
120 | extern void aer_enable_rootport(struct aer_rpc *rpc); | 117 | extern void aer_do_secondary_bus_reset(struct pci_dev *dev); |
121 | extern void aer_delete_rootport(struct aer_rpc *rpc); | ||
122 | extern int aer_init(struct pcie_device *dev); | 118 | extern int aer_init(struct pcie_device *dev); |
123 | extern void aer_isr(struct work_struct *work); | 119 | extern void aer_isr(struct work_struct *work); |
124 | extern void aer_print_error(struct pci_dev *dev, struct aer_err_info *info); | 120 | extern void aer_print_error(struct pci_dev *dev, struct aer_err_info *info); |
diff --git a/drivers/pci/pcie/aer/aerdrv_core.c b/drivers/pci/pcie/aer/aerdrv_core.c index aceb04b67b60..df2d686fe3dd 100644 --- a/drivers/pci/pcie/aer/aerdrv_core.c +++ b/drivers/pci/pcie/aer/aerdrv_core.c | |||
@@ -47,13 +47,12 @@ int pci_enable_pcie_error_reporting(struct pci_dev *dev) | |||
47 | if (!pos) | 47 | if (!pos) |
48 | return -EIO; | 48 | return -EIO; |
49 | 49 | ||
50 | pci_read_config_word(dev, pos+PCI_EXP_DEVCTL, ®16); | 50 | pci_read_config_word(dev, pos + PCI_EXP_DEVCTL, ®16); |
51 | reg16 = reg16 | | 51 | reg16 |= (PCI_EXP_DEVCTL_CERE | |
52 | PCI_EXP_DEVCTL_CERE | | ||
53 | PCI_EXP_DEVCTL_NFERE | | 52 | PCI_EXP_DEVCTL_NFERE | |
54 | PCI_EXP_DEVCTL_FERE | | 53 | PCI_EXP_DEVCTL_FERE | |
55 | PCI_EXP_DEVCTL_URRE; | 54 | PCI_EXP_DEVCTL_URRE); |
56 | pci_write_config_word(dev, pos+PCI_EXP_DEVCTL, reg16); | 55 | pci_write_config_word(dev, pos + PCI_EXP_DEVCTL, reg16); |
57 | 56 | ||
58 | return 0; | 57 | return 0; |
59 | } | 58 | } |
@@ -71,12 +70,12 @@ int pci_disable_pcie_error_reporting(struct pci_dev *dev) | |||
71 | if (!pos) | 70 | if (!pos) |
72 | return -EIO; | 71 | return -EIO; |
73 | 72 | ||
74 | pci_read_config_word(dev, pos+PCI_EXP_DEVCTL, ®16); | 73 | pci_read_config_word(dev, pos + PCI_EXP_DEVCTL, ®16); |
75 | reg16 = reg16 & ~(PCI_EXP_DEVCTL_CERE | | 74 | reg16 &= ~(PCI_EXP_DEVCTL_CERE | |
76 | PCI_EXP_DEVCTL_NFERE | | 75 | PCI_EXP_DEVCTL_NFERE | |
77 | PCI_EXP_DEVCTL_FERE | | 76 | PCI_EXP_DEVCTL_FERE | |
78 | PCI_EXP_DEVCTL_URRE); | 77 | PCI_EXP_DEVCTL_URRE); |
79 | pci_write_config_word(dev, pos+PCI_EXP_DEVCTL, reg16); | 78 | pci_write_config_word(dev, pos + PCI_EXP_DEVCTL, reg16); |
80 | 79 | ||
81 | return 0; | 80 | return 0; |
82 | } | 81 | } |
@@ -99,99 +98,46 @@ int pci_cleanup_aer_uncorrect_error_status(struct pci_dev *dev) | |||
99 | } | 98 | } |
100 | EXPORT_SYMBOL_GPL(pci_cleanup_aer_uncorrect_error_status); | 99 | EXPORT_SYMBOL_GPL(pci_cleanup_aer_uncorrect_error_status); |
101 | 100 | ||
102 | static int set_device_error_reporting(struct pci_dev *dev, void *data) | ||
103 | { | ||
104 | bool enable = *((bool *)data); | ||
105 | |||
106 | if ((dev->pcie_type == PCI_EXP_TYPE_ROOT_PORT) || | ||
107 | (dev->pcie_type == PCI_EXP_TYPE_UPSTREAM) || | ||
108 | (dev->pcie_type == PCI_EXP_TYPE_DOWNSTREAM)) { | ||
109 | if (enable) | ||
110 | pci_enable_pcie_error_reporting(dev); | ||
111 | else | ||
112 | pci_disable_pcie_error_reporting(dev); | ||
113 | } | ||
114 | |||
115 | if (enable) | ||
116 | pcie_set_ecrc_checking(dev); | ||
117 | |||
118 | return 0; | ||
119 | } | ||
120 | |||
121 | /** | 101 | /** |
122 | * set_downstream_devices_error_reporting - enable/disable the error reporting bits on the root port and its downstream ports. | 102 | * add_error_device - list device to be handled |
123 | * @dev: pointer to root port's pci_dev data structure | 103 | * @e_info: pointer to error info |
124 | * @enable: true = enable error reporting, false = disable error reporting. | 104 | * @dev: pointer to pci_dev to be added |
125 | */ | 105 | */ |
126 | static void set_downstream_devices_error_reporting(struct pci_dev *dev, | ||
127 | bool enable) | ||
128 | { | ||
129 | set_device_error_reporting(dev, &enable); | ||
130 | |||
131 | if (!dev->subordinate) | ||
132 | return; | ||
133 | pci_walk_bus(dev->subordinate, set_device_error_reporting, &enable); | ||
134 | } | ||
135 | |||
136 | static inline int compare_device_id(struct pci_dev *dev, | ||
137 | struct aer_err_info *e_info) | ||
138 | { | ||
139 | if (e_info->id == ((dev->bus->number << 8) | dev->devfn)) { | ||
140 | /* | ||
141 | * Device ID match | ||
142 | */ | ||
143 | return 1; | ||
144 | } | ||
145 | |||
146 | return 0; | ||
147 | } | ||
148 | |||
149 | static int add_error_device(struct aer_err_info *e_info, struct pci_dev *dev) | 106 | static int add_error_device(struct aer_err_info *e_info, struct pci_dev *dev) |
150 | { | 107 | { |
151 | if (e_info->error_dev_num < AER_MAX_MULTI_ERR_DEVICES) { | 108 | if (e_info->error_dev_num < AER_MAX_MULTI_ERR_DEVICES) { |
152 | e_info->dev[e_info->error_dev_num] = dev; | 109 | e_info->dev[e_info->error_dev_num] = dev; |
153 | e_info->error_dev_num++; | 110 | e_info->error_dev_num++; |
154 | return 1; | 111 | return 0; |
155 | } | 112 | } |
156 | 113 | return -ENOSPC; | |
157 | return 0; | ||
158 | } | 114 | } |
159 | 115 | ||
160 | |||
161 | #define PCI_BUS(x) (((x) >> 8) & 0xff) | 116 | #define PCI_BUS(x) (((x) >> 8) & 0xff) |
162 | 117 | ||
163 | static int find_device_iter(struct pci_dev *dev, void *data) | 118 | /** |
119 | * is_error_source - check whether the device is source of reported error | ||
120 | * @dev: pointer to pci_dev to be checked | ||
121 | * @e_info: pointer to reported error info | ||
122 | */ | ||
123 | static bool is_error_source(struct pci_dev *dev, struct aer_err_info *e_info) | ||
164 | { | 124 | { |
165 | int pos; | 125 | int pos; |
166 | u32 status; | 126 | u32 status, mask; |
167 | u32 mask; | ||
168 | u16 reg16; | 127 | u16 reg16; |
169 | int result; | ||
170 | struct aer_err_info *e_info = (struct aer_err_info *)data; | ||
171 | 128 | ||
172 | /* | 129 | /* |
173 | * When bus id is equal to 0, it might be a bad id | 130 | * When bus id is equal to 0, it might be a bad id |
174 | * reported by root port. | 131 | * reported by root port. |
175 | */ | 132 | */ |
176 | if (!nosourceid && (PCI_BUS(e_info->id) != 0)) { | 133 | if (!nosourceid && (PCI_BUS(e_info->id) != 0)) { |
177 | result = compare_device_id(dev, e_info); | 134 | /* Device ID match? */ |
178 | if (result) | 135 | if (e_info->id == ((dev->bus->number << 8) | dev->devfn)) |
179 | add_error_device(e_info, dev); | 136 | return true; |
180 | 137 | ||
181 | /* | 138 | /* Continue id comparing if there is no multiple error */ |
182 | * If there is no multiple error, we stop | ||
183 | * or continue based on the id comparing. | ||
184 | */ | ||
185 | if (!e_info->multi_error_valid) | 139 | if (!e_info->multi_error_valid) |
186 | return result; | 140 | return false; |
187 | |||
188 | /* | ||
189 | * If there are multiple errors and id does match, | ||
190 | * We need continue to search other devices under | ||
191 | * the root port. Return 0 means that. | ||
192 | */ | ||
193 | if (result) | ||
194 | return 0; | ||
195 | } | 141 | } |
196 | 142 | ||
197 | /* | 143 | /* |
@@ -200,71 +146,94 @@ static int find_device_iter(struct pci_dev *dev, void *data) | |||
200 | * 2) bus id is equal to 0. Some ports might lose the bus | 146 | * 2) bus id is equal to 0. Some ports might lose the bus |
201 | * id of error source id; | 147 | * id of error source id; |
202 | * 3) There are multiple errors and prior id comparing fails; | 148 | * 3) There are multiple errors and prior id comparing fails; |
203 | * We check AER status registers to find the initial reporter. | 149 | * We check AER status registers to find possible reporter. |
204 | */ | 150 | */ |
205 | if (atomic_read(&dev->enable_cnt) == 0) | 151 | if (atomic_read(&dev->enable_cnt) == 0) |
206 | return 0; | 152 | return false; |
207 | pos = pci_pcie_cap(dev); | 153 | pos = pci_pcie_cap(dev); |
208 | if (!pos) | 154 | if (!pos) |
209 | return 0; | 155 | return false; |
156 | |||
210 | /* Check if AER is enabled */ | 157 | /* Check if AER is enabled */ |
211 | pci_read_config_word(dev, pos+PCI_EXP_DEVCTL, ®16); | 158 | pci_read_config_word(dev, pos + PCI_EXP_DEVCTL, ®16); |
212 | if (!(reg16 & ( | 159 | if (!(reg16 & ( |
213 | PCI_EXP_DEVCTL_CERE | | 160 | PCI_EXP_DEVCTL_CERE | |
214 | PCI_EXP_DEVCTL_NFERE | | 161 | PCI_EXP_DEVCTL_NFERE | |
215 | PCI_EXP_DEVCTL_FERE | | 162 | PCI_EXP_DEVCTL_FERE | |
216 | PCI_EXP_DEVCTL_URRE))) | 163 | PCI_EXP_DEVCTL_URRE))) |
217 | return 0; | 164 | return false; |
218 | pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); | 165 | pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); |
219 | if (!pos) | 166 | if (!pos) |
220 | return 0; | 167 | return false; |
221 | 168 | ||
222 | status = 0; | 169 | /* Check if error is recorded */ |
223 | mask = 0; | ||
224 | if (e_info->severity == AER_CORRECTABLE) { | 170 | if (e_info->severity == AER_CORRECTABLE) { |
225 | pci_read_config_dword(dev, pos + PCI_ERR_COR_STATUS, &status); | 171 | pci_read_config_dword(dev, pos + PCI_ERR_COR_STATUS, &status); |
226 | pci_read_config_dword(dev, pos + PCI_ERR_COR_MASK, &mask); | 172 | pci_read_config_dword(dev, pos + PCI_ERR_COR_MASK, &mask); |
227 | if (status & ~mask) { | ||
228 | add_error_device(e_info, dev); | ||
229 | goto added; | ||
230 | } | ||
231 | } else { | 173 | } else { |
232 | pci_read_config_dword(dev, pos + PCI_ERR_UNCOR_STATUS, &status); | 174 | pci_read_config_dword(dev, pos + PCI_ERR_UNCOR_STATUS, &status); |
233 | pci_read_config_dword(dev, pos + PCI_ERR_UNCOR_MASK, &mask); | 175 | pci_read_config_dword(dev, pos + PCI_ERR_UNCOR_MASK, &mask); |
234 | if (status & ~mask) { | ||
235 | add_error_device(e_info, dev); | ||
236 | goto added; | ||
237 | } | ||
238 | } | 176 | } |
177 | if (status & ~mask) | ||
178 | return true; | ||
239 | 179 | ||
240 | return 0; | 180 | return false; |
181 | } | ||
241 | 182 | ||
242 | added: | 183 | static int find_device_iter(struct pci_dev *dev, void *data) |
243 | if (e_info->multi_error_valid) | 184 | { |
244 | return 0; | 185 | struct aer_err_info *e_info = (struct aer_err_info *)data; |
245 | else | 186 | |
246 | return 1; | 187 | if (is_error_source(dev, e_info)) { |
188 | /* List this device */ | ||
189 | if (add_error_device(e_info, dev)) { | ||
190 | /* We cannot handle more... Stop iteration */ | ||
191 | /* TODO: Should print error message here? */ | ||
192 | return 1; | ||
193 | } | ||
194 | |||
195 | /* If there is only a single error, stop iteration */ | ||
196 | if (!e_info->multi_error_valid) | ||
197 | return 1; | ||
198 | } | ||
199 | return 0; | ||
247 | } | 200 | } |
248 | 201 | ||
249 | /** | 202 | /** |
250 | * find_source_device - search through device hierarchy for source device | 203 | * find_source_device - search through device hierarchy for source device |
251 | * @parent: pointer to Root Port pci_dev data structure | 204 | * @parent: pointer to Root Port pci_dev data structure |
252 | * @err_info: including detailed error information such like id | 205 | * @e_info: including detailed error information such like id |
253 | * | 206 | * |
254 | * Invoked when error is detected at the Root Port. | 207 | * Return true if found. |
208 | * | ||
209 | * Invoked by DPC when error is detected at the Root Port. | ||
210 | * Caller of this function must set id, severity, and multi_error_valid of | ||
211 | * struct aer_err_info pointed by @e_info properly. This function must fill | ||
212 | * e_info->error_dev_num and e_info->dev[], based on the given information. | ||
255 | */ | 213 | */ |
256 | static void find_source_device(struct pci_dev *parent, | 214 | static bool find_source_device(struct pci_dev *parent, |
257 | struct aer_err_info *e_info) | 215 | struct aer_err_info *e_info) |
258 | { | 216 | { |
259 | struct pci_dev *dev = parent; | 217 | struct pci_dev *dev = parent; |
260 | int result; | 218 | int result; |
261 | 219 | ||
220 | /* Must reset in this function */ | ||
221 | e_info->error_dev_num = 0; | ||
222 | |||
262 | /* Is Root Port an agent that sends error message? */ | 223 | /* Is Root Port an agent that sends error message? */ |
263 | result = find_device_iter(dev, e_info); | 224 | result = find_device_iter(dev, e_info); |
264 | if (result) | 225 | if (result) |
265 | return; | 226 | return true; |
266 | 227 | ||
267 | pci_walk_bus(parent->subordinate, find_device_iter, e_info); | 228 | pci_walk_bus(parent->subordinate, find_device_iter, e_info); |
229 | |||
230 | if (!e_info->error_dev_num) { | ||
231 | dev_printk(KERN_DEBUG, &parent->dev, | ||
232 | "can't find device of ID%04x\n", | ||
233 | e_info->id); | ||
234 | return false; | ||
235 | } | ||
236 | return true; | ||
268 | } | 237 | } |
269 | 238 | ||
270 | static int report_error_detected(struct pci_dev *dev, void *data) | 239 | static int report_error_detected(struct pci_dev *dev, void *data) |
@@ -403,43 +372,77 @@ static pci_ers_result_t broadcast_error_message(struct pci_dev *dev, | |||
403 | return result_data.result; | 372 | return result_data.result; |
404 | } | 373 | } |
405 | 374 | ||
406 | struct find_aer_service_data { | 375 | /** |
407 | struct pcie_port_service_driver *aer_driver; | 376 | * aer_do_secondary_bus_reset - perform secondary bus reset |
408 | int is_downstream; | 377 | * @dev: pointer to bridge's pci_dev data structure |
409 | }; | 378 | * |
410 | 379 | * Invoked when performing link reset at Root Port or Downstream Port. | |
411 | static int find_aer_service_iter(struct device *device, void *data) | 380 | */ |
381 | void aer_do_secondary_bus_reset(struct pci_dev *dev) | ||
412 | { | 382 | { |
413 | struct device_driver *driver; | 383 | u16 p2p_ctrl; |
414 | struct pcie_port_service_driver *service_driver; | 384 | |
415 | struct find_aer_service_data *result; | 385 | /* Assert Secondary Bus Reset */ |
386 | pci_read_config_word(dev, PCI_BRIDGE_CONTROL, &p2p_ctrl); | ||
387 | p2p_ctrl |= PCI_BRIDGE_CTL_BUS_RESET; | ||
388 | pci_write_config_word(dev, PCI_BRIDGE_CONTROL, p2p_ctrl); | ||
389 | |||
390 | /* | ||
391 | * we should send hot reset message for 2ms to allow it time to | ||
392 | * propagate to all downstream ports | ||
393 | */ | ||
394 | msleep(2); | ||
395 | |||
396 | /* De-assert Secondary Bus Reset */ | ||
397 | p2p_ctrl &= ~PCI_BRIDGE_CTL_BUS_RESET; | ||
398 | pci_write_config_word(dev, PCI_BRIDGE_CONTROL, p2p_ctrl); | ||
399 | |||
400 | /* | ||
401 | * System software must wait for at least 100ms from the end | ||
402 | * of a reset of one or more device before it is permitted | ||
403 | * to issue Configuration Requests to those devices. | ||
404 | */ | ||
405 | msleep(200); | ||
406 | } | ||
416 | 407 | ||
417 | result = (struct find_aer_service_data *) data; | 408 | /** |
409 | * default_downstream_reset_link - default reset function for Downstream Port | ||
410 | * @dev: pointer to downstream port's pci_dev data structure | ||
411 | * | ||
412 | * Invoked when performing link reset at Downstream Port w/ no aer driver. | ||
413 | */ | ||
414 | static pci_ers_result_t default_downstream_reset_link(struct pci_dev *dev) | ||
415 | { | ||
416 | aer_do_secondary_bus_reset(dev); | ||
417 | dev_printk(KERN_DEBUG, &dev->dev, | ||
418 | "Downstream Port link has been reset\n"); | ||
419 | return PCI_ERS_RESULT_RECOVERED; | ||
420 | } | ||
418 | 421 | ||
419 | if (device->bus == &pcie_port_bus_type) { | 422 | static int find_aer_service_iter(struct device *device, void *data) |
420 | struct pcie_device *pcie = to_pcie_device(device); | 423 | { |
424 | struct pcie_port_service_driver *service_driver, **drv; | ||
421 | 425 | ||
422 | if (pcie->port->pcie_type == PCI_EXP_TYPE_DOWNSTREAM) | 426 | drv = (struct pcie_port_service_driver **) data; |
423 | result->is_downstream = 1; | ||
424 | 427 | ||
425 | driver = device->driver; | 428 | if (device->bus == &pcie_port_bus_type && device->driver) { |
426 | if (driver) { | 429 | service_driver = to_service_driver(device->driver); |
427 | service_driver = to_service_driver(driver); | 430 | if (service_driver->service == PCIE_PORT_SERVICE_AER) { |
428 | if (service_driver->service == PCIE_PORT_SERVICE_AER) { | 431 | *drv = service_driver; |
429 | result->aer_driver = service_driver; | 432 | return 1; |
430 | return 1; | ||
431 | } | ||
432 | } | 433 | } |
433 | } | 434 | } |
434 | 435 | ||
435 | return 0; | 436 | return 0; |
436 | } | 437 | } |
437 | 438 | ||
438 | static void find_aer_service(struct pci_dev *dev, | 439 | static struct pcie_port_service_driver *find_aer_service(struct pci_dev *dev) |
439 | struct find_aer_service_data *data) | ||
440 | { | 440 | { |
441 | int retval; | 441 | struct pcie_port_service_driver *drv = NULL; |
442 | retval = device_for_each_child(&dev->dev, data, find_aer_service_iter); | 442 | |
443 | device_for_each_child(&dev->dev, &drv, find_aer_service_iter); | ||
444 | |||
445 | return drv; | ||
443 | } | 446 | } |
444 | 447 | ||
445 | static pci_ers_result_t reset_link(struct pcie_device *aerdev, | 448 | static pci_ers_result_t reset_link(struct pcie_device *aerdev, |
@@ -447,38 +450,34 @@ static pci_ers_result_t reset_link(struct pcie_device *aerdev, | |||
447 | { | 450 | { |
448 | struct pci_dev *udev; | 451 | struct pci_dev *udev; |
449 | pci_ers_result_t status; | 452 | pci_ers_result_t status; |
450 | struct find_aer_service_data data; | 453 | struct pcie_port_service_driver *driver; |
451 | 454 | ||
452 | if (dev->hdr_type & PCI_HEADER_TYPE_BRIDGE) | 455 | if (dev->hdr_type & PCI_HEADER_TYPE_BRIDGE) { |
456 | /* Reset this port for all subordinates */ | ||
453 | udev = dev; | 457 | udev = dev; |
454 | else | 458 | } else { |
459 | /* Reset the upstream component (likely downstream port) */ | ||
455 | udev = dev->bus->self; | 460 | udev = dev->bus->self; |
461 | } | ||
456 | 462 | ||
457 | data.is_downstream = 0; | 463 | /* Use the aer driver of the component firstly */ |
458 | data.aer_driver = NULL; | 464 | driver = find_aer_service(udev); |
459 | find_aer_service(udev, &data); | ||
460 | 465 | ||
461 | /* | 466 | if (driver && driver->reset_link) { |
462 | * Use the aer driver of the error agent firstly. | 467 | status = driver->reset_link(udev); |
463 | * If it hasn't the aer driver, use the root port's | 468 | } else if (udev->pcie_type == PCI_EXP_TYPE_DOWNSTREAM) { |
464 | */ | 469 | status = default_downstream_reset_link(udev); |
465 | if (!data.aer_driver || !data.aer_driver->reset_link) { | 470 | } else { |
466 | if (data.is_downstream && | 471 | dev_printk(KERN_DEBUG, &dev->dev, |
467 | aerdev->device.driver && | 472 | "no link-reset support at upstream device %s\n", |
468 | to_service_driver(aerdev->device.driver)->reset_link) { | 473 | pci_name(udev)); |
469 | data.aer_driver = | 474 | return PCI_ERS_RESULT_DISCONNECT; |
470 | to_service_driver(aerdev->device.driver); | ||
471 | } else { | ||
472 | dev_printk(KERN_DEBUG, &dev->dev, "no link-reset " | ||
473 | "support\n"); | ||
474 | return PCI_ERS_RESULT_DISCONNECT; | ||
475 | } | ||
476 | } | 475 | } |
477 | 476 | ||
478 | status = data.aer_driver->reset_link(udev); | ||
479 | if (status != PCI_ERS_RESULT_RECOVERED) { | 477 | if (status != PCI_ERS_RESULT_RECOVERED) { |
480 | dev_printk(KERN_DEBUG, &dev->dev, "link reset at upstream " | 478 | dev_printk(KERN_DEBUG, &dev->dev, |
481 | "device %s failed\n", pci_name(udev)); | 479 | "link reset at upstream device %s failed\n", |
480 | pci_name(udev)); | ||
482 | return PCI_ERS_RESULT_DISCONNECT; | 481 | return PCI_ERS_RESULT_DISCONNECT; |
483 | } | 482 | } |
484 | 483 | ||
@@ -495,8 +494,7 @@ static pci_ers_result_t reset_link(struct pcie_device *aerdev, | |||
495 | * error detected message to all downstream drivers within a hierarchy in | 494 | * error detected message to all downstream drivers within a hierarchy in |
496 | * question and return the returned code. | 495 | * question and return the returned code. |
497 | */ | 496 | */ |
498 | static pci_ers_result_t do_recovery(struct pcie_device *aerdev, | 497 | static void do_recovery(struct pcie_device *aerdev, struct pci_dev *dev, |
499 | struct pci_dev *dev, | ||
500 | int severity) | 498 | int severity) |
501 | { | 499 | { |
502 | pci_ers_result_t status, result = PCI_ERS_RESULT_RECOVERED; | 500 | pci_ers_result_t status, result = PCI_ERS_RESULT_RECOVERED; |
@@ -514,10 +512,8 @@ static pci_ers_result_t do_recovery(struct pcie_device *aerdev, | |||
514 | 512 | ||
515 | if (severity == AER_FATAL) { | 513 | if (severity == AER_FATAL) { |
516 | result = reset_link(aerdev, dev); | 514 | result = reset_link(aerdev, dev); |
517 | if (result != PCI_ERS_RESULT_RECOVERED) { | 515 | if (result != PCI_ERS_RESULT_RECOVERED) |
518 | /* TODO: Should panic here? */ | 516 | goto failed; |
519 | return result; | ||
520 | } | ||
521 | } | 517 | } |
522 | 518 | ||
523 | if (status == PCI_ERS_RESULT_CAN_RECOVER) | 519 | if (status == PCI_ERS_RESULT_CAN_RECOVER) |
@@ -538,13 +534,22 @@ static pci_ers_result_t do_recovery(struct pcie_device *aerdev, | |||
538 | report_slot_reset); | 534 | report_slot_reset); |
539 | } | 535 | } |
540 | 536 | ||
541 | if (status == PCI_ERS_RESULT_RECOVERED) | 537 | if (status != PCI_ERS_RESULT_RECOVERED) |
542 | broadcast_error_message(dev, | 538 | goto failed; |
539 | |||
540 | broadcast_error_message(dev, | ||
543 | state, | 541 | state, |
544 | "resume", | 542 | "resume", |
545 | report_resume); | 543 | report_resume); |
546 | 544 | ||
547 | return status; | 545 | dev_printk(KERN_DEBUG, &dev->dev, |
546 | "AER driver successfully recovered\n"); | ||
547 | return; | ||
548 | |||
549 | failed: | ||
550 | /* TODO: Should kernel panic here? */ | ||
551 | dev_printk(KERN_DEBUG, &dev->dev, | ||
552 | "AER driver didn't recover\n"); | ||
548 | } | 553 | } |
549 | 554 | ||
550 | /** | 555 | /** |
@@ -559,7 +564,6 @@ static void handle_error_source(struct pcie_device *aerdev, | |||
559 | struct pci_dev *dev, | 564 | struct pci_dev *dev, |
560 | struct aer_err_info *info) | 565 | struct aer_err_info *info) |
561 | { | 566 | { |
562 | pci_ers_result_t status = 0; | ||
563 | int pos; | 567 | int pos; |
564 | 568 | ||
565 | if (info->severity == AER_CORRECTABLE) { | 569 | if (info->severity == AER_CORRECTABLE) { |
@@ -571,114 +575,8 @@ static void handle_error_source(struct pcie_device *aerdev, | |||
571 | if (pos) | 575 | if (pos) |
572 | pci_write_config_dword(dev, pos + PCI_ERR_COR_STATUS, | 576 | pci_write_config_dword(dev, pos + PCI_ERR_COR_STATUS, |
573 | info->status); | 577 | info->status); |
574 | } else { | 578 | } else |
575 | status = do_recovery(aerdev, dev, info->severity); | 579 | do_recovery(aerdev, dev, info->severity); |
576 | if (status == PCI_ERS_RESULT_RECOVERED) { | ||
577 | dev_printk(KERN_DEBUG, &dev->dev, "AER driver " | ||
578 | "successfully recovered\n"); | ||
579 | } else { | ||
580 | /* TODO: Should kernel panic here? */ | ||
581 | dev_printk(KERN_DEBUG, &dev->dev, "AER driver didn't " | ||
582 | "recover\n"); | ||
583 | } | ||
584 | } | ||
585 | } | ||
586 | |||
587 | /** | ||
588 | * aer_enable_rootport - enable Root Port's interrupts when receiving messages | ||
589 | * @rpc: pointer to a Root Port data structure | ||
590 | * | ||
591 | * Invoked when PCIe bus loads AER service driver. | ||
592 | */ | ||
593 | void aer_enable_rootport(struct aer_rpc *rpc) | ||
594 | { | ||
595 | struct pci_dev *pdev = rpc->rpd->port; | ||
596 | int pos, aer_pos; | ||
597 | u16 reg16; | ||
598 | u32 reg32; | ||
599 | |||
600 | pos = pci_pcie_cap(pdev); | ||
601 | /* Clear PCIe Capability's Device Status */ | ||
602 | pci_read_config_word(pdev, pos+PCI_EXP_DEVSTA, ®16); | ||
603 | pci_write_config_word(pdev, pos+PCI_EXP_DEVSTA, reg16); | ||
604 | |||
605 | /* Disable system error generation in response to error messages */ | ||
606 | pci_read_config_word(pdev, pos + PCI_EXP_RTCTL, ®16); | ||
607 | reg16 &= ~(SYSTEM_ERROR_INTR_ON_MESG_MASK); | ||
608 | pci_write_config_word(pdev, pos + PCI_EXP_RTCTL, reg16); | ||
609 | |||
610 | aer_pos = pci_find_ext_capability(pdev, PCI_EXT_CAP_ID_ERR); | ||
611 | /* Clear error status */ | ||
612 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_ROOT_STATUS, ®32); | ||
613 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_ROOT_STATUS, reg32); | ||
614 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_COR_STATUS, ®32); | ||
615 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_COR_STATUS, reg32); | ||
616 | pci_read_config_dword(pdev, aer_pos + PCI_ERR_UNCOR_STATUS, ®32); | ||
617 | pci_write_config_dword(pdev, aer_pos + PCI_ERR_UNCOR_STATUS, reg32); | ||
618 | |||
619 | /* | ||
620 | * Enable error reporting for the root port device and downstream port | ||
621 | * devices. | ||
622 | */ | ||
623 | set_downstream_devices_error_reporting(pdev, true); | ||
624 | |||
625 | /* Enable Root Port's interrupt in response to error messages */ | ||
626 | pci_write_config_dword(pdev, | ||
627 | aer_pos + PCI_ERR_ROOT_COMMAND, | ||
628 | ROOT_PORT_INTR_ON_MESG_MASK); | ||
629 | } | ||
630 | |||
631 | /** | ||
632 | * disable_root_aer - disable Root Port's interrupts when receiving messages | ||
633 | * @rpc: pointer to a Root Port data structure | ||
634 | * | ||
635 | * Invoked when PCIe bus unloads AER service driver. | ||
636 | */ | ||
637 | static void disable_root_aer(struct aer_rpc *rpc) | ||
638 | { | ||
639 | struct pci_dev *pdev = rpc->rpd->port; | ||
640 | u32 reg32; | ||
641 | int pos; | ||
642 | |||
643 | /* | ||
644 | * Disable error reporting for the root port device and downstream port | ||
645 | * devices. | ||
646 | */ | ||
647 | set_downstream_devices_error_reporting(pdev, false); | ||
648 | |||
649 | pos = pci_find_ext_capability(pdev, PCI_EXT_CAP_ID_ERR); | ||
650 | /* Disable Root's interrupt in response to error messages */ | ||
651 | pci_write_config_dword(pdev, pos + PCI_ERR_ROOT_COMMAND, 0); | ||
652 | |||
653 | /* Clear Root's error status reg */ | ||
654 | pci_read_config_dword(pdev, pos + PCI_ERR_ROOT_STATUS, ®32); | ||
655 | pci_write_config_dword(pdev, pos + PCI_ERR_ROOT_STATUS, reg32); | ||
656 | } | ||
657 | |||
658 | /** | ||
659 | * get_e_source - retrieve an error source | ||
660 | * @rpc: pointer to the root port which holds an error | ||
661 | * | ||
662 | * Invoked by DPC handler to consume an error. | ||
663 | */ | ||
664 | static struct aer_err_source *get_e_source(struct aer_rpc *rpc) | ||
665 | { | ||
666 | struct aer_err_source *e_source; | ||
667 | unsigned long flags; | ||
668 | |||
669 | /* Lock access to Root error producer/consumer index */ | ||
670 | spin_lock_irqsave(&rpc->e_lock, flags); | ||
671 | if (rpc->prod_idx == rpc->cons_idx) { | ||
672 | spin_unlock_irqrestore(&rpc->e_lock, flags); | ||
673 | return NULL; | ||
674 | } | ||
675 | e_source = &rpc->e_sources[rpc->cons_idx]; | ||
676 | rpc->cons_idx++; | ||
677 | if (rpc->cons_idx == AER_ERROR_SOURCES_MAX) | ||
678 | rpc->cons_idx = 0; | ||
679 | spin_unlock_irqrestore(&rpc->e_lock, flags); | ||
680 | |||
681 | return e_source; | ||
682 | } | 580 | } |
683 | 581 | ||
684 | /** | 582 | /** |
@@ -687,11 +585,14 @@ static struct aer_err_source *get_e_source(struct aer_rpc *rpc) | |||
687 | * @info: pointer to structure to store the error record | 585 | * @info: pointer to structure to store the error record |
688 | * | 586 | * |
689 | * Return 1 on success, 0 on error. | 587 | * Return 1 on success, 0 on error. |
588 | * | ||
589 | * Note that @info is reused among all error devices. Clear fields properly. | ||
690 | */ | 590 | */ |
691 | static int get_device_error_info(struct pci_dev *dev, struct aer_err_info *info) | 591 | static int get_device_error_info(struct pci_dev *dev, struct aer_err_info *info) |
692 | { | 592 | { |
693 | int pos, temp; | 593 | int pos, temp; |
694 | 594 | ||
595 | /* Must reset in this function */ | ||
695 | info->status = 0; | 596 | info->status = 0; |
696 | info->tlp_header_valid = 0; | 597 | info->tlp_header_valid = 0; |
697 | 598 | ||
@@ -744,12 +645,6 @@ static inline void aer_process_err_devices(struct pcie_device *p_device, | |||
744 | { | 645 | { |
745 | int i; | 646 | int i; |
746 | 647 | ||
747 | if (!e_info->dev[0]) { | ||
748 | dev_printk(KERN_DEBUG, &p_device->port->dev, | ||
749 | "can't find device of ID%04x\n", | ||
750 | e_info->id); | ||
751 | } | ||
752 | |||
753 | /* Report all before handle them, not to lost records by reset etc. */ | 648 | /* Report all before handle them, not to lost records by reset etc. */ |
754 | for (i = 0; i < e_info->error_dev_num && e_info->dev[i]; i++) { | 649 | for (i = 0; i < e_info->error_dev_num && e_info->dev[i]; i++) { |
755 | if (get_device_error_info(e_info->dev[i], e_info)) | 650 | if (get_device_error_info(e_info->dev[i], e_info)) |
@@ -770,11 +665,10 @@ static void aer_isr_one_error(struct pcie_device *p_device, | |||
770 | struct aer_err_source *e_src) | 665 | struct aer_err_source *e_src) |
771 | { | 666 | { |
772 | struct aer_err_info *e_info; | 667 | struct aer_err_info *e_info; |
773 | int i; | ||
774 | 668 | ||
775 | /* struct aer_err_info might be big, so we allocate it with slab */ | 669 | /* struct aer_err_info might be big, so we allocate it with slab */ |
776 | e_info = kmalloc(sizeof(struct aer_err_info), GFP_KERNEL); | 670 | e_info = kmalloc(sizeof(struct aer_err_info), GFP_KERNEL); |
777 | if (e_info == NULL) { | 671 | if (!e_info) { |
778 | dev_printk(KERN_DEBUG, &p_device->port->dev, | 672 | dev_printk(KERN_DEBUG, &p_device->port->dev, |
779 | "Can't allocate mem when processing AER errors\n"); | 673 | "Can't allocate mem when processing AER errors\n"); |
780 | return; | 674 | return; |
@@ -784,37 +678,72 @@ static void aer_isr_one_error(struct pcie_device *p_device, | |||
784 | * There is a possibility that both correctable error and | 678 | * There is a possibility that both correctable error and |
785 | * uncorrectable error being logged. Report correctable error first. | 679 | * uncorrectable error being logged. Report correctable error first. |
786 | */ | 680 | */ |
787 | for (i = 1; i & ROOT_ERR_STATUS_MASKS ; i <<= 2) { | 681 | if (e_src->status & PCI_ERR_ROOT_COR_RCV) { |
788 | if (i > 4) | 682 | e_info->id = ERR_COR_ID(e_src->id); |
789 | break; | 683 | e_info->severity = AER_CORRECTABLE; |
790 | if (!(e_src->status & i)) | 684 | |
791 | continue; | 685 | if (e_src->status & PCI_ERR_ROOT_MULTI_COR_RCV) |
792 | |||
793 | memset(e_info, 0, sizeof(struct aer_err_info)); | ||
794 | |||
795 | /* Init comprehensive error information */ | ||
796 | if (i & PCI_ERR_ROOT_COR_RCV) { | ||
797 | e_info->id = ERR_COR_ID(e_src->id); | ||
798 | e_info->severity = AER_CORRECTABLE; | ||
799 | } else { | ||
800 | e_info->id = ERR_UNCOR_ID(e_src->id); | ||
801 | e_info->severity = ((e_src->status >> 6) & 1); | ||
802 | } | ||
803 | if (e_src->status & | ||
804 | (PCI_ERR_ROOT_MULTI_COR_RCV | | ||
805 | PCI_ERR_ROOT_MULTI_UNCOR_RCV)) | ||
806 | e_info->multi_error_valid = 1; | 686 | e_info->multi_error_valid = 1; |
687 | else | ||
688 | e_info->multi_error_valid = 0; | ||
689 | |||
690 | aer_print_port_info(p_device->port, e_info); | ||
691 | |||
692 | if (find_source_device(p_device->port, e_info)) | ||
693 | aer_process_err_devices(p_device, e_info); | ||
694 | } | ||
695 | |||
696 | if (e_src->status & PCI_ERR_ROOT_UNCOR_RCV) { | ||
697 | e_info->id = ERR_UNCOR_ID(e_src->id); | ||
698 | |||
699 | if (e_src->status & PCI_ERR_ROOT_FATAL_RCV) | ||
700 | e_info->severity = AER_FATAL; | ||
701 | else | ||
702 | e_info->severity = AER_NONFATAL; | ||
703 | |||
704 | if (e_src->status & PCI_ERR_ROOT_MULTI_UNCOR_RCV) | ||
705 | e_info->multi_error_valid = 1; | ||
706 | else | ||
707 | e_info->multi_error_valid = 0; | ||
807 | 708 | ||
808 | aer_print_port_info(p_device->port, e_info); | 709 | aer_print_port_info(p_device->port, e_info); |
809 | 710 | ||
810 | find_source_device(p_device->port, e_info); | 711 | if (find_source_device(p_device->port, e_info)) |
811 | aer_process_err_devices(p_device, e_info); | 712 | aer_process_err_devices(p_device, e_info); |
812 | } | 713 | } |
813 | 714 | ||
814 | kfree(e_info); | 715 | kfree(e_info); |
815 | } | 716 | } |
816 | 717 | ||
817 | /** | 718 | /** |
719 | * get_e_source - retrieve an error source | ||
720 | * @rpc: pointer to the root port which holds an error | ||
721 | * @e_src: pointer to store retrieved error source | ||
722 | * | ||
723 | * Return 1 if an error source is retrieved, otherwise 0. | ||
724 | * | ||
725 | * Invoked by DPC handler to consume an error. | ||
726 | */ | ||
727 | static int get_e_source(struct aer_rpc *rpc, struct aer_err_source *e_src) | ||
728 | { | ||
729 | unsigned long flags; | ||
730 | int ret = 0; | ||
731 | |||
732 | /* Lock access to Root error producer/consumer index */ | ||
733 | spin_lock_irqsave(&rpc->e_lock, flags); | ||
734 | if (rpc->prod_idx != rpc->cons_idx) { | ||
735 | *e_src = rpc->e_sources[rpc->cons_idx]; | ||
736 | rpc->cons_idx++; | ||
737 | if (rpc->cons_idx == AER_ERROR_SOURCES_MAX) | ||
738 | rpc->cons_idx = 0; | ||
739 | ret = 1; | ||
740 | } | ||
741 | spin_unlock_irqrestore(&rpc->e_lock, flags); | ||
742 | |||
743 | return ret; | ||
744 | } | ||
745 | |||
746 | /** | ||
818 | * aer_isr - consume errors detected by root port | 747 | * aer_isr - consume errors detected by root port |
819 | * @work: definition of this work item | 748 | * @work: definition of this work item |
820 | * | 749 | * |
@@ -824,34 +753,17 @@ void aer_isr(struct work_struct *work) | |||
824 | { | 753 | { |
825 | struct aer_rpc *rpc = container_of(work, struct aer_rpc, dpc_handler); | 754 | struct aer_rpc *rpc = container_of(work, struct aer_rpc, dpc_handler); |
826 | struct pcie_device *p_device = rpc->rpd; | 755 | struct pcie_device *p_device = rpc->rpd; |
827 | struct aer_err_source *e_src; | 756 | struct aer_err_source e_src; |
828 | 757 | ||
829 | mutex_lock(&rpc->rpc_mutex); | 758 | mutex_lock(&rpc->rpc_mutex); |
830 | e_src = get_e_source(rpc); | 759 | while (get_e_source(rpc, &e_src)) |
831 | while (e_src) { | 760 | aer_isr_one_error(p_device, &e_src); |
832 | aer_isr_one_error(p_device, e_src); | ||
833 | e_src = get_e_source(rpc); | ||
834 | } | ||
835 | mutex_unlock(&rpc->rpc_mutex); | 761 | mutex_unlock(&rpc->rpc_mutex); |
836 | 762 | ||
837 | wake_up(&rpc->wait_release); | 763 | wake_up(&rpc->wait_release); |
838 | } | 764 | } |
839 | 765 | ||
840 | /** | 766 | /** |
841 | * aer_delete_rootport - disable root port aer and delete service data | ||
842 | * @rpc: pointer to a root port device being deleted | ||
843 | * | ||
844 | * Invoked when AER service unloaded on a specific Root Port | ||
845 | */ | ||
846 | void aer_delete_rootport(struct aer_rpc *rpc) | ||
847 | { | ||
848 | /* Disable root port AER itself */ | ||
849 | disable_root_aer(rpc); | ||
850 | |||
851 | kfree(rpc); | ||
852 | } | ||
853 | |||
854 | /** | ||
855 | * aer_init - provide AER initialization | 767 | * aer_init - provide AER initialization |
856 | * @dev: pointer to AER pcie device | 768 | * @dev: pointer to AER pcie device |
857 | * | 769 | * |
diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 27c0e6eb7136..b7512cf08c58 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c | |||
@@ -2127,6 +2127,7 @@ DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AMD, 0x9602, quirk_disable_msi); | |||
2127 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASUSTEK, 0x9602, quirk_disable_msi); | 2127 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASUSTEK, 0x9602, quirk_disable_msi); |
2128 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AI, 0x9602, quirk_disable_msi); | 2128 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_AI, 0x9602, quirk_disable_msi); |
2129 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_VIA, 0xa238, quirk_disable_msi); | 2129 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_VIA, 0xa238, quirk_disable_msi); |
2130 | DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, 0x5a3f, quirk_disable_msi); | ||
2130 | 2131 | ||
2131 | /* Go through the list of Hypertransport capabilities and | 2132 | /* Go through the list of Hypertransport capabilities and |
2132 | * return 1 if a HT MSI capability is found and enabled */ | 2133 | * return 1 if a HT MSI capability is found and enabled */ |
@@ -2218,15 +2219,16 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_SERVERWORKS, | |||
2218 | DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_8132_BRIDGE, | 2219 | DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_8132_BRIDGE, |
2219 | ht_enable_msi_mapping); | 2220 | ht_enable_msi_mapping); |
2220 | 2221 | ||
2221 | /* The P5N32-SLI Premium motherboard from Asus has a problem with msi | 2222 | /* The P5N32-SLI motherboards from Asus have a problem with msi |
2222 | * for the MCP55 NIC. It is not yet determined whether the msi problem | 2223 | * for the MCP55 NIC. It is not yet determined whether the msi problem |
2223 | * also affects other devices. As for now, turn off msi for this device. | 2224 | * also affects other devices. As for now, turn off msi for this device. |
2224 | */ | 2225 | */ |
2225 | static void __devinit nvenet_msi_disable(struct pci_dev *dev) | 2226 | static void __devinit nvenet_msi_disable(struct pci_dev *dev) |
2226 | { | 2227 | { |
2227 | if (dmi_name_in_vendors("P5N32-SLI PREMIUM")) { | 2228 | if (dmi_name_in_vendors("P5N32-SLI PREMIUM") || |
2229 | dmi_name_in_vendors("P5N32-E SLI")) { | ||
2228 | dev_info(&dev->dev, | 2230 | dev_info(&dev->dev, |
2229 | "Disabling msi for MCP55 NIC on P5N32-SLI Premium\n"); | 2231 | "Disabling msi for MCP55 NIC on P5N32-SLI\n"); |
2230 | dev->no_msi = 1; | 2232 | dev->no_msi = 1; |
2231 | } | 2233 | } |
2232 | } | 2234 | } |
@@ -2552,6 +2554,19 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_INTEL, 0x1518, quirk_i82576_sriov); | |||
2552 | 2554 | ||
2553 | #endif /* CONFIG_PCI_IOV */ | 2555 | #endif /* CONFIG_PCI_IOV */ |
2554 | 2556 | ||
2557 | /* Allow manual resource allocation for PCI hotplug bridges | ||
2558 | * via pci=hpmemsize=nnM and pci=hpiosize=nnM parameters. For | ||
2559 | * some PCI-PCI hotplug bridges, like PLX 6254 (former HINT HB6), | ||
2560 | * kernel fails to allocate resources when hotplug device is | ||
2561 | * inserted and PCI bus is rescanned. | ||
2562 | */ | ||
2563 | static void __devinit quirk_hotplug_bridge(struct pci_dev *dev) | ||
2564 | { | ||
2565 | dev->is_hotplug_bridge = 1; | ||
2566 | } | ||
2567 | |||
2568 | DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_HINT, 0x0020, quirk_hotplug_bridge); | ||
2569 | |||
2555 | /* | 2570 | /* |
2556 | * This is a quirk for the Ricoh MMC controller found as a part of | 2571 | * This is a quirk for the Ricoh MMC controller found as a part of |
2557 | * some mulifunction chips. | 2572 | * some mulifunction chips. |
diff --git a/drivers/pci/slot.c b/drivers/pci/slot.c index 659eaa0fc48f..e0189cf7c558 100644 --- a/drivers/pci/slot.c +++ b/drivers/pci/slot.c | |||
@@ -97,6 +97,50 @@ static ssize_t cur_speed_read_file(struct pci_slot *slot, char *buf) | |||
97 | return bus_speed_read(slot->bus->cur_bus_speed, buf); | 97 | return bus_speed_read(slot->bus->cur_bus_speed, buf); |
98 | } | 98 | } |
99 | 99 | ||
100 | static void remove_sysfs_files(struct pci_slot *slot) | ||
101 | { | ||
102 | char func[10]; | ||
103 | struct list_head *tmp; | ||
104 | |||
105 | list_for_each(tmp, &slot->bus->devices) { | ||
106 | struct pci_dev *dev = pci_dev_b(tmp); | ||
107 | if (PCI_SLOT(dev->devfn) != slot->number) | ||
108 | continue; | ||
109 | sysfs_remove_link(&dev->dev.kobj, "slot"); | ||
110 | |||
111 | snprintf(func, 10, "function%d", PCI_FUNC(dev->devfn)); | ||
112 | sysfs_remove_link(&slot->kobj, func); | ||
113 | } | ||
114 | } | ||
115 | |||
116 | static int create_sysfs_files(struct pci_slot *slot) | ||
117 | { | ||
118 | int result; | ||
119 | char func[10]; | ||
120 | struct list_head *tmp; | ||
121 | |||
122 | list_for_each(tmp, &slot->bus->devices) { | ||
123 | struct pci_dev *dev = pci_dev_b(tmp); | ||
124 | if (PCI_SLOT(dev->devfn) != slot->number) | ||
125 | continue; | ||
126 | |||
127 | result = sysfs_create_link(&dev->dev.kobj, &slot->kobj, "slot"); | ||
128 | if (result) | ||
129 | goto fail; | ||
130 | |||
131 | snprintf(func, 10, "function%d", PCI_FUNC(dev->devfn)); | ||
132 | result = sysfs_create_link(&slot->kobj, &dev->dev.kobj, func); | ||
133 | if (result) | ||
134 | goto fail; | ||
135 | } | ||
136 | |||
137 | return 0; | ||
138 | |||
139 | fail: | ||
140 | remove_sysfs_files(slot); | ||
141 | return result; | ||
142 | } | ||
143 | |||
100 | static void pci_slot_release(struct kobject *kobj) | 144 | static void pci_slot_release(struct kobject *kobj) |
101 | { | 145 | { |
102 | struct pci_dev *dev; | 146 | struct pci_dev *dev; |
@@ -109,6 +153,8 @@ static void pci_slot_release(struct kobject *kobj) | |||
109 | if (PCI_SLOT(dev->devfn) == slot->number) | 153 | if (PCI_SLOT(dev->devfn) == slot->number) |
110 | dev->slot = NULL; | 154 | dev->slot = NULL; |
111 | 155 | ||
156 | remove_sysfs_files(slot); | ||
157 | |||
112 | list_del(&slot->list); | 158 | list_del(&slot->list); |
113 | 159 | ||
114 | kfree(slot); | 160 | kfree(slot); |
@@ -300,6 +346,8 @@ placeholder: | |||
300 | INIT_LIST_HEAD(&slot->list); | 346 | INIT_LIST_HEAD(&slot->list); |
301 | list_add(&slot->list, &parent->slots); | 347 | list_add(&slot->list, &parent->slots); |
302 | 348 | ||
349 | create_sysfs_files(slot); | ||
350 | |||
303 | list_for_each_entry(dev, &parent->devices, bus_list) | 351 | list_for_each_entry(dev, &parent->devices, bus_list) |
304 | if (PCI_SLOT(dev->devfn) == slot_nr) | 352 | if (PCI_SLOT(dev->devfn) == slot_nr) |
305 | dev->slot = slot; | 353 | dev->slot = slot; |