From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lv Zheng Subject: [PATCH v5 5/5] ACPI 2.0 / AML: Fix module level execution by correctly parsing table as TermList Date: Fri, 23 Sep 2016 11:27:03 +0800 Message-ID: References: <0e65135af51d94db0410c7059f3bc3a2300fc3b5> Return-path: Received: from mga05.intel.com ([192.55.52.43]:41484 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754198AbcIWD1J (ORCPT ); Thu, 22 Sep 2016 23:27:09 -0400 In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Len Brown Cc: Lv Zheng , Lv Zheng , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org This experiment follows de-facto standard behavior, parsing entire table as a single TermList, so that all module level executions are possible during the table loading. If regressions are found against the enabling of this experimental fix, this patch is the only one that should get bisected out. Please report the regressions to the kernel bugzilla for further root causing. Signed-off-by: Lv Zheng --- include/acpi/acpixf.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/include/acpi/acpixf.h b/include/acpi/acpixf.h index 169ec81..2e22eae 100644 --- a/include/acpi/acpixf.h +++ b/include/acpi/acpixf.h @@ -196,10 +196,9 @@ ACPI_INIT_GLOBAL(u8, acpi_gbl_group_module_level_code, FALSE); /* * Optionally support module level code by parsing the entire table as - * a term_list. Default is FALSE, do not execute entire table until some - * lock order issues are fixed. + * a term_list. Default is TRUE, do execute entire table. */ -ACPI_INIT_GLOBAL(u8, acpi_gbl_parse_table_as_term_list, FALSE); +ACPI_INIT_GLOBAL(u8, acpi_gbl_parse_table_as_term_list, TRUE); /* * Optionally use 32-bit FADT addresses if and when there is a conflict -- 1.7.10 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935526AbcIWD1N (ORCPT ); Thu, 22 Sep 2016 23:27:13 -0400 Received: from mga05.intel.com ([192.55.52.43]:41484 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754198AbcIWD1J (ORCPT ); Thu, 22 Sep 2016 23:27:09 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,380,1470726000"; d="scan'208";a="1060805275" From: Lv Zheng To: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Len Brown Cc: Lv Zheng , Lv Zheng , , linux-acpi@vger.kernel.org Subject: [PATCH v5 5/5] ACPI 2.0 / AML: Fix module level execution by correctly parsing table as TermList Date: Fri, 23 Sep 2016 11:27:03 +0800 Message-Id: X-Mailer: git-send-email 1.7.10 In-Reply-To: References: <0e65135af51d94db0410c7059f3bc3a2300fc3b5> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This experiment follows de-facto standard behavior, parsing entire table as a single TermList, so that all module level executions are possible during the table loading. If regressions are found against the enabling of this experimental fix, this patch is the only one that should get bisected out. Please report the regressions to the kernel bugzilla for further root causing. Signed-off-by: Lv Zheng --- include/acpi/acpixf.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/include/acpi/acpixf.h b/include/acpi/acpixf.h index 169ec81..2e22eae 100644 --- a/include/acpi/acpixf.h +++ b/include/acpi/acpixf.h @@ -196,10 +196,9 @@ ACPI_INIT_GLOBAL(u8, acpi_gbl_group_module_level_code, FALSE); /* * Optionally support module level code by parsing the entire table as - * a term_list. Default is FALSE, do not execute entire table until some - * lock order issues are fixed. + * a term_list. Default is TRUE, do execute entire table. */ -ACPI_INIT_GLOBAL(u8, acpi_gbl_parse_table_as_term_list, FALSE); +ACPI_INIT_GLOBAL(u8, acpi_gbl_parse_table_as_term_list, TRUE); /* * Optionally use 32-bit FADT addresses if and when there is a conflict -- 1.7.10