Add --kloadaddr option to utilities

Add --kloadaddr option to utilities

Kernel body load address was hard-coded to CROS_32BIT_ENTRY_ADDR, which
could be an invalid/unavailable memory location on other platforms.

This CL adds an option for setting the load address, and it is default to
CROS_32BIT_ENTRY_ADDR to maintain backward-compatibility.

BUG=chromium-os:1304
TEST=emerge vboot_reference successfully

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=1a0975f

[Randall Spangler, 2011-03-09 17:25:38]

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c
File utility/dump_kernel_config.c (right):

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c...
utility/dump_kernel_config.c:25: { "kernelbodyloadaddress", 1, 0,
OPT_KERNELBODYLOADADDRESS },
When you say long_opts, you're not kidding. :)

--kloadaddr ?

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c...
utility/dump_kernel_config.c:41: uint64_t kernelbodyloadaddress) {
kernel_body_load_address

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c
File utility/vbutil_kernel.c (right):

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c#newc...
utility/vbutil_kernel.c:62: {"kernelbodyloadaddress", 1, 0,    
OPT_KERNELBODYLOADADDRESS   },
kloadaddr

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c#newc...
utility/vbutil_kernel.c:210: static char* BpCmdLineLocation(blob_t *bp, uint64_t
kernelbodyloadaddress)
kernel_body_load_address (elsewhere, too)

[Che-Liang Chiou, 2011-03-10 07:41:28]

Thanks for comments. PTAL.

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c
File utility/dump_kernel_config.c (right):

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c...
utility/dump_kernel_config.c:25: { "kernelbodyloadaddress", 1, 0,
OPT_KERNELBODYLOADADDRESS },
On 2011/03/09 17:25:38, Randall Spangler wrote:
> When you say long_opts, you're not kidding. :)
> 
> --kloadaddr ?

Done.

http://codereview.chromium.org/6651022/diff/3001/utility/dump_kernel_config.c...
utility/dump_kernel_config.c:41: uint64_t kernelbodyloadaddress) {
On 2011/03/09 17:25:38, Randall Spangler wrote:
> kernel_body_load_address

Done.

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c
File utility/vbutil_kernel.c (right):

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c#newc...
utility/vbutil_kernel.c:62: {"kernelbodyloadaddress", 1, 0,    
OPT_KERNELBODYLOADADDRESS   },
On 2011/03/09 17:25:38, Randall Spangler wrote:
> kloadaddr

Done.

http://codereview.chromium.org/6651022/diff/3001/utility/vbutil_kernel.c#newc...
utility/vbutil_kernel.c:210: static char* BpCmdLineLocation(blob_t *bp, uint64_t
kernelbodyloadaddress)
On 2011/03/09 17:25:38, Randall Spangler wrote:
> kernel_body_load_address (elsewhere, too)

Done.

[Randall Spangler, 2011-03-10 16:39:40]

LGTM, thanks