From nobody Wed Apr 8 17:17:07 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F283BFA3741 for ; Fri, 28 Oct 2022 09:01:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229828AbiJ1JB0 (ORCPT ); Fri, 28 Oct 2022 05:01:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56564 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229549AbiJ1JBX (ORCPT ); Fri, 28 Oct 2022 05:01:23 -0400 Received: from mail-yw1-x1149.google.com (mail-yw1-x1149.google.com [IPv6:2607:f8b0:4864:20::1149]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 784A05B9F2 for ; Fri, 28 Oct 2022 02:01:22 -0700 (PDT) Received: by mail-yw1-x1149.google.com with SMTP id 00721157ae682-36f8318e4d0so39273647b3.20 for ; Fri, 28 Oct 2022 02:01:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=hAmqKoUne0t3PzKfRmKr49ptCla0dfe721hyNgLvUyE=; b=cdbdCQIsm/LZ5BUiQ3qFhisI6ul9bB9aozYKXmwBnB2DDANatskHwcMCwULCO23Ktx 8gzAjppfmSgbPykMyYAK0yq/BgGXzadwSLnK1qkhx5HLJwcChIZK8d889jr6VgEAteWD ESNrP9NkMM11rEnhK6OPszQ3wGD4cKLefKj9tsn/e6vnl7m6+YFcjdlNJGCJmANjIw2x P1Fk7eXHs2TyQkM+DfLnlvKfJMe2urAPD+AXIGW6qKAG+GqsNW7nW9U0G6rGSvp2ry86 seUCrRes+9PsVqVO9xewoImgcxKDme9OcibXYSez5AJgsxdZ3i3TanfsMbr+lM4z1fT1 ZqGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=hAmqKoUne0t3PzKfRmKr49ptCla0dfe721hyNgLvUyE=; b=EhnoYYHrhjromlz4GEtATn2FvrlHzTBCfPG1msPM7ORy1gvc6oD/n4qYye7CllKEeq t2C2Sisvqiv6SmkPNWUW8v3EnR5s2vBTd5F54ERW9Tn0SjEyxG32/rsGfIV5Ce+ujFIe 685njB8DQ+qAmmD/dyNSeFG7Lktsro80avKfwK6X+KgIlusi4J0SQbQ2hxiVbfik+o+T r7NzflhlhdjxgL53SPl4wMn0qy4p5WFgG+bQV+RxTGpPWaG+RRoUzlKsuvPFHwqDN35k oT3rV0aJnfWRXsBzC3VoQ0VJmd7mKxFUmjH8R9qBJR4fF4Fy9LJQw67h5XiOxqs4S6Vg IEsw== X-Gm-Message-State: ACrzQf19lpKrJPSgimPxlZYqfUvkjHaSC22iMMTLcX0dQEjxQYGZo65m 3LnaL+pRNVpVSa4G7PVPRUUUH3ypMzY= X-Google-Smtp-Source: AMsMyM6dHm16BB9w0OHIqGP7xzgFMLd6b/b59odbMwdxz2USn5S6oS/M2OAKqikO8ft3lJ5PQVymIH5wxk4= X-Received: from raychi.tao.corp.google.com ([2401:fa00:fc:202:c8cf:1264:21c3:c6ff]) (user=raychi job=sendgmr) by 2002:a25:bbcc:0:b0:6a8:e269:5eec with SMTP id c12-20020a25bbcc000000b006a8e2695eecmr50520258ybk.219.1666947681755; Fri, 28 Oct 2022 02:01:21 -0700 (PDT) Date: Fri, 28 Oct 2022 17:01:16 +0800 Mime-Version: 1.0 X-Mailer: git-send-email 2.38.1.273.g43a17bfeac-goog Message-ID: <20221028090116.199601-1-raychi@google.com> Subject: [PATCH RESEND v4] usb: core: stop USB enumeration if too many retries From: Ray Chi To: gregkh@linuxfoundation.org, stern@rowland.harvard.edu, m.grzeschik@pengutronix.de Cc: albertccwang@google.com, pumahsu@google.com, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, Ray Chi Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When a broken USB accessory connects to a USB host, usbcore might keep doing enumeration retries. If the host has a watchdog mechanism, the kernel panic will happen on the host. This patch provides an attribute early_stop to limit the numbers of retries for each port of a hub. If a port was marked with early_stop attribute, unsuccessful connection attempts will fail quickly. In addition, if an early_stop port has failed to initialize, it will ignore all future connection events until early_stop attribute is clear. Signed-off-by: Ray Chi --- Documentation/ABI/testing/sysfs-bus-usb | 11 +++++ drivers/usb/core/hub.c | 59 +++++++++++++++++++++++++ drivers/usb/core/hub.h | 4 ++ drivers/usb/core/port.c | 27 +++++++++++ 4 files changed, 101 insertions(+) diff --git a/Documentation/ABI/testing/sysfs-bus-usb b/Documentation/ABI/te= sting/sysfs-bus-usb index 568103d3376e..545c2dd97ed0 100644 --- a/Documentation/ABI/testing/sysfs-bus-usb +++ b/Documentation/ABI/testing/sysfs-bus-usb @@ -264,6 +264,17 @@ Description: attached to the port will not be detected, initialized, or enumerated. =20 +What: /sys/bus/usb/devices/...//port/early_stop +Date: Sep 2022 +Contact: Ray Chi +Description: + Some USB hosts have some watchdog mechanisms so that the device + may enter ramdump if it takes a long time during port initialization. + This attribute allows each port just has two attempts so that the + port initialization will be failed quickly. In addition, if a port + which is marked with early_stop has failed to initialize, it will ignore + all future connections until this attribute is clear. + What: /sys/bus/usb/devices/.../power/usb2_lpm_l1_timeout Date: May 2013 Contact: Mathias Nyman diff --git a/drivers/usb/core/hub.c b/drivers/usb/core/hub.c index bbab424b0d55..5510dbef3243 100644 --- a/drivers/usb/core/hub.c +++ b/drivers/usb/core/hub.c @@ -3081,6 +3081,48 @@ static int hub_port_reset(struct usb_hub *hub, int p= ort1, return status; } =20 +/* + * hub_port_stop_enumerate - stop USB enumeration or ignore port events + * @hub: target hub + * @port1: port num of the port + * @retries: port retries number of hub_port_init() + * + * Return: + * true: ignore port actions/events or give up connection attempts. + * false: keep original behavior. + * + * This function will be based on retries to check whether the port which = is + * marked with early_stop attribute would stop enumeration or ignore event= s. + * + * Note: + * This function didn't change anything if early_stop is not set, and it w= ill + * prevent all connection attempts when early_stop is set and the attempts= of + * the port are more than 1. + */ +static bool hub_port_stop_enumerate(struct usb_hub *hub, int port1, int re= tries) +{ + struct usb_port *port_dev =3D hub->ports[port1 - 1]; + + if (port_dev->early_stop) { + if (port_dev->ignore_event) + return true; + + /* + * We want unsuccessful attempts to fail quickly. + * Since some devices may need one failure during + * port initialization, we allow two tries but no + * more. + */ + if (retries < 1) + return false; + + port_dev->ignore_event =3D 1; + } else + port_dev->ignore_event =3D 0; + + return port_dev->ignore_event; +} + /* Check if a port is power on */ int usb_port_is_power_on(struct usb_hub *hub, unsigned int portstatus) { @@ -4855,6 +4897,11 @@ hub_port_init(struct usb_hub *hub, struct usb_device= *udev, int port1, buf->bMaxPacketSize0; kfree(buf); =20 + if (r < 0 && hub_port_stop_enumerate(hub, port1, retries)) { + retval =3D r; + goto fail; + } + retval =3D hub_port_reset(hub, port1, udev, delay, false); if (retval < 0) /* error or disconnect */ goto fail; @@ -5387,6 +5434,9 @@ static void hub_port_connect(struct usb_hub *hub, int= port1, u16 portstatus, if ((status =3D=3D -ENOTCONN) || (status =3D=3D -ENOTSUPP)) break; =20 + if (hub_port_stop_enumerate(hub, port1, i)) + break; + /* When halfway through our retry count, power-cycle the port */ if (i =3D=3D (PORT_INIT_TRIES - 1) / 2) { dev_info(&port_dev->dev, "attempt power cycle\n"); @@ -5614,6 +5664,10 @@ static void port_event(struct usb_hub *hub, int port= 1) if (!pm_runtime_active(&port_dev->dev)) return; =20 + /* skip port actions if ignore_event is true*/ + if (hub_port_stop_enumerate(hub, port1, 0)) + return; + if (hub_handle_remote_wakeup(hub, port1, portstatus, portchange)) connect_change =3D 1; =20 @@ -5934,6 +5988,9 @@ static int usb_reset_and_verify_device(struct usb_dev= ice *udev) ret =3D hub_port_init(parent_hub, udev, port1, i); if (ret >=3D 0 || ret =3D=3D -ENOTCONN || ret =3D=3D -ENODEV) break; + + if (hub_port_stop_enumerate(parent_hub, port1, i)) + goto stop_enumerate; } mutex_unlock(hcd->address0_mutex); =20 @@ -6022,6 +6079,8 @@ static int usb_reset_and_verify_device(struct usb_dev= ice *udev) udev->bos =3D bos; return 0; =20 +stop_enumerate: + mutex_unlock(hcd->address0_mutex); re_enumerate: usb_release_bos_descriptor(udev); udev->bos =3D bos; diff --git a/drivers/usb/core/hub.h b/drivers/usb/core/hub.h index b2925856b4cb..e23833562e4f 100644 --- a/drivers/usb/core/hub.h +++ b/drivers/usb/core/hub.h @@ -90,6 +90,8 @@ struct usb_hub { * @is_superspeed cache super-speed status * @usb3_lpm_u1_permit: whether USB3 U1 LPM is permitted. * @usb3_lpm_u2_permit: whether USB3 U2 LPM is permitted. + * @early_stop: whether port initialization will be stopped earlier. + * @ignore_event: whether events of the port are ignored. */ struct usb_port { struct usb_device *child; @@ -103,6 +105,8 @@ struct usb_port { u32 over_current_count; u8 portnum; u32 quirks; + unsigned int early_stop:1; + unsigned int ignore_event:1; unsigned int is_superspeed:1; unsigned int usb3_lpm_u1_permit:1; unsigned int usb3_lpm_u2_permit:1; diff --git a/drivers/usb/core/port.c b/drivers/usb/core/port.c index 38c1a4f4fdea..126da9408359 100644 --- a/drivers/usb/core/port.c +++ b/drivers/usb/core/port.c @@ -17,6 +17,32 @@ static int usb_port_block_power_off; =20 static const struct attribute_group *port_dev_group[]; =20 +static ssize_t early_stop_show(struct device *dev, + struct device_attribute *attr, char *buf) +{ + struct usb_port *port_dev =3D to_usb_port(dev); + + return sysfs_emit(buf, "%s\n", port_dev->early_stop ? "yes" : "no"); +} + +static ssize_t early_stop_store(struct device *dev, struct device_attribut= e *attr, + const char *buf, size_t count) +{ + struct usb_port *port_dev =3D to_usb_port(dev); + bool value; + + if (kstrtobool(buf, &value)) + return -EINVAL; + + if (value) + port_dev->early_stop =3D 1; + else + port_dev->early_stop =3D 0; + + return count; +} +static DEVICE_ATTR_RW(early_stop); + static ssize_t disable_show(struct device *dev, struct device_attribute *attr, char *buf) { @@ -236,6 +262,7 @@ static struct attribute *port_dev_attrs[] =3D { &dev_attr_quirks.attr, &dev_attr_over_current_count.attr, &dev_attr_disable.attr, + &dev_attr_early_stop.attr, NULL, }; =20 --=20 2.38.1.273.g43a17bfeac-goog