<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/coreboot/vboot.git/README, branch stabilize-11316.146.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>cleanup: DESTDIR refers to the install root, not the bin/</title>
<updated>2014-09-17T01:24:40+00:00</updated>
<author>
<name>Bill Richardson</name>
<email>wfrichar@chromium.org</email>
</author>
<published>2014-09-11T17:41:51+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=efa8756c5e7068e57f5e98f0e7f2f42fb8d969d6'/>
<id>efa8756c5e7068e57f5e98f0e7f2f42fb8d969d6</id>
<content type='text'>
This doesn't have any visible effect. It just brings the meaning
of the where-do-I-put-this variable more in line with common
convention.

BRANCH=ToT
BUG=none
CQ-DEPEND=CL:217940
TEST=lots...

make runtests
make DESTDIR=BAR install
make MINIMAL=1 DESTDIR=FOO install
emerge-$BOARD vboot_reference
sudo emerge vboot_reference
trybots: link-tot-paladin, daisy_spring-paladin

Change-Id: I8d72664da07535f663d8b2f13c872eece37978b9
Signed-off-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/217930
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This doesn't have any visible effect. It just brings the meaning
of the where-do-I-put-this variable more in line with common
convention.

BRANCH=ToT
BUG=none
CQ-DEPEND=CL:217940
TEST=lots...

make runtests
make DESTDIR=BAR install
make MINIMAL=1 DESTDIR=FOO install
emerge-$BOARD vboot_reference
sudo emerge vboot_reference
trybots: link-tot-paladin, daisy_spring-paladin

Change-Id: I8d72664da07535f663d8b2f13c872eece37978b9
Signed-off-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/217930
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cleanup: remove a couple of unused functions and files</title>
<updated>2014-09-12T03:39:46+00:00</updated>
<author>
<name>Bill Richardson</name>
<email>wfrichar@chromium.org</email>
</author>
<published>2014-09-05T19:40:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=bc3f0b74f9e8c8ca043f579d25cbf22a92ea209d'/>
<id>bc3f0b74f9e8c8ca043f579d25cbf22a92ea209d</id>
<content type='text'>
scripts/sign_data.sh is just a wrapper to do this:

  ./signature_digest_utility $1 $3 \
    | openssl rsautl -sign -pkcs -inkey $2

AFAICT, that script is only invoked by the SignatureFile()
function in host/lib/file_keys.c, which is not referenced by
anything. I think I can remove both of those things.

Also remove utility/gbb_utility.cc, which should have been done
long ago in commit 6f39615.

BUG=none
BRANCH=ToT
TEST=make runalltests

Also ran it on daisy_spring-paladin and link-tot-paladin.

Change-Id: I16de5022765806f11bf6144d7ffd8cc849578a68
Signed-off-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/216719
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
scripts/sign_data.sh is just a wrapper to do this:

  ./signature_digest_utility $1 $3 \
    | openssl rsautl -sign -pkcs -inkey $2

AFAICT, that script is only invoked by the SignatureFile()
function in host/lib/file_keys.c, which is not referenced by
anything. I think I can remove both of those things.

Also remove utility/gbb_utility.cc, which should have been done
long ago in commit 6f39615.

BUG=none
BRANCH=ToT
TEST=make runalltests

Also ran it on daisy_spring-paladin and link-tot-paladin.

Change-Id: I16de5022765806f11bf6144d7ffd8cc849578a68
Signed-off-by: Bill Richardson &lt;wfrichar@chromium.org&gt;
Reviewed-on: https://chromium-review.googlesource.com/216719
Reviewed-by: Mike Frysinger &lt;vapier@chromium.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove unused files, and tidy the directory structure of the remaining ones.</title>
<updated>2010-06-17T21:45:22+00:00</updated>
<author>
<name>Randall Spangler</name>
<email>rspangler@chromium.org</email>
</author>
<published>2010-06-17T21:45:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=620c38cf34eadcd222535b01fb71c5e9fbc1cb80'/>
<id>620c38cf34eadcd222535b01fb71c5e9fbc1cb80</id>
<content type='text'>
Review URL: http://codereview.chromium.org/2815011
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Review URL: http://codereview.chromium.org/2815011
</pre>
</div>
</content>
</entry>
<entry>
<title>Rework the vboot_reference make system.</title>
<updated>2010-06-14T22:41:27+00:00</updated>
<author>
<name>vbendeb</name>
<email>vbendeb@chromium.org</email>
</author>
<published>2010-06-14T22:41:27+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=70e9509a2748c0ee8646d77f2a0e6fcbb1446505'/>
<id>70e9509a2748c0ee8646d77f2a0e6fcbb1446505</id>
<content type='text'>
After this change the generated files are placed in a
separate tree (such thet they don't show in the
`git status' output anymore) and the dependencies are
followed properly (if a .h file changes the
appropriate .o files and apps get rebuilt).

Tested as follows:

&gt; $ make clean
&gt; $ make # build succeeds
&gt; $ git status # shows clean directory
&gt; $ RUNTESTS=1 make # (captured test output matches that of the test run before any changes)
&gt; $ touch ./vboot_firmware/include/tlcl.h
&gt; $ make  # make succeeds
&gt; $ find build -type f -newer ./vboot_firmware/include/tlcl.h
build/vboot_firmware/lib/rollback_index.o
build/vboot_firmware/lib/rollback_index.o.d
build/vboot_firmware/a.out
build/vboot_fw.a
build/utility/vbutil_key
build/utility/kernel_utility.d
build/utility/vbutil_key.d
build/utility/verify_data
build/utility/load_kernel_test.d
build/utility/vbutil_keyblock.d
build/utility/vbutil_kernel
build/utility/vbutil_kernel.d
build/utility/firmware_utility
build/utility/signature_digest_utility.d
build/utility/kernel_utility
build/utility/verify_data.d
build/utility/vbutil_keyblock
build/utility/signature_digest_utility
build/utility/load_kernel_test
build/utility/firmware_utility.d
build/tests/vboot_common3_tests
build/tests/vboot_common2_tests
build/host/a.out
$ &gt;

Review URL: http://codereview.chromium.org/2845001
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
After this change the generated files are placed in a
separate tree (such thet they don't show in the
`git status' output anymore) and the dependencies are
followed properly (if a .h file changes the
appropriate .o files and apps get rebuilt).

Tested as follows:

&gt; $ make clean
&gt; $ make # build succeeds
&gt; $ git status # shows clean directory
&gt; $ RUNTESTS=1 make # (captured test output matches that of the test run before any changes)
&gt; $ touch ./vboot_firmware/include/tlcl.h
&gt; $ make  # make succeeds
&gt; $ find build -type f -newer ./vboot_firmware/include/tlcl.h
build/vboot_firmware/lib/rollback_index.o
build/vboot_firmware/lib/rollback_index.o.d
build/vboot_firmware/a.out
build/vboot_fw.a
build/utility/vbutil_key
build/utility/kernel_utility.d
build/utility/vbutil_key.d
build/utility/verify_data
build/utility/load_kernel_test.d
build/utility/vbutil_keyblock.d
build/utility/vbutil_kernel
build/utility/vbutil_kernel.d
build/utility/firmware_utility
build/utility/signature_digest_utility.d
build/utility/kernel_utility
build/utility/verify_data.d
build/utility/vbutil_keyblock
build/utility/signature_digest_utility
build/utility/load_kernel_test
build/utility/firmware_utility.d
build/tests/vboot_common3_tests
build/tests/vboot_common2_tests
build/host/a.out
$ &gt;

Review URL: http://codereview.chromium.org/2845001
</pre>
</div>
</content>
</entry>
<entry>
<title>Rearrange vboot_reference directories to isolate external components.</title>
<updated>2010-05-26T16:18:38+00:00</updated>
<author>
<name>Bill Richardson</name>
<email>wfrichar@chromium.org</email>
</author>
<published>2010-05-26T16:18:38+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=0b8f35c6a56a0e645254dd7f07ca68e01c8ee629'/>
<id>0b8f35c6a56a0e645254dd7f07ca68e01c8ee629</id>
<content type='text'>
This creates a new vboot_firmware subdirectory, and which contains the
entirety of the BIOS code. There shouldn't be anything in this directory
that is NOT required by the BIOS.

Review URL: http://codereview.chromium.org/2219004
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This creates a new vboot_firmware subdirectory, and which contains the
entirety of the BIOS code. There shouldn't be anything in this directory
that is NOT required by the BIOS.

Review URL: http://codereview.chromium.org/2219004
</pre>
</div>
</content>
</entry>
<entry>
<title>Updated README</title>
<updated>2010-03-31T21:09:31+00:00</updated>
<author>
<name>Gaurav Shah</name>
<email>gauravsh@chromium.org</email>
</author>
<published>2010-03-31T21:09:31+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=ef7510fe405ffe3479890f9008730939987df240'/>
<id>ef7510fe405ffe3479890f9008730939987df240</id>
<content type='text'>
I clobbered my branch associated with the last CL. This is a new issue with the same change.

Review URL: http://codereview.chromium.org/1584005
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
I clobbered my branch associated with the last CL. This is a new issue with the same change.

Review URL: http://codereview.chromium.org/1584005
</pre>
</div>
</content>
</entry>
<entry>
<title>VBoot Reference: Refactoring Part 3</title>
<updated>2010-03-31T20:26:55+00:00</updated>
<author>
<name>Gaurav Shah</name>
<email>gauravsh@chromium.org</email>
</author>
<published>2010-03-31T20:26:55+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=fc70d72aaab4d558e39ec43832375267603bfd93'/>
<id>fc70d72aaab4d558e39ec43832375267603bfd93</id>
<content type='text'>
Refactor and restructure reference code into individual self-contain modules. I have revamped the way the code is structured to make it easy to determine which parts belong in the firmware and which are used by userland tools.

common/ - common utilities and stub functions (Firmware)
cryptolib/ - crypto library (Firmware)
misclibs/ - miscellaneous userland libraries (Userland)
sctips/ - Miscellaenous scripts (Userland)
tests/ - Tests (Userland)
vfirmware/ - Verified Firmware Implementation
vfirmware/firmware_image_fw.c (Firmware)
vfirmware/firmware_image.c (Userland)

vkernel/ - Verified Kernel Implementation
vkernel/kernel_image_fw.c (Firmware)
vkernel/kernel_image.c (Userland)

Review URL: http://codereview.chromium.org/1581005
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Refactor and restructure reference code into individual self-contain modules. I have revamped the way the code is structured to make it easy to determine which parts belong in the firmware and which are used by userland tools.

common/ - common utilities and stub functions (Firmware)
cryptolib/ - crypto library (Firmware)
misclibs/ - miscellaneous userland libraries (Userland)
sctips/ - Miscellaenous scripts (Userland)
tests/ - Tests (Userland)
vfirmware/ - Verified Firmware Implementation
vfirmware/firmware_image_fw.c (Firmware)
vfirmware/firmware_image.c (Userland)

vkernel/ - Verified Kernel Implementation
vkernel/kernel_image_fw.c (Firmware)
vkernel/kernel_image.c (Userland)

Review URL: http://codereview.chromium.org/1581005
</pre>
</div>
</content>
</entry>
<entry>
<title>VBoot ReferenceL Add instructions to generate (self-)signed firmware images.</title>
<updated>2010-03-29T19:50:09+00:00</updated>
<author>
<name>Gaurav Shah</name>
<email>gauravsh@chromium.org</email>
</author>
<published>2010-03-29T19:50:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=5b730c4a338e8956851ca3fa0f36260e8bc32218'/>
<id>5b730c4a338e8956851ca3fa0f36260e8bc32218</id>
<content type='text'>
Review URL: http://codereview.chromium.org/1573001
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Review URL: http://codereview.chromium.org/1573001
</pre>
</div>
</content>
</entry>
<entry>
<title>Add a README and turn-on optimizations.</title>
<updated>2010-03-03T21:15:53+00:00</updated>
<author>
<name>Gaurav Shah</name>
<email>gauravsh@google.com</email>
</author>
<published>2010-03-03T21:15:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/coreboot/vboot.git/commit/?id=56c9f4dfa7cd00c8bd87da3813360f73f6df9546'/>
<id>56c9f4dfa7cd00c8bd87da3813360f73f6df9546</id>
<content type='text'>
Review URL: http://codereview.chromium.org/665009
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Review URL: http://codereview.chromium.org/665009
</pre>
</div>
</content>
</entry>
</feed>
