On 2/19/26 2:09 PM, Simon Glass wrote: Hello Simon,
+ /* + * Allow runtime configuration of decompression chunk on + * sandbox to better cover the chunked decompression + * functionality without having to use > 4 GiB files. + */ + const ulong minchunk = 0x400; + const ulong maxchunk = SZ_4G - minchunk; + const ulong chunk = + CONFIG_IS_ENABLED(SANDBOX, + (clamp(env_get_ulong("gzwrite_chunk", 10, maxchunk), + minchunk, maxchunk)), + (maxchunk));As you mentioned in the last version, there is a compression test in compression.c - could you add a version of this function that takes maxchunk as an argument and call it from that test? Reading an environment variable to avoid passing a parameter seems pretty odd to me :-)
Do we not want to test the command line interface ?
