| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <Z6nc_itA5KT1Gk2i@J2N7QTR9R3>
Date: Mon, 10 Feb 2025 11:04:15 +0000
From: Mark Rutland <mark.rutland@....com>
To: Jinjie Ruan <ruanjinjie@...wei.com>
Cc: catalin.marinas@....com, will@...nel.org, oleg@...hat.com,
sstabellini@...nel.org, tglx@...utronix.de, peterz@...radead.org,
luto@...nel.org, mingo@...hat.com, juri.lelli@...hat.com,
vincent.guittot@...aro.org, dietmar.eggemann@....com,
rostedt@...dmis.org, bsegall@...gle.com, mgorman@...e.de,
vschneid@...hat.com, kees@...nel.org, wad@...omium.org,
akpm@...ux-foundation.org, samitolvanen@...gle.com,
masahiroy@...nel.org, hca@...ux.ibm.com, aliceryhl@...gle.com,
rppt@...nel.org, xur@...gle.com, paulmck@...nel.org, arnd@...db.de,
mbenes@...e.cz, puranjay@...nel.org, pcc@...gle.com,
ardb@...nel.org, sudeep.holla@....com, guohanjun@...wei.com,
rafael@...nel.org, liuwei09@...tc.cn, dwmw@...zon.co.uk,
Jonathan.Cameron@...wei.com, liaochang1@...wei.com,
kristina.martsenko@....com, ptosi@...gle.com, broonie@...nel.org,
thiago.bauermann@...aro.org, kevin.brodsky@....com,
joey.gouly@....com, liuyuntao12@...wei.com, leobras@...hat.com,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
xen-devel@...ts.xenproject.org
Subject: Re: [PATCH -next v5 01/22] arm64: ptrace: Replace
interrupts_enabled() with regs_irqs_disabled()
On Fri, Dec 06, 2024 at 06:17:23PM +0800, Jinjie Ruan wrote:
> The generic entry code expects architecture code to provide
> regs_irqs_disabled(regs) function, but arm64 does not have this and
> provides inerrupts_enabled(regs), which has the opposite polarity.
>
> In preparation for moving arm64 over to the generic entry code,
> relace arm64's interrupts_enabled() with regs_irqs_disabled() and
> update its callers under arch/arm64.
>
> For the moment, a definition of interrupts_enabled() is provided for
> the GICv3 driver. Once arch/arm implement regs_irqs_disabled(), this
> can be removed.
>
> No functional changes.
>
> Suggested-by: Mark Rutland <mark.rutland@....com>
> Signed-off-by: Jinjie Ruan <ruanjinjie@...wei.com>
> ---
> arch/arm64/include/asm/daifflags.h | 2 +-
> arch/arm64/include/asm/ptrace.h | 7 +++++++
> arch/arm64/include/asm/xen/events.h | 2 +-
> arch/arm64/kernel/acpi.c | 2 +-
> arch/arm64/kernel/debug-monitors.c | 2 +-
> arch/arm64/kernel/entry-common.c | 4 ++--
> arch/arm64/kernel/sdei.c | 2 +-
> 7 files changed, 14 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm64/include/asm/daifflags.h b/arch/arm64/include/asm/daifflags.h
> index fbb5c99eb2f9..5fca48009043 100644
> --- a/arch/arm64/include/asm/daifflags.h
> +++ b/arch/arm64/include/asm/daifflags.h
> @@ -128,7 +128,7 @@ static inline void local_daif_inherit(struct pt_regs *regs)
> {
> unsigned long flags = regs->pstate & DAIF_MASK;
>
> - if (interrupts_enabled(regs))
> + if (!regs_irqs_disabled(regs))
> trace_hardirqs_on();
>
> if (system_uses_irq_prio_masking())
> diff --git a/arch/arm64/include/asm/ptrace.h b/arch/arm64/include/asm/ptrace.h
> index 47ff8654c5ec..bcfa96880377 100644
> --- a/arch/arm64/include/asm/ptrace.h
> +++ b/arch/arm64/include/asm/ptrace.h
> @@ -214,9 +214,16 @@ static inline void forget_syscall(struct pt_regs *regs)
> (regs)->pmr == GIC_PRIO_IRQON : \
> true)
>
> +/*
> + * Used by the GICv3 driver, can be removed once arch/arm implements
> + * regs_irqs_disabled() directly.
> + */
> #define interrupts_enabled(regs) \
> (!((regs)->pstate & PSR_I_BIT) && irqs_priority_unmasked(regs))
>
> +#define regs_irqs_disabled(regs) \
> + (((regs)->pstate & PSR_I_BIT) || (!irqs_priority_unmasked(regs)))
Please make this:
| static __always_inline bool regs_irqs_disabled(const struct pt_regs *regs)
| {
| return (regs->pstate & PSR_I_BIT) || !irqs_priority_unmasked(regs);
| }
|
| #define interrupts_enabled(regs) (!regs_irqs_disabled(regs))
That way this matches the style of x86 and s390, and with
interrupts_enabled() defined in terms of regs_irqs_disabled(), the two
cannot accidentaly diverge.
> #define fast_interrupts_enabled(regs) \
> (!((regs)->pstate & PSR_F_BIT))
We should probably delete this at the same time; it's unused and we
don't want any new users to show up.
With those changes:
Acked-by: Mark Rutland <mark.rutland@....com>
Mark.
>
> diff --git a/arch/arm64/include/asm/xen/events.h b/arch/arm64/include/asm/xen/events.h
> index 2788e95d0ff0..2977b5fe068d 100644
> --- a/arch/arm64/include/asm/xen/events.h
> +++ b/arch/arm64/include/asm/xen/events.h
> @@ -14,7 +14,7 @@ enum ipi_vector {
>
> static inline int xen_irqs_disabled(struct pt_regs *regs)
> {
> - return !interrupts_enabled(regs);
> + return regs_irqs_disabled(regs);
> }
>
> #define xchg_xen_ulong(ptr, val) xchg((ptr), (val))
> diff --git a/arch/arm64/kernel/acpi.c b/arch/arm64/kernel/acpi.c
> index e6f66491fbe9..732f89daae23 100644
> --- a/arch/arm64/kernel/acpi.c
> +++ b/arch/arm64/kernel/acpi.c
> @@ -403,7 +403,7 @@ int apei_claim_sea(struct pt_regs *regs)
> return_to_irqs_enabled = !irqs_disabled_flags(arch_local_save_flags());
>
> if (regs)
> - return_to_irqs_enabled = interrupts_enabled(regs);
> + return_to_irqs_enabled = !regs_irqs_disabled(regs);
>
> /*
> * SEA can interrupt SError, mask it and describe this as an NMI so
> diff --git a/arch/arm64/kernel/debug-monitors.c b/arch/arm64/kernel/debug-monitors.c
> index 58f047de3e1c..460c09d03a73 100644
> --- a/arch/arm64/kernel/debug-monitors.c
> +++ b/arch/arm64/kernel/debug-monitors.c
> @@ -231,7 +231,7 @@ static void send_user_sigtrap(int si_code)
> if (WARN_ON(!user_mode(regs)))
> return;
>
> - if (interrupts_enabled(regs))
> + if (!regs_irqs_disabled(regs))
> local_irq_enable();
>
> arm64_force_sig_fault(SIGTRAP, si_code, instruction_pointer(regs),
> diff --git a/arch/arm64/kernel/entry-common.c b/arch/arm64/kernel/entry-common.c
> index b260ddc4d3e9..c547e70428d3 100644
> --- a/arch/arm64/kernel/entry-common.c
> +++ b/arch/arm64/kernel/entry-common.c
> @@ -73,7 +73,7 @@ static __always_inline void __exit_to_kernel_mode(struct pt_regs *regs)
> {
> lockdep_assert_irqs_disabled();
>
> - if (interrupts_enabled(regs)) {
> + if (!regs_irqs_disabled(regs)) {
> if (regs->exit_rcu) {
> trace_hardirqs_on_prepare();
> lockdep_hardirqs_on_prepare();
> @@ -569,7 +569,7 @@ static void noinstr el1_interrupt(struct pt_regs *regs,
> {
> write_sysreg(DAIF_PROCCTX_NOIRQ, daif);
>
> - if (IS_ENABLED(CONFIG_ARM64_PSEUDO_NMI) && !interrupts_enabled(regs))
> + if (IS_ENABLED(CONFIG_ARM64_PSEUDO_NMI) && regs_irqs_disabled(regs))
> __el1_pnmi(regs, handler);
> else
> __el1_irq(regs, handler);
> diff --git a/arch/arm64/kernel/sdei.c b/arch/arm64/kernel/sdei.c
> index 255d12f881c2..27a17da635d8 100644
> --- a/arch/arm64/kernel/sdei.c
> +++ b/arch/arm64/kernel/sdei.c
> @@ -247,7 +247,7 @@ unsigned long __kprobes do_sdei_event(struct pt_regs *regs,
> * If we interrupted the kernel with interrupts masked, we always go
> * back to wherever we came from.
> */
> - if (mode == kernel_mode && !interrupts_enabled(regs))
> + if (mode == kernel_mode && regs_irqs_disabled(regs))
> return SDEI_EV_HANDLED;
>
> /*
> --
> 2.34.1
>
Powered by blists - more mailing lists