Re: [PATCH v3 11/14] bloblist: Adjust the bloblist header

On Wed, 27 Dec 2023 at 16:50, Raymond Mao raymond.mao@linaro.org wrote:

Hi Ilias,

On Wed, 27 Dec 2023 at 05:17, Ilias Apalodimas ilias.apalodimas@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