From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from paleale.coelho.fi ([176.9.41.70]:55200 "EHLO farmhouse.coelho.fi" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751537AbdIOTvN (ORCPT ); Fri, 15 Sep 2017 15:51:13 -0400 Message-ID: <1505505062.32032.1.camel@coelho.fi> (sfid-20170915_215118_579916_223F0B9B) From: Luca Coelho To: Jens Axboe , Linus Torvalds Cc: Srinath Mannam , Bjorn Helgaas , Johannes Berg , "Grumbach, Emmanuel" , linuxwifi , "linux-wireless@vger.kernel.org" , "linux-pci@vger.kernel.org" Date: Fri, 15 Sep 2017 22:51:02 +0300 In-Reply-To: References: <04c9b578-693c-1dc6-9f0f-904580231b21@kernel.dk> <1505232673.5400.243.camel@intel.com> <1505234187.5400.249.camel@coelho.fi> <4bcbcbc1-7c79-09f0-5071-bc2f53bf6574@kernel.dk> <1505246657.1974.11.camel@sipsolutions.net> <8de8d1e2-1673-e1ff-d0ac-6d3bb03e9e99@kernel.dk> <93bfbc9b-8829-66fa-f4d6-6d47124e460c@kernel.dk> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Subject: Re: iwlwifi firmware load broken in current -git Sender: linux-wireless-owner@vger.kernel.org List-ID: On Fri, 2017-09-15 at 13:48 -0600, Jens Axboe wrote: > On 09/15/2017 01:38 PM, Linus Torvalds wrote: > > On Fri, Sep 15, 2017 at 12:32 PM, Jens Axboe wrote: > > > > > > > > In any case, your patch introduces a regression on systems. Please get > > > > it reverted now, and then you can come up with a new approach to fix the > > > > double enable of the upstream bridge. > > > > > > Who's sending in the revert? I can certainly do it if no one else does, > > > but it needs to be done. > > > > > > I'm not seeing any patches coming out of Srinath to fix up the > > > situation, so we should revert the broken patch until a better solution > > > exists. > > > > Hmm. I don't have the history here (apparently it never made lkml, for > > example), so I don't even know which commit you're talking about. > > > > From some of the context it looks like commit 40f11adc7cd9 ("PCI: > > Avoid race while enabling upstream bridges"), is that correct? > > Yes, Luca says that Bjorn already sent in the revert request, I just > didn't see it since I wasn't CC'ed on it. So looks like we're all > good, provided that makes it into -rc1. 40f11adc7cd9 is the broken > commit. Strange... AFAICT you *were* CCed on it. And so was everyone else in the original thread (+LKML)... -- Luca.