From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 25C98C31E51 for ; Tue, 18 Jun 2019 09:21:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id EDBF92084B for ; Tue, 18 Jun 2019 09:21:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729171AbfFRJVs (ORCPT ); Tue, 18 Jun 2019 05:21:48 -0400 Received: from mail.steuer-voss.de ([85.183.69.95]:58172 "EHLO mail.steuer-voss.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728810AbfFRJVs (ORCPT ); Tue, 18 Jun 2019 05:21:48 -0400 X-Virus-Scanned: Debian amavisd-new at mail.steuer-voss.de Received: by mail.steuer-voss.de (Postfix, from userid 1000) id C023A4CD7A; Tue, 18 Jun 2019 11:21:44 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by mail.steuer-voss.de (Postfix) with ESMTP id BBDE9449CC; Tue, 18 Jun 2019 11:21:44 +0200 (CEST) Date: Tue, 18 Jun 2019 11:21:44 +0200 (CEST) From: Nikolaus Voss X-X-Sender: nv@fox.voss.local To: "Moore, Robert" cc: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Len Brown , "Schmauss, Erik" , Jacek Anaszewski , Pavel Machek , Dan Murphy , Thierry Reding , ACPI Devel Maling List , "open list:ACPI COMPONENT ARCHITECTURE (ACPICA)" , "linux-leds@vger.kernel.org" , Linux PWM List , Linux Kernel Mailing List Subject: RE: [PATCH v2 1/3] ACPI: Resolve objects on host-directed table loads In-Reply-To: <94F2FBAB4432B54E8AACC7DFDE6C92E3B95F9EC6@ORSMSX110.amr.corp.intel.com> Message-ID: References: <94F2FBAB4432B54E8AACC7DFDE6C92E3B95EFB26@ORSMSX110.amr.corp.intel.com> <94F2FBAB4432B54E8AACC7DFDE6C92E3B95F9EC6@ORSMSX110.amr.corp.intel.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Mon, 17 Jun 2019, Moore, Robert wrote: >> -----Original Message----- >> From: Nikolaus Voss [mailto:nv@vosn.de] >> Sent: Sunday, June 16, 2019 11:24 PM >> To: Moore, Robert >> Cc: Rafael J. Wysocki ; Rafael J. Wysocki >> ; Len Brown ; Schmauss, Erik >> ; Jacek Anaszewski >> ; Pavel Machek ; Dan Murphy >> ; Thierry Reding ; ACPI Devel >> Maling List ; open list:ACPI COMPONENT >> ARCHITECTURE (ACPICA) ; linux-leds@vger.kernel.org; >> Linux PWM List ; Linux Kernel Mailing List >> ; nikolaus.voss@loewensteinmedical.de >> Subject: RE: [PATCH v2 1/3] ACPI: Resolve objects on host-directed table >> loads >> >> Bob, >> >> On Fri, 14 Jun 2019, Moore, Robert wrote: >>> >>> >>> -----Original Message----- >>> From: Nikolaus Voss [mailto:nv@vosn.de] >>> Sent: Friday, June 14, 2019 2:26 AM >>> To: Rafael J. Wysocki >>> Cc: Rafael J. Wysocki ; Len Brown >>> ; Moore, Robert ; Schmauss, >>> Erik ; Jacek Anaszewski >>> ; Pavel Machek ; Dan Murphy >>> ; Thierry Reding ; ACPI >>> Devel Maling List ; open list:ACPI >>> COMPONENT ARCHITECTURE (ACPICA) ; >>> linux-leds@vger.kernel.org; Linux PWM List >>> ; Linux Kernel Mailing List >>> >>> Subject: Re: [PATCH v2 1/3] ACPI: Resolve objects on host-directed >>> table loads >>> >>> Hi Rafael, >>> >>> On Fri, 14 Jun 2019, Rafael J. Wysocki wrote: >>>> On Wed, Jun 12, 2019 at 10:36 AM Nikolaus Voss >>>> wrote: >>>>> >>>>> If an ACPI SSDT overlay is loaded after built-in tables have been >>>>> loaded e.g. via configfs or efivar_ssdt_load() it is necessary to >>>>> rewalk the namespace to resolve references. Without this, relative >>>>> and absolute paths like ^PCI0.SBUS or \_SB.PCI0.SBUS are not >>>>> resolved correctly. >>>>> >>>>> Make configfs load use the same method as efivar_ssdt_load(). >>>>> >>>>> Signed-off-by: Nikolaus Voss >>>> >>>> This is fine by me, so >>>> >>>> Acked-by: Rafael J. Wysocki >>>> >>>> Or if you want me to take this patch (without the other two in the >>>> series), please let me know. >>> >>> thanks. I think it would be the best if you take up this patch as it >>> is an independent topic. In retrospect it wasn't a good idea to put it >>> into this series. >>> >>> Kind regards, >>> Niko >>> >>> I would have to ask, why is additional code needed for package >>> initialization/resolution? It already happens elsewhere in acpica. Bob >> >> for built-in tables loaded via acpi_ex_load_table_op() everything is >> fine, because after acpi_tb_load_table() acpi_ns_walk_namespace() is >> called to resolve references. >> >> My fix only affects tables loaded dynamically via either acpi_configfs >> driver (for debugging purposes) or efivar_ssdt_load() (to specify a >> table on the kernel's command line). They use acpi_load_table() to load >> the table from a caller-owned buffer. To resolve the references, it is >> again necessary to rewalk the namespace, which was simply missing in >> acpi_load_table(). >> > [Moore, Robert] > > Perhaps you should call AcpiInitializeObjects after the call to > AcpiLoadTable, but I will check. My usage of acpi_load_table() is to load a SSDT which is the intended use of this method according to its description. And my expectation is that the package objects of the loaded table are initialized when this function successfully returns so it aligns with the behavior of acpi_ex_load_table_op() for built-in SSDTs. Otherwise there would be no point in having this function at all, because acpi_tb_install_and_load_table() could be called directly without any difference. Niko