<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/vboot.git, branch stabilize-14469.58.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>Add vboot_reference-sys</title>
<updated>2022-01-19T23:35:40+00:00</updated>
<author>
<name>Allen Webb</name>
<email>allenwebb@google.com</email>
</author>
<published>2022-01-14T22:53:54+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=283db281e27e89adabf40fbd8dabd4f489f083f4'/>
<id>283db281e27e89adabf40fbd8dabd4f489f083f4</id>
<content type='text'>
This adds Rust bindings to vboot_reference.

BRANCH=none
BUG=b:214081328
TEST=cargo build

Change-Id: I4f9df36a0de93c0617ead2a75ef2ca5fcf8f5652
Signed-off-by: Allen Webb &lt;allenwebb@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3390514
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This adds Rust bindings to vboot_reference.

BRANCH=none
BUG=b:214081328
TEST=cargo build

Change-Id: I4f9df36a0de93c0617ead2a75ef2ca5fcf8f5652
Signed-off-by: Allen Webb &lt;allenwebb@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3390514
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot_reference/cgpt_nor: Make ForkExecL() static local</title>
<updated>2022-01-13T20:53:42+00:00</updated>
<author>
<name>Edward O'Callaghan</name>
<email>quasisec@google.com</email>
</author>
<published>2021-12-22T05:44:36+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=f445b236579e928354b866f7779ffb630d15311c'/>
<id>f445b236579e928354b866f7779ffb630d15311c</id>
<content type='text'>
BUG=none
TEST=builds

Signed-off-by: Edward O'Callaghan &lt;quasisec@google.com&gt;
Change-Id: Ied809d3c2839f33d738d1938cafd6c634d54ee28
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3353558
Commit-Queue: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Tested-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Auto-Submit: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Reviewed-by: Nikolai Artemiev &lt;nartemiev@google.com&gt;
Commit-Queue: Nikolai Artemiev &lt;nartemiev@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
BUG=none
TEST=builds

Signed-off-by: Edward O'Callaghan &lt;quasisec@google.com&gt;
Change-Id: Ied809d3c2839f33d738d1938cafd6c634d54ee28
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3353558
Commit-Queue: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Tested-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Auto-Submit: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Reviewed-by: Nikolai Artemiev &lt;nartemiev@google.com&gt;
Commit-Queue: Nikolai Artemiev &lt;nartemiev@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>futility: Use crosid to probe firmware manifest key</title>
<updated>2022-01-11T23:41:23+00:00</updated>
<author>
<name>Jack Rosenthal</name>
<email>jrosenth@chromium.org</email>
</author>
<published>2022-01-06T00:40:29+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=2407b678bd2e41422aef6bb79246dbeaf6dfed7f'/>
<id>2407b678bd2e41422aef6bb79246dbeaf6dfed7f</id>
<content type='text'>
Use the API function crosid_get_firmware_manifest_key from libcrosid
instead of "mosys platform model" to get the firmware manifest key.

Note: this leaves inconsistent terminology around "model" and
"firmware manifest key", which is quite intentional at the moment,
since the model is still the firmware manifest key for the time
being.  Should we decide to change that in the future, we can do
further renaming in futility.

BUG=b:213251232
BRANCH=none
TEST=chromeos-firmwareupdate on delbin
     run recovery on delbin

Cq-Depend: chromium:3364974
Signed-off-by: Jack Rosenthal &lt;jrosenth@chromium.org&gt;
Change-Id: I306ae444849e327ba208f52e34b584b571ef6a3c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3368704
Reviewed-by: Raul Rangel &lt;rrangel@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Use the API function crosid_get_firmware_manifest_key from libcrosid
instead of "mosys platform model" to get the firmware manifest key.

Note: this leaves inconsistent terminology around "model" and
"firmware manifest key", which is quite intentional at the moment,
since the model is still the firmware manifest key for the time
being.  Should we decide to change that in the future, we can do
further renaming in futility.

BUG=b:213251232
BRANCH=none
TEST=chromeos-firmwareupdate on delbin
     run recovery on delbin

Cq-Depend: chromium:3364974
Signed-off-by: Jack Rosenthal &lt;jrosenth@chromium.org&gt;
Change-Id: I306ae444849e327ba208f52e34b584b571ef6a3c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3368704
Reviewed-by: Raul Rangel &lt;rrangel@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>futility: Link with libcrosid</title>
<updated>2022-01-10T23:19:15+00:00</updated>
<author>
<name>Jack Rosenthal</name>
<email>jrosenth@chromium.org</email>
</author>
<published>2022-01-06T00:27:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=22076b1036354be65ac694e3ed0657b6f344f9d1'/>
<id>22076b1036354be65ac694e3ed0657b6f344f9d1</id>
<content type='text'>
Use pkg-config to link with libcrosid when it's available.

Note that when futility is compiled from the coreboot tree, libcrosid
won't be available, at least at the moment, so expose -DUSE_CROSID to
the C code so we know when we have it available.

BUG=b:213251232
BRANCH=none
TEST=emerge-volteer -j vboot_reference
     "lddtree /build/volteer/usr/bin/futility" lists libcrosid

Cq-Depend: chromium:3367251
Signed-off-by: Jack Rosenthal &lt;jrosenth@chromium.org&gt;
Change-Id: I8e69c5ce4570ac9df72440d0a34b6b03b5a36e77
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3368703
Reviewed-by: Raul Rangel &lt;rrangel@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Use pkg-config to link with libcrosid when it's available.

Note that when futility is compiled from the coreboot tree, libcrosid
won't be available, at least at the moment, so expose -DUSE_CROSID to
the C code so we know when we have it available.

BUG=b:213251232
BRANCH=none
TEST=emerge-volteer -j vboot_reference
     "lddtree /build/volteer/usr/bin/futility" lists libcrosid

Cq-Depend: chromium:3367251
Signed-off-by: Jack Rosenthal &lt;jrosenth@chromium.org&gt;
Change-Id: I8e69c5ce4570ac9df72440d0a34b6b03b5a36e77
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3368703
Reviewed-by: Raul Rangel &lt;rrangel@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>crossystem: Remove the stale TPM simulator support</title>
<updated>2022-01-10T05:44:22+00:00</updated>
<author>
<name>Yi Chou</name>
<email>yich@google.com</email>
</author>
<published>2021-12-30T06:11:00+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=c91e3152722eaa827ea5b86aaf8da8558f74c29b'/>
<id>c91e3152722eaa827ea5b86aaf8da8558f74c29b</id>
<content type='text'>
We don't need the MOUNT_ENCRYPTED_KEY_PATH after we replaced the stale
TPM simulator with the driver-level TPM simulator.
It's time to remove it.

BUG=b:203195852, b:174807059
BRANCH=none
TEST=crossystem clear_tpm_owner_request=1

Signed-off-by: Yi Chou &lt;yich@google.com&gt;
Change-Id: I7c1c9c16697b1bbda9adea13448fdb9cffd9cc7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358669
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-by: Leo Lai &lt;cylai@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We don't need the MOUNT_ENCRYPTED_KEY_PATH after we replaced the stale
TPM simulator with the driver-level TPM simulator.
It's time to remove it.

BUG=b:203195852, b:174807059
BRANCH=none
TEST=crossystem clear_tpm_owner_request=1

Signed-off-by: Yi Chou &lt;yich@google.com&gt;
Change-Id: I7c1c9c16697b1bbda9adea13448fdb9cffd9cc7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358669
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-by: Leo Lai &lt;cylai@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>crossystem: Add flock to prevent race conditions</title>
<updated>2022-01-10T05:44:21+00:00</updated>
<author>
<name>Yi Chou</name>
<email>yich@google.com</email>
</author>
<published>2021-12-30T06:57:47+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=58efaacd8ad2a4020fbb0dd2dceae2fc8897f6db'/>
<id>58efaacd8ad2a4020fbb0dd2dceae2fc8897f6db</id>
<content type='text'>
The crossystem command would have potential race conditions when we
calling this command in parallel. Add a flock should prevent this
issue.

BUG=b:172876417
BRANCH=none
TEST=Firing multiple crossystem commands, and check result

Signed-off-by: Yi Chou &lt;yich@google.com&gt;
Change-Id: Ia9f8a73d5fd762c79088f350ea05d0dd540f3fc3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358668
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-by: Leo Lai &lt;cylai@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The crossystem command would have potential race conditions when we
calling this command in parallel. Add a flock should prevent this
issue.

BUG=b:172876417
BRANCH=none
TEST=Firing multiple crossystem commands, and check result

Signed-off-by: Yi Chou &lt;yich@google.com&gt;
Change-Id: Ia9f8a73d5fd762c79088f350ea05d0dd540f3fc3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358668
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
Reviewed-by: Leo Lai &lt;cylai@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot_reference/futility: Pass through SERVOD_NAME.</title>
<updated>2022-01-07T04:47:47+00:00</updated>
<author>
<name>Sam McNally</name>
<email>sammc@chromium.org</email>
</author>
<published>2022-01-05T05:12:56+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=e5707317e85f7e8d79c8d9bcddc582ec00f97b69'/>
<id>e5707317e85f7e8d79c8d9bcddc582ec00f97b69</id>
<content type='text'>
SERVOD_PORT and SERVOD_NAME environment variables are both used by
dut-control to identify the servod instance to use. If SERVOD_NAME is
set, query the servo serial number just like when SERVOD_PORT is set.

BUG=None
TEST=futility --servo -i with SERVOD_NAME set
BRANCH=None

Change-Id: I0c656e25d4f5da67570e3ff7d6cf28fdca5ec1cf
Signed-off-by: Sam McNally &lt;sammc@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3367518
Reviewed-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
SERVOD_PORT and SERVOD_NAME environment variables are both used by
dut-control to identify the servod instance to use. If SERVOD_NAME is
set, query the servo serial number just like when SERVOD_PORT is set.

BUG=None
TEST=futility --servo -i with SERVOD_NAME set
BRANCH=None

Change-Id: I0c656e25d4f5da67570e3ff7d6cf28fdca5ec1cf
Signed-off-by: Sam McNally &lt;sammc@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3367518
Reviewed-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot_reference/futility: Support "ccd_gsc" as a servo type fragment.</title>
<updated>2022-01-07T04:47:46+00:00</updated>
<author>
<name>Sam McNally</name>
<email>sammc@chromium.org</email>
</author>
<published>2022-01-05T05:06:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=bf7497d5a55442f0067397c789d6e02b861822cc'/>
<id>bf7497d5a55442f0067397c789d6e02b861822cc</id>
<content type='text'>
For DT, servo_type reports ccd_gsc instead of ccd_cr50. Treat ccd_gsc
the same as ccd_cr50.

BUG=None
TEST=futility update --servo -i
BRANCH=None

Change-Id: I27f5ea7ccc70fd1a247b844a9929aed7a133ebb3
Signed-off-by: Sam McNally &lt;sammc@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3367517
Reviewed-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
For DT, servo_type reports ccd_gsc instead of ccd_cr50. Treat ccd_gsc
the same as ccd_cr50.

BUG=None
TEST=futility update --servo -i
BRANCH=None

Change-Id: I27f5ea7ccc70fd1a247b844a9929aed7a133ebb3
Signed-off-by: Sam McNally &lt;sammc@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3367517
Reviewed-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Makefile: Fix missed boolean flags</title>
<updated>2022-01-04T11:50:59+00:00</updated>
<author>
<name>Jakub Czapiga</name>
<email>jacz@semihalf.com</email>
</author>
<published>2021-12-20T10:33:50+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=80629e7fd1935c51c476fb228b3560f922982cbe'/>
<id>80629e7fd1935c51c476fb228b3560f922982cbe</id>
<content type='text'>
Fix flags missed in CL:3306615.

BUG=b:206031372
TEST=make runtests
TEST=sudo emerge-guybrush sys-boot/depthcharge
BRANCH=none

Signed-off-by: Jakub Czapiga &lt;jacz@semihalf.com&gt;
Change-Id: Idbb205bb9e14f44a10806785fb9f7b29572f0626
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3347667
Reviewed-by: Yu-Ping Wu &lt;yupingso@chromium.org&gt;
Tested-by: Jakub Czapiga &lt;czapiga@google.com&gt;
Commit-Queue: Jakub Czapiga &lt;czapiga@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix flags missed in CL:3306615.

BUG=b:206031372
TEST=make runtests
TEST=sudo emerge-guybrush sys-boot/depthcharge
BRANCH=none

Signed-off-by: Jakub Czapiga &lt;jacz@semihalf.com&gt;
Change-Id: Idbb205bb9e14f44a10806785fb9f7b29572f0626
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3347667
Reviewed-by: Yu-Ping Wu &lt;yupingso@chromium.org&gt;
Tested-by: Jakub Czapiga &lt;czapiga@google.com&gt;
Commit-Queue: Jakub Czapiga &lt;czapiga@google.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot_reference: Allow flashrom_read_image() reading per-region</title>
<updated>2022-01-04T04:32:51+00:00</updated>
<author>
<name>Edward O'Callaghan</name>
<email>quasisec@google.com</email>
</author>
<published>2021-12-30T05:38:04+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=cdbd5769d3b586042555b6211c5cb5160a4ed565'/>
<id>cdbd5769d3b586042555b6211c5cb5160a4ed565</id>
<content type='text'>
BUG=b:207808292
BRANCH=none
TEST=`make`

Signed-off-by: Edward O'Callaghan &lt;quasisec@google.com&gt;
Change-Id: I98f1a2566ef50b2c7b1376141bb2f1c096b710a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358664
Tested-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Auto-Submit: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Reviewed-by: Nikolai Artemiev &lt;nartemiev@google.com&gt;
Commit-Queue: Nikolai Artemiev &lt;nartemiev@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
BUG=b:207808292
BRANCH=none
TEST=`make`

Signed-off-by: Edward O'Callaghan &lt;quasisec@google.com&gt;
Change-Id: I98f1a2566ef50b2c7b1376141bb2f1c096b710a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3358664
Tested-by: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Auto-Submit: Edward O'Callaghan &lt;quasisec@chromium.org&gt;
Reviewed-by: Nikolai Artemiev &lt;nartemiev@google.com&gt;
Commit-Queue: Nikolai Artemiev &lt;nartemiev@google.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
