Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Issue 6685079: Revert "Revert "Add --kloadaddr option to utilities"" (Closed)

Created:
9 years, 9 months ago by Che-Liang Chiou
Modified:
9 years ago
Reviewers:
marcheu, sjg
CC:
chromium-os-reviews_chromium.org, Randall Spangler, gauravsh, Luigi Semenzato, Bill Richardson
Visibility:
Public.

Description

Revert "Revert "Add --kloadaddr option to utilities"" This reverts commit bc7a84d9a1bef3fb8c1e2709033f6c9777599fe9. It was a false alarm that --kloadaddr causes chromeos-install on a x86 targets to fail. The error of chromeos-install cannot be reproduced, and judging by the reported error message, the error should not be attributed to --kloadaddr, which has no effect in x86 targets. So --kloadaddr is restored. Verification process are below: (Verify that --kloadaddr option is restored) $ dump_kernel_config -h Expected argument after options dump_kernel_config - Prints the kernel command line Usage: dump_kernel_config [--kloadaddr <ADDRESS>] <image/blockdevice> (Setup a x86 target with kernel-next profile) $ rm -rf /build/${X86_TARGET} $ ./setup_board --board=${X86_TARGET} --profile=kernel-next $ ./build_packages --board=${X86_TARGET} $ ./build_image --board=${X86_TARGET} (Run chromeos-install on target machine successfully) $ /usr/sbin/chromeos-install (Change directory to where image sits) $ cd ~/trunk/src/build/images/${X86_TARGET}/latest (Unpack Chromium OS image) $ ./unpack_partitions.sh chromiumos_image.bin (Verify that dump_kernel_config runs successfully) $ dump_kernel_config part_2 console=tty2 init=/sbin/init add_efi_memmap boot=local noresume noswap i915.modeset=1 cros_secure kern_guid=%U tpm_tis.force=1 tpm_tis.interrupts=0 nmi_watchdog=panic,lapic i8042.nomux=1 root=/dev/dm-0 quiet loglevel=1 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="vroot none ro,0 1740800 verity %U+1 %U+1 1740800 1 sha1 c357e07395150770ce25ebc0e3c6d15941675c58" (Run load_kernel_test) $ load_kernel_test -b 2 chromiumos_image.bin /usr/share/vboot/devkeys/recovery_key.vbpubk Read 2088 bytes of key from /usr/share/vboot/devkeys/recovery_key.vbpubk bootflags = 6 Reading from image: chromiumos_image.bin Ending LBA: 3989538 Read(1, 1) Read(2, 32) Read(3989506, 32) Read(3989538, 1) Read(4096, 128) Read(4224, 6472) LoadKernel() returned 0 Partition number: 2 Bootloader address: 4345856 Bootloader size: 16384 Partition guid: b2a453b0-a64a-5c4d-a957-1388cea384a5 R=marcheu@chromium.org,sjg@chromium.org BUG=none TEST=see verification process above Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=2c0711b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+94 lines, -25 lines) Patch
M utility/dump_kernel_config.c View 6 chunks +53 lines, -7 lines 0 comments Download
M utility/vbutil_kernel.c View 18 chunks +41 lines, -18 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Che-Liang Chiou
9 years, 9 months ago (2011-03-18 02:24:06 UTC) #1
marcheu
On 2011/03/18 02:24:06, Che-Liang Chiou wrote: LGTM. I can't reproduce any more either.
9 years, 9 months ago (2011-03-18 02:58:35 UTC) #2
sjg
9 years, 9 months ago (2011-03-19 00:28:00 UTC) #3
I have not look at this again- but you have an LGTM. Happy to help if
needed. - Simon

On Thu, Mar 17, 2011 at 7:58 PM,  <marcheu@chromium.org> wrote:
> On 2011/03/18 02:24:06, Che-Liang Chiou wrote:
>
> LGTM. I can't reproduce any more either.
>
>
> http://codereview.chromium.org/6685079/
>

Powered by Google App Engine
This is Rietveld 408576698