Received-SPF: pass (google.com: domain of linux-kernel+bounces-7333-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <5bfaa405-b15e-36ef-a4e0-04b93dd983b1@amd.com>
Date: Wed, 20 Dec 2023 22:31:20 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [Patch v2 2/2] thunderbolt: Teardown tunnels and reset downstream
 ports created by boot firmware
Content-Language: en-US
To: Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: Sanath S <Sanath.S@amd.com>, mario.limonciello@amd.com,
 andreas.noever@gmail.com, michael.jamet@intel.com, YehezkelShB@gmail.com,
 linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20231218104234.GB1074920@black.fi.intel.com>
 <c433f29b-597c-b6d6-aa48-2b84a26dc623@amd.com>
 <20231218113151.GC1074920@black.fi.intel.com>
 <20231218122312.GE1074920@black.fi.intel.com>
 <997f2a94-66d9-fb95-8f75-46d61937f7e8@amd.com>
 <20231218131840.GH1074920@black.fi.intel.com>
 <0fd5c09f-1cf2-8813-a8f9-1bd856e3a298@amd.com>
 <20231219122634.GJ1074920@black.fi.intel.com>
 <0816caa4-81b5-d0f9-2305-80c7fec6abb9@amd.com>
 <20231219180424.GL1074920@black.fi.intel.com>
 <20231220125857.GA2543524@black.fi.intel.com>
From: Sanath S <sanaths2@amd.com>
In-Reply-To: <20231220125857.GA2543524@black.fi.intel.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?dTR3dDdGeExTS3g2UjFSNWJwbGpOcDBJWHZmc0ZsWCtuc2xJNWd0OGQrRTlx?=
 =?utf-8?B?bllYTTJQMFNRd1hvTFBSaUpTVFM0WTg0eFppcW9ibXpKMFpYVGZQZGZmNDRD?=
 =?utf-8?B?SlhnZ0loMkliV0RzRW9tSFg1U3ZEd21QaTdyT2tSZjNSaWM3b2dRbi80cWRt?=
 =?utf-8?B?STlSM3RUTmkvdjRVK1pnUXg3MEt2anF5QzEwV2kxUVRrYnlXUG1RZGJGaFA0?=
 =?utf-8?B?ZkQ3TDhYTE5MS2grd1Z2ZzkydFZrc09XckdyVTd5LzZQWWpqM0NMbGg1ZmEz?=
 =?utf-8?B?UHdvMmx6WkpoRzhqZ0NCUm5XbjNqaFNDWVhERHBNaWFtaGdoanphMWFUazFq?=
 =?utf-8?B?REtySVhoMks4aWZidVltaC9jd3Vyb0NjeHkxamtmSDlJdG5iTmF1eTlsdDlS?=
 =?utf-8?B?L1VPSVFXaTVOMWdvaTc0R0JVdzNDTDk3dERTcDJrWnF2TUl5RnoySlRIelR2?=
 =?utf-8?B?bDJuRVEyVTk5SlhJZHAwQmZQbVRIN0RoYStIUlVJa2RCbU9lTmVxdHptbkpr?=
 =?utf-8?B?ZGxXVmd6T2JnZ1hKNkQ2RGdCZXlXQUlvZjBjbGhNWlZlVC9zUGIrSFk1a2cx?=
 =?utf-8?B?SElDMG54bFdja1YweWRjazJQU2owaTEvRFdnTWh2OUwxQWNZU2ZtYm1KNTNG?=
 =?utf-8?B?VmVhazZmTVFseDBpbiticXJkenpSc1M5cytGbzZwSXdyUlZ3MmlJSmppUTJD?=
 =?utf-8?B?RHBnb2t2WXl1aWh1ak1RQ01id3Q2OExRTm1tQ3duaTIxWUNrb08zeC9zOU0y?=
 =?utf-8?B?UG9QSU1pV3I1TzcyZ2FmcDdld2NhQU9HSWEyVlVmQm0zYThzU3RUZUEyeHNJ?=
 =?utf-8?B?RUVYUTgwL3VUejVXYmlLUExVM0JLSDZMemFwZWlEaDFwbmhKN0VCYzhiSnRO?=
 =?utf-8?B?bDlWcUgrMFV3SXFBTXZra1JRYmxyZjdtcjg0bDE3Z1VGZGJFaXVlMzJMVEh0?=
 =?utf-8?B?TkV3dVQ4R3lTVXNmeEF0TjBCZURydFRnT01WVGRVZ3B4WDZlNWxCcTJIOVI0?=
 =?utf-8?B?cUF2NGg4QnFzSDRVelZteUl3UkhaQ0RuM28xOU5vRGVoYUxYbVBKUG1iZ3VX?=
 =?utf-8?B?TTAwek5tMm9LbXpFYVVaRWZpTmpuYldkV2luNzl1VUFDbCtZVWUxMHVQc2w0?=
 =?utf-8?B?ZERzZzcvL04xKzU0MTFnTkhuTk9SNjY0YXVFdlNjZG9KK0VROXBhZHVSYnpS?=
 =?utf-8?B?d0dwdDc2dVlVTER0ditkWVRDRDFpZmFQL29OM3hMRld1S0xySFJpT1BFTFBj?=
 =?utf-8?B?N2oya1JPMlRtcDZqR0JuTHR5aThaS2QyYVo5MTExazRZTTEwMytablc4T2xr?=
 =?utf-8?B?NnY3dkI0Y2pSY3BXbXFGRjBRZnhsMk4zSU9wWC9WWlhnUkxYUWFNeDdIdnQ1?=
 =?utf-8?B?UDMwL3VxUG52QjR6ZTRHbzNVRCswMjZzbzBXeStOZ1Q0a3VmMmRVNktCRTRB?=
 =?utf-8?B?Y3c5SmVPRVY0empwbzczWnRraHJ4bUNzTHJncTVsTm02UndOR0pDYXFZMkEr?=
 =?utf-8?B?TkJFQUNidUFEZXNBdk02ajBjaFpydEFzMWJodzJ4dHhFaUtzWDRFMWlQZDly?=
 =?utf-8?B?ODFiTUJIcVFHRDhxSFJEWlhxdWRTVjhKY2tuZ2pEYjVlNlZVMXE0bWtaNkln?=
 =?utf-8?B?czRuZjkrOXpzSzNMR2lMMnhjTjZSQmUzYnk5eExROW9TZXdYbkdGN1hBMmds?=
 =?utf-8?B?R0NJWEtXV0FZTG95STlXcm01RjIzOHF3Q1hQMk9xaFRaWjgvR0pST0FwczlC?=
 =?utf-8?B?U3hWQmZlcUZ2ZWE3UG5IRHpDcmZZUzIrcFhwRE9kR1hPNVFCR1BSTnBod05W?=
 =?utf-8?B?SkRFTjJpZkZqSjRzWGdwVTdhU3NxLzc2YUxSdGEyQzUxZTBBK0JiMmNPVjI5?=
 =?utf-8?B?QlFrQ1o2NC81RFpydUkzWDNweSs3ZFBPMERPSEpvVjBuUG9NUmpKTkw3ck5D?=
 =?utf-8?B?b3l1RGpqbUM2UXIzUGYrUmRyMTZMaTRyZE41UC9nMWtVVWtqNEdSWnJlOTlV?=
 =?utf-8?B?alpDUi9VN29UWm1IWThFRmowcEhoRlZvSlNBNVRxOGkwdXB4UVVPVS9CTjVO?=
 =?utf-8?B?OUtyVWZpVHE5RnRSVnZYeE5HYjFsMmJOQUwwL0FJemVOakNCZ1d6a1JNZVZD?=
 =?utf-8?Q?mDDgX8Es2hAvYqsCspkHrTSFG?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5c6ec236-b48b-4245-2b99-08dc017d4ee3
X-MS-Exchange-CrossTenant-AuthSource: MW4PR12MB7016.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Dec 2023 17:01:29.3139
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: uiRKiBql7Y/g3yGgkEwV7jiyZsaw76xFERQDimIEteKBHezyuL0oectUpMHHJ6NAqB+uPpL5tpHy4EB7oqBJjA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB8323


On 12/20/2023 6:28 PM, Mika Westerberg wrote:
> On Tue, Dec 19, 2023 at 08:04:24PM +0200, Mika Westerberg wrote:
>>>> One additional question though, say we have PCIe tunnel established by
>>>> the BIOS CM and we do the "reset", that means there will be hot-remove
>>>> on the PCIe side and then hotplug again, does this slow down the boot
>>>> considerably? We have some delays there in the PCIe code that might hit
>>>> us here although I agree that we definitely prefer working system rather
>>>> than fast-booting non-working system but perhaps the delays are not
>>>> noticeable by the end-user?
>>> I've not observed any delay which is noticeable. As soon as I get the login
>>> screen
>>> and check dmesg, it would already be enumerated.
>> Okay, I need to try it on my side too.
> One additional thing that came to mind. Please check with some device
> with a real PCIe endpoint. For instance there is the integrated xHCI
> controller on Intel Titan Ridge and Goshen Ridge based docks. With TR it
> is easy because it does not support USB4 so xHCI is brought up
> immediately once there is PCIe tunnel. For GR (the OWC dock you have) it
> is disabled when the link is USB4 (because USB 3.x is tunneled as well)
> but you can get it enabled too if you connect it with an active TBT3
> cable.
Sure. I'll check with these combinations.