On Mon, Aug 13, 2018 at 8:51 PM Greg KH <greg@xxxxxxxxx> wrote: > > On Mon, Aug 13, 2018 at 05:52:49PM +0000, Schmauss, Erik wrote: > > > > > > > -----Original Message----- > > > From: Rafael J. Wysocki [mailto:rafael@xxxxxxxxxx] > > > Sent: Sunday, August 12, 2018 2:47 AM > > > To: Schmauss, Erik <erik.schmauss@xxxxxxxxx> > > > Cc: ACPI Devel Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Rafael J. Wysocki > > > <rjw@xxxxxxxxxxxxx> > > > Subject: Re: [PATCH 05/11] ACPICA: AML Parser: skip opcodes that open a scope > > > upon parse failure > > > > > > On Fri, Aug 10, 2018 at 11:45 PM Erik Schmauss <erik.schmauss@xxxxxxxxx> > > > wrote: > > > > > > > > This change skips the entire length of opcodes that open a scope > > > > (Device, Scope, Processor, etc) if the creation of the op fails. The > > > > failure could be caused by various errors including AE_ALREADY_EXISTS > > > > and AE_NOT_FOUND. > > > > > > > > Reported-by: Jeremy Linton <jeremy.linton@xxxxxxx> > > > > Tested-by: Jeremy Linton <jeremy.linton@xxxxxxx> > > > > Signed-off-by: Erik Schmauss <erik.schmauss@xxxxxxxxx> > > > > > > I think that we should propagate this fix to the "stable" kernel series, at least > > > 4.17.y and newer. Do you agree? > > > > Yes, I agree. > > > > Hi Greg, please add this to the stable kernel > > What commit id should I add to the stable kernel? It's not in the mainline yet. I asked whether or not I should add a "Cc: stable" tag to this patch when applying it. :-) Cheers, Rafael