From patchwork Thu Mar 15 16:49:40 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 10285213 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 3BB4D60386 for ; Thu, 15 Mar 2018 16:50:04 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2E6FE28B6E for ; Thu, 15 Mar 2018 16:50:04 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 22D8228B7E; Thu, 15 Mar 2018 16:50:04 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id B89B928B6E for ; Thu, 15 Mar 2018 16:50:03 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E9B2E6E93F; Thu, 15 Mar 2018 16:50:01 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mout.kundenserver.de (mout.kundenserver.de [212.227.17.13]) by gabe.freedesktop.org (Postfix) with ESMTPS id EAC206E93F for ; Thu, 15 Mar 2018 16:50:00 +0000 (UTC) Received: from wuerfel.lan ([95.208.111.237]) by mrelayeu.kundenserver.de (mreue104 [212.227.15.145]) with ESMTPA (Nemesis) id 0MXp03-1fAahh2jaR-00WrCv; Thu, 15 Mar 2018 17:49:54 +0100 From: Arnd Bergmann To: Oded Gabbay , Alex Deucher , =?UTF-8?q?Christian=20K=C3=B6nig?= , "David (ChunMing) Zhou" , David Airlie Subject: [PATCH] drm/amdkfd: fix uninitialized variable use Date: Thu, 15 Mar 2018 17:49:40 +0100 Message-Id: <20180315164952.2168794-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:DXDVWqdHUhpt8IYnCpEKdljdDE2cuj2D+r14qiL5n9JWv7iTjH7 ttBJZgoHWR9hzuiJndtzrPOo3y3sm/puK2TSfOr5OWxFyX2arAozWmotarw/V/ijwwIaFEv tDd4srvix8WonST20bKqFY/IK4GrVwXQstyQJBlTT6mX6FnOUJ2iVx7TA72jiJVKPatd7C+ Tzb0d2JJ4CJX88A1sZe5Q== X-UI-Out-Filterresults: notjunk:1; V01:K0:CJd0uI0sx7A=:wgPxTFgt+32Gc11OjIm1tm qxRo+ai8MTYH/aBwrA6FvD7DoI4ZehokGMOXbbp0+WEmTbuFyTYJQbLifewzR0tJOBUI25zJW /xej21sRaLJydXQi0/cguSq5vbPNKSv1dGUKPM5GNKY6vHrD6tA2I6ganICODuRQ8FeDI5IiN akze1TQDZ4VXc6IHOvnGEyxp0O7t8Bs+q4VaYOrVf2t6tQp1WB9GMRvDIYj9FKkAzSdYHtIc+ e5A0dv4nlJfariFeHaS4Jq9/RAob6yCRv5k2JzZG7iweczvvpOWfkzEvJurHiGfYgTbU16U+h hf4ueENW6zBZ2xcPVF4DdQmt78bckbtkpMStZGWcmmQ64VoTUZqc6SpYwHo+6Owb1UB18L26c dVaajpvMGLNBrjLesxNfX8DKuwvQ0jpf4IsuB1zbKtKnC+0q5PK+7pdzhLElBwfe4KwGrCyl1 Mg3pxs8c85Mms2lUQbOufIvxyTFBGciaEaxwB1fyVPCcnPV8+SXd/iFlCXq1rVe3DLsQNvTD1 fgtSub6MN7ihOZpbp0ZH1cLQuJGWokHkhM0NzvVFhFkE1H6/X6lfT3wUdaP5M7t9wQUMiEoen uzQX8g8F+q3JRhg5OgKiYYP5dHxV7VO0iUap5RrQzxEnFbdwAinEhrIvZxCc4pTnejA5XCa8A yRl+fR7sKMh/a2WPGEsytkppZtCNQFvC25MaLWFPFWeVmuXjlcU0Ughqy8BWFx5RcGiE1U21G XjpcmO6GtC5B7SIF/zNcB6davwlyb40uc1W7Sw== X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: amd-gfx@lists.freedesktop.org, Arnd Bergmann , Amber Lin , Felix Kuehling , Harish Kasiviswanathan , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Kent Russell MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP When CONFIG_ACPI is disabled, we never initialize the acpi_table structure in kfd_create_crat_image_virtual: drivers/gpu/drm/amd/amdkfd/kfd_crat.c: In function 'kfd_create_crat_image_virtual': drivers/gpu/drm/amd/amdkfd/kfd_crat.c:888:40: error: 'acpi_table' may be used uninitialized in this function [-Werror=maybe-uninitialized] The undefined behavior also happens for any other acpi_get_table() failure, but then the compiler can't warn about it. This adds an error check that prevents the structure from being used in error, avoiding both the undefined behavior and the warning about it. Fixes: 520b8fb755cc ("drm/amdkfd: Add topology support for CPUs") Signed-off-by: Arnd Bergmann --- drivers/gpu/drm/amd/amdkfd/kfd_crat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c index 7493f47e7fe1..d85112224f1d 100644 --- a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c +++ b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c @@ -882,7 +882,7 @@ static int kfd_create_vcrat_image_cpu(void *pcrat_image, size_t *size) crat_table->length = sizeof(struct crat_header); status = acpi_get_table("DSDT", 0, &acpi_table); - if (status == AE_NOT_FOUND) + if (status != AE_OK) pr_warn("DSDT table not found for OEM information\n"); else { crat_table->oem_revision = acpi_table->revision;