Return-path: Received: from mail-ey0-f174.google.com ([209.85.215.174]:42826 "EHLO mail-ey0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933154Ab2DLNcm convert rfc822-to-8bit (ORCPT ); Thu, 12 Apr 2012 09:32:42 -0400 MIME-Version: 1.0 In-Reply-To: <20120412011313.GA23764@kroah.com> References: <20120411231102.GA6404@kroah.com> <20120412002927.GA23167@kroah.com> <20120412011313.GA23764@kroah.com> Date: Thu, 12 Apr 2012 16:32:40 +0300 Message-ID: (sfid-20120412_153306_907994_8B5732EE) Subject: Re: [ 00/78] 3.3.2-stable review From: Felipe Contreras To: Greg KH Cc: Sergio Correia , linux-kernel@vger.kernel.org, stable@vger.kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, alan@lxorguk.ukuu.org.uk, linux-wireless Mailing List , Sujith Manoharan , "ath9k-devel@lists.ath9k.org" , "John W. Linville" Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, Apr 12, 2012 at 4:13 AM, Greg KH wrote: > On Thu, Apr 12, 2012 at 04:03:59AM +0300, Felipe Contreras wrote: >> On Thu, Apr 12, 2012 at 3:29 AM, Greg KH wrote: >> > On Wed, Apr 11, 2012 at 07:59:14PM -0400, Sergio Correia wrote: >> >> Hello Greg, >> >> >> >> >> >> On Wed, Apr 11, 2012 at 7:11 PM, Greg KH wrote: >> >> > This is the start of the stable review cycle for the 3.3.2 release. >> >> > There are 78 patches in this series, all will be posted as a response >> >> > to this one.  If anyone has any issues with these being applied, please >> >> > let me know. >> >> > >> >> > Responses should be made by Fri Apr 13 23:10:16 UTC 2012. >> >> > Anything received after that time might be too late. >> >> > >> >> >> >> is there any chance for this one to be included in this review cycle? >> >> >> >> http://www.spinics.net/lists/linux-wireless/msg87999.html >> >> I was going to ask for exactly the same thing. My system is completely >> unusable without this patch; not only does the network doesn't work, >> but quite often the kernel is stuck consuming 100% of the CPU. >> >> > Have you read Documentation/stable_kernel_rules.txt?  Based on that, I >> > don't think it can, yet, right? >> >> Why not? This patch makes the code go back to a previous state, it is >> obviously more stable than the current state, and the code already >> exists in Linus's tree (in previous releases). > > It does?  What is the git commit id of the patch?  Based in the email > above, I assumed it had not made it to Linus's tree already. It's a revert of c1afdaff90538ef085b756454f12b29575411214, so so just take a look at the code in c1afdaff90538ef085b756454f12b29575411214^. >> But hey, I guess it's OK that 3.3.x is stuck in and endless loop right >> after booting, because rules are more important than fixing obvious >> breakage. > > What rule did you think I was saying this was not acceptable for? The fact that the patch as not been applied/reviewed/accepted upstream. Personally I don't see what is the problem with reverts; we already know the previous code was working. Sure, in theory it might behave different due to other changes, but that doesn't seem to be the case here, plus, it can't be worst than the current situation of staying in an endless loop. It appears in 3.4 there are more issues, so the fix there might look completely different, but regardless, the real issue is that the proper fix is not yet here. So the question is do we want 3.3.2 to be completely broken for these machines, or not? -- Felipe Contreras