From patchwork Fri Jan 15 13:09:06 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anders Roxell X-Patchwork-Id: 363623 Delivered-To: patch@linaro.org Received: by 2002:a02:ccad:0:0:0:0:0 with SMTP id t13csp278885jap; Fri, 15 Jan 2021 05:10:30 -0800 (PST) X-Google-Smtp-Source: ABdhPJy2pyQl0gdOHl7dzOq1KU5avk4aW/MbosKd6kOv4XdgIRG4T2hYfMUfDnBj/1d7cU07+kXK X-Received: by 2002:a50:fb97:: with SMTP id e23mr9712871edq.208.1610716229881; Fri, 15 Jan 2021 05:10:29 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1610716229; cv=none; d=google.com; s=arc-20160816; b=BbBok1JY9qIpd9cp93NEeQ+1x7ZVM3BnCDdCmijVlUAWbfH1mlXJHpJqxH4cWrSPzE 1xUZFxesI3JI6UM/73k6rozgklq1TvcTZOkj1rs8SyXbp67ynh2JAgzdU7oeRyI75vIQ olJwLnRn8ffgKT5KHRyGJi+AK5SF3ESl01t8e0u/orid/7VTIHqyV4t+QFOnxl44Brac WJ+AOqNd8n5qbeczg9EDwGhDIsXZSzs28iSW3+7TkT0eymgK0TZQ8/22QH6O01Anpadu VoZ/oV/3wPqPu/wBYFSiLXGSKyb1NVkx/ueEG4h65CNzhiVuJ8zXECgGillDiwgXGNhx 0NOg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=nj25LfTkFrAAE+qx8en0KB7gSAPCw/LDljzLCu37kJw=; b=JFR1uXKC7Av/qrab1ueXIoZbdqNxakhjwR1kWtwLsHLrWGaPD47bx1ekt+NvkCP625 GAN1zd/iSf50Dqadm6iyB13kXg5wb86hk+DDDML2iN4n0nuIw41HCNaLcuo0RPMrvi2N zCVCZjKDoV28KVvhrz3i5ZtBvh07AHP5H5ax73OnwlDUSA9vnjOL2tyvNosgImeihqK7 ma/gOg0SHNnkeSCLGC7247L+Bw7ESolK9safOSSYOv58S1oRvqau3E+0a6oBKViZqnMT Wr+rl7r1wkYr9r+puUY0b02u9XSRMEFaHcCo7TRzqnalM/P//r2FHJZEmwXXdR1Qs2uV Q37g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=RbBIDKsG; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v1si4418894edr.598.2021.01.15.05.10.29; Fri, 15 Jan 2021 05:10:29 -0800 (PST) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=RbBIDKsG; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732536AbhAONJq (ORCPT + 13 others); Fri, 15 Jan 2021 08:09:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46958 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732463AbhAONJq (ORCPT ); Fri, 15 Jan 2021 08:09:46 -0500 Received: from mail-lj1-x236.google.com (mail-lj1-x236.google.com [IPv6:2a00:1450:4864:20::236]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 54F6AC061799 for ; Fri, 15 Jan 2021 05:09:28 -0800 (PST) Received: by mail-lj1-x236.google.com with SMTP id n11so10327405lji.5 for ; Fri, 15 Jan 2021 05:09:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=nj25LfTkFrAAE+qx8en0KB7gSAPCw/LDljzLCu37kJw=; b=RbBIDKsG5AyXSoGKHLolqmP9CsgzbCWO7vIeUc586k5RKCpmBkSNZvo+UIZ+HX7Xmh 9814SPTXCc7VQ4hvKBI9H+F46fmpyTOy9r1/A1Ja9DHKRRU7jqj8gG3RARLIGSeSCI6S AKH/pa1h635Me4UmzBSKO1y4Zvq53UnfOjlqalouLYWy4mhD+vmZ3K+3hdoauVpKMCJJ ROWvhachqr9PexOH3QFs8xyi4zQTufu5RPiMPkOn5CbzmIqiKLwhElSPAl7YKxleeBMM iAhg510vTb4oZ6Uys/oYjHA5Oc0CFDjm7Qg0Mb5BI3TyQ+wCCiu6jo8222X6AVwAOaOW KL+A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=nj25LfTkFrAAE+qx8en0KB7gSAPCw/LDljzLCu37kJw=; b=ZBX+qYXVqQ5Wjgj21t2FX6mO98O3E5REePRlRktVdMeo9Nqd/Z4CcMSoqYDMa5w1oA 59AAflVOheKrwqR9x3E/svw+XSK6v9fXpE6APTB9mRagO5zG6s0ocqGCs7DUU5m3MEOp g68bofXwmqCzbr0EucA8dXL/B7aTVmXMLGo8YHV6R3Z1kWs7cC2lGZlx7MDyH3vhmGJY 5DpV0KrsPSenaul4bfE/UeuVz3i3NZd+1JBnC/H9UpZXlNMNiUue+HinEnRf3vIvmLgP 6Eu7dDovL5m6EDB4etGoN8EWctMBLSf6PJWJ6eZydTnMvXVC+xBEnmn/Fd5Pgy3xjLQ8 JHbA== X-Gm-Message-State: AOAM531a4pG1mC3Axvz/9gikZc7Baoix0k9zDQ9AkSZ130D+pHt/9gUJ ZuMi2FdoPyTSToch5eBc3X2krCqM6n4BbhZJ X-Received: by 2002:a2e:87d2:: with SMTP id v18mr5339872ljj.464.1610716166861; Fri, 15 Jan 2021 05:09:26 -0800 (PST) Received: from localhost (c-9b28e555.07-21-73746f28.bbcust.telenor.se. [85.229.40.155]) by smtp.gmail.com with ESMTPSA id i11sm894372lfl.297.2021.01.15.05.09.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Jan 2021 05:09:26 -0800 (PST) From: Anders Roxell To: stable@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Anders Roxell , Nathan Chancellor , Naresh Kamboju , Nick Desaulniers , Thomas Bogendoerfer Subject: [PATCH] mips: lib: uncached: fix non-standard usage of variable 'sp' Date: Fri, 15 Jan 2021 14:09:06 +0100 Message-Id: <20210115130906.1084281-2-anders.roxell@linaro.org> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210115130906.1084281-1-anders.roxell@linaro.org> References: <20210115130906.1084281-1-anders.roxell@linaro.org> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org commit 5b058973d3205578aa6c9a71392e072a11ca44ef upstream. When building mips tinyconfig with clang the following warning show up: arch/mips/lib/uncached.c:45:6: warning: variable 'sp' is uninitialized when used here [-Wuninitialized] if (sp >= (long)CKSEG0 && sp < (long)CKSEG2) ^~ arch/mips/lib/uncached.c:40:18: note: initialize the variable 'sp' to silence this warning register long sp __asm__("$sp"); ^ = 0 1 warning generated. Rework to make an explicit inline move, instead of the non-standard use of specifying registers for local variables. This is what's written from the gcc-10 manual [1] about specifying registers for local variables: "6.47.5.2 Specifying Registers for Local Variables ................................................. [...] "The only supported use for this feature is to specify registers for input and output operands when calling Extended 'asm' (*note Extended Asm::). [...]". Cc: # v5.4+ [1] https://docs.w3cub.com/gcc~10/local-register-variables Signed-off-by: Anders Roxell Reported-by: Nathan Chancellor Reported-by: Naresh Kamboju Reviewed-by: Nick Desaulniers Signed-off-by: Thomas Bogendoerfer --- arch/mips/lib/uncached.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) -- 2.29.2 diff --git a/arch/mips/lib/uncached.c b/arch/mips/lib/uncached.c index 09d5deea747f..f80a67c092b6 100644 --- a/arch/mips/lib/uncached.c +++ b/arch/mips/lib/uncached.c @@ -37,10 +37,12 @@ */ unsigned long run_uncached(void *func) { - register long sp __asm__("$sp"); register long ret __asm__("$2"); long lfunc = (long)func, ufunc; long usp; + long sp; + + __asm__("move %0, $sp" : "=r" (sp)); if (sp >= (long)CKSEG0 && sp < (long)CKSEG2) usp = CKSEG1ADDR(sp);