public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mika Westerberg <mika.westerberg@linux.intel.com>
To: Brad Campbell <lists2009@fnarfbargle.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Apple Thunderbolt Display chaining
Date: Sat, 6 Aug 2022 09:13:46 +0300	[thread overview]
Message-ID: <Yu4Gmkous7asng6h@lahna> (raw)
In-Reply-To: <87c1a001-ef79-6390-dfe2-06d2850f6e84@fnarfbargle.com>

Hi Brad,

On Fri, Aug 05, 2022 at 10:43:54PM +0800, Brad Campbell wrote:
> On 5/8/22 22:21, Mika Westerberg wrote:
> 
> > They are pretty standard so I suspect myself the display side of things.
> > Not sure if it is possible (I think it is from sysfs /sys/class/drm/*)
> > to disable the tunneled DP connections and see if that makes it not
> > hang. Alternatively you can try to comment out the call to
> > tb_tunnel_dp() from the driver. Let me know if you want me to make hack
> > patch that does it for you.
> > 
> 
> I used this :
> 
> iff --git a/drivers/thunderbolt/tb.c b/drivers/thunderbolt/tb.c
> index 9a3214fb5038..eae7523af78b 100644
> --- a/drivers/thunderbolt/tb.c
> +++ b/drivers/thunderbolt/tb.c
> @@ -865,7 +865,7 @@ static void tb_tunnel_dp(struct tb *tb)
>         struct tb_cm *tcm = tb_priv(tb);
>         struct tb_port *port, *in, *out;
>         struct tb_tunnel *tunnel;
> -
> +       return;
>         if (!tb_acpi_may_tunnel_dp()) {
>                 tb_dbg(tb, "DP tunneling disabled, not creating tunnel\n");
>                 return;
> 
> I'm now using Linus GIT head. No change in behaviour really.
> 
> No tunnels were created. None of the TB displays light up in Linux or BIOS. System still locks
> up on reboot and the first time I attempted to compose this reply it locked up hard before I
> had a chance to finish it because I attempted to verify the devices were present with an lspci -tv.
> 
> If I do an lspci -tv after the module load, it locks hard instantly and reproducibly.

Okay, let's try to deal with one issue at the time so first the hang
that appears after the OS is booted up. Do you still have the
"pcie_port_pm=off" in the kernel command line? The hang that happens
afterwards sounds exactly like that the runtime PM would kick in but the
parameter should prevent that from happening. Since you are connecting
Thunderbolt 1 devices there is really no PM we can do for them at all
and the TBT driver should block it too.

Is it possible to narrow this down to a single device connected and then
get me the full dmesg output (with CONFIG_PCI_DEBUG=y) and output of
'sudo lspci -vv'?

  reply	other threads:[~2022-08-06  6:13 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <acbb3a86-ea15-47ec-90fa-72fbd94921b1@fnarfbargle.com>
2022-03-29 11:31 ` Apple Thunderbolt Display chaining Mika Westerberg
2022-03-29 12:35   ` Brad Campbell
2022-03-29 13:00     ` Mika Westerberg
2022-03-29 14:06       ` Brad Campbell
2022-03-30 10:18         ` Mika Westerberg
2022-03-30 13:19           ` Brad Campbell
2022-03-30 13:43             ` Mika Westerberg
2022-03-30 14:24               ` Brad Campbell
2022-03-30 14:47                 ` Mika Westerberg
2022-03-31  9:02                   ` Brad Campbell
2022-03-31 16:36                     ` Mika Westerberg
2022-04-01  5:48                       ` Brad Campbell
2022-04-01 14:30                         ` Mika Westerberg
2022-04-01 15:05                           ` Brad Campbell
2022-04-04 10:10                             ` Mika Westerberg
2022-04-04 11:38                               ` Brad Campbell
2022-04-04 12:53                                 ` Mika Westerberg
2022-04-06  2:51                                   ` Brad Campbell
2022-04-06 14:56                                     ` Mika Westerberg
2022-08-05  7:41                                       ` Brad Campbell
2022-08-05 11:30                                         ` Mika Westerberg
2022-08-05 12:43                                           ` Brad Campbell
2022-08-05 13:01                                             ` Mika Westerberg
2022-08-05 14:13                                               ` Brad Campbell
2022-08-05 14:21                                                 ` Mika Westerberg
2022-08-05 14:43                                                   ` Brad Campbell
2022-08-06  6:13                                                     ` Mika Westerberg [this message]
2022-08-06  9:41                                                       ` Brad Campbell
2022-08-08  9:51                                                         ` Mika Westerberg
2022-08-08 11:55                                                           ` Brad Campbell
2022-08-08 12:25                                                             ` Brad Campbell
2022-08-08 12:46                                                               ` Mika Westerberg
2022-08-08 13:27                                                                 ` Brad Campbell
2022-08-09 10:23                                                                   ` Mika Westerberg
2022-08-09 10:40                                                                     ` Brad Campbell
2022-08-09 10:55                                                                       ` Mika Westerberg
2022-08-09 11:03                                                                         ` Brad Campbell
2022-08-09 11:08                                                                         ` Brad Campbell
2022-08-09 14:42                                                                           ` Mika Westerberg
2022-08-09 15:16                                                                             ` Brad Campbell
2022-08-09 15:50                                                                               ` Mika Westerberg
2022-08-10  7:40                                                                                 ` Brad Campbell
2022-08-11  9:50                                                                                   ` Mika Westerberg
2022-08-11 14:17                                                                                     ` Brad Campbell
2022-08-12  9:35                                                                                       ` Mika Westerberg
2022-08-12 10:16                                                                                         ` Brad Campbell
2022-08-08 12:42                                                             ` Mika Westerberg

Reply instructions:

You may reply publicly 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=Yu4Gmkous7asng6h@lahna \
    --to=mika.westerberg@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lists2009@fnarfbargle.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox