<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/vboot.git, branch stabilize-RUST-14057.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>crossystem: update --help text</title>
<updated>2021-06-26T05:43:42+00:00</updated>
<author>
<name>Mike Frysinger</name>
<email>vapier@chromium.org</email>
</author>
<published>2021-06-16T15:04:25+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=5c0fcf0f8725b98043a4b18f36e21489375d133a'/>
<id>5c0fcf0f8725b98043a4b18f36e21489375d133a</id>
<content type='text'>
Fix styling of earlier sections and link to the CrOS os_config guide.

BUG=None
TEST=`crossystem --help` looks nice
BRANCH=None

Signed-off-by: Mike Frysinger &lt;vapier@chromium.org&gt;
Change-Id: I1d5d9b080ee288541619ec4e0e8d550985051558
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2966239
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix styling of earlier sections and link to the CrOS os_config guide.

BUG=None
TEST=`crossystem --help` looks nice
BRANCH=None

Signed-off-by: Mike Frysinger &lt;vapier@chromium.org&gt;
Change-Id: I1d5d9b080ee288541619ec4e0e8d550985051558
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2966239
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: change recovery behaviour on kernel failure</title>
<updated>2021-06-26T00:39:15+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-05-26T06:13:37+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=600c679cd66265a54ee63b1747abfff6f94a25ad'/>
<id>600c679cd66265a54ee63b1747abfff6f94a25ad</id>
<content type='text'>
On load kernel failure, only call vb2api_fail when in normal
mode.

Previously, the behaviour here was to only call when attempting
to load a kernel from a fixed disk.  This maps to (1) normal
mode, and (2) developer mode when booting from an internal disk.

Excluding (2) creates a more consistent experience in developer
mode, and also prepares for a world where recovery kernels might
exist on disk.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551, b:188121855
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ic2c55a073b036be98f4ce9b2e0c7fb3209de74c8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2917623
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
On load kernel failure, only call vb2api_fail when in normal
mode.

Previously, the behaviour here was to only call when attempting
to load a kernel from a fixed disk.  This maps to (1) normal
mode, and (2) developer mode when booting from an internal disk.

Excluding (2) creates a more consistent experience in developer
mode, and also prepares for a world where recovery kernels might
exist on disk.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551, b:188121855
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ic2c55a073b036be98f4ce9b2e0c7fb3209de74c8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2917623
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: break disk check out to separate function</title>
<updated>2021-06-15T19:35:56+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-04-29T04:02:16+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=4e982f1c39da417100e4021fb1c2c370da5f8dd6'/>
<id>4e982f1c39da417100e4021fb1c2c370da5f8dd6</id>
<content type='text'>
Move disk validity check to static function is_valid_disk().

If multiple disk types are selected (e.g. REMOVABLE | FIXED),
is_valid_disk() will now check that exactly *one* of those flags
is selected by VbDiskInfo.flags.

Also, split disk flags into two 16-bit sections:
 - Disk selection in the lower 16 bits (where the disk lives)
 - Disk attributes in the higher 16 bits (extra information about
   the disk needed to access it correctly)

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Icf76ab6e92cca40810071def66aed13cdb3a7ec7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2872251
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Move disk validity check to static function is_valid_disk().

If multiple disk types are selected (e.g. REMOVABLE | FIXED),
is_valid_disk() will now check that exactly *one* of those flags
is selected by VbDiskInfo.flags.

Also, split disk flags into two 16-bit sections:
 - Disk selection in the lower 16 bits (where the disk lives)
 - Disk attributes in the higher 16 bits (extra information about
   the disk needed to access it correctly)

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Icf76ab6e92cca40810071def66aed13cdb3a7ec7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2872251
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: remove LoadKernelParams struct</title>
<updated>2021-06-15T19:35:55+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-04-22T13:11:50+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=da50d8587ae24b1a5e7528dde1ead5523e78f6b2'/>
<id>da50d8587ae24b1a5e7528dde1ead5523e78f6b2</id>
<content type='text'>
LoadKernelParams struct is identical to
VbSelectAndLoadKernelParams.  Remove it, and use the public
interface internally.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I8b820d18c1e9a66404a7a091aa3ccc1b050a559d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846282
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
LoadKernelParams struct is identical to
VbSelectAndLoadKernelParams.  Remove it, and use the public
interface internally.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I8b820d18c1e9a66404a7a091aa3ccc1b050a559d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846282
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: pass VbDiskInfo into LoadKernel</title>
<updated>2021-06-15T19:35:54+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-04-22T13:03:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=43fa783cbf6014d23d8b23b5eb7b4e63b17864ec'/>
<id>43fa783cbf6014d23d8b23b5eb7b4e63b17864ec</id>
<content type='text'>
Pass VbDiskInfo struct into LoadKernel, rather than copying all
of its members into LoadKernelParams.  Remove the unused members
from LoadKernelParams.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I60957426388c88b16e570b717addb5eaf65b5e4f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846281
Reviewed-by: Joel Kitching &lt;kitching@chromium.org&gt;
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Pass VbDiskInfo struct into LoadKernel, rather than copying all
of its members into LoadKernelParams.  Remove the unused members
from LoadKernelParams.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I60957426388c88b16e570b717addb5eaf65b5e4f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846281
Reviewed-by: Joel Kitching &lt;kitching@chromium.org&gt;
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: move workbuf init into vb2_load_partition</title>
<updated>2021-06-15T19:35:53+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-05-10T04:53:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=39ea3ade28b733521b480da161c61848764a3096'/>
<id>39ea3ade28b733521b480da161c61848764a3096</id>
<content type='text'>
Not needed prior to that.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ia60aa05384384bc1e2981266c33b960115734fae
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2882527
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Not needed prior to that.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ia60aa05384384bc1e2981266c33b960115734fae
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2882527
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: move kernel/fw struct functions</title>
<updated>2021-06-15T19:35:52+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-04-14T07:13:28+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=c6cf8890532a7b78b6cb3dc534145f500444dae2'/>
<id>c6cf8890532a7b78b6cb3dc534145f500444dae2</id>
<content type='text'>
Relocate currently-in-use kernel struct functions out
of lib20 namespace, and into:
* 2struct.c for functions required at runtime
* host_common.c for functions required by host

Relocate firmware struct functions from 2common.c
into 2struct.c

vb2_common.h may be deleted as a result.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ic162d9633b6112ddc4a819b3e58d313dc484f304
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2825269
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Relocate currently-in-use kernel struct functions out
of lib20 namespace, and into:
* 2struct.c for functions required at runtime
* host_common.c for functions required by host

Relocate firmware struct functions from 2common.c
into 2struct.c

vb2_common.h may be deleted as a result.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: Ic162d9633b6112ddc4a819b3e58d313dc484f304
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2825269
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>vboot/vboot_kernel: stop printing keyblock version error when in dev mode</title>
<updated>2021-06-15T19:35:50+00:00</updated>
<author>
<name>Joel Kitching</name>
<email>kitching@google.com</email>
</author>
<published>2021-04-22T10:55:10+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=ba23df230a9efe7f6aed8faab4dda944f62f76da'/>
<id>ba23df230a9efe7f6aed8faab4dda944f62f76da</id>
<content type='text'>
Previously, the "Key version too old." error message is printed
even when in dev mode (!need_keyblock_valid).  Stop printing this
message when the key version check is not applicable.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I1c4605342a2c53c45b74be37f2ba597670654a8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846252
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Previously, the "Key version too old." error message is printed
even when in dev mode (!need_keyblock_valid).  Stop printing this
message when the key version check is not applicable.

This CL is part of a series to merge vboot1 and vboot2.0
kernel verification code; see b/181739551.

BUG=b:181739551
TEST=make clean &amp;&amp; make runtests
BRANCH=none

Signed-off-by: Joel Kitching &lt;kitching@google.com&gt;
Change-Id: I1c4605342a2c53c45b74be37f2ba597670654a8e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2846252
Tested-by: Joel Kitching &lt;kitching@chromium.org&gt;
Commit-Queue: Joel Kitching &lt;kitching@chromium.org&gt;
Reviewed-by: Julius Werner &lt;jwerner@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>futility: updater: correct the rootkey in manifest.json</title>
<updated>2021-06-06T06:06:30+00:00</updated>
<author>
<name>Isaac Lee</name>
<email>isaaclee@google.com</email>
</author>
<published>2021-06-01T07:47:34+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=0bffea9ab8c6acc30af0cb613ae4cdd48f865834'/>
<id>0bffea9ab8c6acc30af0cb613ae4cdd48f865834</id>
<content type='text'>
For customization projects, the bios image should be patched by
model first, then the gbb got from the image could be correct.

Reason:
In the function, patch_image_by_model, it will apply the rootkey,
vblock_a and vblock_b of the customization project to the gbb part
of the image.

This change moves the function "find_gbb" to the position after the
function "patch_image_by_model" is called.

BUG=b:188861251
BRANCH=none
TEST=build

Change-Id: I5d80405a2a3b0ecc45e86ca3322b328479b83702
Signed-off-by: Isaac Lee &lt;isaaclee@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2929924
Reviewed-by: Hung-Te Lin &lt;hungte@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
For customization projects, the bios image should be patched by
model first, then the gbb got from the image could be correct.

Reason:
In the function, patch_image_by_model, it will apply the rootkey,
vblock_a and vblock_b of the customization project to the gbb part
of the image.

This change moves the function "find_gbb" to the position after the
function "patch_image_by_model" is called.

BUG=b:188861251
BRANCH=none
TEST=build

Change-Id: I5d80405a2a3b0ecc45e86ca3322b328479b83702
Signed-off-by: Isaac Lee &lt;isaaclee@google.com&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2929924
Reviewed-by: Hung-Te Lin &lt;hungte@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>image_signing: fix rootfs mount when checking kernel params</title>
<updated>2021-06-05T20:14:11+00:00</updated>
<author>
<name>Mike Frysinger</name>
<email>vapier@chromium.org</email>
</author>
<published>2021-05-28T08:11:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=dadb53ee4c27c3a4423765e0998c7a175d2b0822'/>
<id>dadb53ee4c27c3a4423765e0998c7a175d2b0822</id>
<content type='text'>
Use the existing mount helper so we clean up automatically when exiting.

BUG=None
TEST=CQ passes
BRANCH=None

Change-Id: I882c7f5ea3b54e08745c48378cc50702550cdc71
Signed-off-by: Mike Frysinger &lt;vapier@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2923828
Reviewed-by: George Engelbrecht &lt;engeg@google.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Use the existing mount helper so we clean up automatically when exiting.

BUG=None
TEST=CQ passes
BRANCH=None

Change-Id: I882c7f5ea3b54e08745c48378cc50702550cdc71
Signed-off-by: Mike Frysinger &lt;vapier@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2923828
Reviewed-by: George Engelbrecht &lt;engeg@google.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
