On 10.12.2025 17:57, Jason Andryuk wrote: > On 2025-12-10 03:17, Jan Beulich wrote: >> On 09.12.2025 22:47, Jason Andryuk wrote: >>> --- a/xen/common/Kconfig >>> +++ b/xen/common/Kconfig >>> @@ -680,4 +680,13 @@ config PM_STATS >>> Enable collection of performance management statistics to aid in >>> analyzing and tuning power/performance characteristics of the system >>> >>> +config GC_SECTIONS >>> + bool "Garbage Collect Sections" >>> + select CC_SPLIT_SECTIONS >>> + help >>> + During final linking, garbage collect unused sections. This will >>> + reduce the size of the final Xen binary >>> + >>> + Only supported for ELF/Multiboot xen/xen.gz, not EFI xen.efi. >> >> This last sentence is x86-centric, which it shouldn't be here (or it should >> say that this is an x86-only aspect). >> >> I also wonder whether this wouldn't better live next to CC_SPLIT_SECTIONS. > > If I put it immediately after CC_SPLIT_SECTIONS, menuconfig puts it as a > top level option: > > │ │ [*] Garbage Collect Sections > │ │ Architecture Features ---> > │ │ Common Features ---> > > I thought Common Features was a better place for it.
Oh, right. I didn't recall CC_SPLIT_SECTIONS wouldn't even have a prompt. I wonder if it shouldn't gain one, move somewhere here, and then this new option could be placed next to it. > Also, I think it should probably gain " if EXPERT" as well. Not sure there. If this works reliably, I don't see why it would take an expert to enable. Eventually, after all, this may want to be the default. Jan
