From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: usb: dwc3: gadget: fix miss isoc issue introduced by IRQ latency From: Felipe Balbi Message-Id: <878t0sl5br.fsf@linux.intel.com> Date: Fri, 14 Dec 2018 13:23:52 +0200 To: "Zengtao \(B\)" Cc: liangshengjun , Greg Kroah-Hartman , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Thinh Nguyen List-ID: SGksCgoiWmVuZ3RhbyAoQikiIDxwcmltZS56ZW5nQGhpc2lsaWNvbi5jb20+IHdyaXRlczoKPj4t LS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+PkZyb206IEZlbGlwZSBCYWxiaSBbbWFpbHRvOmJh bGJpQGtlcm5lbC5vcmddCj4+U2VudDogRnJpZGF5LCBEZWNlbWJlciAxNCwgMjAxOCA0OjUyIFBN Cj4+VG86IFplbmd0YW8gKEIpIDxwcmltZS56ZW5nQGhpc2lsaWNvbi5jb20+Cj4+Q2M6IGxpYW5n c2hlbmdqdW4gPGxpYW5nc2hlbmdqdW5AaGlzaWxpY29uLmNvbT47IFplbmd0YW8gKEIpCj4+PHBy aW1lLnplbmdAaGlzaWxpY29uLmNvbT47IEdyZWcgS3JvYWgtSGFydG1hbgo+PjxncmVna2hAbGlu dXhmb3VuZGF0aW9uLm9yZz47IGxpbnV4LXVzYkB2Z2VyLmtlcm5lbC5vcmc7Cj4+bGludXgta2Vy bmVsQHZnZXIua2VybmVsLm9yZwo+PlN1YmplY3Q6IFJlOiBbUEFUQ0hdIHVzYjogZHdjMzogZ2Fk Z2V0OiBmaXggbWlzcyBpc29jIGlzc3VlIGludHJvZHVjZWQgYnkKPj5JUlEgbGF0ZW5jeQo+Pgo+ PiogUEdQIFNpZ25lZCBieSBhbiB1bmtub3duIGtleQo+Pgo+PlplbmcgVGFvIDxwcmltZS56ZW5n QGhpc2lsaWNvbi5jb20+IHdyaXRlczoKPj4KPj4+IElmIGl0J3MgYSBidXN5IHN5c3RlbSwgc29t ZSB0aW1lcyB3aGVuIHdlIHN0YXJ0IGFuIGlzb2MgdHJhbnNmZXIsIHRoZQo+Pj4gZnJhbWVudW1i ZXIgZ2V0IGZyb20gdGhlIGV2ZW50IGJ1ZmZlciBtYXkgYmUgYWxyZWFkeSBlbGFzcGVkLCBpbiB0 aGlzCj4+PiBjYXNlLCB3ZSB3aWxsIGdldCBhbGwgdGhlIHBhY2tldHMgZHJvcHBlZCBkdWUgdG8g bWlzcyBpc29jLiBBbmQgd2UKPj4+IHR1cm4gaW50byB0cmFuc2ZlciBub3RoaW5nLCB0byBmaXgg dGhpcyBpc3N1ZSwgd2UgbmVlZCB0byBmaXggdGhlCj4+PiBmcmFtZW51bWJlciB0byBtYWtlIHN1 cmUgdGhhdCBpdCdzIG5vdCBvdXQgb2YgZGF0ZS4KPj4+Cj4+PiBTaWduZWQtb2ZmLWJ5OiBMaWFu ZyBTaGVuZ2p1biA8bGlhbmdzaGVuZ2p1bkBoaXNpbGljb24uY29tPgo+Pj4gU2lnbmVkLW9mZi1i eTogWmVuZyBUYW8gPHByaW1lLnplbmdAaGlzaWxpY29uLmNvbT4KPj4KPj5UaGVyZSdzIGEgcGF0 Y2ggZ29pbmcgdXBzdHJlYW0gYWxyZWFkeSBkb2luZyB0aGlzOgo+Pgo+Pmh0dHBzOi8vZ2l0Lmtl cm5lbC5vcmcvcHViL3NjbS9saW51eC9rZXJuZWwvZ2l0L2JhbGJpL3VzYi5naXQvY29tbWl0Lz9o Cj4+PW5leHQmaWQ9ZDUzNzAxMDY3ZjA0OGI4YjExNjM1ZTk2NGI2ZDNiZDlhMjQ4YzYyMgo+Pgo+ Cj4gU29ycnksIEkgdGhpbmsgSSBtaXNzZWQgdG8gdXBkYXRlIHRvIHRoZSBsYXRlc3QgdmVyc2lv bi4gQnV0IEkgdGhpbmsgbXkgCj4gcGF0Y2ggaXMgbW9yZSBlZmZpY2llbnQuIEJlY2F1c2UgSSBq dXN0IHN5bmMgdGhlIGZyYW1lIGZyb20gdGhlIEhXLCBpdAo+ICB3b24ndCBmYWlsIGFuZCB0aGVy ZSBpcyBubyBuZWVkIHRvIGV4dHJhIHRyaWVzLgo+Cj4gV2hhdCBkbyB5b3UgdGhpbmsgYWJvdXQg aXQ/Cgp0aGUgMTQgYml0cyB5b3UgdXNlIGZvciB5b3VyIGNoZWNrIGlzIG5vdCByZXByZXNlbnRh dGl2ZSBvZiB0aGUgYWN0dWFsCm1pY3JvLWZyYW1lIHlvdSdyZSBjdXJyZW50bHkgaW4uIFRoaW5o IGV4cGxhaW5lZCB0aGF0IGluIHRoZSBkaXNjdXNzaW9uCndlIGhhZCB1bnRpbCB3ZSBjYW1lIHRv IHRoZSBwYXRjaCBJIHBvaW50ZWQgeW91IHRvIGFib3ZlLiBQbGVhc2UgbG9vayBhdAp0aGUgbWFp bGluZyBsaXN0IGFyY2hpdmVzIGZvciBkZXRhaWxzLgo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SIGNED_OFF_BY,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 67D72C67839 for ; Fri, 14 Dec 2018 11:24:01 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 218852080F for ; Fri, 14 Dec 2018 11:24:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1544786641; bh=80hzkteHQDRw8gcNklsCyGzNwTw5t6gq6qtqZj42q3Q=; h=From:To:Cc:Subject:In-Reply-To:References:Date:List-ID:From; b=JbJgU5ODjeCeA9ObKBkQrSTPH8NkcvK3Qzl/XvdVKwjVoA++DZCK9jMf0TrNs+K+O niGDeFIMvwqhHwswWFY9cRxGJZLZlZMNoHSrkOVpobmxSotjya7wx2tH2G7Kz62R+p eQxQeVlwkzNsLmTZB7c50CG+/4ZOCcNZ2CoFI3Ls= DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 218852080F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729483AbeLNLX7 (ORCPT ); Fri, 14 Dec 2018 06:23:59 -0500 Received: from mga11.intel.com ([192.55.52.93]:61428 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728612AbeLNLX7 (ORCPT ); Fri, 14 Dec 2018 06:23:59 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Dec 2018 03:23:58 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,352,1539673200"; d="asc'?scan'208";a="125901293" Received: from pipin.fi.intel.com (HELO localhost) ([10.237.72.175]) by fmsmga002.fm.intel.com with ESMTP; 14 Dec 2018 03:23:57 -0800 From: Felipe Balbi To: "Zengtao \(B\)" Cc: liangshengjun , Greg Kroah-Hartman , "linux-usb\@vger.kernel.org" , "linux-kernel\@vger.kernel.org" , Thinh Nguyen Subject: RE: [PATCH] usb: dwc3: gadget: fix miss isoc issue introduced by IRQ latency In-Reply-To: <678F3D1BB717D949B966B68EAEB446ED24E20BE5@dggemm526-mbx.china.huawei.com> References: <1544805179-2248-1-git-send-email-prime.zeng@hisilicon.com> <87efaklcd1.fsf@linux.intel.com> <678F3D1BB717D949B966B68EAEB446ED24E20BE5@dggemm526-mbx.china.huawei.com> Date: Fri, 14 Dec 2018 13:23:52 +0200 Message-ID: <878t0sl5br.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, "Zengtao (B)" writes: >>-----Original Message----- >>From: Felipe Balbi [mailto:balbi@kernel.org] >>Sent: Friday, December 14, 2018 4:52 PM >>To: Zengtao (B) >>Cc: liangshengjun ; Zengtao (B) >>; Greg Kroah-Hartman >>; linux-usb@vger.kernel.org; >>linux-kernel@vger.kernel.org >>Subject: Re: [PATCH] usb: dwc3: gadget: fix miss isoc issue introduced by >>IRQ latency >> >>* PGP Signed by an unknown key >> >>Zeng Tao writes: >> >>> If it's a busy system, some times when we start an isoc transfer, the >>> framenumber get from the event buffer may be already elasped, in this >>> case, we will get all the packets dropped due to miss isoc. And we >>> turn into transfer nothing, to fix this issue, we need to fix the >>> framenumber to make sure that it's not out of date. >>> >>> Signed-off-by: Liang Shengjun >>> Signed-off-by: Zeng Tao >> >>There's a patch going upstream already doing this: >> >>https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git/commit/?h >>=3Dnext&id=3Dd53701067f048b8b11635e964b6d3bd9a248c622 >> > > Sorry, I think I missed to update to the latest version. But I think my=20 > patch is more efficient. Because I just sync the frame from the HW, it > won't fail and there is no need to extra tries. > > What do you think about it? the 14 bits you use for your check is not representative of the actual micro-frame you're currently in. Thinh explained that in the discussion we had until we came to the patch I pointed you to above. Please look at the mailing list archives for details. =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEElLzh7wn96CXwjh2IzL64meEamQYFAlwTksgACgkQzL64meEa mQZJng//XfRK6HOJ2tT9+GNmNvtyo8h1BNhMhEZvXkn4WTvU2ecqq3GLOphr9Fts ux0T2be5pbjJ0xdBCbiMjL9XKDs3BxKTP5kzfjDikGFeZyjwPDqjlOC2HGgiJt40 GXorTFOBfEsGSP9nVNYL1OFPYsTTwZSYypmo5FpRNpq4Ey9izfUvu/NYN1y6l5RA MH6gOdasaMZL98mETXNP8hK4m3pjz5RbcBUJW9uDlw53XjWlQ3L1lbaFCsT8f0lY GCDVVUNYoRr4vxrsZ1WkaHa7BcU4ONLnWhK1S5s0CtMv77nXlzzy6nQ2IOEfZJEJ VQYVBbst02PTQPlACB32Knt367ajrH+Xh0k4vCRvJ6SxgAwMzjwq1QgUGg9hV881 zj9ZNESbNC/+KGIK83w1uKoUZ6S4MELD1tbyD6wTtIOUHJZ5KSUqvk2b0DAqiiqw RWr+UVj54J5mx+viwICw1MJvQS489oh39uLznB+ODXSXkFBQZDXrt/ZTkd7+XKyp zzDbJIyFLCJtSnP1xg8FrHTu99dbQtHwi33xLryP0V5OmbRzkvG7Xsl+zLiRQsuJ gdd2rmy07gAkeN+Qnfa7HE/RSYt1/OD2DBB1OIegBsIgOd2fe34WhfSxov333906 ZfhEjwk7KKchpwIfKvdj66WIJzPF5jhic/jAW4Vr9pKvvFOyeSA= =Sr1X -----END PGP SIGNATURE----- --=-=-=--