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
| ||
|
Date: Mon, 26 Sep 2022 07:18:26 +0800 From: Guo Ren <guoren@...nel.org> To: Jisheng Zhang <jszhang@...nel.org> Cc: Paul Walmsley <paul.walmsley@...ive.com>, Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, Nathan Chancellor <nathan@...nel.org>, Nick Desaulniers <ndesaulniers@...gle.com>, linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org, llvm@...ts.linux.dev Subject: Re: [PATCH 1/4] riscv: remove extra level wrappers of trace_hardirqs_{on,off} Reviewed-by: Guo Ren <guoren@...nel.org> I would involve the patch in the generic entry series, okay? On Mon, Sep 26, 2022 at 2:03 AM Jisheng Zhang <jszhang@...nel.org> wrote: > > Since riscv is converted to generic entry, there's no need for the > extra wrappers of trace_hardirqs_{on,off}. > > Tested with llvm + irqsoff. > > Signed-off-by: Jisheng Zhang <jszhang@...nel.org> > --- > arch/riscv/kernel/Makefile | 2 -- > arch/riscv/kernel/trace_irq.c | 27 --------------------------- > arch/riscv/kernel/trace_irq.h | 11 ----------- > 3 files changed, 40 deletions(-) > delete mode 100644 arch/riscv/kernel/trace_irq.c > delete mode 100644 arch/riscv/kernel/trace_irq.h > > diff --git a/arch/riscv/kernel/Makefile b/arch/riscv/kernel/Makefile > index 01da14e21019..11ee206cc235 100644 > --- a/arch/riscv/kernel/Makefile > +++ b/arch/riscv/kernel/Makefile > @@ -69,8 +69,6 @@ obj-$(CONFIG_CPU_PM) += suspend_entry.o suspend.o > obj-$(CONFIG_FUNCTION_TRACER) += mcount.o ftrace.o > obj-$(CONFIG_DYNAMIC_FTRACE) += mcount-dyn.o > > -obj-$(CONFIG_TRACE_IRQFLAGS) += trace_irq.o > - > obj-$(CONFIG_PERF_EVENTS) += perf_callchain.o > obj-$(CONFIG_HAVE_PERF_REGS) += perf_regs.o > obj-$(CONFIG_RISCV_SBI) += sbi.o > diff --git a/arch/riscv/kernel/trace_irq.c b/arch/riscv/kernel/trace_irq.c > deleted file mode 100644 > index 095ac976d7da..000000000000 > --- a/arch/riscv/kernel/trace_irq.c > +++ /dev/null > @@ -1,27 +0,0 @@ > -// SPDX-License-Identifier: GPL-2.0 > -/* > - * Copyright (C) 2022 Changbin Du <changbin.du@...il.com> > - */ > - > -#include <linux/irqflags.h> > -#include <linux/kprobes.h> > -#include "trace_irq.h" > - > -/* > - * trace_hardirqs_on/off require the caller to setup frame pointer properly. > - * Otherwise, CALLER_ADDR1 might trigger an pagging exception in kernel. > - * Here we add one extra level so they can be safely called by low > - * level entry code which $fp is used for other purpose. > - */ > - > -void __trace_hardirqs_on(void) > -{ > - trace_hardirqs_on(); > -} > -NOKPROBE_SYMBOL(__trace_hardirqs_on); > - > -void __trace_hardirqs_off(void) > -{ > - trace_hardirqs_off(); > -} > -NOKPROBE_SYMBOL(__trace_hardirqs_off); > diff --git a/arch/riscv/kernel/trace_irq.h b/arch/riscv/kernel/trace_irq.h > deleted file mode 100644 > index 99fe67377e5e..000000000000 > --- a/arch/riscv/kernel/trace_irq.h > +++ /dev/null > @@ -1,11 +0,0 @@ > -/* SPDX-License-Identifier: GPL-2.0 */ > -/* > - * Copyright (C) 2022 Changbin Du <changbin.du@...il.com> > - */ > -#ifndef __TRACE_IRQ_H > -#define __TRACE_IRQ_H > - > -void __trace_hardirqs_on(void); > -void __trace_hardirqs_off(void); > - > -#endif /* __TRACE_IRQ_H */ > -- > 2.34.1 > -- Best Regards Guo Ren
Powered by blists - more mailing lists