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: 131824 Delivered-To: patch@linaro.org Received: by 10.46.84.17 with SMTP id i17csp1350562ljb; Thu, 15 Mar 2018 09:50:09 -0700 (PDT) X-Google-Smtp-Source: AG47ELu+Bj5ZKrAbMYpanRDrzoHHFybu35IMdSCOlkQO+AYnbg6dAIRzypx76lt2QEQ42w2rsXON X-Received: by 2002:a17:902:bc3:: with SMTP id 61-v6mr8775189plr.398.1521132609469; Thu, 15 Mar 2018 09:50:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521132609; cv=none; d=google.com; s=arc-20160816; b=YbplfTEl0tikAU8egt8LEbtXdP/gF4G200fVvJlrnrAI7wztMGkJFmMZhGX4pYPPyn FNtDjFSYYuma6DNnHJ9BLKPtQuvNDOdGKwq7sVuBgKLlBFzRBWtvwcgoxHbXHQY8ug8I s7i0WMa/N38JzZKuoi0vUBeDgWe3/htqFSaPFZ9mVQ5erI+nuHHrOFTBEdlgBM5w3QR1 mT5TDVoGjFG/4/I+Oz4E/YP2JFmOydVvCOkXffKLhPMu1mHe26NNOeAcofXD3CoqHYen wIOfBU2GAiFffeHeqeb4dRYQZ71MFlAOBY0eeXMskij7C1Ixy7aoGSv9L4jUD5XdtrxS PA1g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=hvJhmpCxQE7LEis12y+Rxqrx8y9iAfqvrwnr6aB0q4A=; b=qyBpSOQqkLq5VNqNXz9xXuETouwAFh2eX/qRJI0TJORYdRQgUabjF76lrm28Vew7TN cZcynkaidXYBaX5zP+zcmVXVceDTVxYgSeA8rq7xNtwN67FwhT2iobdtdGtMmshKDXkL 4NyAgHiHmdgY53IlwZeoshO8lFJU42LSoo99GiIkHrgykpjSNIsGOoaMneNFHDYxkkLm Sil3QZIbpMyjcDxxxS439EhNp1jsBf7nWTgn5WY3OzYQdm1yIEn6DSBiRB42LolkIuwx 1tgkNgdZBgmaPLtKbnezcO0ovioShVwErWBkgPdsOhURr6tYruui+AmHQpUwv6k1LkHO 27xA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id q6-v6si706655plk.234.2018.03.15.09.50.09; Thu, 15 Mar 2018 09:50:09 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752452AbeCOQuE (ORCPT + 28 others); Thu, 15 Mar 2018 12:50:04 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:33515 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750731AbeCOQuD (ORCPT ); Thu, 15 Mar 2018 12:50:03 -0400 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_K=C3=B6nig?= , "David (ChunMing) Zhou" , David Airlie Cc: Arnd Bergmann , Felix Kuehling , Harish Kasiviswanathan , Kent Russell , Amber Lin , dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org 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== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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(-) -- 2.9.0 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;