[coreboot-gerrit] Change in coreboot[master]: vboot: use vboot persistent context

[Joel Kitching (Code Review)]

Joel Kitching has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36300 )

Change subject: vboot: use vboot persistent context
......................................................................

Patch Set 9:

(1 comment)

https://review.coreboot.org/c/coreboot/+/36300/7/src/security/vboot/common.c 
File src/security/vboot/common.c:

https://review.coreboot.org/c/coreboot/+/36300/7/src/security/vboot/common.c... 
PS7, Line 142: 			cbmem_size - wd_preram->buffer_offset,
> Done

Well that's awkward.  It turns out buffer_size is a uint16_t, which is too small to store VB2_KERNEL_WORKBUF_RECOMMENDED_SIZE.  The one place that it's actually used is in coreboot_table.c to give a size for the lb_range struct.  Which probably isn't necessary to fill out, but could be nice for consistency.  So maybe we should change it to uint32_t?