[PATCH v5 00/29] pxe: Precursor series for supporting read_all() in extlinux / PXE

[Jonas Karlman]

Hi Tom and Simon,

On 2025-03-19 00:21, Tom Rini wrote:
> On Wed, 05 Mar 2025 17:24:54 -0700, Simon Glass wrote:
> 
>> This series includes some patches related to allowing read_all() to be
>> used with the extlinux / PXE bootmeths.
>>
>> These patches were split out from the stb4 series, since it will need to
>> have additional patches for LWIP, to avoid breaking PXE booting when
>> LWIP is used.
>>
>> [...]
> 
> Applied to u-boot/next, thanks!

This series broke booting a compressed arm64 defconfig Linux kernel
(without module loading) due to changes in decompression buffer length.

My arm64 defconfig kernel (Image.gz) is ~24 MiB compressed and ~85 MiB
uncompressed.

Before this series the decompression buffer was 10x the kernel_comp_size
and now it is instead limited by the SYS_BOOTM_LEN Kconfig symbol.

A broken boot using current next branch:

  Retrieving file: /Image.gz
  Retrieving file: /initramfs.cpio.gz
  append: earlycon
  cmd 'booti' states 1f1f addr_img '0x02080000' conf_ramdisk '0x06000000:394d3c' conf_fdt '3df16350' images 000000003ffe53a0
     kernel data at 0x02080000, len = 0x00000000 (0)
  load 2080000 start 2080000 len 0 ep 0 os 5 comp 1
  find_other type 2 os 5
  ## Flattened Device Tree blob at 3df16350
     Booting using the fdt blob at 0x3df16350
  Working FDT set to 3df16350
  load_os load 8000000 image_start 2080000 image_len 2000000
     Uncompressing Kernel Image to 8000000
  Error: inflate() returned -5
  Image too large: increase CONFIG_SYS_BOOTM_LEN
  Must RESET board to recover
  Resetting the board...

Changing SYS_BOOTM_LEN from default 64 MiB to 128 MiB fixed the boot:

  Retrieving file: /Image.gz
  Retrieving file: /initramfs.cpio.gz
  append: earlycon
  cmd 'booti' states 1f1f addr_img '0x02080000' conf_ramdisk '0x06000000:394d3c' conf_fdt '3df16430' images 000000003ffe5480
     kernel data at 0x02080000, len = 0x00000000 (0)
  load 2080000 start 2080000 len 0 ep 0 os 5 comp 1
  find_other type 2 os 5
  ## Flattened Device Tree blob at 3df16430
     Booting using the fdt blob at 0x3df16430
  Working FDT set to 3df16430
  load_os load 8000000 image_start 2080000 image_len 2000000
     Uncompressing Kernel Image to 8000000
     kernel loaded at 0x08000000, end = 0x0d3b5a00
     Loading Ramdisk to 3cb51000, end 3cee5d3c ... OK
     Loading Device Tree to 000000003cb3f000, end 000000003cb509df ... OK
  Working FDT set to 3cb3f000

  Starting kernel ...

Do we need to increase the default SYS_BOOTM_LEN for ARM64 now?

Regards,
Jonas

> 
> [01/29] x86: Make do_zboot_states() static
>         commit: 299d24eddfeb18a7576187b3010d5462317335e9
> [02/29] x86: Rename zboot_run() to zboot_run_args()
>         commit: 95641f4bf98d3c90bfc4ae94515c98c440ffb2e1
> [03/29] x86: Drop duplicate definition of zimage_dump()
>         commit: 2de073527bb92b47d49366249cd3fdea5016bcaf
> [04/29] x86: Move x86 zboot state into struct bootm_info
>         commit: 75e85df7963f57e4bb80b3d805ba2295b1843911
> [05/29] x86: Rename state to bmi
>         commit: 00cfb598e740d8bad79097e7e069ad71d86bbd5a
> [06/29] x86: Move the bootm state for zimage into cmd/
>         commit: 4e36b1739b03e81ff395959b58fe33e67c4d2233
> [07/29] bootstd: Correct display of kernel version
>         commit: 1592ff27d502efdc79992f57c07ff0cf81bc9305
> [08/29] x86: Drop the unnecessary base_ptr argument to zboot_dump()
>         commit: c73da92304280b229e3d8dfd565fae5a24fe3ce8
> [09/29] boot: Use strlcpy() in label_boot()
>         commit: 0fd3ed1cd73d6b262c204c27cbe6dc1ad1c09e91
> [10/29] boot: Split pxe label_boot() into two parts
>         commit: ff9fef41fe4d329a5c76746cc45f9fbf8fdb9b12
> [11/29] boot: Split pxe label_run_boot() into two parts
>         commit: f6aa262f55b78e3a21dd020e47601320c2edec4e
> [12/29] boot: Pass just the FDT argument to label_process_fdt()
>         commit: 600bc21da54a300f2825a5e3088719b1ffd5ee53
> [13/29] bootm: Allow building bootm.c without CONFIG_SYS_BOOTM_LEN
>         commit: 7f10a7fe126dce5644d933af693eda40497d7755
> [14/29] boot: Convert IMAGE_FORMAT into an enum
>         commit: 3c7b13b075488ebcff2923b0a7b46cc11f39285e
> [15/29] boot: arm: riscv: sandbox: Add a format for the booti file
>         commit: 098407e67390ed0c369029bab0777a51e5a7bad2
> [16/29] boot: Support booti format in bootm
>         commit: d6bb0ea535e4384ed1975ee9c755488f5036a79e
> [17/29] boot: Support compressed booti images in bootm
>         commit: ecd50bb4643c9052e5f8b6171ab6c3905ed0ca70
> [18/29] boot: pxe: Use bootm_...() functions where possible
>         commit: b13408021d361fbae1c254307f139ad1e6def3d2
> [19/29] pxe_utils: Simplify default fdt in label_run_boot()
>         commit: feb8d7fd749413e889ba2c396654f473ba32034d
> [20/29] boot: pxe: Refactor label_run_boot() to avoid cmdline
>         commit: e2e87b840162ddf4ec8df3f235be98a74a964509
> [21/29] net: Keep the bootstage functions together
>         commit: 2c04afbc957d18e61813315a0694db903754a280
> [22/29] net: Tidy up the comments to parse_args()
>         commit: 3ccbc10cd9e3b28d67fe5008943a648b870db1e6
> [23/29] net: Simplify parse_args()
>         commit: f1ece5d86e5577ae9bb3325d65ed7003f3cbaf9e
> [24/29] net: Return the load address from parse_args()
>         commit: 4b6070e056fc2c11b7621c32151f95f7a045ac01
> [25/29] net: Return the address and size from parse_addr_size()
>         commit: f60421204816bd0aa40e9e43c18d4bc5861d3521
> [26/29] net: Return the size from parse_args()
>         commit: 25d51d3c798f6e256daa628bb2a9fcc36fde0382
> [27/29] net: Refactor part of netboot_common() into a function
>         commit: bfffdfaaf6f492865100197a5b141482799afc09
> [28/29] net: Drop #ifdef in parse_args()
>         commit: f278f0cb4996398720328ce33e057678f6ea4109
> [29/29] net: Provide a function to run network operations
>         commit: 0f094b8b146679c3980cd2febde4e902bbc4405d