Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932244AbeAHKCS (ORCPT + 1 other); Mon, 8 Jan 2018 05:02:18 -0500 Received: from osg.samsung.com ([64.30.133.232]:47637 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932089AbeAHKCO (ORCPT ); Mon, 8 Jan 2018 05:02:14 -0500 Date: Mon, 8 Jan 2018 08:02:00 -0200 From: Mauro Carvalho Chehab To: Linus Torvalds Cc: Ingo Molnar , Josef Griebichler , Greg Kroah-Hartman , Alan Stern , USB list , 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: <20180108080200.77d374c2@vento.lan> In-Reply-To: References: <20171217120634.pmmuhdqyqmbkxrvl@gofer.mess.org> <20171217112738.4f3a4f9b@recife.lan> <20180106175420.275e24e7@recife.lan> Organization: Samsung X-Mailer: Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: Hi Linus, Em Sun, 7 Jan 2018 13:23:39 -0800 Linus Torvalds escreveu: > On Sat, Jan 6, 2018 at 11:54 AM, Mauro Carvalho Chehab > wrote: > > > > Em Sat, 6 Jan 2018 16:04:16 +0100 > > "Josef Griebichler" escreveu: > >> > >> the causing commit has been identified. > >> After reverting commit https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4cd13c21b207e80ddb1144c576500098f2d5f882 > >> its working again. > > > > Just replying to me won't magically fix this. The ones that were involved on > > this patch should also be c/c, plus USB people. Just added them. > > Actually, you seem to have added an odd subset of the people involved. > > For example, Ingo - who actually committed that patch - wasn't on the cc. Sorry, my fault. I forgot to add him to it. > I do think we need to simply revert that patch. It's very simple: it > has been reported to lead to actual problems for people, and we don't > fix one problem and then say "well, it fixed something else" when > something breaks. > > When something breaks, we either unbreak it, or we revert the change > that caused the breakage. > > It's really that simple. That's what "no regressions" means. We don't > accept changes that cause regressions. This one did. Yeah, we should either unbreak or revert it. In the specific case of media devices, Alan came with a proposal of increasing the number of buffers. This is an one line change, and increase a capture delay from 0.63 ms to 5 ms on this specific case (Digital TV) shouldn't make much harm. So, I guess it would worth trying it before reverting the patch. It is hard to foresee the consequences of the softirq changes for other devices, though. For example, we didn't have any reports about this issue affecting cameras, Most cameras use ISOC nowadays, but some only provide bulk transfers. We usually try to use the minimum number of buffers possible, as increasing latency on cameras can be very annoying, specially on videoconference applications. Thanks, Mauro