Skip to content

Cleanup options (part 1)#388

Merged
ptr1337 merged 2 commits intomasterfrom
cleanup-options-part1
Feb 8, 2025
Merged

Cleanup options (part 1)#388
ptr1337 merged 2 commits intomasterfrom
cleanup-options-part1

Conversation

@ventureoo
Copy link
Member

We've talked about this before in #367, and this time I've limited myself to removing options that won't cause doubt.

First commit removes _makegconfig and _makemenuconfig:

We have 4 interfaces for kernel configuration, most of which have the
same functionality and differ only in the rendering backend. gconfig
uses GTK2 which is EOL, and unlikely anyone will install it just for
gconfig when there are alternatives like xconfig which is based on Qt5 /
Qt 6. Similarly in the case of TUI, instead of menuconfig, here we have
ncurses based nconfig which is supported by most terminals.

Second commit removes _NUMAdisable:

The first reports of positive impact of disabling NUMA for desktop
systems date back to 2012 [1], and even then they were extremely minor
and at margin of error level of ~300ms at kernel compilation. Now more
than 10 years have passed, there have been many NUMA-related changes in
kernel over the years and it probably makes no sense to say that
disabling NUMA provides any benefits. I haven't found any relevant
performance comparisons with NUMA disabled, if someone provides them and
shows evidence that it may still be necessary - I'm happy to revert this
change.

I should also note that some schedulers like PDS and BMQ forcibly
disable NUMA because they do not have a proper implementation of its
support.

[1] - https://bugs.archlinux.org/task/31187

I should add that NUMA can also be disabled during the boot time via numa=off.

We have 4 interfaces for kernel configuration, most of which have the
same functionality and differ only in the rendering backend. gconfig
uses GTK2 which is EOL, and unlikely anyone will install it just for
gconfig when there are alternatives like xconfig which is based on Qt5 /
Qt 6. Similarly in the case of TUI, instead of menuconfig, here we have
ncurses based nconfig which is supported by most terminals.

Signed-off-by: Vasiliy Stelmachenok <ventureo@cachyos.org>
The first reports of positive impact of disabling NUMA for desktop
systems date back to 2012 [1], and even then they were extremely minor
and at margin of error level of ~300ms at kernel compilation. Now more
than 10 years have passed, there have been many NUMA-related changes in
kernel over the years and it probably makes no sense to say that
disabling NUMA provides any benefits. I haven't found any relevant
performance comparisons with NUMA disabled, if someone provides them and
shows evidence that it may still be necessary - I'm happy to revert this
change.

I should also note that some schedulers like PDS and BMQ forcibly
disable NUMA because they do not have a proper implementation of its
support.

[1] - https://bugs.archlinux.org/task/31187

Signed-off-by: Vasiliy Stelmachenok <ventureo@cachyos.org>
@ptr1337 ptr1337 merged commit 5588a00 into master Feb 8, 2025
2 checks passed
@ptr1337 ptr1337 deleted the cleanup-options-part1 branch February 8, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants