Linux-ACPI Archive on lore.kernel.org
 help / color / Atom feed
From: <Mario.Limonciello@dell.com>
To: <mika.westerberg@linux.intel.com>
Cc: <linux-kernel@vger.kernel.org>, <andreas.noever@gmail.com>,
	<michael.jamet@intel.com>, <YehezkelShB@gmail.com>,
	<rjw@rjwysocki.net>, <lenb@kernel.org>, <lukas@wunner.de>,
	<anthony.wong@canonical.com>, <rajmohan.mani@intel.com>,
	<raanan.avargil@intel.com>, <David.Laight@aculab.com>,
	<linux-acpi@vger.kernel.org>
Subject: RE: [PATCH v3 0/8] thunderbolt: Intel Ice Lake support
Date: Mon, 19 Aug 2019 18:22:26 +0000
Message-ID: <c56451e3ae244acd8a824fbca1322c0c@AUSX13MPC101.AMER.DELL.COM> (raw)
In-Reply-To: <20190819175730.GX19908@lahna.fi.intel.com>

> > I've tested this on a pre-production ICL platform.  This was on top of 5.3-rc5,
> both of
> > your patches from -next and this series.
> >
> > I've run into a problem when using
> > a WD19TB that after unplugging it will cause the following to spew in dmesg:
> >
> > [ 2198.017003] ============================================
> > [ 2198.017005] WARNING: possible recursive locking detected
> > [ 2198.017008] 5.3.0-rc5+ #75 Not tainted
> > [ 2198.017009] --------------------------------------------
> > [ 2198.017012] irq/122-pciehp/121 is trying to acquire lock:
> > [ 2198.017015] 00000000801d4de8 (&ctrl->reset_lock){.+.+}, at:
> pciehp_check_presence+0x1b/0x80
> > [ 2198.017026]
> >                but task is already holding lock:
> > [ 2198.017028] 000000000899e2eb (&ctrl->reset_lock){.+.+}, at:
> pciehp_ist+0xaf/0x1c0
> > [ 2198.017033]
> >                other info that might help us debug this:
> > [ 2198.017035]  Possible unsafe locking scenario:
> >
> > [ 2198.017037]        CPU0
> > [ 2198.017038]        ----
> > [ 2198.017041]   lock(&ctrl->reset_lock);
> > [ 2198.017044]   lock(&ctrl->reset_lock);
> > [ 2198.017046]
> >                 *** DEADLOCK ***
> >
> > [ 2198.017048]  May be due to missing lock nesting notation
> >
> > [ 2198.017051] 3 locks held by irq/122-pciehp/121:
> > [ 2198.017052]  #0: 000000000899e2eb (&ctrl->reset_lock){.+.+}, at:
> pciehp_ist+0xaf/0x1c0
> > [ 2198.017057]  #1: 0000000069e1fd47 (pci_rescan_remove_lock){+.+.}, at:
> pciehp_unconfigure_device+0x41/0x130
> > [ 2198.017064]  #2: 0000000046a5bfbf (&dev->mutex){....}, at:
> device_release_driver_internal+0x1c/0x1a0
> > [ 2198.017071]
> >                stack backtrace:
> > [ 2198.017075] CPU: 3 PID: 121 Comm: irq/122-pciehp Not tainted 5.3.0-rc5+ #75
> > [ 2198.017079] Call Trace:
> > [ 2198.017089]  dump_stack+0x5e/0x8b
> > [ 2198.017096]  __lock_acquire+0x12fe/0x1520
> > [ 2198.017102]  lock_acquire+0x9d/0x1a0
> > [ 2198.017105]  ? pciehp_check_presence+0x1b/0x80
> > [ 2198.017113]  ? pci_restore_standard_config+0x40/0x40
> > [ 2198.017118]  down_read+0x3b/0x160
> > [ 2198.017121]  ? pciehp_check_presence+0x1b/0x80
> > [ 2198.017123]  pciehp_check_presence+0x1b/0x80
> > [ 2198.017128]  ? quirk_disable_msi.part.29+0x31/0x31
> > [ 2198.017131]  pciehp_resume+0x27/0x30
> > [ 2198.017136]  device_for_each_child+0x47/0x90
> > [ 2198.017140]  ? pci_restore_standard_config+0x40/0x40
> > [ 2198.017145]  pcie_port_device_runtime_resume+0x30/0x50
> > [ 2198.017150]  pci_pm_runtime_resume+0x6e/0xc0
> > [ 2198.017156]  __rpm_callback+0xb0/0x110
> > [ 2198.017161]  ? pci_restore_standard_config+0x40/0x40
> > [ 2198.017165]  rpm_callback+0x1f/0x70
> > [ 2198.017169]  ? pci_restore_standard_config+0x40/0x40
> > [ 2198.017173]  rpm_resume+0x4fe/0x690
> > [ 2198.017179]  ? irq_forced_thread_fn+0x70/0x70
> > [ 2198.017183]  __pm_runtime_resume+0x47/0x80
> > [ 2198.017187]  device_release_driver_internal+0x7c/0x1a0
> > [ 2198.017191]  pci_stop_bus_device+0x5d/0x80
> > [ 2198.017195]  pci_stop_bus_device+0x2b/0x80
> > [ 2198.017198]  pci_stop_and_remove_bus_device+0xe/0x20
> > [ 2198.017202]  pciehp_unconfigure_device+0x7c/0x130
> > [ 2198.017206]  pciehp_disable_slot+0x69/0x120
> > [ 2198.017211]  pciehp_handle_presence_or_link_change+0x7d/0x4b0
> > [ 2198.017215]  pciehp_ist+0x12a/0x1c0
> > [ 2198.017219]  irq_thread_fn+0x1b/0x60
> > [ 2198.017222]  irq_thread+0x15b/0x1c0
> > [ 2198.017226]  ? lockdep_hardirqs_on+0x115/0x180
> > [ 2198.017229]  ? wake_threads_waitq+0x30/0x30
> > [ 2198.017233]  ? irq_thread_dtor+0x90/0x90
> > [ 2198.017237]  kthread+0x101/0x140
> > [ 2198.017241]  ? kthread_cancel_delayed_work_sync+0x10/0x10
> > [ 2198.017245]  ret_from_fork+0x24/0x30
> > [ 2198.017897] xhci_hcd 0000:03:00.0: remove, state 1
> > [ 2198.017917] usb usb6: USB disconnect, device number 1
> > [ 2198.017925] usb 6-2: USB disconnect, device number 2
> > [ 2198.017932] usb 6-2.3: USB disconnect, device number 3
> > [ 2198.017939] usb 6-2.3.1: USB disconnect, device number 5
> > [ 2198.019565] sd 0:0:0:0: [sda] Synchronizing SCSI cache
> > [ 2198.019671] sd 0:0:0:0: [sda] Synchronize Cache(10) failed: Result:
> hostbyte=DID_NO_CONNECT driverbyte=DRIVER_OK
> > [ 2198.057200] xhci_hcd 0000:03:00.0: xHCI host controller not responding,
> assume dead
> > [ 2198.057227] r8152 6-2.4:1.0 enxd8d090034dcb: Stop submitting intr, status -
> 108
> > [ 2198.057574] usb 6-2.4: USB disconnect, device number 4
> > [ 2198.074541] xhci_hcd 0000:03:00.0: USB bus 6 deregistered
> > [ 2198.074558] xhci_hcd 0000:03:00.0: remove, state 1
> > [ 2198.074564] usb usb5: USB disconnect, device number 1
> > [ 2198.074566] usb 5-2: USB disconnect, device number 2
> > [ 2198.074569] usb 5-2.3: USB disconnect, device number 3
> > [ 2198.074571] usb 5-2.3.4: USB disconnect, device number 5
> > [ 2198.077367] usb 5-2.3.5: USB disconnect, device number 6
> > [ 2198.078488] usb 5-2.5: USB disconnect, device number 4
> > [ 2198.080340] xhci_hcd 0000:03:00.0: Host halt failed, -19
> > [ 2198.080344] xhci_hcd 0000:03:00.0: Host not accessible, reset failed.
> > [ 2198.080770] xhci_hcd 0000:03:00.0: USB bus 5 deregistered
> > [ 2198.081153] pci 0000:03:00.0: Removing from iommu group 3
> > [ 2198.081180] pci_bus 0000:03: busn_res: [bus 03] is released
> > [ 2198.081287] pci 0000:02:02.0: Removing from iommu group 3
> > [ 2198.081396] pci_bus 0000:04: busn_res: [bus 04-2d] is released
> > [ 2198.081509] pci 0000:02:04.0: Removing from iommu group 3
> > [ 2198.082273] pci_bus 0000:02: busn_res: [bus 02-2d] is released
> > [ 2198.085759] pci 0000:01:00.0: Removing from iommu group 3
> > [ 2198.630098] thunderbolt 0-1: device disconnected
> >
> > Additionally I've found after this I can't get the dock to work by hotplugging
> again
> > until I reboot the system.
> 
> Can you try if the following patch helps?
> 
> https://lore.kernel.org/patchwork/patch/1113961/

I just checked with that patch on top of my previous test kernel, same call trace
situation.

  reply index

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-19 11:22 Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 1/8] thunderbolt: Correct path indices for PCIe tunnel Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 2/8] thunderbolt: Move NVM upgrade support flag to struct icm Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 3/8] thunderbolt: Use 32-bit writes when writing ring producer/consumer Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 4/8] thunderbolt: Do not fail adding switch if some port is not implemented Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 5/8] thunderbolt: Hide switch attributes that are not set Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 6/8] thunderbolt: Expose active parts of NVM even if upgrade is not supported Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 7/8] thunderbolt: Add support for Intel Ice Lake Mika Westerberg
2019-08-19 11:22 ` [PATCH v3 8/8] ACPI / property: Add two new Thunderbolt property GUIDs to the list Mika Westerberg
2019-08-19 16:29 ` [PATCH v3 0/8] thunderbolt: Intel Ice Lake support Mario.Limonciello
2019-08-19 17:57   ` Mika Westerberg
2019-08-19 18:22     ` Mario.Limonciello [this message]
2019-08-19 19:36       ` Mario.Limonciello
2019-08-20 10:01         ` Mika Westerberg
2019-08-20 11:34   ` Lukas Wunner
2019-08-20 17:00     ` Mario.Limonciello
2019-08-26  9:20 ` Mika Westerberg

Reply instructions:

You may reply publically to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c56451e3ae244acd8a824fbca1322c0c@AUSX13MPC101.AMER.DELL.COM \
    --to=mario.limonciello@dell.com \
    --cc=David.Laight@aculab.com \
    --cc=YehezkelShB@gmail.com \
    --cc=andreas.noever@gmail.com \
    --cc=anthony.wong@canonical.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lukas@wunner.de \
    --cc=michael.jamet@intel.com \
    --cc=mika.westerberg@linux.intel.com \
    --cc=raanan.avargil@intel.com \
    --cc=rajmohan.mani@intel.com \
    --cc=rjw@rjwysocki.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-ACPI Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-acpi/0 linux-acpi/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-acpi linux-acpi/ https://lore.kernel.org/linux-acpi \
		linux-acpi@vger.kernel.org linux-acpi@archiver.kernel.org
	public-inbox-index linux-acpi

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-acpi


AGPL code for this site: git clone https://public-inbox.org/ public-inbox