diff options
author | Nicolas Boichat <drinkcat@google.com> | 2017-03-08 17:49:25 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-04-06 03:29:38 -0700 |
commit | ca0e82685989a08c6adaa52fc8d33546cd3bb1a2 (patch) | |
tree | 14a2af14564998a5b26e3ae2de1666507b28a8a7 /include/version.h | |
parent | 72306c7e020818e31d0fb351b20649b4f8cd732d (diff) | |
download | chrome-ec-ca0e82685989a08c6adaa52fc8d33546cd3bb1a2.tar.gz |
common/rollback: Add support for rollback protection
Implement actual rollback protection. First, we add a new field
in the version structure, which is an incrementing integer
(we'll start by shipping images with version 0, and gradually
increase the number as required). This allows us to release
new versions of the EC without necessarily bumping the rollback
protection.
For the rollback protection block itself, it contains 2 sub-blocks
of equal size (normally, 2k), that are individually erasable.
The rollback code looks at both, and takes the most restrictive one
to determine the desired rollback minimum version. The blocks
are also allowed to be erased (full of 1's), in which case the
rollback minimum version is assumed to be 0.
We also add an FMAP entry, in case we later decide to allow the
signer to increment the rollback version.
Also note that, like any version_data struct change, this change
breaks compatibility between old and new RO/RW.
Follow-up code will take care of auto-updating the rollback block
as required, and properly manage block protection.
BRANCH=none
BUG=b:35586219
TEST=Flash hammer
rollbackinfo => 1 version 0 block, 1 empty block, RW verifies
correctly.
rollbackupdate 0; rollbackinfo => No change
rollbackupdate 1; reboot => RO refuses to jump to RW
rollbackupdate 2, 3, 4; rollbackinfo => Writes alternate
between the 2 blocks.
rollbackupdate 2 => Refuses to downgrade version
Change-Id: Ia969afb481a93deb912b9153bdd95ace01ad8fa7
Reviewed-on: https://chromium-review.googlesource.com/452815
Commit-Ready: Nicolas Boichat <drinkcat@chromium.org>
Tested-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'include/version.h')
-rw-r--r-- | include/version.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/version.h b/include/version.h index 94e8dd9ed5..14aa61a0e2 100644 --- a/include/version.h +++ b/include/version.h @@ -11,12 +11,13 @@ #include "common.h" #define CROS_EC_IMAGE_DATA_COOKIE1 0xce778899 -#define CROS_EC_IMAGE_DATA_COOKIE2 0xceaabbcc +#define CROS_EC_IMAGE_DATA_COOKIE2 0xceaabbdd struct image_data { uint32_t cookie1; char version[32]; uint32_t size; + int32_t rollback_version; uint32_t cookie2; } __packed; |