From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f49.google.com (mail-pj1-f49.google.com [209.85.216.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E361A30E858 for ; Sun, 23 Aug 2026 15:18:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787498282; cv=none; b=mTrD0E06ABFWr833wFDN3sNIl1Al76uaznsmqBgYJ38FAqL7srHFxiu68NbxAsP4lDvWkUQdlcqEqnsE57d7T7jbHx484PwrPWAXDJYVuQmvqjoZM0Y0x8XfZ4Qzh/e3vwyHH/PPoOlGFxwmS/xZd2rZMUa3y47CP9MjYfGD27Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787498282; c=relaxed/simple; bh=6lN12AIgQiPJz2Kow1rLb4w6YVrECt7pOuPq56ji09c=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=gRqS1ZDa6wEJM9grCSSjaRW1G9+AZrraaj0Ao8XRHQlmAbMqALZ0nVukFSR6YAkdHblBqtRKxlW2vD0k5ncJJbdE9kRVDe81TZ/RQI66scY/0nV7oplcRTDmydKU248wnphBIs+LwR9VJ/Cyv9kXYNlHOpQvCptfi4O4UFPLzwg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=iz5uzO+f; arc=none smtp.client-ip=209.85.216.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="iz5uzO+f" Received: by mail-pj1-f49.google.com with SMTP id 98e67ed59e1d1-38e69bdb0fcso2202945a91.1 for ; Sun, 23 Aug 2026 08:18:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787498280; x=1788103080; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=8+kkSUH6z+rcpiejvViicwcB0HlF0dRs5GEUbWGBoZ0=; b=iz5uzO+f/JjrZnOHtid3n7AtIomaL56xyEjyFGS2oGmdk2VYNYsr3VSCC12Nr8g2bQ 5JbPM6xaUZEvZ7xHmwO8sB/vNHulqs/MgMnHY73TMY/I+1IR/eBSqEC98Xw1AzeBLBSF wYS2nzBAEiWxn16KET/i0+sE7Oz/Nr8s9BE6PH6C8Ev2NmjG0F87wqaOmtUc2N3zrwW6 pXj2w0zYwz/wQzcEcexCQfaZrOkhYHW+NOZMebFhq1i1vM8JxwJbGZr+0upmLy2d6dFH BkMhV8i4rjK02jxIJlYi6gWCVPWumjoiFbKkICbhr+s77u8lI/KH/40n6kHMVNZclEtT sYtw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787498280; x=1788103080; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=8+kkSUH6z+rcpiejvViicwcB0HlF0dRs5GEUbWGBoZ0=; b=QREpm6MyRPVjAxmA/YKuDa5KsCj9YzswezfpIq+uMWMhYFOHbxCVfoEVB37dCt8wzJ Ynsvms1mKkuVVDjUAFON2qtZCbXZLFnu4XttPnVCjkhAXarIV4vr+uiAzU1FXQgfl1TH P16Q7f5fgoFAFUYAvPq8Y2603UZE6Mjv3wk4iiHWEbJbshGj0moJyTQG/8Nspc3MwGL7 xJNwZPvIYq9i+bDdST4T1dqpmcyMLrt6v/q3ja4PCS81blpAu1ic39xzCs4LIgCJDEP8 8BZ/KJzjkUEfoqI8nedYTZaKAQ7KLyj5zEnl+zMWX41QbJQz78bgtnOe9rU56uppt0Ag DkXQ== X-Forwarded-Encrypted: i=1; AHgh+RpQMVMGc6BpcmG+A25LprfWxOF4foSCmUaqwS/37xhjePqhVePFZo8ycSVQkK2DGtC6m7lWnnrfVhrJocs=@vger.kernel.org X-Gm-Message-State: AFuF++kUg9qKWzX34Q/ZT8OC4TxUIHGKDHhfkgwcf5/UMcgOSWBMgJvh +GAnuKMts9+Dz8Xe3QsxPXqUAVUtW4XDITbiR9+7M+N43QTD4b7Bh+zj X-Gm-Gg: AR+sD12Gt2LaHrko4Bx83RCq/lbTwRd5LFkD31da/XX7gLC/sDfaVkqqy5TeJq8F9Ld LjfMxSEuWB4yoabKA51YMBkleMk90L5NuOm71nhz50ZrzdnUScSYqn7IQACGWnrAxHZ4FRFqKnN YfuFSI01jwIeUedoQbN/ahLw++Ch4oVXZnPjxrbUYJMDeTTZzBD9aPcszR2eY0uic07WBkWsr3S SkHSK37GkwoOig6yUw5Af4centkb5+z5KJoq+07S0bdjvRE0yf1KVGDDvjwozi4m6epYtd62t0p PDk0d373EPr4hQ+JgOuewalQMVmEdkaMxM8fSJn94ZhMQ3NaIuHYNDM9cTZF1K8VwqneX8YcZoD y1W5RAZiTuwb4b74u/sQnICYRyosiZ4rWv8kDvUD3R/5w5df6F5A5E+7MW6BEXoMcpSYDf13DT4 fFHdPhx9LeydUvasC1MD8OZUT6HyjL6S/h2Duyubb5TNxJhUjOLSTxsMhw39oRecqpM6nJFOFPb Q7Omuqk3hon17kWzR8cL5W/L6NyOzc= X-Received: by 2002:a17:90b:3883:b0:380:540:d499 with SMTP id 98e67ed59e1d1-395def03e79mr20461038a91.6.1787498279919; Sun, 23 Aug 2026 08:17:59 -0700 (PDT) Received: from localhost ([2409:40c4:10ff:376f:bea1:1f87:a96a:c2c3]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-141860ec7f7sm17786260c88.6.2026.08.23.08.17.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 23 Aug 2026 08:17:59 -0700 (PDT) Date: Sun, 23 Aug 2026 20:47:55 +0530 From: Lovekesh Solanki To: Thorsten Leemhuis Cc: Mathias Nyman , linux-usb@vger.kernel.org, regressions@lists.linux.dev, stable@vger.kernel.org, linux-kernel@vger.kernel.org, Mario Limonciello , Forest , Slavik Dev , Michal Pecio , Mathieu Fluhr Subject: Re: [REGRESSION] 6.12.36+: usb: hub: post-resume delayed work triggers > uncorrected MCE / data fabric sync flood on Threadripper 7970X > (bisected to aec11e5f9c45) Message-ID: References: <07435e6b-ee30-4c85-8c8b-0ce3a4ead1d9@leemhuis.info> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <07435e6b-ee30-4c85-8c8b-0ce3a4ead1d9@leemhuis.info> On Sun, Aug 23, 2026 at 12:36:55PM +0200, Thorsten Leemhuis wrote: > that ticket apparently didn't help. And no other fix is in sight, or am > I missing something? > > Mathias reverting the culprit in mainline a option to resolve this (I > assume it is not, but I'm asking just to be sure)? I don't think a revert is needed here, we're just trading one regression for another. Commit 8f5b7e2bec1c introduced the 200ms hold for all superspeed hubs, but it's only useful for external ones. Root hubs are superspeed hubs too so they go through the same code, but there's nothing useful for the hold to do there: A root hub has no upstream suspended hub whose wake propagation we need to wait for and xhci already handles late USB3 link training itself. Also hubs have a 0 second autosuspend delay (596d789a211d), so this hold is the only thing stretching the awake window. and anything opening or closing /dev/bus/usb nodes auto resumes and auto suspends the whole host controller, adb's periodic enumeration does that so on the reported affected systems every SS roothub cycle grows from ~30 to ~235ms (from the dynamic debug traces in https://lore.kernel.org/all/qc0nhk9c6l0a08bkfeplrm3qjssgrjkvkp@sonic.net/) which makes suspend move from close() call into delayed work and results in the ~1Hz stress loop described upthread. I think skipping the hold for hubs without a parent device, i.e. keep TB dock behaviour everywhere it matters, should fix this. Patch below could any of the reporters test it? Thanks, DrEggCake diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index d92bf887739d..f642ea0c31dc 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c @@ -1346,7 +1346,8 @@ static void hub_activate(struct usb_hub *hub, enum hub_activation_type type) device_unlock(&hdev->dev); } - if (type == HUB_RESUME && hub_is_superspeed(hub->hdev)) { + if (type == HUB_RESUME && hub->hdev->parent && + hub_is_superspeed(hub->hdev)) { /* give usb3 downstream links training time after hub resume */ usb_autopm_get_interface_no_resume( to_usb_interface(hub->intfdev));