Received: by 2002:a05:7412:8521:b0:e2:908c:2ebd with SMTP id t33csp554429rdf; Fri, 3 Nov 2023 08:16:24 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFBVUbCKssUXRgw+6UivWrFks1glgLAPY0RxAoVjk1vrrrKODvq4hHl8CGuAh6YRWgSygRt X-Received: by 2002:a05:6e02:743:b0:359:47b9:7be6 with SMTP id x3-20020a056e02074300b0035947b97be6mr9966055ils.10.1699024584553; Fri, 03 Nov 2023 08:16:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1699024584; cv=none; d=google.com; s=arc-20160816; b=KT8urlPt1Reqf3SBVqqwbaoQKRCIOz0zxy2/fCQgm08DpAHBvjMu53YtT78VynJ9zs 6VgCiRU7U09snreEwoEnRDv/zTETjZm9DzPwu6acVQWm2/wu2MJYjPu1D6pojXJNMito FeRHPziyKyGUXDlzSRfP2HYFveD1uGIhKzU66rlQYZbU6x9TUtPBdQAj0LR2CqkuctPh /Z+nUNMcSFF6weLTpfYNQd8fzFdrEzD67DtqUqFGEX/K5lrDfGst7tklSERyBFsVYwuO 0jFRdj9wgw6+I1TBR98e6otGytQ1OZSDRrx/ia/WxsRgNUKd9ri8y8gGLuphw3AFOPGU ZNZQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date; bh=FAH3tK7dDrDkd+Wfp98ub9fAA5IQdJoE17WjyuOn4gQ=; fh=O5oRIbJVYOCb5eSqk6UT+v6hqHvZinmjRfvUP4uNKk4=; b=TkxN5nr7fF5uyUp0HTWk3Fk3Fnrsl5I2fwffqzhoPs6NAL71QqJ+TrDL0qobEoSls3 +Nqo97JqwoilNMpzryjPdm0lK6bZ33DqVOTgpxf2NFI0GmIdh7sAFFWXlZKdSglhbV/y LEbRcPaCfa1bUxDMJ/MKCv3hjban+dj11TBLi/RtmTIf0UzoEyVEM4VKPdHFy6sHqUcq yqwXpUL4QFsjZGO2GKHx7XkBudjv5kkXuseIaB0RGwWEJCdS2mbaq2/ni8iAkXgxwirJ FPsikvgfN6I4vz8zgjCbHOuedMfoYlOWItcMS0MOEdk0Va+jj21XsZbGyfOEn6y0oJ8G cAdw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from snail.vger.email (snail.vger.email. [23.128.96.37]) by mx.google.com with ESMTPS id cf8-20020a056a02084800b005859c255ce9si1762729pgb.861.2023.11.03.08.16.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Nov 2023 08:16:24 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 4D2D3803E487; Fri, 3 Nov 2023 08:16:02 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233297AbjKCPQA (ORCPT + 99 others); Fri, 3 Nov 2023 11:16:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41528 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229491AbjKCPP6 (ORCPT ); Fri, 3 Nov 2023 11:15:58 -0400 Received: from hi1smtp01.de.adit-jv.com (smtp1.de.adit-jv.com [93.241.18.167]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5A2BCD47; Fri, 3 Nov 2023 08:15:55 -0700 (PDT) Received: from hi2exch02.adit-jv.com (hi2exch02.adit-jv.com [10.72.92.28]) by hi1smtp01.de.adit-jv.com (Postfix) with ESMTP id 2F9F45201D3; Fri, 3 Nov 2023 16:15:53 +0100 (CET) Received: from vmlxhi-118.adit-jv.com (10.72.93.77) by hi2exch02.adit-jv.com (10.72.92.28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.34; Fri, 3 Nov 2023 16:15:53 +0100 Date: Fri, 3 Nov 2023 16:15:48 +0100 From: Hardik Gajjar To: Alan Stern CC: Hardik Gajjar , , , , , , Subject: Re: [PATCH v7 2/2] usb: new quirk to reduce the SET_ADDRESS request timeout Message-ID: <20231103151548.GA101464@vmlxhi-118.adit-jv.com> References: <20231027152029.104363-1-hgajjar@de.adit-jv.com> <20231027152029.104363-2-hgajjar@de.adit-jv.com> <7cbc1a70-2dcc-4a2a-99e6-fdd92bd5cc9b@rowland.harvard.edu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <7cbc1a70-2dcc-4a2a-99e6-fdd92bd5cc9b@rowland.harvard.edu> User-Agent: Mutt/1.9.4 (2018-02-28) X-Originating-IP: [10.72.93.77] X-ClientProxiedBy: hi2exch02.adit-jv.com (10.72.92.28) To hi2exch02.adit-jv.com (10.72.92.28) X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Fri, 03 Nov 2023 08:16:02 -0700 (PDT) On Fri, Oct 27, 2023 at 11:49:13AM -0400, Alan Stern wrote: > On Fri, Oct 27, 2023 at 05:20:29PM +0200, Hardik Gajjar wrote: > > This patch introduces a new USB quirk, > > USB_QUIRK_SHORT_SET_ADDRESS_REQ_TIMEOUT, which modifies the timeout value > > for the SET_ADDRESS request. The standard timeout for USB request/command > > is 5000 ms, as recommended in the USB 3.2 specification (section 9.2.6.1). > > > > However, certain scenarios, such as connecting devices through an APTIV > > hub, can lead to timeout errors when the device enumerates as full speed > > initially and later switches to high speed during chirp negotiation. > > > > In such cases, USB analyzer logs reveal that the bus suspends for > > 5 seconds due to incorrect chirp parsing and resumes only after two > > consecutive timeout errors trigger a hub driver reset. > > > > Packet(54) Dir(?) Full Speed J(997.100 us) Idle( 2.850 us) > > _______| Time Stamp(28 . 105 910 682) > > _______|_____________________________________________________________Ch0 > > Packet(55) Dir(?) Full Speed J(997.118 us) Idle( 2.850 us) > > _______| Time Stamp(28 . 106 910 632) > > _______|_____________________________________________________________Ch0 > > Packet(56) Dir(?) Full Speed J(399.650 us) Idle(222.582 us) > > _______| Time Stamp(28 . 107 910 600) > > _______|_____________________________________________________________Ch0 > > Packet(57) Dir Chirp J( 23.955 ms) Idle(115.169 ms) > > _______| Time Stamp(28 . 108 532 832) > > _______|_____________________________________________________________Ch0 > > Packet(58) Dir(?) Full Speed J (Suspend)( 5.347 sec) Idle( 5.366 us) > > _______| Time Stamp(28 . 247 657 600) > > _______|_____________________________________________________________Ch0 > > > > This 5-second delay in device enumeration is undesirable, particularly > > in automotive applications where quick enumeration is crucial > > (ideally within 3 seconds). > > > > The newly introduced quirks provide the flexibility to align with a > > 3-second time limit, as required in specific contexts like automotive > > applications. > > > > By reducing the SET_ADDRESS request timeout to 500 ms, the > > system can respond more swiftly to errors, initiate rapid recovery, and > > ensure efficient device enumeration. This change is vital for scenarios > > where rapid smartphone enumeration and screen projection are essential. > > > > To use the quirk, please write "vendor_id:product_id:p" to > > /sys/bus/usb/drivers/hub/module/parameter/quirks > > > > For example, > > echo "0x2c48:0x0132:p" > /sys/bus/usb/drivers/hub/module/parameters/quirks" > > > > Signed-off-by: Hardik Gajjar > > --- > > Reviewed-by: Alan Stern @Greg KH, Friendly reminder. Thanks, Hardik