From: Vadim Lomovtsev <Vadim.Lomovtsev@caviumnetworks.com>
To: robert.moore@intel.com, lv.zheng@intel.com, rafael.j.wysocki@intel.com,
        lenb@kernel.org, linux-acpi@vger.kernel.org, devel@acpica.org,
        linux-kernel@vger.kernel.org
Cc: vadim.lomovtsev@cavium.com
Subject: [PATCH] acpi: acpica: add acpi status check prior walking through namespace
Date: Tue, 12 Dec 2017 07:59:20 -0800
Message-Id: <1513094360-1414-2-git-send-email-Vadim.Lomovtsev@caviumnetworks.com>
In-Reply-To: <1513094360-1414-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com>
References: <1513094360-1414-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com>
MIME-Version: 1.0
Content-Type: text/plain
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Dec 2017 15:59:37.4990 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: f119cf1c-6d5a-4fdf-b0a2-08d5417958ba
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR07MB2998
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1305
Lines: 41

From: Vadim Lomovtsev <vadim.lomovtsev@cavium.com>

While having kernel built with ACPI support enabled and booted over FDT,
the ltp_acpi test from LTP suite causes kernel crash while calling
acpi_ns_walk_namespace(). The acpi_get_devices is high level wrapper for
it, so we need to protect kernel from crashes by adding acpi status
check before walking through namespace which is not loaded because of
acpi is disabled.

Signed-off-by: Vadim Lomovtsev <vadim.lomovtsev@cavium.com>
---
 drivers/acpi/acpica/nsxfeval.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/acpi/acpica/nsxfeval.c b/drivers/acpi/acpica/nsxfeval.c
index 783f4c8..e0eb9ae 100644
--- a/drivers/acpi/acpica/nsxfeval.c
+++ b/drivers/acpi/acpica/nsxfeval.c
@@ -52,6 +52,8 @@
 #define _COMPONENT          ACPI_NAMESPACE
 ACPI_MODULE_NAME("nsxfeval")
 
+extern int acpi_disabled;
+
 /* Local prototypes */
 static void acpi_ns_resolve_references(struct acpi_evaluate_info *info);
 
@@ -812,6 +814,11 @@ static void acpi_ns_resolve_references(struct acpi_evaluate_info *info)
 
 	ACPI_FUNCTION_TRACE(acpi_get_devices);
 
+	/* check if ACPI disabled to prevent kernel crash later */
+	if (acpi_disabled) {
+		return_ACPI_STATUS(AE_NOT_CONFIGURED);
+	}
+
 	/* Parameter validation */
 
 	if (!user_function) {
-- 
1.8.3.1