<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/vboot.git, branch stabilize-7077.134.B</title>
<subtitle>review.coreboot.org: vboot.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/'/>
<entry>
<title>GBB: Add missing flag LID_SHUTDOWN to vb2_gbb_flag structure</title>
<updated>2015-05-16T04:17:26+00:00</updated>
<author>
<name>Furquan Shaikh</name>
<email>furquan@google.com</email>
</author>
<published>2015-05-16T01:15:18+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=ea71df260e9041dd5260425dae3d6f5412516b9b'/>
<id>ea71df260e9041dd5260425dae3d6f5412516b9b</id>
<content type='text'>
BUG=None
BRANCH=None
TEST=Compiles successfully

Change-Id: I80a501efc3940ca5657dc143c0ab3c6b020dc1e0
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271620
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
Reviewed-by: Aaron Durbin &lt;adurbin@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
BUG=None
BRANCH=None
TEST=Compiles successfully

Change-Id: I80a501efc3940ca5657dc143c0ab3c6b020dc1e0
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271620
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
Reviewed-by: Aaron Durbin &lt;adurbin@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>GBB: Add flag for forcing full fastboot capability in firmware</title>
<updated>2015-05-16T04:17:21+00:00</updated>
<author>
<name>Furquan Shaikh</name>
<email>furquan@google.com</email>
</author>
<published>2015-05-15T07:44:41+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=138bb3c004154f67f3b37aafef3a6efa8bf0b338'/>
<id>138bb3c004154f67f3b37aafef3a6efa8bf0b338</id>
<content type='text'>
This flag is equivalent to FORCE_DEV_BOOT_USB. It allows full fastboot
capability in firmware for developer mode.

BUG=chrome-os-partner:40196
BRANCH=None
TEST=Compiles successfully for smaug.

Change-Id: I82a2ebe7a8b3bbf38694ab81ca2678624f77fca1
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271410
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This flag is equivalent to FORCE_DEV_BOOT_USB. It allows full fastboot
capability in firmware for developer mode.

BUG=chrome-os-partner:40196
BRANCH=None
TEST=Compiles successfully for smaug.

Change-Id: I82a2ebe7a8b3bbf38694ab81ca2678624f77fca1
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271410
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>nvstorage: Add new flag VBNV_DEV_BOOT_FASTBOOT_FULL_CAP</title>
<updated>2015-05-16T04:17:16+00:00</updated>
<author>
<name>Furquan Shaikh</name>
<email>furquan@google.com</email>
</author>
<published>2015-05-14T19:53:19+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=9101df2fe3ed4f0e8b65121ffc1c7e8b2d65b35d'/>
<id>9101df2fe3ed4f0e8b65121ffc1c7e8b2d65b35d</id>
<content type='text'>
Add a new flag to nvstorage for controlling fastboot capabilities
offered in firmware in dev-mode. By default, value of this flag would
be ignored in normal mode. Thus, when fastboot-based recovery is
entered from normal mode, only limited capability would be available
in firmware.

After switching to dev-mode, this flag can be set automatically by
user script after performing the wipe or it can be set manually using
crossystem. When fastboot-based recovery is entered from dev mode and
this flag is set, it will provide full fastboot capability in the
firmware.

BUG=chrome-os-partner:40196
BRANCH=None
TEST=Compiles successfully for smaug. make runalltests successful.

Change-Id: I761a9ab304dd90f0b73081acc9ce1f8d9052325f
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271369
Reviewed-by: Aaron Durbin &lt;adurbin@chromium.org&gt;
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a new flag to nvstorage for controlling fastboot capabilities
offered in firmware in dev-mode. By default, value of this flag would
be ignored in normal mode. Thus, when fastboot-based recovery is
entered from normal mode, only limited capability would be available
in firmware.

After switching to dev-mode, this flag can be set automatically by
user script after performing the wipe or it can be set manually using
crossystem. When fastboot-based recovery is entered from dev mode and
this flag is set, it will provide full fastboot capability in the
firmware.

BUG=chrome-os-partner:40196
BRANCH=None
TEST=Compiles successfully for smaug. make runalltests successful.

Change-Id: I761a9ab304dd90f0b73081acc9ce1f8d9052325f
Signed-off-by: Furquan Shaikh &lt;furquan@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/271369
Reviewed-by: Aaron Durbin &lt;adurbin@chromium.org&gt;
Reviewed-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Commit-Queue: Furquan Shaikh &lt;furquan@chromium.org&gt;
Trybot-Ready: Furquan Shaikh &lt;furquan@chromium.org&gt;
Tested-by: Furquan Shaikh &lt;furquan@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>chromeos-tpm-recovery: Convert to manual TPM reset script for developers</title>
<updated>2015-05-16T01:42:28+00:00</updated>
<author>
<name>Julius Werner</name>
<email>jwerner@chromium.org</email>
</author>
<published>2015-01-06T18:02:46+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=07e56f22b852f4d96af2b2df57da11f6938778dd'/>
<id>07e56f22b852f4d96af2b2df57da11f6938778dd</id>
<content type='text'>
chromeos-tpm-recovery has not been used for anything in forever (see
CL:238236), but it is still installed on every image. Resetting the TPM
(e.g. to resolve rollback issues when reflashing an MP-signed device to
dev firmware) is a common request by developers, and I get tired of
always digging out the required tpmc commands manually again. Let's
repurpose this script as a simple one-shot tool for developers to reset
their TPM, so the next time someone asks we can just tell them 'boot a
test image in recovery mode and run chromeos-tpm-recovery'.

BRANCH=none
BUG=chromium:419942
TEST=Ran on a Jerry, confirmed that TPM spaces were reset.

Change-Id: Ia95246cfed3dc9b0c6fdb0481218e3ae14d8318a
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/271512
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-by: Luigi Semenzato &lt;semenzato@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
chromeos-tpm-recovery has not been used for anything in forever (see
CL:238236), but it is still installed on every image. Resetting the TPM
(e.g. to resolve rollback issues when reflashing an MP-signed device to
dev firmware) is a common request by developers, and I get tired of
always digging out the required tpmc commands manually again. Let's
repurpose this script as a simple one-shot tool for developers to reset
their TPM, so the next time someone asks we can just tell them 'boot a
test image in recovery mode and run chromeos-tpm-recovery'.

BRANCH=none
BUG=chromium:419942
TEST=Ran on a Jerry, confirmed that TPM spaces were reset.

Change-Id: Ia95246cfed3dc9b0c6fdb0481218e3ae14d8318a
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/271512
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-by: Luigi Semenzato &lt;semenzato@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot2: Support VB2_GBB_FLAG_DISABLE_FW_ROLLBACK_CHECK</title>
<updated>2015-05-16T01:42:20+00:00</updated>
<author>
<name>Julius Werner</name>
<email>jwerner@chromium.org</email>
</author>
<published>2015-05-15T19:50:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=fb4e4080112d9005f83b57551ab19cbf478da36a'/>
<id>fb4e4080112d9005f83b57551ab19cbf478da36a</id>
<content type='text'>
Looks like the DISABLE_FW_ROLLBACK_CHECK GBB flag (0x200) was forgotten
in the vboot2 implementation. It's too late for Veyron now, but let's at
least fix it for future devices.

BRANCH=none
BUG=None
TEST=make runtests

Change-Id: I867f7aada28be3897efda73a6bdc3b0848c23dca
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/271419
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Looks like the DISABLE_FW_ROLLBACK_CHECK GBB flag (0x200) was forgotten
in the vboot2 implementation. It's too late for Veyron now, but let's at
least fix it for future devices.

BRANCH=none
BUG=None
TEST=make runtests

Change-Id: I867f7aada28be3897efda73a6bdc3b0848c23dca
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/271419
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Detect GBB 1.1 also as impcompatible version</title>
<updated>2015-05-14T02:25:57+00:00</updated>
<author>
<name>Daisuke Nojiri</name>
<email>dnojiri@chromium.org</email>
</author>
<published>2015-05-13T22:32:14+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=dc49a6827670abc0f2dc45178c2623e070ff5961'/>
<id>dc49a6827670abc0f2dc45178c2623e070ff5961</id>
<content type='text'>
Older GBB headers (e.g. 1.0 and 1.1) do not have hwid_digest. In such cases,
PCR1 is currently extended from 0, causing a remote attestation failure.
This change makes all GBB headers older than 1.2 incompatible.

BUG=none
BRANCH=tot
TEST=make -j runtests

Change-Id: I7a3b19c2da325a3fa4b9c1fe06ed6f43cb51fb9e
Signed-off-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270796
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Older GBB headers (e.g. 1.0 and 1.1) do not have hwid_digest. In such cases,
PCR1 is currently extended from 0, causing a remote attestation failure.
This change makes all GBB headers older than 1.2 incompatible.

BUG=none
BRANCH=tot
TEST=make -j runtests

Change-Id: I7a3b19c2da325a3fa4b9c1fe06ed6f43cb51fb9e
Signed-off-by: Daisuke Nojiri &lt;dnojiri@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270796
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot2: Add support for kernel version secure data space</title>
<updated>2015-05-13T22:23:42+00:00</updated>
<author>
<name>Randall Spangler</name>
<email>rspangler@chromium.org</email>
</author>
<published>2015-05-12T20:44:30+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=bf9c2760d2657e29e481150fb3fab8809c3ab755'/>
<id>bf9c2760d2657e29e481150fb3fab8809c3ab755</id>
<content type='text'>
Holds kernel rollback information.  Will be used by vboot 2.0 kernel
verification.

BUG=chromium:487699
BRANCH=none
TEST=make -j runtests

Change-Id: Ib4a70e943ebd79aac06404df09cf4ce62d719201
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270626
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Holds kernel rollback information.  Will be used by vboot 2.0 kernel
verification.

BUG=chromium:487699
BRANCH=none
TEST=make -j runtests

Change-Id: Ib4a70e943ebd79aac06404df09cf4ce62d719201
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270626
Reviewed-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot1: Condition default legacy boot on dev_boot_legacy</title>
<updated>2015-05-12T01:17:23+00:00</updated>
<author>
<name>Julius Werner</name>
<email>jwerner@chromium.org</email>
</author>
<published>2015-05-09T06:00:16+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=0140cd2906b1107ef9ea60ed7a81f8cbdc9162f8'/>
<id>0140cd2906b1107ef9ea60ed7a81f8cbdc9162f8</id>
<content type='text'>
This patch fixes what I think is an inconsistency in the existing legacy
boot behavior: when the GBB flag that defaults to legacy boot is set,
running out the 30 second timer would still boot legacy mode even if
dev_boot_legacy is not actually set (whereas pressing CTRL+L in the
same configuration would beep and refuse).

This patch makes both legacy boot trgiggers check the same condition
before boot. This does not restrict functionality since anyone who sets
the DEFAULT_DEV_BOOT_LEGACY GBB flag could simply set
FORCE_DEV_BOOT_LEGACY at the same time. It does however open up an
interesting new use case of using NVRAM to change back-and-forth between
legacy and normal developer mode (after GBB flags are changed once and
write-protection is enabled again).

If this is updated in the field it might lock existing devices out of
legacy mode... however, since by far the most common GBB flag
combination recommended on the internet seems to be 0x489 (including
FORCE_DEV_BOOT_LEGACY), I doubt this would be a problem in practice.

BRANCH=tbd
BUG=chrome-os-partner:39999
TEST=Booted with GBB flags 0x4b9 and 0x439, observed difference.

Change-Id: If6a6d99ab2cf116db2237fdc3df97fc22a68251c
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270182
Reviewed-by: Duncan Laurie &lt;dlaurie@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch fixes what I think is an inconsistency in the existing legacy
boot behavior: when the GBB flag that defaults to legacy boot is set,
running out the 30 second timer would still boot legacy mode even if
dev_boot_legacy is not actually set (whereas pressing CTRL+L in the
same configuration would beep and refuse).

This patch makes both legacy boot trgiggers check the same condition
before boot. This does not restrict functionality since anyone who sets
the DEFAULT_DEV_BOOT_LEGACY GBB flag could simply set
FORCE_DEV_BOOT_LEGACY at the same time. It does however open up an
interesting new use case of using NVRAM to change back-and-forth between
legacy and normal developer mode (after GBB flags are changed once and
write-protection is enabled again).

If this is updated in the field it might lock existing devices out of
legacy mode... however, since by far the most common GBB flag
combination recommended on the internet seems to be 0x489 (including
FORCE_DEV_BOOT_LEGACY), I doubt this would be a problem in practice.

BRANCH=tbd
BUG=chrome-os-partner:39999
TEST=Booted with GBB flags 0x4b9 and 0x439, observed difference.

Change-Id: If6a6d99ab2cf116db2237fdc3df97fc22a68251c
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270182
Reviewed-by: Duncan Laurie &lt;dlaurie@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot1: Lock TPM physical presence (kernel rollback) on legacy boot</title>
<updated>2015-05-12T01:17:16+00:00</updated>
<author>
<name>Julius Werner</name>
<email>jwerner@chromium.org</email>
</author>
<published>2015-05-09T05:54:14+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=957b424c52d1cb0075f2a464e6845fcb59da41df'/>
<id>957b424c52d1cb0075f2a464e6845fcb59da41df</id>
<content type='text'>
Even though legacy boot is an unsafe mode that has to be manually
initiated by the user, we should still lock the kernel TPM space to be
consistent with existing developer mode practice.

BRANCH=tbd
BUG=chrome-os-partner:39999
TEST=Spent over an hour unsuccessfully trying to get SeaBIOS to boot a
Chromium test image on my Falco. Decided that's not worth it an just
tested the firmware side of this (pressing CTRL+L when legacy mode is
enabled and disabled, multiple times, with and without GBB flag
DEFAULT_DEV_BOOT_LEGACY).

Change-Id: I3b02b59a9055431d222c0c7446de2cd7d2e0bb82
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270181
Reviewed-by: Duncan Laurie &lt;dlaurie@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Even though legacy boot is an unsafe mode that has to be manually
initiated by the user, we should still lock the kernel TPM space to be
consistent with existing developer mode practice.

BRANCH=tbd
BUG=chrome-os-partner:39999
TEST=Spent over an hour unsuccessfully trying to get SeaBIOS to boot a
Chromium test image on my Falco. Decided that's not worth it an just
tested the firmware side of this (pressing CTRL+L when legacy mode is
enabled and disabled, multiple times, with and without GBB flag
DEFAULT_DEV_BOOT_LEGACY).

Change-Id: I3b02b59a9055431d222c0c7446de2cd7d2e0bb82
Signed-off-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/270181
Reviewed-by: Duncan Laurie &lt;dlaurie@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Make SHA library accessible to calling firmware</title>
<updated>2015-05-07T00:00:36+00:00</updated>
<author>
<name>Randall Spangler</name>
<email>rspangler@chromium.org</email>
</author>
<published>2015-05-05T23:06:50+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=f81fce91bf885293f1447c1197bb0c4143d8fced'/>
<id>f81fce91bf885293f1447c1197bb0c4143d8fced</id>
<content type='text'>
And add a vb2_digest_buffer() call which produces the hash of a buffer
all in a single function call.  That function actually already
existed, but was in a unit test file rather than in the library
itself.  It's a small function, so adding it won't increase the size
of the library significantly - or at all, on platforms which compile
with -ffunction-sections.

This allows coreboot to reuse this SHA library for hashing CBFS
entries and file data.  All it has to do is #define
NEED_VB2_SHA_LIBRARY and then #include "vb2_api.h".

BUG=chromium:482652
BRANCH=none
TEST=make -j runtests

Change-Id: Ice2d0929324b58b2665f3989b5b887225f6ef61e
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/269523
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
And add a vb2_digest_buffer() call which produces the hash of a buffer
all in a single function call.  That function actually already
existed, but was in a unit test file rather than in the library
itself.  It's a small function, so adding it won't increase the size
of the library significantly - or at all, on platforms which compile
with -ffunction-sections.

This allows coreboot to reuse this SHA library for hashing CBFS
entries and file data.  All it has to do is #define
NEED_VB2_SHA_LIBRARY and then #include "vb2_api.h".

BUG=chromium:482652
BRANCH=none
TEST=make -j runtests

Change-Id: Ice2d0929324b58b2665f3989b5b887225f6ef61e
Signed-off-by: Randall Spangler &lt;rspangler@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/269523
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
</feed>
