> -----Original Message----- > From: Rafael J. Wysocki [mailto:[email protected]] > Sent: Friday, April 13, 2018 12:43 AM > To: Schmauss, Erik <[email protected]> > Cc: Williams, Dan J <[email protected]>; Linux ACPI <linux- > [email protected]>; Moore, Robert <[email protected]>; linux- > nvdimm <[email protected]> > Subject: Re: [PATCH v2 5/7] ACPICA: Integrate package handling with > module-level code > > On Friday, April 13, 2018 6:54:04 AM CEST Schmauss, Erik wrote: > > > > > -----Original Message----- > > > From: Dan Williams [mailto:[email protected]] > > > Sent: Thursday, April 12, 2018 5:57 PM > > > To: Schmauss, Erik <[email protected]> > > > Cc: Linux ACPI <[email protected]>; Rafael J. Wysocki > > > <[email protected]>; Moore, Robert <[email protected]>; > > > linux-nvdimm <[email protected]> > > > Subject: Re: [PATCH v2 5/7] ACPICA: Integrate package handling with > > > module- level code > > > > > > On Thu, Apr 12, 2018 at 3:50 PM, Dan Williams > > > <[email protected]> > > > wrote: > > > > [ adding linux-nvdimm ] > > > > > > > > On Fri, Feb 16, 2018 at 7:20 AM Erik Schmauss > > > > <[email protected]> > > > > wrote: > > > > > > > >> ACPICA commit 8faf6fca445eb7219963d80543fb802302a7a8c7 > > > > > > > >> This change completes the integration of the recent changes to > > > >> package object handling with the module-level code support. > > > > > > > >> For acpi_exec, the -ep flag is removed. > > > > > > > >> This change allows table load to behave as if it were a method > > > >> invocation. Before this, the definition block definition below > > > >> would have loaded all named objects at the root scope. After > > > >> loading, it would execute the if statements at the root scope. > > > > > > > >> DefinitionBlock (...) > > > >> { > > > >> Name(OBJ1, 0) > > > > > > > >> if (1) > > > >> { > > > >> Device (DEV1) > > > >> { > > > >> Name (_HID,0x0) > > > >> } > > > >> } > > > >> Scope (DEV1) > > > >> { > > > >> Name (OBJ2) > > > >> } > > > >> } > > > > > > > >> The above code would load OBJ1 to the namespace, defer the > > > >> execution of the if statement and attempt to add OBJ2 within the > scope of DEV1. > > > >> Since DEV1 is not in scope, this would incur an AE_NOT_FOUND > error. > > > >> After this error is emitted, the if block is invoked and DEV1 and > > > >> its _HID is added to the namespace. > > > > > > > >> This commit changes the behavior to execute the if block in place > > > >> rather than deferring it until all tables are loaded. The new > > > >> behavior is as follows: insert OBJ1 in the namespace, invoke the > > > >> if statement and add DEV1 and its _HID to the namespace, add OBJ2 > > > >> to the scope of DEV1. > > > > > > > >> Bug report links: > > > >> Link: https://bugs.acpica.org/show_bug.cgi?id=963 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=153541 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=196165 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=192621 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=197207 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=198051 > > > >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=198515 > > > > > > > >> ACPICA repo: > > > >> Link: https://github.com/acpica/acpica/commit/8faf6fca > > > > > > > >> Tested-by: Kai-Heng Feng <[email protected]> > > > >> Signed-off-by: Bob Moore <[email protected]> > > > >> Signed-off-by: Erik Schmauss <[email protected]> > > > > > > > > Hi, > > > > > > > > This commit 5a8361f7ecce ("ACPICA: Integrate package handling with > > > > module-level code") regresses the detection of persistent memory > > > > in my qemu-kvm setup. > > > > > > > > > Hi Dan, > > > > Thanks for figuring this out. Do you think it's possible for you to > send me a full acpidump or some sort hexdump of the QEMU environment? > > > > > With the following set of clean reverts on top of latest-Linus I'm > > > back to a working configuration: > > > > > > Revert "ACPICA: Change a compile-time option to a runtime option" > > > Revert "ACPICA: Cleanup/simplify module-level code support" > > > Revert "ACPICA: Rename a global for clarity, no functional change" > > > Revert "ACPICA: Add option to disable Package object name > resolution errors" > > > Revert "ACPICA: Integrate package handling with module-level code" > > > > > > > Rafael, we may want to hold back on the module-level code changes (the > > patches below) for rc1. Between this and the strange _TSS issue, it > > seems like there are a few more things to resolve before this is ready > > for kernel upstream. > > It looks like you are asking me to queue up reverts as per the Dan's > report, is that correct? > > > If this is the case, I wonder if we could change the ACPICA version > > number so that it is one digit lower or append something to the end of > > the version number. This would be helpful in reminding us that it's > > not the full release in ACPICA upstream. Either that or remove the > > ACPICA release all together for this rc.. > > I can revert the commit updating the ACPICA version number easily > enough. :-) [Moore, Robert]
We should be able to figure this one out soon. Need acpidump Bob > > > > ...i.e. > > > > > > git revert 34f206fd757c > > > git revert a406dea82af8 > > > git revert e7d970f6fca8 > > > git revert 959c38a7e128 > > > git revert 5a8361f7ecce > > > _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
