| Message ID | 2622847.3aWjiMW5oK@vostro.rjw.lan (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Thu, Jan 30, 2014 at 4:38 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > On Friday, January 31, 2014 12:59:06 AM Rafael J. Wysocki wrote: >> On Thursday, January 30, 2014 03:39:02 PM Yinghai Lu wrote: >> > On Thu, Jan 30, 2014 at 3:39 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: >> > > On Thursday, January 30, 2014 08:56:05 AM Yinghai Lu wrote: >> > >> >> > >> --047d7b5d2ea4eb937804f132eedf >> > >> Content-Type: text/plain; charset=ISO-8859-1 >> > >> >> > >> >> The latest mainline kernel "hangs" when Thunderbolt devices are >> > >> >> hot-unplugged to the system. I can't see any oops but after hot-unplug I'm >> > >> >> getting huge amounts of messages like: >> > >> >> >> > >> >> [ 352.717001] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717011] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717021] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717032] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717041] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717051] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717061] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717070] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717083] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717094] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717104] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717113] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717124] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717133] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717143] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717153] pci 0000:02:00.0: PME# disabled >> > >> >> [ 352.717162] pci 0000:02:00.0: PME# disabled >> > >> > >> > >> > that mean pci_stop_dev() get called again and again ? >> > >> >> > >> please check if attached patch could help. >> > > >> > > Well, it looks like what happens is an endless loop in >> > > acpiphp_glue.c:disable_slot(). >> > > >> > > dev_in_slot() returns the first device in the list, so >> > > pci_stop_and_remove_bus_device() is called for it, but it >> > > doesn't remove the device from bus->devices any more, so >> > > dev_in_slot() will return the same device next time and >> > > so on forever. >> > > >> > ... >> > > >> > > So the above won't help in my opinion. >> > > >> > > I wonder, however, if this patch helps instead: >> > > >> > > https://patchwork.kernel.org/patch/3540701/ >> > > >> > > I thought it would be 3.15 material, but it very well can go in earlier if >> > > it happens to address this particular problem. >> > >> > Agree, that should fix the problem. >> > >> > but please use list_for_each_entry_safe_reverse >> > instead. >> >> OK, I will. > > Mika, below is an updated patch to try. > > --- > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > Subject: ACPI / hotplug / PCI: Simplify disable_slot() > > After recent PCI core changes related to the rescan/remove locking, > the ACPIPHP's disable_slot() function is only called under the > general PCI rescan/remove lock, so it doesn't have to use > dev_in_slot() any more to avoid race conditions. Make it simply > walk the devices on the bus and drop the ones in the slot being > disabled and drop dev_in_slot() which has no more users. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > --- > drivers/pci/hotplug/acpiphp_glue.c | 28 +++++----------------------- > 1 file changed, 5 insertions(+), 23 deletions(-) > > Index: linux-pm/drivers/pci/hotplug/acpiphp_glue.c > =================================================================== > --- linux-pm.orig/drivers/pci/hotplug/acpiphp_glue.c > +++ linux-pm/drivers/pci/hotplug/acpiphp_glue.c > @@ -604,32 +604,15 @@ static void __ref enable_slot(struct acp > } > } > > -/* return first device in slot, acquiring a reference on it */ > -static struct pci_dev *dev_in_slot(struct acpiphp_slot *slot) > -{ > - struct pci_bus *bus = slot->bus; > - struct pci_dev *dev; > - struct pci_dev *ret = NULL; > - > - down_read(&pci_bus_sem); > - list_for_each_entry(dev, &bus->devices, bus_list) > - if (PCI_SLOT(dev->devfn) == slot->device) { > - ret = pci_dev_get(dev); > - break; > - } > - up_read(&pci_bus_sem); > - > - return ret; > -} > - > /** > * disable_slot - disable a slot > * @slot: ACPI PHP slot > */ > static void disable_slot(struct acpiphp_slot *slot) > { > + struct pci_bus *bus = slot->bus; > + struct pci_dev *dev, *prev; > struct acpiphp_func *func; > - struct pci_dev *pdev; > > /* > * enable_slot() enumerates all functions in this device via > @@ -637,10 +620,9 @@ static void disable_slot(struct acpiphp_ > * methods (_EJ0, etc.) or not. Therefore, we remove all functions > * here. > */ > - while ((pdev = dev_in_slot(slot))) { > - pci_stop_and_remove_bus_device(pdev); > - pci_dev_put(pdev); > - } > + list_for_each_entry_safe_reverse(dev, prev, &bus->devices, bus_list) > + if (PCI_SLOT(dev->devfn) == slot->device) > + pci_stop_and_remove_bus_device(dev); > > list_for_each_entry(func, &slot->funcs, sibling) > acpiphp_bus_trim(func_to_handle(func)); > Acked-by: Yinghai Lu <yinghai@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Jan 31, 2014 at 01:38:42AM +0100, Rafael J. Wysocki wrote: > On Friday, January 31, 2014 12:59:06 AM Rafael J. Wysocki wrote: > > On Thursday, January 30, 2014 03:39:02 PM Yinghai Lu wrote: > > > On Thu, Jan 30, 2014 at 3:39 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > > > > On Thursday, January 30, 2014 08:56:05 AM Yinghai Lu wrote: > > > >> > > > >> --047d7b5d2ea4eb937804f132eedf > > > >> Content-Type: text/plain; charset=ISO-8859-1 > > > >> > > > >> >> The latest mainline kernel "hangs" when Thunderbolt devices are > > > >> >> hot-unplugged to the system. I can't see any oops but after hot-unplug I'm > > > >> >> getting huge amounts of messages like: > > > >> >> > > > >> >> [ 352.717001] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717011] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717021] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717032] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717041] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717051] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717061] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717070] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717083] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717094] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717104] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717113] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717124] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717133] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717143] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717153] pci 0000:02:00.0: PME# disabled > > > >> >> [ 352.717162] pci 0000:02:00.0: PME# disabled > > > >> > > > > >> > that mean pci_stop_dev() get called again and again ? > > > >> > > > >> please check if attached patch could help. > > > > > > > > Well, it looks like what happens is an endless loop in > > > > acpiphp_glue.c:disable_slot(). > > > > > > > > dev_in_slot() returns the first device in the list, so > > > > pci_stop_and_remove_bus_device() is called for it, but it > > > > doesn't remove the device from bus->devices any more, so > > > > dev_in_slot() will return the same device next time and > > > > so on forever. > > > > > > > ... > > > > > > > > So the above won't help in my opinion. > > > > > > > > I wonder, however, if this patch helps instead: > > > > > > > > https://patchwork.kernel.org/patch/3540701/ > > > > > > > > I thought it would be 3.15 material, but it very well can go in earlier if > > > > it happens to address this particular problem. > > > > > > Agree, that should fix the problem. > > > > > > but please use list_for_each_entry_safe_reverse > > > instead. > > > > OK, I will. > > Mika, below is an updated patch to try. > > --- > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > Subject: ACPI / hotplug / PCI: Simplify disable_slot() > > After recent PCI core changes related to the rescan/remove locking, > the ACPIPHP's disable_slot() function is only called under the > general PCI rescan/remove lock, so it doesn't have to use > dev_in_slot() any more to avoid race conditions. Make it simply > walk the devices on the bus and drop the ones in the slot being > disabled and drop dev_in_slot() which has no more users. > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> Thanks for the fix. Unfortunately, it now crashes here after I re-plug the TBT chain (I have both of your patches applied): int sysfs_create_bin_file(struct kobject *kobj, const struct bin_attribute *attr) { BUG_ON(!kobj || !kobj->sd || !attr); <-- Since I don't have proper serial console to that machine, all I see is the end of the backtrace :-( Here is a hand copied backtrace from the screen: pci_create_sysfs_dev_files() pci_bus_add_device() pci_bus_add_devices() enable_slot() acpiphp_check_bridge() hotplug_event() ... -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Friday, January 31, 2014 12:53:01 PM Mika Westerberg wrote: > On Fri, Jan 31, 2014 at 01:38:42AM +0100, Rafael J. Wysocki wrote: > > On Friday, January 31, 2014 12:59:06 AM Rafael J. Wysocki wrote: > > > On Thursday, January 30, 2014 03:39:02 PM Yinghai Lu wrote: > > > > On Thu, Jan 30, 2014 at 3:39 PM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > > > > > On Thursday, January 30, 2014 08:56:05 AM Yinghai Lu wrote: > > > > >> > > > > >> --047d7b5d2ea4eb937804f132eedf > > > > >> Content-Type: text/plain; charset=ISO-8859-1 > > > > >> > > > > >> >> The latest mainline kernel "hangs" when Thunderbolt devices are > > > > >> >> hot-unplugged to the system. I can't see any oops but after hot-unplug I'm > > > > >> >> getting huge amounts of messages like: > > > > >> >> > > > > >> >> [ 352.717001] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717011] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717021] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717032] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717041] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717051] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717061] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717070] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717083] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717094] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717104] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717113] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717124] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717133] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717143] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717153] pci 0000:02:00.0: PME# disabled > > > > >> >> [ 352.717162] pci 0000:02:00.0: PME# disabled > > > > >> > > > > > >> > that mean pci_stop_dev() get called again and again ? > > > > >> > > > > >> please check if attached patch could help. > > > > > > > > > > Well, it looks like what happens is an endless loop in > > > > > acpiphp_glue.c:disable_slot(). > > > > > > > > > > dev_in_slot() returns the first device in the list, so > > > > > pci_stop_and_remove_bus_device() is called for it, but it > > > > > doesn't remove the device from bus->devices any more, so > > > > > dev_in_slot() will return the same device next time and > > > > > so on forever. > > > > > > > > > ... > > > > > > > > > > So the above won't help in my opinion. > > > > > > > > > > I wonder, however, if this patch helps instead: > > > > > > > > > > https://patchwork.kernel.org/patch/3540701/ > > > > > > > > > > I thought it would be 3.15 material, but it very well can go in earlier if > > > > > it happens to address this particular problem. > > > > > > > > Agree, that should fix the problem. > > > > > > > > but please use list_for_each_entry_safe_reverse > > > > instead. > > > > > > OK, I will. > > > > Mika, below is an updated patch to try. > > > > --- > > From: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Subject: ACPI / hotplug / PCI: Simplify disable_slot() > > > > After recent PCI core changes related to the rescan/remove locking, > > the ACPIPHP's disable_slot() function is only called under the > > general PCI rescan/remove lock, so it doesn't have to use > > dev_in_slot() any more to avoid race conditions. Make it simply > > walk the devices on the bus and drop the ones in the slot being > > disabled and drop dev_in_slot() which has no more users. > > > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Thanks for the fix. > > Unfortunately, it now crashes here after I re-plug the TBT chain (I have > both of your patches applied): > > int sysfs_create_bin_file(struct kobject *kobj, > const struct bin_attribute *attr) > { > BUG_ON(!kobj || !kobj->sd || !attr); <-- > > Since I don't have proper serial console to that machine, all I see is the > end of the backtrace :-( > > Here is a hand copied backtrace from the screen: > > pci_create_sysfs_dev_files() > pci_bus_add_device() > pci_bus_add_devices() > enable_slot() > acpiphp_check_bridge() > hotplug_event() > ... So I think what happens is that we leak the struct pci_dev during removal and the proper cleanup is never done. Can you please add a debug printk into pci_release_dev() and see if that's ever called after TBT unplug?
On Fri, Jan 31, 2014 at 12:52:43PM +0100, Rafael J. Wysocki wrote: > So I think what happens is that we leak the struct pci_dev during removal and > the proper cleanup is never done. > > Can you please add a debug printk into pci_release_dev() and see if that's > ever called after TBT unplug? OK, I added the debug print (still on top of your two patches) and was able to capture a bit more from /var/log/messages before it crashes. Here's the log. I added dev_info(dev, "RELEASE\n") to pci_release_dev(). Unplug: Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.557920] pcieport 0000:06:03.0: PME# disabled Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.559483] pcieport 0000:05:00.0: PME# disabled Jan 31 20:05:57 buildroot kern.info kernel: [ 439.561074] pci 0000:07:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.562536] pci_bus 0000:07: busn_res: [bus 07] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.563993] pci 0000:06:03.0: RELEASE Jan 31 20:05:57 buildroot kern.info kernel: [ 439.570345] pci 0000:0a:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.571734] pci_bus 0000:0a: busn_res: [bus 0a] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.573154] pci 0000:09:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.574528] pci_bus 0000:09: busn_res: [bus 09-2e] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.575939] pci 0000:08:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.577316] pci_bus 0000:08: busn_res: [bus 08-2e] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.578721] pci 0000:06:04.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.580081] pci_bus 0000:2f: busn_res: [bus 2f] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.581487] pci 0000:06:05.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.582873] pci_bus 0000:06: busn_res: [bus 06-2f] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.584322] pci 0000:05:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.585727] pcieport 0000:03:00.0: PME# disabled Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.587225] pci_bus 0000:04: busn_res: [bus 04] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.588723] pci 0000:03:00.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.660389] pci_bus 0000:05: busn_res: [bus 05-2f] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.661993] pci 0000:03:03.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.663527] pci_bus 0000:30: busn_res: [bus 30-38] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.665103] pci 0000:03:04.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.666641] pci_bus 0000:39: busn_res: [bus 39] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.668210] pci 0000:03:05.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.669764] pci_bus 0000:3a: busn_res: [bus 3a] is released Jan 31 20:05:57 buildroot kern.info kernel: [ 439.671350] pci 0000:03:06.0: RELEASE Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released Plug: Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.609684] acpiphp_glue: hotplug_event: Bus check notify on \_SB_.PCI0.RP05 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.611339] acpiphp_glue: hotplug_event: re-enumerating slots under \_SB_.PCI0.RP05 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.614625] pci 0000:02:00.0: scanning [bus 03-3a] behind bridge, pass 0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.616434] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.618102] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.619797] kobject_add_internal failed for pci_bus (error: -2 parent: 0000:02:00.0) Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.621491] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.623191] CPU: 1 PID: 956 Comm: kworker/u8:5 Not tainted 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.624912] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.626649] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.628395] 0000000000000009 ffff88006de4d9f8 ffffffff818129e3 ffff88006de4da40 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.630164] ffff88006de4da30 ffffffff81047228 ffff88006dfd1000 00000000fffffffe Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.631933] ffff88006de140a8 ffff88006d582918 ffff88006d582918 ffff88006de4da90 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.633691] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.635428] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.637138] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.638879] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.640579] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.642297] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.643986] [<ffffffff81141852>] ? kmem_cache_alloc_trace+0xe2/0x130 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.645694] [<ffffffff81455584>] get_device_parent+0x174/0x1e0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.647377] [<ffffffff81455a33>] device_add+0xe3/0x610 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.649062] [<ffffffff81460ac4>] ? device_pm_sleep_init+0x44/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.650729] [<ffffffff81455f75>] device_register+0x15/0x20 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.652409] [<ffffffff8180c1a7>] pci_add_new_bus+0x167/0x3e0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.654064] [<ffffffff81303057>] ? pci_find_next_bus+0x47/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.655724] [<ffffffff812fc692>] pci_scan_bridge+0x5c2/0x630 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.657372] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.659057] [<ffffffff8180d116>] enable_slot+0xb6/0x320 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.660703] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.662387] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.664049] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.665746] [<ffffffff81316685>] hotplug_event+0x105/0x260 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.667417] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.669118] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.670816] [<ffffffff8106128a>] process_one_work+0x17a/0x440 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.672537] [<ffffffff81061e89>] worker_thread+0x119/0x390 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.674239] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.675976] [<ffffffff81067dfd>] kthread+0xcd/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.677689] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.679446] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.681174] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.682942] ---[ end trace 84e80bde4d2086ef ]--- Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.684679] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.686450] WARNING: CPU: 1 PID: 956 at drivers/pci/probe.c:711 pci_add_new_bus+0x3db/0x3e0() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.688245] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.690032] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.691883] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.693703] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.695531] 0000000000000009 ffff88006de4db88 ffffffff818129e3 0000000000000000 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.697377] ffff88006de4dbc0 ffffffff81047228 ffff88006d582800 ffff88006eac9000 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.699233] ffff88006de14000 ffff88006de14000 ffff88006d582918 ffff88006de4dbd0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.701114] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.702989] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.704871] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.706767] [<ffffffff81047305>] warn_slowpath_null+0x15/0x20 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.708637] [<ffffffff8180c41b>] pci_add_new_bus+0x3db/0x3e0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.710518] [<ffffffff81303057>] ? pci_find_next_bus+0x47/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.712381] [<ffffffff812fc692>] pci_scan_bridge+0x5c2/0x630 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.714258] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.716109] [<ffffffff8180d116>] enable_slot+0xb6/0x320 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.717973] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.719824] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.721685] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.723527] [<ffffffff81316685>] hotplug_event+0x105/0x260 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.725378] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.727195] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.729016] [<ffffffff8106128a>] process_one_work+0x17a/0x440 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.730815] [<ffffffff81061e89>] worker_thread+0x119/0x390 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.732622] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.734403] [<ffffffff81067dfd>] kthread+0xcd/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.736193] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.737962] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.739730] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.741471] ---[ end trace 84e80bde4d2086f0 ]--- Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.743215] pci_bus 0000:03: scanning bus Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.744993] pci 0000:03:00.0: [8086:1548] type 01 class 0x060400 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.746859] pci 0000:03:00.0: calling pci_fixup_transparent_bridge+0x0/0x30 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.748767] pci 0000:03:00.0: supports D1 D2 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.750433] pci 0000:03:00.0: PME# supported from D0 D1 D2 D3hot D3cold Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.752141] pci 0000:03:00.0: PME# disabled Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.753848] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.755500] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.757195] kobject_add_internal failed for 0000:03:00.0 (error: -2 parent: 0000:02:00.0) Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.758885] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.760589] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.762328] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.764082] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.765862] 0000000000000009 ffff88006de4d9c0 ffffffff818129e3 ffff88006de4da08 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.767661] ffff88006de4d9f8 ffffffff81047228 ffff88006de170a8 00000000fffffffe Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.769476] ffff88006de140a8 ffff88006de17098 ffff88006eac9000 ffff88006de4da58 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.771289] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.773091] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.774886] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.776712] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.778510] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.780332] [<ffffffff8163ea05>] ? pci_conf1_read+0xb5/0x110 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.782134] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.783959] [<ffffffff814558fe>] ? device_private_init+0x1e/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.785760] [<ffffffff81455a61>] device_add+0x111/0x610 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.787573] [<ffffffff812fb89d>] pci_device_add+0x10d/0x140 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.789362] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.791171] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.792958] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.794759] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.796535] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.798329] [<ffffffff8180d116>] enable_slot+0xb6/0x320 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.800097] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.801896] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.803671] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.805471] [<ffffffff81316685>] hotplug_event+0x105/0x260 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.807246] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.809033] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.810801] [<ffffffff8106128a>] process_one_work+0x17a/0x440 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.812573] [<ffffffff81061e89>] worker_thread+0x119/0x390 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.814307] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.816053] [<ffffffff81067dfd>] kthread+0xcd/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.817755] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.819467] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.821144] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.822829] ---[ end trace 84e80bde4d2086f1 ]--- Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.824500] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.826182] WARNING: CPU: 1 PID: 956 at drivers/pci/probe.c:1397 pci_device_add+0x13c/0x140() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.827872] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.829572] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.831314] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.833072] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.834845] 0000000000000009 ffff88006de4db10 ffffffff818129e3 0000000000000000 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.836648] ffff88006de4db48 ffffffff81047228 ffff88006de17000 ffff88006d582828 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.838466] ffff88006de17098 0000000000000000 ffff88006eac9000 ffff88006de4db58 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.840280] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.842081] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.843877] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.845694] [<ffffffff81047305>] warn_slowpath_null+0x15/0x20 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.847482] [<ffffffff812fb8cc>] pci_device_add+0x13c/0x140 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.849290] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.851090] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.852897] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.854674] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.856470] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.858251] [<ffffffff8180d116>] enable_slot+0xb6/0x320 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.860044] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.861819] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.863612] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.865389] [<ffffffff81316685>] hotplug_event+0x105/0x260 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.867187] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.868956] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.870739] [<ffffffff8106128a>] process_one_work+0x17a/0x440 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.872493] [<ffffffff81061e89>] worker_thread+0x119/0x390 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.874268] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.876024] [<ffffffff81067dfd>] kthread+0xcd/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.877794] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.879532] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.881273] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.882984] ---[ end trace 84e80bde4d2086f2 ]--- Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.884782] pci 0000:03:03.0: [8086:1548] type 01 class 0x060400 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.886627] pci 0000:03:03.0: calling pci_fixup_transparent_bridge+0x0/0x30 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.888494] pci 0000:03:03.0: supports D1 D2 Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.890141] pci 0000:03:03.0: PME# supported from D0 D1 D2 D3hot D3cold Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.891805] pci 0000:03:03.0: PME# disabled Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.893490] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.895118] WARNING: CPU: 3 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.896778] kobject_add_internal failed for 0000:03:03.0 (error: -2 parent: 0000:02:00.0) Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.898453] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.900135] CPU: 3 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.901841] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.903569] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.905310] 0000000000000009 ffff88006de4d9c0 ffffffff818129e3 ffff88006de4da08 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.907080] ffff88006de4d9f8 ffffffff81047228 ffff88006d6260a8 00000000fffffffe Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.908866] ffff88006de140a8 ffff88006d626098 ffff88006eac9000 ffff88006de4da58 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.910654] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.912412] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.914189] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.915980] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.917764] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.919557] [<ffffffff8164189e>] ? raw_pci_read+0x1e/0x40 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.921344] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.923127] [<ffffffff814558fe>] ? device_private_init+0x1e/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.924910] [<ffffffff81455a61>] device_add+0x111/0x610 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.926683] [<ffffffff812fb89d>] pci_device_add+0x10d/0x140 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.928453] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.930234] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.932004] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.933773] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.935636] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.937399] [<ffffffff8180d116>] enable_slot+0xb6/0x320 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.939148] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.940901] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.942659] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.944419] [<ffffffff81316685>] hotplug_event+0x105/0x260 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.946166] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.947917] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.949648] [<ffffffff8106128a>] process_one_work+0x17a/0x440 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.951366] [<ffffffff81061e89>] worker_thread+0x119/0x390 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.953070] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.954764] [<ffffffff81067dfd>] kthread+0xcd/0xf0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.956425] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.958084] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.959727] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.961378] ---[ end trace 84e80bde4d2086f3 ]--- Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.963016] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.964646] WARNING: CPU: 3 PID: 956 at drivers/pci/probe.c:1397 pci_device_add+0x13c/0x140() and then it crashes. The PCI tree looks like: 00:00.0 Host bridge: Intel Corporation 3rd Gen Core processor DRAM Controller (rev 09) 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) 00:16.0 Communication controller: Intel Corporation 7 Series/C210 Series Chipset Family MEI Controller #1 (rev 04) 00:1a.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB Enhanced Host Controller #2 (rev 04) 00:1b.0 Audio device: Intel Corporation 7 Series/C210 Series Chipset Family High Definition Audio Controller (rev 04) 00:1c.0 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 1 (rev c4) 00:1c.4 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 5 (rev c4) 00:1d.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB Enhanced Host Controller #1 (rev 04) 00:1f.0 ISA bridge: Intel Corporation QS77 Express Chipset LPC Controller (rev 04) 00:1f.2 SATA controller: Intel Corporation 7 Series Chipset Family 6-port SATA Controller [AHCI mode] (rev 04) 00:1f.3 SMBus: Intel Corporation 7 Series/C210 Series Chipset Family SMBus Controller (rev 04) 02:00.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 03:00.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 03:03.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 03:04.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 03:05.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 03:06.0 PCI bridge: Intel Corporation Device 1548 (rev 03) 05:00.0 PCI bridge: Intel Corporation Device 1513 06:03.0 PCI bridge: Intel Corporation Device 1513 06:04.0 PCI bridge: Intel Corporation Device 1513 06:05.0 PCI bridge: Intel Corporation Device 1513 07:00.0 SATA controller: Marvell Technology Group Ltd. Device 9182 (rev 11) 08:00.0 PCI bridge: Intel Corporation DSL3510 Thunderbolt Controller [Cactus Ridge] 09:00.0 PCI bridge: Intel Corporation DSL3510 Thunderbolt Controller [Cactus Ridge] 0a:00.0 Ethernet controller: Broadcom Corporation NetXtreme BCM57762 Gigabit Ethernet PCIe -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Friday, January 31, 2014 02:36:07 PM Mika Westerberg wrote: > On Fri, Jan 31, 2014 at 12:52:43PM +0100, Rafael J. Wysocki wrote: > > So I think what happens is that we leak the struct pci_dev during removal and > > the proper cleanup is never done. > > > > Can you please add a debug printk into pci_release_dev() and see if that's > > ever called after TBT unplug? > > OK, I added the debug print (still on top of your two patches) and was able > to capture a bit more from /var/log/messages before it crashes. Here's the > log. I added dev_info(dev, "RELEASE\n") to pci_release_dev(). > > Unplug: > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.557920] pcieport 0000:06:03.0: PME# disabled > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.559483] pcieport 0000:05:00.0: PME# disabled > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.561074] pci 0000:07:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.562536] pci_bus 0000:07: busn_res: [bus 07] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.563993] pci 0000:06:03.0: RELEASE > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.570345] pci 0000:0a:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.571734] pci_bus 0000:0a: busn_res: [bus 0a] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.573154] pci 0000:09:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.574528] pci_bus 0000:09: busn_res: [bus 09-2e] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.575939] pci 0000:08:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.577316] pci_bus 0000:08: busn_res: [bus 08-2e] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.578721] pci 0000:06:04.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.580081] pci_bus 0000:2f: busn_res: [bus 2f] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.581487] pci 0000:06:05.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.582873] pci_bus 0000:06: busn_res: [bus 06-2f] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.584322] pci 0000:05:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.585727] pcieport 0000:03:00.0: PME# disabled > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.587225] pci_bus 0000:04: busn_res: [bus 04] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.588723] pci 0000:03:00.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.660389] pci_bus 0000:05: busn_res: [bus 05-2f] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.661993] pci 0000:03:03.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.663527] pci_bus 0000:30: busn_res: [bus 30-38] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.665103] pci 0000:03:04.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.666641] pci_bus 0000:39: busn_res: [bus 39] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.668210] pci 0000:03:05.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.669764] pci_bus 0000:3a: busn_res: [bus 3a] is released > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.671350] pci 0000:03:06.0: RELEASE > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released OK, so my guess wasn't right. We seem to call pci_release_dev for all of the devices that go away after unplug. Do I think correctly that the below doesn't happen with the Yinghai's commit reverted? > Plug: > > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.609684] acpiphp_glue: hotplug_event: Bus check notify on \_SB_.PCI0.RP05 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.611339] acpiphp_glue: hotplug_event: re-enumerating slots under \_SB_.PCI0.RP05 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.614625] pci 0000:02:00.0: scanning [bus 03-3a] behind bridge, pass 0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.616434] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.618102] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.619797] kobject_add_internal failed for pci_bus (error: -2 parent: 0000:02:00.0) create_dir() fails here and that's not because it already exists. Interesting. > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.621491] Modules linked in: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.623191] CPU: 1 PID: 956 Comm: kworker/u8:5 Not tainted 3.13.0+ #156 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.624912] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.626649] Workqueue: kacpi_hotplug acpi_hotplug_work_fn > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.628395] 0000000000000009 ffff88006de4d9f8 ffffffff818129e3 ffff88006de4da40 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.630164] ffff88006de4da30 ffffffff81047228 ffff88006dfd1000 00000000fffffffe > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.631933] ffff88006de140a8 ffff88006d582918 ffff88006d582918 ffff88006de4da90 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.633691] Call Trace: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.635428] [<ffffffff818129e3>] dump_stack+0x45/0x56 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.637138] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.638879] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.640579] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.642297] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.643986] [<ffffffff81141852>] ? kmem_cache_alloc_trace+0xe2/0x130 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.645694] [<ffffffff81455584>] get_device_parent+0x174/0x1e0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.647377] [<ffffffff81455a33>] device_add+0xe3/0x610 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.649062] [<ffffffff81460ac4>] ? device_pm_sleep_init+0x44/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.650729] [<ffffffff81455f75>] device_register+0x15/0x20 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.652409] [<ffffffff8180c1a7>] pci_add_new_bus+0x167/0x3e0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.654064] [<ffffffff81303057>] ? pci_find_next_bus+0x47/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.655724] [<ffffffff812fc692>] pci_scan_bridge+0x5c2/0x630 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.657372] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.659057] [<ffffffff8180d116>] enable_slot+0xb6/0x320 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.660703] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.662387] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.664049] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.665746] [<ffffffff81316685>] hotplug_event+0x105/0x260 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.667417] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.669118] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.670816] [<ffffffff8106128a>] process_one_work+0x17a/0x440 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.672537] [<ffffffff81061e89>] worker_thread+0x119/0x390 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.674239] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.675976] [<ffffffff81067dfd>] kthread+0xcd/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.677689] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.679446] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.681174] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.682942] ---[ end trace 84e80bde4d2086ef ]--- > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.684679] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.686450] WARNING: CPU: 1 PID: 956 at drivers/pci/probe.c:711 pci_add_new_bus+0x3db/0x3e0() That's failing device_register(), probably because of the earlier sysfs issue. > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.688245] Modules linked in: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.690032] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.691883] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.693703] Workqueue: kacpi_hotplug acpi_hotplug_work_fn > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.695531] 0000000000000009 ffff88006de4db88 ffffffff818129e3 0000000000000000 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.697377] ffff88006de4dbc0 ffffffff81047228 ffff88006d582800 ffff88006eac9000 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.699233] ffff88006de14000 ffff88006de14000 ffff88006d582918 ffff88006de4dbd0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.701114] Call Trace: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.702989] [<ffffffff818129e3>] dump_stack+0x45/0x56 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.704871] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.706767] [<ffffffff81047305>] warn_slowpath_null+0x15/0x20 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.708637] [<ffffffff8180c41b>] pci_add_new_bus+0x3db/0x3e0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.710518] [<ffffffff81303057>] ? pci_find_next_bus+0x47/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.712381] [<ffffffff812fc692>] pci_scan_bridge+0x5c2/0x630 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.714258] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.716109] [<ffffffff8180d116>] enable_slot+0xb6/0x320 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.717973] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.719824] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.721685] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.723527] [<ffffffff81316685>] hotplug_event+0x105/0x260 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.725378] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.727195] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.729016] [<ffffffff8106128a>] process_one_work+0x17a/0x440 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.730815] [<ffffffff81061e89>] worker_thread+0x119/0x390 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.732622] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.734403] [<ffffffff81067dfd>] kthread+0xcd/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.736193] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.737962] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.739730] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.741471] ---[ end trace 84e80bde4d2086f0 ]--- > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.743215] pci_bus 0000:03: scanning bus > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.744993] pci 0000:03:00.0: [8086:1548] type 01 class 0x060400 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.746859] pci 0000:03:00.0: calling pci_fixup_transparent_bridge+0x0/0x30 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.748767] pci 0000:03:00.0: supports D1 D2 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.750433] pci 0000:03:00.0: PME# supported from D0 D1 D2 D3hot D3cold > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.752141] pci 0000:03:00.0: PME# disabled > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.753848] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.755500] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.757195] kobject_add_internal failed for 0000:03:00.0 (error: -2 parent: 0000:02:00.0) And here it repeats for the next device and so on. > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.758885] Modules linked in: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.760589] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.762328] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.764082] Workqueue: kacpi_hotplug acpi_hotplug_work_fn > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.765862] 0000000000000009 ffff88006de4d9c0 ffffffff818129e3 ffff88006de4da08 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.767661] ffff88006de4d9f8 ffffffff81047228 ffff88006de170a8 00000000fffffffe > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.769476] ffff88006de140a8 ffff88006de17098 ffff88006eac9000 ffff88006de4da58 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.771289] Call Trace: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.773091] [<ffffffff818129e3>] dump_stack+0x45/0x56 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.774886] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.776712] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.778510] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.780332] [<ffffffff8163ea05>] ? pci_conf1_read+0xb5/0x110 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.782134] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.783959] [<ffffffff814558fe>] ? device_private_init+0x1e/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.785760] [<ffffffff81455a61>] device_add+0x111/0x610 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.787573] [<ffffffff812fb89d>] pci_device_add+0x10d/0x140 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.789362] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.791171] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.792958] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.794759] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.796535] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.798329] [<ffffffff8180d116>] enable_slot+0xb6/0x320 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.800097] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.801896] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.803671] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.805471] [<ffffffff81316685>] hotplug_event+0x105/0x260 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.807246] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.809033] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.810801] [<ffffffff8106128a>] process_one_work+0x17a/0x440 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.812573] [<ffffffff81061e89>] worker_thread+0x119/0x390 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.814307] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.816053] [<ffffffff81067dfd>] kthread+0xcd/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.817755] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.819467] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.821144] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.822829] ---[ end trace 84e80bde4d2086f1 ]--- > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.824500] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.826182] WARNING: CPU: 1 PID: 956 at drivers/pci/probe.c:1397 pci_device_add+0x13c/0x140() > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.827872] Modules linked in: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.829572] CPU: 1 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.831314] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.833072] Workqueue: kacpi_hotplug acpi_hotplug_work_fn > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.834845] 0000000000000009 ffff88006de4db10 ffffffff818129e3 0000000000000000 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.836648] ffff88006de4db48 ffffffff81047228 ffff88006de17000 ffff88006d582828 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.838466] ffff88006de17098 0000000000000000 ffff88006eac9000 ffff88006de4db58 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.840280] Call Trace: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.842081] [<ffffffff818129e3>] dump_stack+0x45/0x56 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.843877] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.845694] [<ffffffff81047305>] warn_slowpath_null+0x15/0x20 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.847482] [<ffffffff812fb8cc>] pci_device_add+0x13c/0x140 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.849290] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.851090] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.852897] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.854674] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.856470] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.858251] [<ffffffff8180d116>] enable_slot+0xb6/0x320 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.860044] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.861819] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.863612] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.865389] [<ffffffff81316685>] hotplug_event+0x105/0x260 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.867187] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.868956] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.870739] [<ffffffff8106128a>] process_one_work+0x17a/0x440 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.872493] [<ffffffff81061e89>] worker_thread+0x119/0x390 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.874268] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.876024] [<ffffffff81067dfd>] kthread+0xcd/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.877794] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.879532] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.881273] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.882984] ---[ end trace 84e80bde4d2086f2 ]--- > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.884782] pci 0000:03:03.0: [8086:1548] type 01 class 0x060400 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.886627] pci 0000:03:03.0: calling pci_fixup_transparent_bridge+0x0/0x30 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.888494] pci 0000:03:03.0: supports D1 D2 > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.890141] pci 0000:03:03.0: PME# supported from D0 D1 D2 D3hot D3cold > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.891805] pci 0000:03:03.0: PME# disabled > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.893490] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.895118] WARNING: CPU: 3 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.896778] kobject_add_internal failed for 0000:03:03.0 (error: -2 parent: 0000:02:00.0) > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.898453] Modules linked in: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.900135] CPU: 3 PID: 956 Comm: kworker/u8:5 Tainted: G W 3.13.0+ #156 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.901841] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.903569] Workqueue: kacpi_hotplug acpi_hotplug_work_fn > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.905310] 0000000000000009 ffff88006de4d9c0 ffffffff818129e3 ffff88006de4da08 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.907080] ffff88006de4d9f8 ffffffff81047228 ffff88006d6260a8 00000000fffffffe > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.908866] ffff88006de140a8 ffff88006d626098 ffff88006eac9000 ffff88006de4da58 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.910654] Call Trace: > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.912412] [<ffffffff818129e3>] dump_stack+0x45/0x56 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.914189] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.915980] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.917764] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.919557] [<ffffffff8164189e>] ? raw_pci_read+0x1e/0x40 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.921344] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.923127] [<ffffffff814558fe>] ? device_private_init+0x1e/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.924910] [<ffffffff81455a61>] device_add+0x111/0x610 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.926683] [<ffffffff812fb89d>] pci_device_add+0x10d/0x140 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.928453] [<ffffffff8180c011>] pci_scan_single_device+0x91/0xc0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.930234] [<ffffffff812fb919>] pci_scan_slot+0x49/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.932004] [<ffffffff812fc73d>] pci_scan_child_bus+0x3d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.933773] [<ffffffff812fc53b>] pci_scan_bridge+0x46b/0x630 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.935636] [<ffffffff812fb9dd>] ? pci_scan_slot+0x10d/0x150 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.937399] [<ffffffff8180d116>] enable_slot+0xb6/0x320 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.939148] [<ffffffff812fa273>] ? pci_bus_read_dev_vendor_id+0x23/0xe0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.940901] [<ffffffff81315814>] ? trim_stale_devices+0xc4/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.942659] [<ffffffff81315cf8>] acpiphp_check_bridge.part.9+0xe8/0x100 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.944419] [<ffffffff81316685>] hotplug_event+0x105/0x260 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.946166] [<ffffffff8131680a>] hotplug_event_work+0x2a/0x70 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.947917] [<ffffffff8132fc09>] acpi_hotplug_work_fn+0x17/0x22 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.949648] [<ffffffff8106128a>] process_one_work+0x17a/0x440 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.951366] [<ffffffff81061e89>] worker_thread+0x119/0x390 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.953070] [<ffffffff81061d70>] ? manage_workers.isra.25+0x2a0/0x2a0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.954764] [<ffffffff81067dfd>] kthread+0xcd/0xf0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.956425] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.958084] [<ffffffff81823a3c>] ret_from_fork+0x7c/0xb0 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.959727] [<ffffffff81067d30>] ? kthread_create_on_node+0x180/0x180 > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.961378] ---[ end trace 84e80bde4d2086f3 ]--- > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.963016] ------------[ cut here ]------------ > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.964646] WARNING: CPU: 3 PID: 956 at drivers/pci/probe.c:1397 pci_device_add+0x13c/0x140() > > and then it crashes. > > The PCI tree looks like: > > 00:00.0 Host bridge: Intel Corporation 3rd Gen Core processor DRAM Controller (rev 09) > 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) > 00:16.0 Communication controller: Intel Corporation 7 Series/C210 Series Chipset Family MEI Controller #1 (rev 04) > 00:1a.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB Enhanced Host Controller #2 (rev 04) > 00:1b.0 Audio device: Intel Corporation 7 Series/C210 Series Chipset Family High Definition Audio Controller (rev 04) > 00:1c.0 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 1 (rev c4) > 00:1c.4 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 5 (rev c4) > 00:1d.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB Enhanced Host Controller #1 (rev 04) > 00:1f.0 ISA bridge: Intel Corporation QS77 Express Chipset LPC Controller (rev 04) > 00:1f.2 SATA controller: Intel Corporation 7 Series Chipset Family 6-port SATA Controller [AHCI mode] (rev 04) > 00:1f.3 SMBus: Intel Corporation 7 Series/C210 Series Chipset Family SMBus Controller (rev 04) > 02:00.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 03:00.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 03:03.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 03:04.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 03:05.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 03:06.0 PCI bridge: Intel Corporation Device 1548 (rev 03) > 05:00.0 PCI bridge: Intel Corporation Device 1513 > 06:03.0 PCI bridge: Intel Corporation Device 1513 > 06:04.0 PCI bridge: Intel Corporation Device 1513 > 06:05.0 PCI bridge: Intel Corporation Device 1513 > 07:00.0 SATA controller: Marvell Technology Group Ltd. Device 9182 (rev 11) > 08:00.0 PCI bridge: Intel Corporation DSL3510 Thunderbolt Controller [Cactus Ridge] > 09:00.0 PCI bridge: Intel Corporation DSL3510 Thunderbolt Controller [Cactus Ridge] > 0a:00.0 Ethernet controller: Broadcom Corporation NetXtreme BCM57762 Gigabit Ethernet PCIe Can you please check how the PCI sysfs directory structure changes after unplug with the Yinghai's commit present and reverted and what the difference is?
On Fri, Jan 31, 2014 at 02:49:21PM +0100, Rafael J. Wysocki wrote: > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released > > OK, so my guess wasn't right. We seem to call pci_release_dev for all of the > devices that go away after unplug. > > Do I think correctly that the below doesn't happen with the Yinghai's commit > reverted? Yes, with that commit reverted everything works fine. > Can you please check how the PCI sysfs directory structure changes after unplug > with the Yinghai's commit present and reverted and what the difference is? OK, I'll check that and report back. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Friday, January 31, 2014 02:49:21 PM Rafael J. Wysocki wrote: > On Friday, January 31, 2014 02:36:07 PM Mika Westerberg wrote: > > On Fri, Jan 31, 2014 at 12:52:43PM +0100, Rafael J. Wysocki wrote: > > > So I think what happens is that we leak the struct pci_dev during removal and > > > the proper cleanup is never done. > > > > > > Can you please add a debug printk into pci_release_dev() and see if that's > > > ever called after TBT unplug? > > > > OK, I added the debug print (still on top of your two patches) and was able > > to capture a bit more from /var/log/messages before it crashes. Here's the > > log. I added dev_info(dev, "RELEASE\n") to pci_release_dev(). > > > > Unplug: > > > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.557920] pcieport 0000:06:03.0: PME# disabled > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.559483] pcieport 0000:05:00.0: PME# disabled > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.561074] pci 0000:07:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.562536] pci_bus 0000:07: busn_res: [bus 07] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.563993] pci 0000:06:03.0: RELEASE > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.570345] pci 0000:0a:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.571734] pci_bus 0000:0a: busn_res: [bus 0a] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.573154] pci 0000:09:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.574528] pci_bus 0000:09: busn_res: [bus 09-2e] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.575939] pci 0000:08:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.577316] pci_bus 0000:08: busn_res: [bus 08-2e] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.578721] pci 0000:06:04.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.580081] pci_bus 0000:2f: busn_res: [bus 2f] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.581487] pci 0000:06:05.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.582873] pci_bus 0000:06: busn_res: [bus 06-2f] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.584322] pci 0000:05:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.585727] pcieport 0000:03:00.0: PME# disabled > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.587225] pci_bus 0000:04: busn_res: [bus 04] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.588723] pci 0000:03:00.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.660389] pci_bus 0000:05: busn_res: [bus 05-2f] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.661993] pci 0000:03:03.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.663527] pci_bus 0000:30: busn_res: [bus 30-38] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.665103] pci 0000:03:04.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.666641] pci_bus 0000:39: busn_res: [bus 39] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.668210] pci 0000:03:05.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.669764] pci_bus 0000:3a: busn_res: [bus 3a] is released > > Jan 31 20:05:57 buildroot kern.info kernel: [ 439.671350] pci 0000:03:06.0: RELEASE > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released > > OK, so my guess wasn't right. We seem to call pci_release_dev for all of the > devices that go away after unplug. > > Do I think correctly that the below doesn't happen with the Yinghai's commit > reverted? > > > Plug: > > > > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.609684] acpiphp_glue: hotplug_event: Bus check notify on \_SB_.PCI0.RP05 > > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.611339] acpiphp_glue: hotplug_event: re-enumerating slots under \_SB_.PCI0.RP05 > > Jan 31 20:06:11 buildroot kern.debug kernel: [ 453.614625] pci 0000:02:00.0: scanning [bus 03-3a] behind bridge, pass 0 > > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.616434] ------------[ cut here ]------------ > > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.618102] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/0x400() > > Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.619797] kobject_add_internal failed for pci_bus (error: -2 parent: 0000:02:00.0) > > create_dir() fails here and that's not because it already exists. > Interesting. That's more interesting than I thought. So the error is -2, which is -ENOENT. Let's see when create_dir() returns -ENOENT, then. Evidently, it calls sysfs_create_dir_ns() and returns the error code returned by that, but if it is 0, it returns the return value of populate_dir(). sysfs_create_dir_ns() tries to use kobj->parent->sd and returns -ENOENT when that is NULL. There you go. So it looks like the sysfs dir of PCI device 0000:02:00.0 doesn't exist at this point. Yinghai, any ideas?
On Fri, Jan 31, 2014 at 03:49:25PM +0200, Mika Westerberg wrote: > On Fri, Jan 31, 2014 at 02:49:21PM +0100, Rafael J. Wysocki wrote: > > > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released > > > > OK, so my guess wasn't right. We seem to call pci_release_dev for all of the > > devices that go away after unplug. > > > > Do I think correctly that the below doesn't happen with the Yinghai's commit > > reverted? > > Yes, with that commit reverted everything works fine. > > > Can you please check how the PCI sysfs directory structure changes after unplug > > with the Yinghai's commit present and reverted and what the difference is? > > OK, I'll check that and report back. I now tried so that I have your two patches and then did hotplug/unplug with and without Yinghai's patch and then compared the resulting PCI sysfs structure. Unfortunately I didn't find any difference the layout in both cases is this: /sys/bus/pci/devices: 0000:00:00.0@ 0000:00:02.0@ 0000:00:16.0@ 0000:00:1a.0@ 0000:00:1b.0@ 0000:00:1c.0@ 0000:00:1c.4@ 0000:00:1d.0@ 0000:00:1f.0@ 0000:00:1f.2@ 0000:00:1f.3@ Then I checked 0000:00:1c.4 as that's the root port that hosts the TBT stuff: 0000:00:1c.4/ 0000:00:1c.4/irq 0000:00:1c.4/subsystem_vendor 0000:00:1c.4/broken_parity_status 0000:00:1c.4/class 0000:00:1c.4/power 0000:00:1c.4/power/wakeup_abort_count 0000:00:1c.4/power/wakeup_active 0000:00:1c.4/power/wakeup_total_time_ms 0000:00:1c.4/power/wakeup_active_count 0000:00:1c.4/power/wakeup_max_time_ms 0000:00:1c.4/power/wakeup_count 0000:00:1c.4/power/wakeup_last_time_ms 0000:00:1c.4/power/wakeup 0000:00:1c.4/power/wakeup_expire_count 0000:00:1c.4/reset 0000:00:1c.4/resource 0000:00:1c.4/enabled 0000:00:1c.4/consistent_dma_mask_bits 0000:00:1c.4/modalias 0000:00:1c.4/dma_mask_bits 0000:00:1c.4/local_cpus 0000:00:1c.4/config 0000:00:1c.4/device 0000:00:1c.4/driver 0000:00:1c.4/subsystem 0000:00:1c.4/msi_bus 0000:00:1c.4/local_cpulist 0000:00:1c.4/remove 0000:00:1c.4/rescan 0000:00:1c.4/uevent 0000:00:1c.4/vendor 0000:00:1c.4/pci_bus 0000:00:1c.4/pci_bus/0000:02 0000:00:1c.4/pci_bus/0000:02/power 0000:00:1c.4/pci_bus/0000:02/device 0000:00:1c.4/pci_bus/0000:02/subsystem 0000:00:1c.4/pci_bus/0000:02/cpulistaffinity 0000:00:1c.4/pci_bus/0000:02/cpuaffinity 0000:00:1c.4/pci_bus/0000:02/rescan 0000:00:1c.4/pci_bus/0000:02/uevent 0000:00:1c.4/subsystem_device 0000:00:1c.4/numa_node 0000:00:1c.4/firmware_node Both cases the structure is the same. -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Jan 31, 2014 at 5:49 AM, Mika Westerberg <mika.westerberg@linux.intel.com> wrote: > On Fri, Jan 31, 2014 at 02:49:21PM +0100, Rafael J. Wysocki wrote: >> > Jan 31 20:05:57 buildroot kern.debug kernel: [ 439.672933] pci_bus 0000:03: busn_res: [bus 03-3a] is released >> >> OK, so my guess wasn't right. We seem to call pci_release_dev for all of the >> devices that go away after unplug. >> >> Do I think correctly that the below doesn't happen with the Yinghai's commit >> reverted? > > Yes, with that commit reverted everything works fine. can you make it clear ? after my commit is reverted, the warning does not happen any more? Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.616434] ------------[ cut here ]------------ Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.618102] WARNING: CPU: 1 PID: 956 at lib/kobject.c:244 kobject_add_internal+0x12d/ 0x400() Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.619797] kobject_add_internal failed for pci_bus (error: -2 parent: 0000:02:00.0) Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.621491] Modules linked in: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.623191] CPU: 1 PID: 956 Comm: kworker/u8:5 Not tainted 3.13.0+ #156 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.624912] Hardware name: /D33217CK, BIOS GKPPT10H.86A.0042.2013.0422.1439 04/22/2013 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.626649] Workqueue: kacpi_hotplug acpi_hotplug_work_fn Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.628395] 0000000000000009 ffff88006de4d9f8 ffffffff818129e3 ffff88006de4da40 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.630164] ffff88006de4da30 ffffffff81047228 ffff88006dfd1000 00000000fffffffe Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.631933] ffff88006de140a8 ffff88006d582918 ffff88006d582918 ffff88006de4da90 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.633691] Call Trace: Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.635428] [<ffffffff818129e3>] dump_stack+0x45/0x56 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.637138] [<ffffffff81047228>] warn_slowpath_common+0x78/0xa0 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.638879] [<ffffffff81047297>] warn_slowpath_fmt+0x47/0x50 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.640579] [<ffffffff812d81ad>] kobject_add_internal+0x12d/0x400 Jan 31 20:06:11 buildroot kern.warn kernel: [ 453.642297] [<ffffffff812d88b5>] kobject_add+0x65/0xb0 -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Index: linux-pm/drivers/pci/hotplug/acpiphp_glue.c =================================================================== --- linux-pm.orig/drivers/pci/hotplug/acpiphp_glue.c +++ linux-pm/drivers/pci/hotplug/acpiphp_glue.c @@ -604,32 +604,15 @@ static void __ref enable_slot(struct acp } } -/* return first device in slot, acquiring a reference on it */ -static struct pci_dev *dev_in_slot(struct acpiphp_slot *slot) -{ - struct pci_bus *bus = slot->bus; - struct pci_dev *dev; - struct pci_dev *ret = NULL; - - down_read(&pci_bus_sem); - list_for_each_entry(dev, &bus->devices, bus_list) - if (PCI_SLOT(dev->devfn) == slot->device) { - ret = pci_dev_get(dev); - break; - } - up_read(&pci_bus_sem); - - return ret; -} - /** * disable_slot - disable a slot * @slot: ACPI PHP slot */ static void disable_slot(struct acpiphp_slot *slot) { + struct pci_bus *bus = slot->bus; + struct pci_dev *dev, *prev; struct acpiphp_func *func; - struct pci_dev *pdev; /* * enable_slot() enumerates all functions in this device via @@ -637,10 +620,9 @@ static void disable_slot(struct acpiphp_ * methods (_EJ0, etc.) or not. Therefore, we remove all functions * here. */ - while ((pdev = dev_in_slot(slot))) { - pci_stop_and_remove_bus_device(pdev); - pci_dev_put(pdev); - } + list_for_each_entry_safe_reverse(dev, prev, &bus->devices, bus_list) + if (PCI_SLOT(dev->devfn) == slot->device) + pci_stop_and_remove_bus_device(dev); list_for_each_entry(func, &slot->funcs, sibling) acpiphp_bus_trim(func_to_handle(func));