Joel Kitching has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36300 )
Change subject: vboot: use vboot persistent context ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36300/1/src/security/vboot/common.c File src/security/vboot/common.c:
https://review.coreboot.org/c/coreboot/+/36300/1/src/security/vboot/common.c... PS1, Line 131: /* TODO: We don't know the workbuf size with the current API... */
We should probably just use KERNEL_WORKBUF_RECOMMENDED_SIZE here right away, then depthcharge can ju […]
@Julius, yes, this is actually the current plan as listed in the design document. It seems extremely awkward to provide a vb2api_relocate function which can squash the workbuf to be smaller, when you don't actually know how small it can get. But I suppose if we don't have that use case at the moment, there is no need to expose workbuf_used?