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: dvb usb issues since kernel 4.9 From: Mauro Carvalho Chehab Message-Id: <20180108074324.3c153189@vento.lan> Date: Mon, 8 Jan 2018 07:43:24 -0200 To: Alan Stern Cc: Josef Griebichler , Greg Kroah-Hartman , linux-usb@vger.kernel.org, Eric Dumazet , Rik van Riel , Paolo Abeni , Hannes Frederic Sowa , Jesper Dangaard Brouer , linux-kernel , netdev , Jonathan Corbet , LMML , Peter Zijlstra , David Miller , torvalds@linux-foundation.org List-ID: RW0gU3VuLCA3IEphbiAyMDE4IDEwOjQxOjM3IC0wNTAwIChFU1QpCkFsYW4gU3Rlcm4gPHN0ZXJu QHJvd2xhbmQuaGFydmFyZC5lZHU+IGVzY3JldmV1OgoKPiBPbiBTdW4sIDcgSmFuIDIwMTgsIE1h dXJvIENhcnZhbGhvIENoZWhhYiB3cm90ZToKPiAKPiA+ID4gPiBJdCBzZWVtcyB0aGF0IHRoZSBv cmlnaW5hbCBwYXRjaCB3ZXJlIGRlc2lnbmVkIHRvIHNvbHZlIHNvbWUgSVJRIGlzc3Vlcwo+ID4g PiA+IHdpdGggbmV0d29yayBjYXJkcyB3aXRoIGNhdXNlcyBkYXRhIGxvc3NlcyBvbiBoaWdoIHRy YWZmaWMuIEhvd2V2ZXIsCj4gPiA+ID4gaXQgaXMgYWxzbyBjYXVzaW5nIGJhZCBlZmZlY3RzIG9u IHN1c3RhaW5lZCBoaWdoIGJhbmR3aWR0aCBkZW1hbmRzCj4gPiA+ID4gcmVxdWlyZWQgYnkgRFZC IGNhcmRzLCBhdCBsZWFzdCBvbiBzb21lIFVTQiBob3N0IGRyaXZlcnMuCj4gPiA+ID4gCj4gPiA+ ID4gQWxhbi9HcmVnL0VyaWMvRGF2aWQ6Cj4gPiA+ID4gCj4gPiA+ID4gQW55IGlkZWFzIGFib3V0 IGhvdyB0byBmaXggaXQgd2l0aG91dCBjYXVzaW5nIHJlZ3Jlc3Npb25zIHRvCj4gPiA+ID4gbmV0 d29yaz8gICAgCj4gPiA+IAo+ID4gPiBJdCB3b3VsZCBiZSBnb29kIHRvIGtub3cgd2hhdCBoYXJk d2FyZSB3YXMgaW52b2x2ZWQgb24gdGhlIHg4NiBzeXN0ZW0KPiA+ID4gYW5kIHRvIGhhdmUgc29t ZSB0aW1pbmcgZGF0YS4gIENhbiB3ZSBzZWUgdGhlIG91dHB1dCBmcm9tIGxzdXNiIGFuZAo+ID4g PiB1c2Jtb24sIHJ1bm5pbmcgb24gYSB2YW5pbGxhIGtlcm5lbCB0aGF0IGdldHMgcGxlbnR5IG9m IHZpZGVvIGdsaXRjaGVzPyAgCj4gPiAKPiA+IEZyb20gSm9zZWYncyByZXBvcnQsIGFuZCBmcm9t IHRoZSBCWiwgdGhlIGFmZmVjdGVkIGhhcmR3YXJlIHNlZW1zCj4gPiB0byBiZSBiYXNlZCBvbiBN b250YWdlIFRlY2hub2xvZ3kgTTg4RFMzMTAzL004OFRTMjAyMiBjaGlwc2V0LiAgCj4gCj4gV2hh dCB0eXBlIG9mIFVTQiBob3N0IGNvbnRyb2xsZXIgZG9lcyB0aGUgeDg2XzY0IHN5c3RlbSB1c2U/ ICBFSENJIG9yCj4geEhDST8KCkknbGwgbGV0IEpvc2VmIGFuc3dlciB0aGlzLgoKPiAKPiA+IFRo ZSBkcml2ZXIgaXQgdXNlcyBpcyBhdCBkcml2ZXJzL21lZGlhL3VzYi9kdmItdXNiLXYyL2R2YnNr eS5jLAo+ID4gd2l0aCBzaGFyZXMgYSBVU0IgaW1wbGVtZW50YXRpb24gdGhhdCBpcyB1c2VkIGJ5 IGEgbG90IG1vcmUgZHJpdmVycy4KPiA+IFRoZSBVUkIgaGFuZGxpbmcgY29kZSBpcyBhdDoKPiA+ IAo+ID4gCWRyaXZlcnMvbWVkaWEvdXNiL2R2Yi11c2ItdjIvdXNiX3VyYi5jCj4gPiAKPiA+IFRo aXMgcGFydGljdWxhciBkcml2ZXIgYWxsb2NhdGVzIDggYnVmZmVycyB3aXRoIDQwOTYgYnl0ZXMg ZWFjaAo+ID4gZm9yIGJ1bGsgdHJhbnNmZXJzLCB1c2luZyB0cmFuc2Zlcl9mbGFncyA9IFVSQl9O T19UUkFOU0ZFUl9ETUFfTUFQLgo+ID4gCj4gPiBUaGlzIGJlY29tZSBhIHBvcHVsYXIgVVNCIGhh cmR3YXJlIG5vd2FkYXlzLiBJIGhhdmUgb25lIFM5NjBjCj4gPiBteXNlbGYsIHNvIEkgY2FuIHNl bmQgeW91IHRoZSBsc3VzYiBmcm9tIGl0LiBZb3Ugc2hvdWxkIG5vdGljZSwgaG93ZXZlciwKPiA+ IHRoYXQgYSBEVkItQy9EVkItUzIgY2hhbm5lbCBjYW4gZWFzaWx5IHByb3ZpZGUgdmVyeSBoaWdo IHN1c3RhaW5lZCBiaXQKPiA+IHJhdGVzLiBIZXJlLCBvbiBteSBEVkItUzIgcHJvdmlkZXIsIGEg dHlwaWNhbCB0cmFuc3BvbmRlciBwcm9kdWNlcyA1OCBNcHBzCj4gPiBvZiBwYXlsb2FkIGFmdGVy IHJlbW92aW5nIFVSQiBoZWFkZXJzLiAgCj4gCj4gWW91IG1lbnRpb25lZCBlYXJsaWVyIHRoYXQg dGhlIGRyaXZlciB1c2VzIGJ1bGsgdHJhbnNmZXJzLiAgSW4gVVNCLTIuMCwKPiB0aGUgbWF4aW11 bSBwb3NzaWJsZSBwYXlsb2FkIGRhdGEgdHJhbnNmZXIgcmF0ZSB1c2luZyBidWxrIHRyYW5zZmVy cyBpcwo+IDUzMjQ4IGJ5dGVzL21zLCB3aGljaCBpcyA1My4yNDggTUIvcyAoaS5lLiwgbG93ZXIg dGhhbiA1OCBNQi9zKS4gIEFuZAo+IGV2ZW4gdGhpcyBpcyBwb3NzaWJsZSBvbmx5IGlmIGFsbW9z dCBub3RoaW5nIGVsc2UgaXMgdXNpbmcgdGhlIGJ1cyBhdAo+IHRoZSBzYW1lIHRpbWUuCgpObywg SSBzYWlkIDU4IE1iaXRzL3MgKG5vdCBieXRlcykuCgpPbiBEVkItQyBhbmQgRFZCLVMyIHNwZWNz LCBBRkFJS1QsIHRoZXJlJ3Mgbm8gaGFyZCBsaW1pdCBmb3IgdGhlIG1heGltdW0KcGF5bG9hZCBk YXRhIHJhdGUsIGFsdGhvdWdoIGluZHVzdHJ5IHNlZW1zIHRvIGxpbWl0IGl0IHRvIGJlIGFyb3Vu ZAo2MCBNYml0cy9zLiBPbiB0aG9zZSBzdGFuZGFyZHMsIHRoZSBtYXhpbWFsIGJpdCByYXRlIGlz IGRlZmluZWQgYnkgdGhlCm1vZHVsYXRpb24gdHlwZSBhbmQgYnkgdGhlIGNoYW5uZWwgc3ltYm9s IHJhdGUuCgpUbyBnaXZlIHlvdSBhIHByYWN0aWNhbCBleGFtcGxlLCBteSBEVkItUzIgcHJvdmlk ZXIgbW9kdWxhdGVzIGVhY2gKdHJhbnNwb25kZXIgd2l0aCA4L1BTSyAoMyBiaXRzL3N5bWJvbCks IGFuZCBkZWZpbmUgY2hhbm5lbHMgd2l0aCBhCnN5bWJvbCByYXRlIG9mIDMwIE1iYXVkcy9zLiBT bywgaXQgY291bGQsIHRoZW9yZXRpY2FsbHksIHRyYW5zcG9ydAphIE1QRUctVFMgc3RyZWFtIHVw IHRvIDkwIE1iaXRzL3MgKG1pbnVzIGhlYWRlcnMgYW5kIGd1YXJkIGludGVydmFscykuCkluIHBy YWN0aWNlLCB0aGUgc3RyZWFtcyB0aGVyZSBhcmUgdHJhbnNtaXR0ZWQgd2l0aCA1OCwwMjYuNSBL Yml0cy9zLgoKPiA+IEEgMTAgbWludXRlcyByZWNvcmQgd2l0aCB0aGUKPiA+IGVudGlyZSBkYXRh ICh3aXRoIHR5cGljYWxseSBjb250YWlucyA1LTEwIGNoYW5uZWxzKSBjYW4gZWFzaWx5IGdvCj4g PiBhYm92ZSA0IEdCLCBqdXN0IHRvIHJlcHJvZHVjZSAxLTIgZ2xpdGNoZXMuIFNvLCBJJ20gbm90 IHN1cmUgaWYKPiA+IGEgdXNibW9uIGR1bXAgd291bGQgYmUgdXNlZnVsLiAgCj4gCj4gSXQgbWln aHQgbm90IGJlIGhlbHBmdWwgYXQgYWxsLiAgSG93ZXZlciwgSSdtIG5vdCBpbnRlcmVzdGVkIGlu IHRoZSAKPiBwYXlsb2FkIGRhdGEgKHdoaWNoIHdvdWxkIGJlIHVuaW50ZWxsaWdpYmxlIHRvIG1l IGFueXdheSkgYnV0IHJhdGhlciAKPiB0aGUgdGltaW5nIG9mIFVSQiBzdWJtaXNzaW9ucyBhbmQg Y29tcGxldGlvbnMuICBBIHVzYm1vbiB0cmFjZSB3aGljaCAKPiBkaWRuJ3Qga2VlcCBtdWNoIG9m IHRoZSBwYXlsb2FkIGRhdGEgd291bGQgb25seSByZXF1aXJlIG9uIHRoZSBvcmRlciBvZiAKPiA1 MCBNQiBwZXIgbWludXRlIC0tIGFuZCBKb3NlZiBzYWlkIHRoYXQgZ2xpdGNoZXMgdXN1YWxseSB3 b3VsZCBzaG93IHVwIAo+IHdpdGhpbiBhIG1pbnV0ZSBvciBzby4KClllYWgsIHRoaXMgY291bGQg aGVscC4KCkpvc2VmLAoKWW91IGNhbiBnZXQgaXQgd2l0aCB3aXJlc2hhcmsvdHNoYXJrIG9yIHRj cGR1bXAuIFNlZToKCWh0dHBzOi8vdGVjaG5vbGluY2hwaW4ud29yZHByZXNzLmNvbS8yMDE1LzEw LzIzL3VzYi1idXMtc25pZmZlcnMtZm9yLWxpbnV4LXN5c3RlbS8KCWh0dHBzOi8vd2lraS53aXJl c2hhcmsub3JnL0NhcHR1cmVTZXR1cC9VU0IKCj4gPiBJJ20gZW5jbG9zaW5nIHRoZSBsc3VzYiBm cm9tIGEgUzk2MEMgZGV2aWNlLCB3aXRoIGlzIGJhc2VkIG9uIHRob3NlCj4gPiBNb250YWdlIGNo aXBzZXRzOiAgCj4gCj4gV2hhdCBJIHdhbnRlZCB0byBzZWUgd2FzIHRoZSBvdXRwdXQgZnJvbSAi bHN1c2IiIG9uIHRoZSBhZmZlY3RlZAo+IHN5c3RlbSwgbm90IHRoZSBvdXRwdXQgZnJvbSAibHN1 c2IgLXYgLXMgQjpEIiBvbiB5b3VyIHN5c3RlbS4KPiAKPiA+ID4gT3ZlcmFsbCwgdGhpcyBtYXkg YmUgYSB2ZXJ5IGRpZmZpY3VsdCBwcm9ibGVtIHRvIHNvbHZlLiAgVGhlCj4gPiA+IDRjZDEzYzIx YjIwNyBjb21taXQgd2FzIGludGVuZGVkIHRvIGltcHJvdmUgdGhyb3VnaHB1dCBhdCB0aGUgY29z dCBvZgo+ID4gPiBpbmNyZWFzZWQgbGF0ZW5jeS4gIEJ1dCB0aGVuIHdoYXQgZG8geW91IGRvIHdo ZW4gdGhlIGxhdGVuY3kgYmVjb21lcwo+ID4gPiB0b28gaGlnaCBmb3IgdGhlIHZpZGVvIHN1YnN5 c3RlbSB0byBoYW5kbGU/ICAKPiA+IAo+ID4gTGF0ZW5jeSBjYW4ndCBiZSB0b28gaGlnaCwgb3Ro ZXJ3aXNlIGZyYW1lcyB3aWxsIGJlIGRyb3BwZWQuICAKPiAKPiBZZXMsIHRoYXQncyB0aGUgd2hv bGUgcG9pbnQuCj4gCj4gPiBFdmVuIGlmIHRoZSBLZXJuZWwgaXRzZWxmIGRvZXNuJ3QgZHJvcCwg aWYgdGhlIGRlbGF5IGdvZXMgaGlnaGVyCj4gPiB0aGFuIGEgY2VydGFpbiB0aHJlc2hvbGQsIHVz ZXJzcGFjZSB3aWxsIG5lZWQgdG8gZHJvcCwgYXMgaXQKPiA+IHNob3VsZCBiZSBwcmVzZW50aW5n IGF1ZGlvIGFuZCB2aWRlbyBvbiByZWFsIHRpbWUuIFlldCwgdHlwaWNhbGx5LAo+ID4gdXNlcnNw YWNlIHdpbGwgZGVsYXkgaXQgYnkgb25lIG9yIHR3byBzZWNvbmRzLCB3aXRoIHdvdWxkIG1lYW4K PiA+IDE1MDAtMzUwMCBidWZmZXJzLCB3aXRoIEkgc3VzcGVjdCBpdCBpcyBhIGxvdCBtb3JlIHRo YW4gdGhlIGhhcmR3YXJlCj4gPiBsaW1pdHMuIFNvIEkgc3VzcGVjdCB0aGF0IHRoZSBoYXJkd2Fy ZSBzdGFydmVzIGZyZWUgYnVmZmVycyBhIHdheSAKPiA+IGJlZm9yZSB1c2Vyc3BhY2UsIGFzIG1l ZGlhIGhhcmR3YXJlIGRvbid0IGhhdmUgdW5saW1pdGVkIGJ1ZmZlcnMKPiA+IGluc2lkZSB0aGVt LCBhcyB0aGV5IGFzc3VtZSB0aGF0IHRoZSBLZXJuZWwvdXNlcnNwYWNlIHdpbGwgYmUgZmFzdAo+ ID4gZW5vdWdoIHRvIHN1c3RhaW4gYml0IHJhdGVzIHVwIHRvIDY2IE1icHMgb2YgcGF5bG9hZC4g IAo+IAo+IFRoZSB0aW1pbmcgaW5mb3JtYXRpb24gd291bGQgdGVsbCB1cyBob3cgbGFyZ2UgdGhl IGxhdGVuY3kgaXMuCj4gCj4gSW4gYW55IGNhc2UsIHlvdSBtaWdodCBiZSBhYmxlIHRvIGF0dGFj ayB0aGUgcHJvYmxlbSBzaW1wbHkgYnkgdXNpbmcKPiBtb3JlIHRoYW4gOCBidWZmZXJzLiAgV2l0 aCBqdXN0IGVpZ2h0IDQwOTYtYnl0ZSBidWZmZXJzLCB0aGUgdG90YWwKPiBwaXBlbGluZSBjYXBh Y2l0eSBpcyBvbmx5IGFib3V0IDAuNjIgbXMgKGF0IHRoZSBtYXhpbXVtIHBvc3NpYmxlCj4gdHJh bnNmZXIgcmF0ZSkuICBJbmNyZWFzaW5nIHRoZSBudW1iZXIgb2YgYnVmZmVycyB0byA2NSB3b3Vs ZCBnaXZlIGEKPiBjYXBhY2l0eSBvZiA1IG1zLCB3aGljaCBpcyBwcm9iYWJseSBhIGxvdCBiZXR0 ZXIgc3VpdGVkIGZvciBzaXR1YXRpb25zCj4gd2hlcmUgY29tcGxldGlvbnMgYXJlIGhhbmRsZWQg YnkgdGhlIGtzb2Z0aXJxZCB0aHJlYWQuCgpJbmNyZWFzaW5nIGl0IHRvIDY1IHNob3VsZG4ndCBi ZSBoYXJkLiBOb3Qgc3VyZSwgaG93ZXZlciwgaWYgdGhlIGhhcmR3YXJlCndpbGwgYWN0dWFsbHkg ZmlsbCB0aGUgNjUgYnVmZmVycywgYnV0IGl0IGlzIHdvcnRoIHRvIHRyeS4KCj4gPiBQZXJoYXBz IG1lZGlhIGRyaXZlcnMgY291bGQgcGFzcyBzb21lIHF1aXJrIHNpbWlsYXIgdG8gVVJCX0lTT19B U0FQLAo+ID4gaW4gb3JkZXIgdG8gcmV2ZXJ0IHRoZSBrZXJuZWwgbG9naWMgdG8gcHJpb3JpdGl6 ZSBsYXRlbmN5IGluc3RlYWQgb2YKPiA+IHRocm91Z2hwdXQuICAKPiAKPiBJdCBjYW4ndCBiZSBk b25lIHdpdGhvdXQgcGVydmFzaXZlIGNoYW5nZXMgdG8gdGhlIFVTQiBzdWJzeXN0ZW0sIHdoaWNo Cj4gSSB3b3VsZCBncmVhdGx5IHByZWZlciB0byBhdm9pZC4gIEJlc2lkZXMsIHRoaXMgd291bGRu J3QgcmVhbGx5IHNvbHZlCj4gdGhlIHByb2JsZW0uICBEZWNyZWFzaW5nIHRoZSBsYXRlbmN5IGZv ciBvbmUgZGV2aWNlIHdpbGwgY2F1c2UgaXQgdG8gYmUKPiBpbmNyZWFzZWQgZm9yIG90aGVycy4K CklmIHRoZXJlIGlzIGEgVFYgc3RyZWFtaW5nIHRyYWZmaWMgYXQgYSBVU0IgYnVzLCBpdCBtZWFu cyB0aGF0IHRoZQp1c2VyIHdhbnRzIHRvIGVpdGhlciB3YXRjaCBhbmQvb3IgcmVjb3JkIGEgVFYg cHJvZ3JhbS4gT24gc3VjaAp1c2VjYXNlIHNjZW5hcmlvLCBhIGxvdyBsYXRlbmN5IGlzIGhpZ2hs eSBkZXNpcmVkIGZvciB0aGUgVFYgY2FwdHVyZQooYW5kIGRpc3BsYXksIGlmIHRoZSBHUFUgaXMg VVNCKSwgZXZlbiBpdCBtZWFucyBhIGhpZ2hlciBsYXRlbmN5IGZvcgpvdGhlciB0cmFmZmljLgoK Sm9zZWYsCgpDb3VsZCB5b3UgcGxlYXNlIHRyeSB0aGUgZm9sbG93aW5nIHBhdGNoIG9uIEtlcm5l bCA0LjE0LjEwICh3aXRob3V0CnJldmVydGluZyBhbnkgY2hhbmdlc2V0cyksIGFuZCBzZWUgaWYg aXQgZml4ZXMgdGhlIGlzc3VlPwoKCm1lZGlhOiBkdmJza3k6IEluY3JlYXNlIHRoZSBudW1iZXIg b2YgYnVmZmVycwoKUmlnaHQgbm93LCBUaGlzIGRyaXZlciBleHBlY3RzIGEgMC42MiBtcyBkZWxh eSB3aXRoIDggYnVmZmVycyBvbiBhbiBVU0IgMi4wCmhpZ2ggc3BlZWQgYnVzLiBJbmNyZWFzZSBp dCB0byA2NSBidWZmZXJzLCBpbiBvcmRlciB0byBnaXZlIG1vcmUgdGltZSBmb3IKdGhlIHRvcCBo YWxmIG9mIHRoZSBVU0IgdHJhbnNmZXIgaGFuZGxlciB0byBjb21wbGV0ZSBpdHMgdGFzay4KClN1 Z2dlc3RlZC1ieTogQWxhbiBTdGVybiA8c3Rlcm5Acm93bGFuZC5oYXJ2YXJkLmVkdT4KU2lnbmVk LW9mZi1ieTogTWF1cm8gQ2FydmFsaG8gQ2hlaGFiIDxtY2hlaGFiQHMtb3BlbnNvdXJjZS5jb20+ CgoKCj4gCgoKClRoYW5rcywKTWF1cm8KLS0tClRvIHVuc3Vic2NyaWJlIGZyb20gdGhpcyBsaXN0 OiBzZW5kIHRoZSBsaW5lICJ1bnN1YnNjcmliZSBsaW51eC11c2IiIGluCnRoZSBib2R5IG9mIGEg bWVzc2FnZSB0byBtYWpvcmRvbW9Admdlci5rZXJuZWwub3JnCk1vcmUgbWFqb3Jkb21vIGluZm8g YXQgIGh0dHA6Ly92Z2VyLmtlcm5lbC5vcmcvbWFqb3Jkb21vLWluZm8uaHRtbAoKZGlmZiAtLWdp dCBhL2RyaXZlcnMvbWVkaWEvdXNiL2R2Yi11c2ItdjIvZHZic2t5LmMgYi9kcml2ZXJzL21lZGlh L3VzYi9kdmItdXNiLXYyL2R2YnNreS5jCmluZGV4IDEzMWI2YzA4ZTE5OS4uZDNmNWZmYzU0YjI1 IDEwMDY0NAotLS0gYS9kcml2ZXJzL21lZGlhL3VzYi9kdmItdXNiLXYyL2R2YnNreS5jCisrKyBi L2RyaXZlcnMvbWVkaWEvdXNiL2R2Yi11c2ItdjIvZHZic2t5LmMKQEAgLTc0MCw3ICs3NDAsNyBA QCBzdGF0aWMgc3RydWN0IGR2Yl91c2JfZGV2aWNlX3Byb3BlcnRpZXMgZHZic2t5X3M5NjBfcHJv cHMgPSB7CiAJLm51bV9hZGFwdGVycyA9IDEsCiAJLmFkYXB0ZXIgPSB7CiAJCXsKLQkJCS5zdHJl YW0gPSBEVkJfVVNCX1NUUkVBTV9CVUxLKDB4ODIsIDgsIDQwOTYpLAorCQkJLnN0cmVhbSA9IERW Ql9VU0JfU1RSRUFNX0JVTEsoMHg4MiwgNjUsIDQwOTYpLAogCQl9CiAJfQogfTsK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from osg.samsung.com ([64.30.133.232]:53500 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756236AbeAHJnk (ORCPT ); Mon, 8 Jan 2018 04:43:40 -0500 Date: Mon, 8 Jan 2018 07:43:24 -0200 From: Mauro Carvalho Chehab To: Alan Stern Cc: Josef Griebichler , Greg Kroah-Hartman , , Eric Dumazet , Rik van Riel , Paolo Abeni , Hannes Frederic Sowa , Jesper Dangaard Brouer , linux-kernel , netdev , Jonathan Corbet , LMML , Peter Zijlstra , David Miller , Subject: Re: dvb usb issues since kernel 4.9 Message-ID: <20180108074324.3c153189@vento.lan> In-Reply-To: References: <20180107090336.03826df2@vento.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Em Sun, 7 Jan 2018 10:41:37 -0500 (EST) Alan Stern escreveu: > On Sun, 7 Jan 2018, Mauro Carvalho Chehab wrote: > > > > > It seems that the original patch were designed to solve some IRQ issues > > > > with network cards with causes data losses on high traffic. However, > > > > it is also causing bad effects on sustained high bandwidth demands > > > > required by DVB cards, at least on some USB host drivers. > > > > > > > > Alan/Greg/Eric/David: > > > > > > > > Any ideas about how to fix it without causing regressions to > > > > network? > > > > > > It would be good to know what hardware was involved on the x86 system > > > and to have some timing data. Can we see the output from lsusb and > > > usbmon, running on a vanilla kernel that gets plenty of video glitches? > > > > From Josef's report, and from the BZ, the affected hardware seems > > to be based on Montage Technology M88DS3103/M88TS2022 chipset. > > What type of USB host controller does the x86_64 system use? EHCI or > xHCI? I'll let Josef answer this. > > > The driver it uses is at drivers/media/usb/dvb-usb-v2/dvbsky.c, > > with shares a USB implementation that is used by a lot more drivers. > > The URB handling code is at: > > > > drivers/media/usb/dvb-usb-v2/usb_urb.c > > > > This particular driver allocates 8 buffers with 4096 bytes each > > for bulk transfers, using transfer_flags = URB_NO_TRANSFER_DMA_MAP. > > > > This become a popular USB hardware nowadays. I have one S960c > > myself, so I can send you the lsusb from it. You should notice, however, > > that a DVB-C/DVB-S2 channel can easily provide very high sustained bit > > rates. Here, on my DVB-S2 provider, a typical transponder produces 58 Mpps > > of payload after removing URB headers. > > You mentioned earlier that the driver uses bulk transfers. In USB-2.0, > the maximum possible payload data transfer rate using bulk transfers is > 53248 bytes/ms, which is 53.248 MB/s (i.e., lower than 58 MB/s). And > even this is possible only if almost nothing else is using the bus at > the same time. No, I said 58 Mbits/s (not bytes). On DVB-C and DVB-S2 specs, AFAIKT, there's no hard limit for the maximum payload data rate, although industry seems to limit it to be around 60 Mbits/s. On those standards, the maximal bit rate is defined by the modulation type and by the channel symbol rate. To give you a practical example, my DVB-S2 provider modulates each transponder with 8/PSK (3 bits/symbol), and define channels with a symbol rate of 30 Mbauds/s. So, it could, theoretically, transport a MPEG-TS stream up to 90 Mbits/s (minus headers and guard intervals). In practice, the streams there are transmitted with 58,026.5 Kbits/s. > > A 10 minutes record with the > > entire data (with typically contains 5-10 channels) can easily go > > above 4 GB, just to reproduce 1-2 glitches. So, I'm not sure if > > a usbmon dump would be useful. > > It might not be helpful at all. However, I'm not interested in the > payload data (which would be unintelligible to me anyway) but rather > the timing of URB submissions and completions. A usbmon trace which > didn't keep much of the payload data would only require on the order of > 50 MB per minute -- and Josef said that glitches usually would show up > within a minute or so. Yeah, this could help. Josef, You can get it with wireshark/tshark or tcpdump. See: https://technolinchpin.wordpress.com/2015/10/23/usb-bus-sniffers-for-linux-system/ https://wiki.wireshark.org/CaptureSetup/USB > > I'm enclosing the lsusb from a S960C device, with is based on those > > Montage chipsets: > > What I wanted to see was the output from "lsusb" on the affected > system, not the output from "lsusb -v -s B:D" on your system. > > > > Overall, this may be a very difficult problem to solve. The > > > 4cd13c21b207 commit was intended to improve throughput at the cost of > > > increased latency. But then what do you do when the latency becomes > > > too high for the video subsystem to handle? > > > > Latency can't be too high, otherwise frames will be dropped. > > Yes, that's the whole point. > > > Even if the Kernel itself doesn't drop, if the delay goes higher > > than a certain threshold, userspace will need to drop, as it > > should be presenting audio and video on real time. Yet, typically, > > userspace will delay it by one or two seconds, with would mean > > 1500-3500 buffers, with I suspect it is a lot more than the hardware > > limits. So I suspect that the hardware starves free buffers a way > > before userspace, as media hardware don't have unlimited buffers > > inside them, as they assume that the Kernel/userspace will be fast > > enough to sustain bit rates up to 66 Mbps of payload. > > The timing information would tell us how large the latency is. > > In any case, you might be able to attack the problem simply by using > more than 8 buffers. With just eight 4096-byte buffers, the total > pipeline capacity is only about 0.62 ms (at the maximum possible > transfer rate). Increasing the number of buffers to 65 would give a > capacity of 5 ms, which is probably a lot better suited for situations > where completions are handled by the ksoftirqd thread. Increasing it to 65 shouldn't be hard. Not sure, however, if the hardware will actually fill the 65 buffers, but it is worth to try. > > Perhaps media drivers could pass some quirk similar to URB_ISO_ASAP, > > in order to revert the kernel logic to prioritize latency instead of > > throughput. > > It can't be done without pervasive changes to the USB subsystem, which > I would greatly prefer to avoid. Besides, this wouldn't really solve > the problem. Decreasing the latency for one device will cause it to be > increased for others. If there is a TV streaming traffic at a USB bus, it means that the user wants to either watch and/or record a TV program. On such usecase scenario, a low latency is highly desired for the TV capture (and display, if the GPU is USB), even it means a higher latency for other traffic. Josef, Could you please try the following patch on Kernel 4.14.10 (without reverting any changesets), and see if it fixes the issue? media: dvbsky: Increase the number of buffers Right now, This driver expects a 0.62 ms delay with 8 buffers on an USB 2.0 high speed bus. Increase it to 65 buffers, in order to give more time for the top half of the USB transfer handler to complete its task. Suggested-by: Alan Stern Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/usb/dvb-usb-v2/dvbsky.c b/drivers/media/usb/dvb-usb-v2/dvbsky.c index 131b6c08e199..d3f5ffc54b25 100644 --- a/drivers/media/usb/dvb-usb-v2/dvbsky.c +++ b/drivers/media/usb/dvb-usb-v2/dvbsky.c @@ -740,7 +740,7 @@ static struct dvb_usb_device_properties dvbsky_s960_props = { .num_adapters = 1, .adapter = { { - .stream = DVB_USB_STREAM_BULK(0x82, 8, 4096), + .stream = DVB_USB_STREAM_BULK(0x82, 65, 4096), } } }; > Thanks, Mauro