[PATCH 0/2] tools/sched_ext: Sync configuration with upstream

[PATCH 0/2] tools/sched_ext: Sync configuration with upstream

[Cheng-Yang Chou]

This series synchronizes the sched-ext tool's Kconfig and README.md 
with the upstream scx repository. 

The first patch streamlines README.md by removing redundant entries. 
The second patch adds the missing Kconfig file synced from upstream.

Thanks,
-chengyang

---

Cheng-Yang Chou (2):
  tools/sched_ext: Sync README.md Kconfig with upstream scx
  tools/sched_ext: Add Kconfig to sync with upstream

 tools/sched_ext/Kconfig   | 71 +++++++++++++++++++++++++++++++++++++++
 tools/sched_ext/README.md |  9 +----
 2 files changed, 72 insertions(+), 8 deletions(-)
 create mode 100644 tools/sched_ext/Kconfig

-- 
2.48.1

[PATCH 1/2] tools/sched_ext: Sync README.md Kconfig with upstream scx

[Cheng-Yang Chou]

Sync the documentation with the upstream scx repository to
reflect the current recommended configuration.

Ref: https://github.com/sched-ext/scx/blob/main/README.md#build--install

Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
---
 tools/sched_ext/README.md | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/tools/sched_ext/README.md b/tools/sched_ext/README.md
index 16a42e4060f6..bcd2c49d777c 100644
--- a/tools/sched_ext/README.md
+++ b/tools/sched_ext/README.md
@@ -54,19 +54,12 @@ Kconfig options:
 
 ```
 CONFIG_BPF=y
-CONFIG_SCHED_CLASS_EXT=y
 CONFIG_BPF_SYSCALL=y
 CONFIG_BPF_JIT=y
 CONFIG_DEBUG_INFO_BTF=y
-```
-
-It's also recommended that you also include the following Kconfig options:
-
-```
 CONFIG_BPF_JIT_ALWAYS_ON=y
 CONFIG_BPF_JIT_DEFAULT_ON=y
-CONFIG_PAHOLE_HAS_SPLIT_BTF=y
-CONFIG_PAHOLE_HAS_BTF_TAG=y
+CONFIG_SCHED_CLASS_EXT=y
 ```
 
 There is a `Kconfig` file in this directory whose contents you can append to
-- 
2.48.1

[PATCH 2/2] tools/sched_ext: Add Kconfig to sync with upstream

[Cheng-Yang Chou]

Add the missing Kconfig file to tools/sched_ext/ as referenced in 
the README.

Ref: https://github.com/sched-ext/scx/blob/main/kernel.config

Signed-off-by: Cheng-Yang Chou <yphbchou0911@gmail.com>
---
 tools/sched_ext/Kconfig | 71 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 71 insertions(+)
 create mode 100644 tools/sched_ext/Kconfig

diff --git a/tools/sched_ext/Kconfig b/tools/sched_ext/Kconfig
new file mode 100644
index 000000000000..65b6485d0bd9
--- /dev/null
+++ b/tools/sched_ext/Kconfig
@@ -0,0 +1,71 @@
+# sched-ext mandatory options
+#
+CONFIG_BPF=y
+CONFIG_BPF_SYSCALL=y
+CONFIG_BPF_JIT=y
+CONFIG_DEBUG_INFO_BTF=y
+CONFIG_BPF_JIT_ALWAYS_ON=y
+CONFIG_BPF_JIT_DEFAULT_ON=y
+CONFIG_SCHED_CLASS_EXT=y
+
+# Required by some rust schedulers (e.g. scx_p2dq)
+#
+CONFIG_KALLSYMS_ALL=y
+
+# Required on arm64
+#
+# CONFIG_DEBUG_INFO_REDUCED is not set
+
+# LAVD tracks futex to give an additional time slice for futex holder
+# (i.e., avoiding lock holder preemption) for better system-wide progress.
+# LAVD first tries to use ftrace to trace futex function calls.
+# If that is not available, it tries to use a tracepoint.
+CONFIG_FUNCTION_TRACER=y
+
+# Enable scheduling debugging
+#
+CONFIG_SCHED_DEBUG=y
+
+# Enable extra scheduling features (for a better code coverage while testing
+# the schedulers)
+#
+CONFIG_SCHED_AUTOGROUP=y
+CONFIG_SCHED_CORE=y
+CONFIG_SCHED_MC=y
+
+# Enable fully preemptible kernel for a better test coverage of the schedulers
+#
+# CONFIG_PREEMPT_NONE is not set
+# CONFIG_PREEMPT_VOLUNTARY is not set
+CONFIG_PREEMPT=y
+CONFIG_PREEMPT_COUNT=y
+CONFIG_PREEMPTION=y
+CONFIG_PREEMPT_DYNAMIC=y
+CONFIG_PREEMPT_RCU=y
+
+# Additional debugging information (useful to catch potential locking issues)
+CONFIG_DEBUG_LOCKDEP=y
+CONFIG_DEBUG_ATOMIC_SLEEP=y
+CONFIG_PROVE_LOCKING=y
+
+# Bpftrace headers (for additional debug info)
+CONFIG_BPF=y
+CONFIG_BPF_SYSCALL=y
+CONFIG_BPF_JIT=y
+CONFIG_HAVE_EBPF_JIT=y
+CONFIG_BPF_EVENTS=y
+CONFIG_FTRACE_SYSCALLS=y
+CONFIG_HAVE_DYNAMIC_FTRACE=y
+CONFIG_DYNAMIC_FTRACE=y
+CONFIG_HAVE_KPROBES=y
+CONFIG_KPROBES=y
+CONFIG_KPROBE_EVENTS=y
+CONFIG_ARCH_SUPPORTS_UPROBES=y
+CONFIG_UPROBES=y
+CONFIG_UPROBE_EVENTS=y
+CONFIG_DEBUG_FS=y
+
+# more bpftrace to make that work
+CONFIG_IKHEADERS=y
+CONFIG_IKCONFIG_PROC=y
+CONFIG_IKCONFIG=y
-- 
2.48.1

Re: [PATCH 0/2] tools/sched_ext: Sync configuration with upstream

[Tejun Heo]

Hello,

AI-assisted review follows. Please verify the points.

- Patch 1 doesn't apply. The pre-image includes CONFIG_PAHOLE_HAS_SPLIT_BTF
  which has already been removed. Please rebase.

- Patch 2 has duplicate entries. CONFIG_BPF, CONFIG_BPF_SYSCALL,
  CONFIG_BPF_JIT appear in both the mandatory section and the bpftrace
  section.

- Patch 2 includes a number of options that are not user-selectable - they are
  auto-selected by arch or other dependencies and have no effect in .config
  fragments: CONFIG_HAVE_EBPF_JIT, CONFIG_HAVE_DYNAMIC_FTRACE,
  CONFIG_HAVE_KPROBES, CONFIG_ARCH_SUPPORTS_UPROBES, CONFIG_PREEMPT_COUNT,
  CONFIG_PREEMPTION, CONFIG_PREEMPT_RCU. These should be dropped.

- "more bpftrace to make that work" is a bit too casual as a comment. Please
  clean it up.

Thanks.

--
tejun