Re: [PATCH] kprobes: Reduce preempt disable scope in check_kprobe_access_safe()

From: Google
Date: Mon Nov 25 2024 - 00:25:11 EST


On Thu, 21 Nov 2024 09:04:32 +0100
Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> wrote:

> On 2024-11-21 08:38:08 [+0100], Thomas Weißschuh wrote:
>
> > Signed-off-by: Thomas Weißschuh <thomas.weissschuh@xxxxxxxxxxxxx>
>
> Reviewed-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx>

Thanks for your review!

>
> > diff --git a/kernel/kprobes.c b/kernel/kprobes.c
> > index da59c68df8412c4662d39d3e286d516e8cee9a69..369020170e832a34ea9c05eda6693bded02ec505 100644
> > --- a/kernel/kprobes.c
> > +++ b/kernel/kprobes.c
> > @@ -39,6 +39,7 @@
> > #include <linux/static_call.h>
> > #include <linux/perf_event.h>
> > #include <linux/execmem.h>
> > +#include <linux/cleanup.h>
>
> This hunk can be dropped. The definition for guard(preempt)() comes from
> preempt.h.
>
> > #include <asm/sections.h>
> > #include <asm/cacheflush.h>
>
> Sebastian


--
Masami Hiramatsu (Google) <mhiramat@xxxxxxxxxx>