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

[Patrick Georgi (Code Review)]

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

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

Patch Set 15:

(3 comments)

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

https://review.coreboot.org/c/coreboot/+/36300/2/src/security/vboot/common.c... 
PS2, Line 146: VB2_KERNEL_WORKBUF_RECOMMENDED_SIZE
> Right now it's guaranteed to be 32 byte aligned (via imd_create_tiered_empty() setting up alignment  […]

Ack

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 49: memset(wd, 0, VB2_FIRMWARE_WORKBUF_RECOMMENDED_SIZE);
> coreboot takes care of zeroing the vboot_working_data struct. […]

Ack

https://review.coreboot.org/c/coreboot/+/36300/6/src/security/vboot/misc.h 
File src/security/vboot/misc.h:

https://review.coreboot.org/c/coreboot/+/36300/6/src/security/vboot/misc.h@3... 
PS6, Line 35: vboot_init_context
> can you make a follow-up CL to fix the comment?

See CB:36862