Message ID | 20201106103602.9849-2-will@kernel.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show
Return-Path: <SRS0=s5lf=EM=lists.infradead.org=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@kernel.org> X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CA341C55178 for <linux-arm-kernel@archiver.kernel.org>; Fri, 6 Nov 2020 11:10:26 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 52BD920735 for <linux-arm-kernel@archiver.kernel.org>; Fri, 6 Nov 2020 11:10:26 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="eClRzpSp"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="C0j/VB3s" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 52BD920735 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Message-Id:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/AWwl+q+vbIN6/JeZtP/NtjO22Oq4mfeP0Oa1pKVABs=; b=eClRzpSpjQsfBsY/Dk30Rtttq dobTAcIDjuOrbVGa5o2M1+plvMoQdJuS14mS8qM/ZdD+Ld/w6gDpoR8JybG0IA2j8zBxozv9HQ55M eXPshsDMqsqcvImP5tQsR95cyzIVGLOPB0AM+4SkwH/9QoB/R/MtOp+T1Ws30XCrVctWK4x79bbsi j6GBu1S6tAr14j+XU36Wi4vL9LE+KZH9fJLzUcSupGtnSprsE278JvhA5L8JxXY5zWeRMhM/AXJHp 4piGAb0c6BtZ77EeqJKhCh+kWTPXk+M/oglzI8E8kuheBmQa1WQJ8E7ol/wiMqX7fQq3wpChLFE+8 4qJOk3Mpg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kaz6M-0004Vw-7X; Fri, 06 Nov 2020 10:36:14 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kaz6J-0004US-Eq for linux-arm-kernel@lists.infradead.org; Fri, 06 Nov 2020 10:36:12 +0000 Received: from localhost.localdomain (236.31.169.217.in-addr.arpa [217.169.31.236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 49821221FA; Fri, 6 Nov 2020 10:36:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1604658970; bh=c8VhSjexJh87smF8lFF4bobVi/ptsexAmWhReQCmBlI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=C0j/VB3sGPYfq+QC06pPn6kTewoZ7RHbPIMLsAqMbjQKZ8LcjxAkLsx5fJMw6y85z JhsbfImsn08r70Y9HTimznLDZz2HnsnpiGWmh7GTAX+tRMkk5+G2JtR9bu0n35D6io gMpwgq7LLrBbaWfi6LTkqUyU3oPgs7dBTE+bdDgg= From: Will Deacon <will@kernel.org> To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/2] arm64: psci: Avoid printing in cpu_psci_cpu_die() Date: Fri, 6 Nov 2020 10:36:01 +0000 Message-Id: <20201106103602.9849-2-will@kernel.org> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20201106103602.9849-1-will@kernel.org> References: <20201106103602.9849-1-will@kernel.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201106_053611_617529_D1AF0D98 X-CRM114-Status: GOOD ( 13.46 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: <linux-arm-kernel.lists.infradead.org> List-Unsubscribe: <http://lists.infradead.org/mailman/options/linux-arm-kernel>, <mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe> List-Archive: <http://lists.infradead.org/pipermail/linux-arm-kernel/> List-Post: <mailto:linux-arm-kernel@lists.infradead.org> List-Help: <mailto:linux-arm-kernel-request@lists.infradead.org?subject=help> List-Subscribe: <http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>, <mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe> Cc: "Paul E. McKenney" <paulmck@kernel.org>, Qian Cai <cai@redhat.com>, kernel-team@android.com, linux-kernel@vger.kernel.org, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will@kernel.org> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org> Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org |
Series |
arm64: Fix issues with CPU hotplug and RCU
|
expand
|
diff --git a/arch/arm64/kernel/psci.c b/arch/arm64/kernel/psci.c index 43ae4e0c968f..6a4f3e37c3b4 100644 --- a/arch/arm64/kernel/psci.c +++ b/arch/arm64/kernel/psci.c @@ -75,8 +75,6 @@ static void cpu_psci_cpu_die(unsigned int cpu) PSCI_0_2_POWER_STATE_TYPE_SHIFT; ret = psci_ops.cpu_off(state); - - pr_crit("unable to power off CPU%u (%d)\n", cpu, ret); } static int cpu_psci_cpu_kill(unsigned int cpu)
cpu_psci_cpu_die() is called in the context of the dying CPU, which will no longer be online or tracked by RCU. It is therefore not generally safe to call printk() if the PSCI "cpu off" request fails, so remove the pr_crit() invocation. Cc: Qian Cai <cai@redhat.com> Cc: "Paul E. McKenney" <paulmck@kernel.org> Cc: Catalin Marinas <catalin.marinas@arm.com> Signed-off-by: Will Deacon <will@kernel.org> --- arch/arm64/kernel/psci.c | 2 -- 1 file changed, 2 deletions(-)