Question about config UPROBES and UPROBE_EVENTS

From: Tiezhu Yang
Date: Wed Sep 11 2024 - 02:41:15 EST


Hi Masami,

I am a little confused about config UPROBES and UPROBE_EVENTS.
Uprobes is the user-space counterpart to kprobes, I want to do
some small changes:

(1) since config KPROBES can be selectable, just make config UPROBES
selectable too.

(2) since config KPROBE_EVENTS depends on KPROBES rather than select
KPROBES, just make config UPROBE_EVENTS depends on UPROBES rather
than select UPROBES.

Could you please let me know are you OK with the following changes?
If yes, I will send formal patches later.

-- >8 --
diff --git a/arch/Kconfig b/arch/Kconfig
index 975dd22a2dbd..5de2187d3440 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -124,7 +124,8 @@ config KPROBES_ON_FTRACE
optimize on top of function tracing.

config UPROBES
- def_bool n
+ bool "Uprobes"
+ default n
depends on ARCH_SUPPORTS_UPROBES
help
Uprobes is the user-space counterpart to kprobes: they
diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig
index 721c3b221048..7db0462a5d11 100644
--- a/kernel/trace/Kconfig
+++ b/kernel/trace/Kconfig
@@ -732,10 +732,9 @@ config KPROBE_EVENTS_ON_NOTRACE

config UPROBE_EVENTS
bool "Enable uprobes-based dynamic events"
- depends on ARCH_SUPPORTS_UPROBES
+ depends on UPROBES
depends on MMU
depends on PERF_EVENTS
- select UPROBES
select PROBE_EVENTS
select DYNAMIC_EVENTS
select TRACING

Thanks,
Tiezhu