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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id AFA4FC4167D for ; Wed, 8 Nov 2023 15:45:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 2BFE66B0283; Wed, 8 Nov 2023 10:45:29 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 26FFB6B0287; Wed, 8 Nov 2023 10:45:29 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 137226B028B; Wed, 8 Nov 2023 10:45:29 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 089F46B0283 for ; Wed, 8 Nov 2023 10:45:29 -0500 (EST) Received: from smtpin22.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id CF04B40CA4 for ; Wed, 8 Nov 2023 15:45:28 +0000 (UTC) X-FDA: 81435211536.22.13111A5 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf05.hostedemail.com (Postfix) with ESMTP id E99E9100014 for ; Wed, 8 Nov 2023 15:45:25 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf05.hostedemail.com: domain of mark.rutland@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=mark.rutland@arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1699458327; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=inLIhR3BcFQrjCiUXdefmhhR3TxwQ6yZ+F7/b8qfVcs=; b=XxxclDT6s4Y6Y5uMmDq2KJFOVJd8HTloRcNFR6zwNmG1e0oVTZ01qOEMklBVog3zeUrWLk F9e0ff15Z57RIcC2FTO0o4onQA6hrP4GK4ffbXO4wdjA//1/E8hnF3fRZsWhLb/S7BBpIc XRdN+W6uxVRht0EgPCIZ5AO6ca1zA+0= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf05.hostedemail.com: domain of mark.rutland@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=mark.rutland@arm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1699458327; a=rsa-sha256; cv=none; b=TYSaR84Sq63tX+81DxxXDCy1Ow1+aZpM3qxbdYDS8tfGZv0ZpyHv02HyYzTVmsrf0XaebK Y4hSEbho9jG3QiLByMcOdm2sw02r34SFmYc/+bXGm7H67t4U8NSkuaWj7jTU53eIxuTB+U b9YEl/998YjL9vy3YCobPby/NDZzymI= Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 39CF11476; Wed, 8 Nov 2023 07:46:09 -0800 (PST) Received: from FVFF77S0Q05N (unknown [10.57.37.113]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 819713F64C; Wed, 8 Nov 2023 07:45:19 -0800 (PST) Date: Wed, 8 Nov 2023 15:44:30 +0000 From: Mark Rutland To: Ankur Arora Cc: linux-kernel@vger.kernel.org, tglx@linutronix.de, peterz@infradead.org, torvalds@linux-foundation.org, paulmck@kernel.org, linux-mm@kvack.org, x86@kernel.org, akpm@linux-foundation.org, luto@kernel.org, bp@alien8.de, dave.hansen@linux.intel.com, hpa@zytor.com, mingo@redhat.com, juri.lelli@redhat.com, vincent.guittot@linaro.org, willy@infradead.org, mgorman@suse.de, jon.grimm@amd.com, bharata@amd.com, raghavendra.kt@amd.com, boris.ostrovsky@oracle.com, konrad.wilk@oracle.com, jgross@suse.com, andrew.cooper3@citrix.com, mingo@kernel.org, bristot@kernel.org, mathieu.desnoyers@efficios.com, geert@linux-m68k.org, glaubitz@physik.fu-berlin.de, anton.ivanov@cambridgegreys.com, mattst88@gmail.com, krypton@ulrich-teichert.org, rostedt@goodmis.org, David.Laight@aculab.com, richard@nod.at, mjguzik@gmail.com Subject: Re: [RFC PATCH 08/86] Revert "arm64: Support PREEMPT_DYNAMIC" Message-ID: References: <20231107215742.363031-1-ankur.a.arora@oracle.com> <20231107215742.363031-9-ankur.a.arora@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231107215742.363031-9-ankur.a.arora@oracle.com> X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: E99E9100014 X-Stat-Signature: wjdtoksyb3pweahkomdhoxxcj1dh73r1 X-HE-Tag: 1699458325-547715 X-HE-Meta: U2FsdGVkX18j9h/vK1cv4YkkiaucC2O9gNXb4cHJEL5DGzxdoqmcRpSZdWn3EvNneebg+YEOanLcSyXAXwJVqHifGMsKmsMldwtGWwgxrT0GTB8DTjzI6mlApaT6Fc6UOlrDsLXWFRnThAht8q8Qpj22y5Ac9T+FBb2OGxzahOWgDeeUMxwWQCwNAbPxqLjGsqe0empn7BCzpDUT0ih6gBgqX67emnbs0+Krz1Z9ePKvW5Y5aFEr5KkZLSgOOsbyBK5V2WX3K9FNZ3PRhVHSBofDPopkzh2Mgpj60Uc9vengPCTjYt+Bie+N1frrKaUvOoAmc13T9uD9YI4tkXt0QsG762zhnwwFbHzwbz/RvI/ryaaaKUwEt1PO6GTUbZj4hz5/Ck2zog27HFvr5dF1kIH/uqYwBGCok/xN9POFWaS6frRwU6Cef+y24CtVTD95Bngf3hFErYqPeMwiBORDj4dw7r61JNiKzKRP4Nwd4iFpG25moiiTWicHkCyHxQF7/X73UQOzr6L0rmiGARFFUWAnHVXIVW+LrZJ+7qpViWJuVg48xe6ub4bLILhI8g1GXLXFXn7sWEcXlAolFIkaGUUrPekSpBlPXJDyXdyqgHkIB1DkGLQE77jLdBuP8sCXBTZ03UsSJ7ND98XpS5xblwZTNKll2I3mt8tU3NtRP5BdFdHLFsiRYBc1UVWPIYQ+935p1AXhnAI6Arwg7K8R240SW+M7WRKdNZSnjSKd2w8s35UpDmvkXYJ7h1dVoYNhhbI+tR8qXcsDXxiHTW/4h1DLOea3Bo0j7qRw2ldyj5MuCx0TnndK3XtKJnAbNwByZUkhs9/9caUuEizUF4TSbb7y1wkRaR3NbPABniantEURzARBNyqO1vT+PmrvyptM1o823R87XhZlR46FWYh7sLJfK0JvTPNT9QsNIYwh9QsCCNeFWBzDqdZcTovrIBwH+MGS5dYoOim+oI2upPR 0O3aICZH +LQa/0Z9R7EzswC1eWCuEgGqpPe3ns70sS5ZbFJkWAt7ZLuWW3FOZbh0XY+6kOaLdD8t3hLpAVVeKmPeUP+DF+lmwcifyfq98oNC9tcacKsJfZBsOHPBluRozOOD+AV1P4ejcFJFlccFTN3HupKse/RrkNmM9QsYslWqpg94Um9fySr0gTzIbaUNxHw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Tue, Nov 07, 2023 at 01:56:54PM -0800, Ankur Arora wrote: > This reverts commit 1b2d3451ee50a0968cb9933f726e50b368ba5073. As the author of the commit being reverted, I'd appreciate being Cc'd on subsequent versions of this patch (and ideally, for the series as a whole). Mark. > > Signed-off-by: Ankur Arora > --- > arch/arm64/Kconfig | 1 - > arch/arm64/include/asm/preempt.h | 19 ++----------------- > arch/arm64/kernel/entry-common.c | 10 +--------- > 3 files changed, 3 insertions(+), 27 deletions(-) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index 78f20e632712..856d7be2ee45 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -221,7 +221,6 @@ config ARM64 > select HAVE_PERF_EVENTS_NMI if ARM64_PSEUDO_NMI > select HAVE_PERF_REGS > select HAVE_PERF_USER_STACK_DUMP > - select HAVE_PREEMPT_DYNAMIC_KEY > select HAVE_REGS_AND_STACK_ACCESS_API > select HAVE_POSIX_CPU_TIMERS_TASK_WORK > select HAVE_FUNCTION_ARG_ACCESS_API > diff --git a/arch/arm64/include/asm/preempt.h b/arch/arm64/include/asm/preempt.h > index 0159b625cc7f..e83f0982b99c 100644 > --- a/arch/arm64/include/asm/preempt.h > +++ b/arch/arm64/include/asm/preempt.h > @@ -2,7 +2,6 @@ > #ifndef __ASM_PREEMPT_H > #define __ASM_PREEMPT_H > > -#include > #include > > #define PREEMPT_NEED_RESCHED BIT(32) > @@ -81,24 +80,10 @@ static inline bool should_resched(int preempt_offset) > } > > #ifdef CONFIG_PREEMPTION > - > void preempt_schedule(void); > +#define __preempt_schedule() preempt_schedule() > void preempt_schedule_notrace(void); > - > -#ifdef CONFIG_PREEMPT_DYNAMIC > - > -DECLARE_STATIC_KEY_TRUE(sk_dynamic_irqentry_exit_cond_resched); > -void dynamic_preempt_schedule(void); > -#define __preempt_schedule() dynamic_preempt_schedule() > -void dynamic_preempt_schedule_notrace(void); > -#define __preempt_schedule_notrace() dynamic_preempt_schedule_notrace() > - > -#else /* CONFIG_PREEMPT_DYNAMIC */ > - > -#define __preempt_schedule() preempt_schedule() > -#define __preempt_schedule_notrace() preempt_schedule_notrace() > - > -#endif /* CONFIG_PREEMPT_DYNAMIC */ > +#define __preempt_schedule_notrace() preempt_schedule_notrace() > #endif /* CONFIG_PREEMPTION */ > > #endif /* __ASM_PREEMPT_H */ > diff --git a/arch/arm64/kernel/entry-common.c b/arch/arm64/kernel/entry-common.c > index 0fc94207e69a..5d9c9951562b 100644 > --- a/arch/arm64/kernel/entry-common.c > +++ b/arch/arm64/kernel/entry-common.c > @@ -225,17 +225,9 @@ static void noinstr arm64_exit_el1_dbg(struct pt_regs *regs) > lockdep_hardirqs_on(CALLER_ADDR0); > } > > -#ifdef CONFIG_PREEMPT_DYNAMIC > -DEFINE_STATIC_KEY_TRUE(sk_dynamic_irqentry_exit_cond_resched); > -#define need_irq_preemption() \ > - (static_branch_unlikely(&sk_dynamic_irqentry_exit_cond_resched)) > -#else > -#define need_irq_preemption() (IS_ENABLED(CONFIG_PREEMPTION)) > -#endif > - > static void __sched arm64_preempt_schedule_irq(void) > { > - if (!need_irq_preemption()) > + if (!IS_ENABLED(CONFIG_PREEMPTION)) > return; > > /* > -- > 2.31.1 >