[PATCH v2] gunzip: Implement chunked decompression

[Marek Vasut]

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 ?