Message-ID: <27bdf845-9cbf-406e-b156-472ca79c94a8@linux.intel.com>
Date: Fri, 6 Sep 2024 16:41:22 +0300
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 0/4] Add device links between tunneled USB3 devices and
 USB4 Host
To: Mario Limonciello <mario.limonciello@amd.com>,
 Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: linux-usb@vger.kernel.org, "Rangoju, Raju" <Raju.Rangoju@amd.com>,
 Sanath S <Sanath.S@amd.com>, gregkh@linuxfoundation.org
References: <20240830152630.3943215-1-mathias.nyman@linux.intel.com>
 <321cf204-cf74-4f51-8ed3-8c5738444359@amd.com>
 <20240904050013.GZ1532424@black.fi.intel.com>
 <7a3fc5cd-1705-40ab-8d93-d9f286009aa7@amd.com>
 <20240905055330.GH1532424@black.fi.intel.com>
 <830cc833-24a7-4e19-8804-6b4135e122d1@amd.com>
Content-Language: en-US
From: Mathias Nyman <mathias.nyman@linux.intel.com>
In-Reply-To: <830cc833-24a7-4e19-8804-6b4135e122d1@amd.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

On 5.9.2024 21.57, Mario Limonciello wrote:
> On 9/5/2024 00:53, Mika Westerberg wrote:
>> On Wed, Sep 04, 2024 at 12:38:15PM -0500, Mario Limonciello wrote:
>>> ❯ ls -l /sys/bus/pci/drivers/thunderbolt/*/consumer*
>>> lrwxrwxrwx 1 root root 0 Sep  4 12:33
>>> /sys/bus/pci/drivers/thunderbolt/0000:c4:00.5/consumer:pci:0000:00:03.1 ->
>>> ../../../virtual/devlink/pci:0000:c4:00.5--pci:0000:00:03.1/
>>> lrwxrwxrwx 1 root root 0 Sep  4 12:33
>>> /sys/bus/pci/drivers/thunderbolt/0000:c4:00.6/consumer:pci:0000:00:04.1 ->
>>> ../../../virtual/devlink/pci:0000:c4:00.6--pci:0000:00:04.1/
>>> ❯ cat /sys/bus/pci/drivers/thunderbolt/*/power/runtime_status
>>> suspended
>>> active
>>>
>>> I would have expected the USB4 host router to go back into runtime PM, but
>>> it doesn't anymore until I reboot the system.
>>
>> Yes, it should enter runtime suspend after a while. Would you mind
>> sharing dmesg around this?
> 
> I was capturing artifacts when all of a sudden it started to work.  Then I remembered I updated the BIOS on this system very recently.  This is a pre-production BIOS.
> 
> I downgraded back to old BIOS and everything works as you expect, so there is some BIOS bug at play.  I'll see if others can reproduce my result and drive a BIOS solution.
> 
> Sorry for the noise on the patch, all is great!
> 
> Tested-by: Mario Limonciello <mario.limonciello@amd.com>
> 

Thanks for testing it

-Mathias