On Wed, 27 Dec 2023 at 16:50, Raymond Mao <raymond....@linaro.org> wrote:
>
> Hi Ilias,
>
> On Wed, 27 Dec 2023 at 05:17, Ilias Apalodimas <ilias.apalodi...@linaro.org>
> wrote:
>>
>>
>> [...]
>>
>> > - * @chksum: checksum for the entire bloblist allocated area. Since any of
>> > the
>> > - * blobs can be altered after being created, this checksum is only valid
>> > - * when the bloblist is finalised before jumping to the next stage of
>> > boot.
>> > - * This is the value needed to make all checksummed bytes sum to 0
>> > */
>> > struct bloblist_hdr {
>> > u32 magic;
>> > - u32 version;
>> > - u32 hdr_size;
>> > - u32 flags;
>> > -
>> > - u32 size;
>> > + u8 chksum;
>> > + u8 version;
>> > + u8 hdr_size;
>> > + u8 align_log2;
>> > u32 alloced;
>> > + u32 size;
>> > + u32 flags;
>> > u32 spare;
>> > - u32 chksum;
>> > };
>> >
>>
>> Aren't fields still missing from the current version?
>> e.g max_size and reserved?
>>
> They are all in. Please see:
> [PATCH v3 14/14] bloblist: Align bloblist used_size and total_size to spec
Ok. But why aren't we squashing this to #14 then?
Thanks
/Ilias
>
> [...]
>
> Regards,
> Raymond
