From 1d2e8355516aad01236acf9486dd21b8526ece42 Mon Sep 17 00:00:00 2001 From: Joel Kitching Date: Thu, 13 Feb 2020 19:02:16 +0800 Subject: vboot: remove vb2_packed_key reference from vboot_struct.h Since CL:1651221, the vb2_packed_key "kernel_subkey" field in VbSharedDataHeader is no longer used. Replace the field with a reserved field of the same size, so that we can remove the awkward relative 2struct.h header include. BUG=b:124141368, chromium:1038260 TEST=make clean && make runtests BRANCH=none Change-Id: Ic755ae102cd3928f1dbec8f1e76f723a4411c2a4 Cq-Depend: chrome-internal:2576432 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2054269 Tested-by: Joel Kitching Reviewed-by: Julius Werner Commit-Queue: Joel Kitching --- firmware/include/vboot_struct.h | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/firmware/include/vboot_struct.h b/firmware/include/vboot_struct.h index 246d6c5b..312165fc 100644 --- a/firmware/include/vboot_struct.h +++ b/firmware/include/vboot_struct.h @@ -11,14 +11,6 @@ #include -/* - * Needed for vb2_packed_key. Use relative path to place nicely with - * depthcharge and coreboot. - * TODO(kitching): This include should disappear once everything in - * this file has either been deprecated or has found a better home. - */ -#include "../2lib/include/2struct.h" - #ifdef __cplusplus extern "C" { #endif /* __cplusplus */ @@ -235,8 +227,9 @@ typedef struct VbSharedDataHeader { uint32_t flags; /* Reserved for padding */ uint32_t reserved0; - /* Kernel subkey, from firmware */ - struct vb2_packed_key kernel_subkey; + /* Previously, kernel subkey, from firmware (struct vb2_packed_key). + Now we use vboot2 workbuf for storage. */ + uint8_t reserved1[32]; /* Offset of kernel subkey data from start of this struct */ uint64_t kernel_subkey_data_offset; /* Size of kernel subkey data */ @@ -269,7 +262,7 @@ typedef struct VbSharedDataHeader { /* Firmware index returned by LoadFirmware() or 0xFF if failure */ uint8_t firmware_index; /* Reserved for padding */ - uint8_t reserved1; + uint8_t reserved2; /* Firmware TPM version at start of VbSelectFirmware() */ uint32_t fw_version_tpm_start; /* Firmware lowest version found */ @@ -298,7 +291,7 @@ typedef struct VbSharedDataHeader { /* Recovery reason for current boot */ uint8_t recovery_reason; /* Reserved for padding */ - uint8_t reserved2[7]; + uint8_t reserved3[7]; /* Flags from firmware keyblock */ uint64_t fw_keyblock_flags; /* Kernel TPM version at start of VbSelectAndLoadKernel() */ -- cgit v1.2.1