From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 23FE4D16279 for ; Mon, 14 Oct 2024 14:49:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=MqoCmC3mlROLDTAJ6KuaV6jIyh5atTCVgAtrqA7ejkA=; b=QbGVJj/sETwvfY4zhBQRRs3+fV TAmtJSnGWSNPlDbNwDdCsLqyr2W4DvJrBobHU4Bmlpi3P9pQHLK/tWFD/kn0Y9Lsz4QzBMmT10sn+ 2FeiqUFz16M5pM6jKSH+N8bYuIYP0jLpwlUCf+VorB0NSKsgXOJfxixhjUVuqcMS3Cc+LbUAlsb+q YHOwaAUF2hYYM01ySeI34yKFh7do+F9UzNOMOzNhcOEsTImSrgFRFOHqGEvscu+KdtUWvp7q1yfbt xcAiT86xBDgBlW9pzh2QNUai36qbeokK4U1ptSXscb7alOUXGuV97AMSTkyu1L63iDPqebXbMDoOT EC9YsPAQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t0MO2-00000005Y83-1LZL; Mon, 14 Oct 2024 14:49:30 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t0KqW-00000005DAG-2Op5 for linux-arm-kernel@lists.infradead.org; Mon, 14 Oct 2024 13:10:54 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 9207E5C5A72; Mon, 14 Oct 2024 13:10:43 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7ECABC4CEC6; Mon, 14 Oct 2024 13:10:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1728911447; bh=YlnMitl1nLOteyh8gA+2LBlWbZHa6L6A0MLvuZ7ayak=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=W/N1SIdCG1aBBJJORv5NeHppZz0CcrlCsVaqL5XsMwJANKnLFSeaDYGJ28SVevTtA wmKbAXRv5Almy7vC5SHjDBZdkkuIkqCds7XUaM1ThVIAG28grmZWopCIPlyJZ1tIHn 98oMgDpG3uPPMlDcuMf1NPMmPgGHkBCDoBEz5ABEzvkwhXzDqndK+aeT0Bc3HmgaJ7 YcfkwnBl3mpkIHkOoHWgYCJnuC1CEU+Jmae/cVLN497GcfhnYmGV1j5Z8j94VckNHi 5jNv6/yR/V9vb2l5fIobcC2d7EhHnD7q5TKQOCnoqxvmy1N1ZJRB4rw0CJm0iHIjXD OlQ5X7OZCT2NQ== Date: Mon, 14 Oct 2024 14:10:43 +0100 From: Will Deacon To: Ryo Takakura Cc: catalin.marinas@arm.com, broonie@kernel.org, mark.rutland@arm.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] arm64: Remove the check for CONFIG_TINY_RCU Message-ID: <20241014131042.GA17353@willie-the-truck> References: <20240914090040.166671-1-ryotkkr98@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240914090040.166671-1-ryotkkr98@gmail.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241014_061048_715279_8132876F X-CRM114-Status: GOOD ( 20.75 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sat, Sep 14, 2024 at 06:00:40PM +0900, Ryo Takakura wrote: > Since the commit 4b3dc9679cf77 ("arm64: force CONFIG_SMP=y and remove > redundant #ifdefs"), arm64 defaults to CONFIG_SMP but TINY_RCU is cofigured > only for !SMP systems. > > Remove the check for CONFIG_TINY_RCU as it should always be false. > > Signed-off-by: Ryo Takakura > --- > arch/arm64/kernel/entry-common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/kernel/entry-common.c b/arch/arm64/kernel/entry-common.c > index b77a15955f28..a9765364fc67 100644 > --- a/arch/arm64/kernel/entry-common.c > +++ b/arch/arm64/kernel/entry-common.c > @@ -40,7 +40,7 @@ static __always_inline void __enter_from_kernel_mode(struct pt_regs *regs) > { > regs->exit_rcu = false; > > - if (!IS_ENABLED(CONFIG_TINY_RCU) && is_idle_task(current)) { > + if (is_idle_task(current)) { > lockdep_hardirqs_off(CALLER_ADDR0); > ct_irq_enter(); > trace_hardirqs_off_finish(); I think this code was deliberately written to follow kernel/entry/common.c as closely as possible, as we should be able to switch over to that at some point. Come to think of it, Mark, what work is needed before we can move to the generic code? Is there anything you need a hand with? Will