From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E20F222526485 for ; Thu, 12 Apr 2018 21:54:06 -0700 (PDT) From: "Schmauss, Erik" Subject: RE: [PATCH v2 5/7] ACPICA: Integrate package handling with module-level code Date: Fri, 13 Apr 2018 04:54:04 +0000 Message-ID: References: <20180215210932.15490-1-erik.schmauss@intel.com> <20180215210932.15490-6-erik.schmauss@intel.com> In-Reply-To: Content-Language: en-US MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: "Williams, Dan J" , "Rafael J. Wysocki" Cc: Linux ACPI , Moore, List-ID: > -----Original Message----- > From: Dan Williams [mailto:dan.j.williams@intel.com] > Sent: Thursday, April 12, 2018 5:57 PM > To: Schmauss, Erik > Cc: Linux ACPI ; Rafael J. Wysocki > ; Moore, Robert ; linux-nvdimm > > 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 > wrote: > > [ adding linux-nvdimm ] > > > > On Fri, Feb 16, 2018 at 7:20 AM Erik Schmauss > > > > 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 > >> Signed-off-by: Bob Moore > >> Signed-off-by: Erik Schmauss > > > > 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. 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.. Thanks, Erik > ...i.e. > > git revert 34f206fd757c > git revert a406dea82af8 > git revert e7d970f6fca8 > git revert 959c38a7e128 > git revert 5a8361f7ecce _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm