From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wout2-smtp.messagingengine.com ([64.147.123.25]:53119 "EHLO wout2-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725999AbeKUSiD (ORCPT ); Wed, 21 Nov 2018 13:38:03 -0500 Date: Wed, 21 Nov 2018 09:04:27 +0100 From: Greg KH To: "Schmauss, Erik" Cc: Jean Delvare , "stable@vger.kernel.org" , Guenter Roeck , "Wysocki, Rafael J" , "Moore, Robert" Subject: Re: Please revert "ACPICA: AML interpreter: add region addresses in global list during initialization" Message-ID: <20181121080427.GA6999@kroah.com> References: <20181120094649.243f9670@endymion> <20181120085419.GA19171@kroah.com> <20181120100359.3986a611@endymion> <20181120091136.GA20291@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Tue, Nov 20, 2018 at 06:30:22PM +0000, Schmauss, Erik wrote: > > > -----Original Message----- > > From: Greg KH [mailto:greg@kroah.com] > > Sent: Tuesday, November 20, 2018 1:12 AM > > To: Jean Delvare > > Cc: stable@vger.kernel.org; Guenter Roeck ; > > Schmauss, Erik ; Wysocki, Rafael J > > > > Subject: Re: Please revert "ACPICA: AML interpreter: add region addresses in > > global list during initialization" > > > > On Tue, Nov 20, 2018 at 10:03:59AM +0100, Jean Delvare wrote: > > > On Tue, 20 Nov 2018 09:54:19 +0100, Greg KH wrote: > > > > On Tue, Nov 20, 2018 at 09:46:49AM +0100, Jean Delvare wrote: > > > > > Dear stable kernel team, > > > > > > > > > > In stable kernel 4.19.2, the following upstream commit was included: > > > > > > > > > > commit 4abb951b73ff0a8a979113ef185651aa3c8da19b > > > > > Author: Erik Schmauss > > > > > Date: Wed Oct 17 14:09:35 2018 -0700 > > > > > > > > > > ACPICA: AML interpreter: add region addresses in global list > > > > > during initialization > > > > > > > > > > (...) > Hi, > > > > > > Please revert this commit from future stable kernels on all > > > > > affected branches (I think only 4.18.19 and 4.19.2 are affected at > > > > > the moment, but maybe other affected releases are in the works > > already). > > > > > > > > Ok, I'll go revert this, but shouldn't it also be reverted in > > > > Linus's tree as well? > > > > > Hello, > > > No. As I understand it (with my limited knowledge of ACPICA), the > > > change itself is correct. The problem is that it will detect resource > > > conflicts which were unnoticed before, and that will prevent drivers > > > from loading. Some of them may be addressed with driver fixes or new > > > drivers. Others are false positives (due to bogus BIOS) which users > > > will have to work around with acpi_resource_conflicts=lax. We have > > > been through this before, nothing new really, but it takes years to > > > address such problems. This just can't be done in stable kernel series. > > I would like to give you more context. > > There was a fairly complicated change that occurred in 4.17 and we > caused a regression by forgetting to add region addresses in a global list > during operation region initialization. We found the regression when bug > reporters tried to boot their macbook pro and asus laptop and saw that > there was a difference in behavior when drivers are being loaded > > So what I am trying to say is that we have been emitting these errors for a > while before we caused the regression. The goal with this patch is to keep > the behavior the same as kernels older than 4.17 where warnings are > printed to dmesg due to resource conflicts. Ok, so does this mean it belongs in 4.19.y? (4.18.y is about to go end-of-life in a few hours) Or does this mean it should just stay in 4.20? confused, greg k-h