[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <D034M18D63EC.2Y11D954YSZYK@kernel.org>
Date: Mon, 25 Mar 2024 22:50:40 +0200
From: "Jarkko Sakkinen" <jarkko@...nel.org>
To: "Jarkko Sakkinen" <jarkko@...nel.org>, <linux-riscv@...ts.infradead.org>
Cc: "Masami Hiramatsu" <mhiramat@...nel.org>, "Paul Walmsley"
<paul.walmsley@...ive.com>, "Palmer Dabbelt" <palmer@...belt.com>, "Albert
Ou" <aou@...s.berkeley.edu>, <linux-kernel@...r.kernel.org>, "Luis
Chamberlain" <mcgrof@...nel.org>, <linux-modules@...r.kernel.org>, "Naveen
N . Rao" <naveen.n.rao@...ux.ibm.com>, "Anil S Keshavamurthy"
<anil.s.keshavamurthy@...el.com>, "David S . Miller" <davem@...emloft.net>
Subject: Re: [PATCH v3 1/2] kprobes: textmem API
On Mon Mar 25, 2024 at 10:37 PM EET, Jarkko Sakkinen wrote:
> - if (ret == -ENOENT && !trace_kprobe_module_exist(tk)) {
> +#ifdef CONFIG_MODULES
> + if (ret == -ENOENT && trace_kprobe_module_exist(tk))
> + ret = 0;
> +#endif /* CONFIG_MODULES */
For this we could have
#ifndef CONFIG_MODULES
#define trace_kprobe_module_exist(tk) false
#endif
That would clean up at least two locations requiring no changes. Should
I go forward this or not?
BR, Jarkko
Powered by blists - more mailing lists