From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: dvb usb issues since kernel 4.9 Date: Mon, 8 Jan 2018 18:55:51 +0100 Message-ID: <20180108175551.wp6thxmiozrz4yp2@gmail.com> References: <20171217120634.pmmuhdqyqmbkxrvl@gofer.mess.org> <20171217112738.4f3a4f9b@recife.lan> <20180106175420.275e24e7@recife.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Mauro Carvalho Chehab , 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 To: Linus Torvalds Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org * Linus Torvalds wrote: > 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. > > 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, absolutely - for the revert: Acked-by: Ingo Molnar as I doubt we have enough time to root-case this properly. Thanks, Ingo