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: [BUG] mt76x0u: Probing issues on Raspberry Pi 3 B+ From: Lorenzo Bianconi Message-Id: <20190211151055.GC8128@localhost.localdomain> Date: Mon, 11 Feb 2019 16:10:56 +0100 To: Stefan Wahren Cc: Stanislaw Gruszka , Felix Fietkau , Doug Anderson , Minas Harutyunyan , linux-wireless , linux-usb@vger.kernel.org List-ID: PiBIaSBMb3JlbnpvLAo+IAo+IEFtIDExLjAyLjE5IHVtIDEyOjA2IHNjaHJpZWIgTG9yZW56byBC aWFuY29uaToKPiA+PiBIaSwKPiA+Pgo+ID4+IEFtIDExLjAyLjE5IHVtIDExOjA0IHNjaHJpZWIg TG9yZW56byBCaWFuY29uaToKPiA+Pj4+IE9uIFN1biwgRmViIDEwLCAyMDE5IGF0IDExOjIyOjI1 QU0gKzAxMDAsIExvcmVuem8gQmlhbmNvbmkgd3JvdGU6Cj4gPj4+Pj4+IE9uIFNhdCwgRmViIDA5 LCAyMDE5IGF0IDA5OjI5OjA1UE0gKzAxMDAsIFN0ZWZhbiBXYWhyZW4gd3JvdGU6Cj4gPj4+Pj4+ Pj4gY291bGQgeW91IHBsZWFzZSB0ZXN0IHRoZSBmb2xsb3dpbmcgc2VyaWVzOgo+ID4+Pj4+Pj4+ IGh0dHBzOi8vcGF0Y2h3b3JrLmtlcm5lbC5vcmcvY292ZXIvMTA3NjQ0NTMvCj4gPj4+Pj4+PiB5 ZWFoIHRoaXMgZml4ZWQgdGhlIHByb2JpbmcgdGltZW91dCBhbmQgdGhlIGRyaXZlciB3aWxsIHBy b2JlIHN1Y2Nlc3NmdWwuIEFGQUlLIHRoZSBkd2MyIGhvc3QgbW9kZSBkb2Vzbid0IHN1cHBvcnQg c2NhdHRlci1nYXRoZXIgeWV0Lgo+ID4+Pj4+PiBTbyB0aGlzIGlzIGVpdGhlciBkd2MyIHNjYXR0 ZXItZ2F0aGVyIHByb2JsZW0gd2hpY2ggc2hvdWxkIGJlIGFkZHJlc3NlZCBpbgo+ID4+Pj4+PiB0 aGlzIGRyaXZlciBvciBtdDc2eDB1IGRvZXMgc29tZXRoaW5nIHdyb25nIHdoZW4gY29uZmlndXJp bmcgU0cuCj4gPj4+Pj4+Cj4gPj4+Pj4+IERpc2FibGluZyBTRyBpcyBqdXN0IHdvcmthcm91bmQs IHdoaWNoIGRvIG5vdCBhZGRyZXNzIGFjdHVhbCBwcm9ibGVtLgo+ID4+Pj4+Pgo+ID4+Pj4+PiBJ IHRoaW5rIEkgZm91bmQgbXQ3NngwdSBpc3N1ZSB0aGF0IGNvdWxkIGNhdXNlIHRoaXMgVVNCIHBy b2JlIGVycm9yCj4gPj4+Pj4+IChhbmQgcG9zc2libHkgYWxzbyBhZGRyZXNzIEFNRCBJT01NVSBp c3N1ZSkuIFdlIHNlZW1zIGRvIG5vdCBjb3JyZWN0bHkKPiA+Pj4+Pj4gc2V0IFVSQiB0cmFuc2Zl ciBsZW5ndGggc21hbGxlciB0aGFuIHNnIGJ1ZmZlciBsZW5ndGguIEF0dGFjaGVkICBwYXRjaAo+ ID4+Pj4+PiBzaG91bGQgY29ycmVjdCB0aGF0Lgo+ID4+Pj4+IEhpIFN0YW5pc2xhdywKPiA+Pj4+ Pgo+ID4+Pj4+IEkgdGhpbmsgJ3NnWzBdLmxlbmd0aCcgaXMgYWxyZWFkeSBzZXQgaW4gbXQ3NnVf ZmlsbF9yeF9zZygpLgo+ID4+Pj4gSXQgaXMsIGJ1Zi0+bGVuIGFuZCBzZ1swXS5sZW5ndGggYXJl IGluaXRpYWxpemVkIHRvIHRoZSBzYW1lIHZhbHVlIGZvciAxCj4gPj4+PiBzZWdtZW50LiBCdXQg dGhlbiBidWYtPmxlbiAoYXNzaWduZWQgdG8gdXJiLT5idWZmZXJfdHJhbnNmZXJfbGVuZ3RoKSBj aGFuZ2UKPiA+Pj4+IHRvIHNtYWxsZXIgdmFsdWUgLCBidXQgc2dbMF0ubGVuZ3RoIHN0YXkgdGhl IHNhbWUuIFdoYXQgSSB0aGluayBjYW4gYmUKPiA+Pj4+IHByb2JsZW0gZm9yIHVzYiBob3N0IGRy aXZlci4KPiA+Pj4+Cj4gPj4+Pj4gTW9yZW92ZXIgYXBwbHlpbmcgdGhpcyBwYXRjaCBJIGdvdCB0 aGUgZm9sbG93aW5nIGNyYXNoIChycGktNS4wLnkpOgo+ID4+Pj4gT2ssIHNvIHdpdGggcGF0Y2gg cHJvYmUgZmFpbCBpbnN0YW50bHkgYW5kIHRyaWdnZXIgeWV0IGFub3RoZXIgYnVnKHMpCj4gPj4+ PiBvbiBlcnJvciBwYXRoLiBZb3Ugc2VlbXMgdG8gYWRkcmVzcyB0aGF0IGFscmVhZHkuIAo+ID4+ Pj4KPiA+Pj4+PiBNb3Jlb3ZlciBmb3IgbXQ3NngwdSBTRyBpcyAnYWxyZWFkeScgZGlzYWJsZWQg c2luY2Ugd2UgdXNlIGp1c3Qgb25lCj4gPj4+Pj4gYnVmZmVyIHNvIGZyb20gcGVyZm9ybWFuY2Ug cG9pbnQgb2YgdmlldyBJIGRvIG5vdCBzZWUgYW55IGRpZmZlcmVuY2UKPiA+Pj4+PiBvZiB1c2lu ZyBhIHN0YW5kYXJkIHVzYiBidWZmZXIuCj4gPj4+Pj4gVGhpcyBwYXRjaCBoYXMgYmVlbiB0ZXN0 ZWQgaW4gbXVsdGlwbGUgc2NlbmFyaW9zIGFuZCBzZWVtcyB0byBmaXgKPiA+Pj4+PiByZXBvcnRl ZCBpc3N1ZXMgKGZvciB1c2IyLjApLgo+ID4+Pj4gT2ssIHNvIHBhc3NpbmcgYnVmZmVyIHZpYSB1 cmItPnRyYW5zZmVyX2J1ZmZlciB3b3Jrcy4gQnV0IHdoeSB1cmItPnNnCj4gPj4+PiBkb2VzIG5v dCB3b3JrIGZvciAxIHNlZ21lbnQgPwo+ID4+PiBIZXJlIGl0IGlzIGEgZGlmZmVyZW50IGlzc3Vl IHJlc3BlY3QgdG8gdGhlIEFNRCBJT01NVSBvbmUsIGR3YzIgaG9zdCBkcml2ZXIKPiA+Pj4gZG9l cyBub3QgaW1wbGVtZW50IFNHIEkvTyBzbyBwcm9iaW5nIGZhaWxzLiBJIGd1ZXNzIGl0IGlzIHN0 aWxsIHVzZWZ1bCB0bwo+ID4+PiBpbXBsZW1lbnQgYSAnbGVnYWN5JyBtb2RlIHRoYXQgZW5hYmxl IG10NzYgb24gaG9zdCBjb250cm9sbGVycyB0aGF0IGRvIG5vdCBpbXBsZW1lbnQKPiA+Pj4gU0cg SS9PIChycGkgaXMgYSB2ZXJ5IGNvbW1vbiBkZXZpY2Ugc28gaXQgd2lsbCBiZSBjb29sIHRvIGhh dmUgbXQ3NiB3b3JraW5nIG9uCj4gPj4+IGl0KS4gTW9yZW92ZXIgd2UgYXJlIG5vdCByZW1vdmlu ZyBmdW5jdGlvbmFsaXRpZXMsIHVzZXIgZXhwZXJpZW5jZSB3aWxsIHJlbWFpbgo+ID4+PiB0aGUg c2FtZQo+ID4+Pgo+ID4+IGknbSBub3Qgc3VyZSB0aGF0IHlvdSB1bmRlcnN0YW5kIG15IG1haWwg WzFdIHdpdGggdGhlIHN1bW1hcnkgb2YgbXkgdGVzdAo+ID4+IHJlc3VsdHMuCj4gPj4KPiA+IFll cyByaWdodCwgSSBkaWQgbm90IGdldCBpdCBzb3JyeSA6KQo+ID4gYXMgaW5kaWNhdGVkIGhlcmUg aHR0cHM6Ly93d3cucmFzcGJlcnJ5cGkub3JnL2RvY3VtZW50YXRpb24vbGludXgva2VybmVsL2J1 aWxkaW5nLm1kCj4gPiBJIGFtIHVzaW5nIGJjbTI3MDlfZGVmY29uZmlnIGNvbmZpZyAodXNpbmcg aXQgSSBzcG90dGVkIHRoZSBtdDc2IGNyYXNoZXMgYW5kCj4gPiBwcm9iZSBmYWlsdXJlKQo+IAo+ IG5vIHByb2JsZW0sIGF0IHRoZSBiZWdpbm5pbmcgdGhpcyBjb3VsZCBiZSB2ZXJ5IGNvbmZ1c2lu Zy4gSSBvbmx5IHdhbnQKPiB0byBjbGFyaWZ5IHRoYXQgdGhpcyBkb2N1bWVudGF0aW9uIHJlZmVy cyB0byB0aGUgdmVuZG9yIGtlcm5lbCAod2l0aCBhCj4gZGlmZmVyZW50IFVTQiBob3N0IGRyaXZl cikgb2YgdGhlIFJhc3BiZXJyeSBQaSBGb3VuZGF0aW9uLgo+IAo+IEFsbCBteSByZXN1bHRzIHJl ZmVycyB0byB0aGUgbWFpbmxpbmUga2VybmVsIHdlIGFsbCBzaG91bGQgdGFsayBhYm91dC4gSQo+ IHN0YXJ0ZWQgYSBnaXN0IHdoaWNoIHRyeSB0byBkZXNjcmliZSB0aGUgbWFpbmxpbmUgdmFyaWFu dDoKPiBodHRwczovL2dpc3QuZ2l0aHViLmNvbS9sYXRlZ29vZGJ5ZS9jNzMxN2E0MmJmN2Y5YzA3 ZjVhOTFiYWVkOGM2OGY3NQoKU28gdG8gc3VtbWFyaXplOgotIFJhc3BiZXJyeSBQaSBGb3VuZGF0 aW9uIGtlcm5lbCB3b3JrcyBqdXN0IHdpdGggUkZDIHNlcmllcwotIG1haW5saW5lIGtlcm5lbCB3 b3JrcyBvdXQgb2YgdGhlIGJveAoKaXMgbXkgdW5kZXJzdGFuZGluZyBjb3JyZWN0PyBJIGFtIHN0 aWxsIGNvbnNpZGVyaW5nIGFkZGluZyBhIGxlZ2FjeSBtb2RlIHNpbmNlCnRoZXJlIHdpbGwgbm90 IGJlIGFueSByZWdyZXNzaW9uIHVzaW5nIGl0IGluc3RlYWQgb2YgU0cgSS9PIHdpdGgganVzdCBv bmUgU0cKYnVmZmVyCgpSZWdhcmRzLApMb3JlbnpvCgo+IAo+IE1heWJlIHRoaXMgY291bGQgYmUg aGVscGZ1bC4KPiAKPiBTdGVmYW4KPiAKPiA+IFJlZ2FyZHMsCj4gPiBMb3JlbnpvCj4gPgo+ID4+ IEluIGNhc2Ugb2YgdXNpbmcgdGhlIGFybS9tdWx0aV92N19kZWZjb25maWcgKDMyIGJpdCkgdGhl IG10NzYgd29ya3MgbGlrZQo+ID4+IGEgY2hhcm0gd2l0aG91dCB5b3VyIHNnIGF2b2lkIHBhdGNo IHNlcmllcywgYnV0IHRoZSBhcm02NC9kZWZjb25maWcgKDY0Cj4gPj4gYml0KSByZXF1aXJlcyB0 aGUgc2VyaWVzIHRvIHByb2JlIGF0IGxlYXN0LiBTbyBpIHdvdWxkbid0IGNvbmNsdWRlIGZyb20K PiA+PiB0aGUgZmFjdCB0aGF0IGR3YzIgZG9lc24ndCBzdXBwb3J0IFNHIGFueSBwcm9iaW5nIGlz c3VlcyBvbiBhcm02NC4gU28gd2UKPiA+PiBuZWVkIHRvIGludmVzdGlnYXRlIHdoaWNoIGNvbmZp ZyBvcHRpb24gdHJpZ2dlcnMgdGhlIHByb2JsZW0uCj4gPj4KPiA+PiBTdGVmYW4KPiA+Pgo+ID4+ IFsxXSAtIGh0dHBzOi8vbWFyYy5pbmZvLz9sPWxpbnV4LXVzYiZtPTE1NDk4MTY3NTcyNDA3OAo+ ID4+Cg== 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=-7.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_PASS,URIBL_BLOCKED, USER_AGENT_MUTT 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 AC83CC282DA for ; Mon, 11 Feb 2019 15:11:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8B80F222B0 for ; Mon, 11 Feb 2019 15:11:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404035AbfBKPLE (ORCPT ); Mon, 11 Feb 2019 10:11:04 -0500 Received: from mail-wm1-f65.google.com ([209.85.128.65]:52750 "EHLO mail-wm1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2391502AbfBKPLC (ORCPT ); Mon, 11 Feb 2019 10:11:02 -0500 Received: by mail-wm1-f65.google.com with SMTP id m1so18097246wml.2 for ; Mon, 11 Feb 2019 07:11:01 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=CTCVlB/ozOOzsnfd5Rz/lfSd8ugZtp3aH083Ab6zJPg=; b=DoYxOertSayAznKP0bZiDgpizBQg4hhUzk6W6OxtLm/U4cvNLQWZdIFrgo/iKllzzb NdeXfMcp02LEQx13wk37Tf/3E9wnh6TTOIoXNsBnb9b2NNxkfvteg+kfCRH/KK+uXSuG kCfVWmvyJcBdVKkU3e2FALg3rI8founVsla5cxEQMfL1zNdnPYfQLhqVutXpw3Lj7jxH g4Vg+1OTNenJMzCbQP4snyUWzPZUSGSx2M7M0fVrFeTg4CyzS51YxbikOo+LOLrFIrAU nXHMGfufb/x70S02nM3bE0u+oSY4acMoT9NBtukzW3UuQshhW8KI6hFMLj92mSM9ciZl SRag== X-Gm-Message-State: AHQUAuYpv2ERIvVNBTdFhJ3LDwEGYrFgGMnVIKbMjoxhSug+ulojXd5v ojWICMawqNm/5tpBLAiluW0Wpw== X-Google-Smtp-Source: AHgI3IabNCGn9B9ehjR7Bw6lfIGteCsxVzor2EOe4V/3Dtbtjx0zzN64GrNFpr1vlOgU8WsFoXMVJg== X-Received: by 2002:a1c:a104:: with SMTP id k4mr9404938wme.54.1549897860369; Mon, 11 Feb 2019 07:11:00 -0800 (PST) Received: from localhost.localdomain (nat-pool-mxp-t.redhat.com. [149.6.153.186]) by smtp.gmail.com with ESMTPSA id c1sm10055614wmb.14.2019.02.11.07.10.59 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 11 Feb 2019 07:10:59 -0800 (PST) Date: Mon, 11 Feb 2019 16:10:56 +0100 From: Lorenzo Bianconi To: Stefan Wahren Cc: Stanislaw Gruszka , Felix Fietkau , Doug Anderson , Minas Harutyunyan , linux-wireless , linux-usb@vger.kernel.org Subject: Re: [BUG] mt76x0u: Probing issues on Raspberry Pi 3 B+ Message-ID: <20190211151055.GC8128@localhost.localdomain> References: <2003727085.234456.1549714119945@email.ionos.de> <165515185.283024.1549744145982@email.ionos.de> <20190210094123.GB2913@redhat.com> <20190211074455.GA6292@redhat.com> <20190211100405.GD3467@localhost.localdomain> <20190211110635.GD2293@localhost.localdomain> <8a996497-ed69-4087-2c37-7f73633cbb4e@i2se.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8a996497-ed69-4087-2c37-7f73633cbb4e@i2se.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org > Hi Lorenzo, > > Am 11.02.19 um 12:06 schrieb Lorenzo Bianconi: > >> Hi, > >> > >> Am 11.02.19 um 11:04 schrieb Lorenzo Bianconi: > >>>> On Sun, Feb 10, 2019 at 11:22:25AM +0100, Lorenzo Bianconi wrote: > >>>>>> On Sat, Feb 09, 2019 at 09:29:05PM +0100, Stefan Wahren wrote: > >>>>>>>> could you please test the following series: > >>>>>>>> https://patchwork.kernel.org/cover/10764453/ > >>>>>>> yeah this fixed the probing timeout and the driver will probe successful. AFAIK the dwc2 host mode doesn't support scatter-gather yet. > >>>>>> So this is either dwc2 scatter-gather problem which should be addressed in > >>>>>> this driver or mt76x0u does something wrong when configuring SG. > >>>>>> > >>>>>> Disabling SG is just workaround, which do not address actual problem. > >>>>>> > >>>>>> I think I found mt76x0u issue that could cause this USB probe error > >>>>>> (and possibly also address AMD IOMMU issue). We seems do not correctly > >>>>>> set URB transfer length smaller than sg buffer length. Attached patch > >>>>>> should correct that. > >>>>> Hi Stanislaw, > >>>>> > >>>>> I think 'sg[0].length' is already set in mt76u_fill_rx_sg(). > >>>> It is, buf->len and sg[0].length are initialized to the same value for 1 > >>>> segment. But then buf->len (assigned to urb->buffer_transfer_length) change > >>>> to smaller value , but sg[0].length stay the same. What I think can be > >>>> problem for usb host driver. > >>>> > >>>>> Moreover applying this patch I got the following crash (rpi-5.0.y): > >>>> Ok, so with patch probe fail instantly and trigger yet another bug(s) > >>>> on error path. You seems to address that already. > >>>> > >>>>> Moreover for mt76x0u SG is 'already' disabled since we use just one > >>>>> buffer so from performance point of view I do not see any difference > >>>>> of using a standard usb buffer. > >>>>> This patch has been tested in multiple scenarios and seems to fix > >>>>> reported issues (for usb2.0). > >>>> Ok, so passing buffer via urb->transfer_buffer works. But why urb->sg > >>>> does not work for 1 segment ? > >>> Here it is a different issue respect to the AMD IOMMU one, dwc2 host driver > >>> does not implement SG I/O so probing fails. I guess it is still useful to > >>> implement a 'legacy' mode that enable mt76 on host controllers that do not implement > >>> SG I/O (rpi is a very common device so it will be cool to have mt76 working on > >>> it). Moreover we are not removing functionalities, user experience will remain > >>> the same > >>> > >> i'm not sure that you understand my mail [1] with the summary of my test > >> results. > >> > > Yes right, I did not get it sorry :) > > as indicated here https://www.raspberrypi.org/documentation/linux/kernel/building.md > > I am using bcm2709_defconfig config (using it I spotted the mt76 crashes and > > probe failure) > > no problem, at the beginning this could be very confusing. I only want > to clarify that this documentation refers to the vendor kernel (with a > different USB host driver) of the Raspberry Pi Foundation. > > All my results refers to the mainline kernel we all should talk about. I > started a gist which try to describe the mainline variant: > https://gist.github.com/lategoodbye/c7317a42bf7f9c07f5a91baed8c68f75 So to summarize: - Raspberry Pi Foundation kernel works just with RFC series - mainline kernel works out of the box is my understanding correct? I am still considering adding a legacy mode since there will not be any regression using it instead of SG I/O with just one SG buffer Regards, Lorenzo > > Maybe this could be helpful. > > Stefan > > > Regards, > > Lorenzo > > > >> In case of using the arm/multi_v7_defconfig (32 bit) the mt76 works like > >> a charm without your sg avoid patch series, but the arm64/defconfig (64 > >> bit) requires the series to probe at least. So i wouldn't conclude from > >> the fact that dwc2 doesn't support SG any probing issues on arm64. So we > >> need to investigate which config option triggers the problem. > >> > >> Stefan > >> > >> [1] - https://marc.info/?l=linux-usb&m=154981675724078 > >>