On Thu, Oct 09, 2025 at 10:29:17PM -0700, Tony Dinh wrote: > Make it possible to disable CMD_PXE. > Remove unnecessary PXE_UTILS selection in BOOTMETH_EXTLINUX config. > In extlinux_boot(), invoke pxe utils only when > CONFIG_BOOTMETH_EXTLINUX_PXE is enabled. > > This patch results in about 9K reduction in image size when > PXE boot is disabled. > > Signed-off-by: Tony Dinh <[email protected]> > --- > > boot/Kconfig | 3 +-- > boot/bootmeth_extlinux.c | 18 ++++++++++-------- > 2 files changed, 11 insertions(+), 10 deletions(-)
Is some part of the symbol logic here wrong? A challenge is that "PXE" is also where the logic to parse extlinux.conf style files came from, and I thought we had split those two out. And then there's this: > > diff --git a/boot/Kconfig b/boot/Kconfig > index 2993cd7f9ba..403ce4c3d46 100644 > --- a/boot/Kconfig > +++ b/boot/Kconfig > @@ -421,10 +421,10 @@ config BOOT_DEFAULTS_CMDS > select CMD_PART if PARTITIONS > select CMD_DHCP if CMD_NET > select CMD_PING if CMD_NET > - select CMD_PXE if CMD_NET > select CMD_BOOTI if ARM64 > select CMD_BOOTZ if ARM && !ARM64 > imply CMD_MII if NET > + imply CMD_PXE if CMD_NET This is one of the things where defaults isn't supposed to be so easy to get out of. How many platforms are you wanting to then disable CMD_PXE on to save space? -- Tom
signature.asc
Description: PGP signature
