diff options
author | Nam T. Nguyen <namnguyen@chromium.org> | 2014-11-13 19:30:46 -0800 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-11-14 22:46:59 +0000 |
commit | ab899591808dd3e5f955ab7693b54a83389cd35f (patch) | |
tree | 8d12700b5be62ac132fd09e3cb17f6b6cce4b70c /cgpt/cmd_legacy.c | |
parent | a524a3a51591618c1395cb9e1238ee72b3f5e767 (diff) | |
download | vboot-ab899591808dd3e5f955ab7693b54a83389cd35f.tar.gz |
vboot: cgpt: Treat drive_path as the GPT storage
Previously, "cgpt" called out to "flashrom" directly to read and write
NOR area. This CL removes that dependency and always treats "drive_path"
as the storage of GPT structs. This makes it consistent that whatever
device that cgpt reads from or writes to is always the device that
stores GPT structs. We only need to pass in the size of the drive that
contains the partitions, but we do not need to access to that drive.
More information is in the bug.
BUG=chromium:432611
BRANCH=none
TEST=unittest
CQ-DEPEND=CL:228942
Change-Id: Id0139adf70463cec4f2924de8b9a4725dbec822b
Reviewed-on: https://chromium-review.googlesource.com/229736
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
Commit-Queue: Nam Nguyen <namnguyen@chromium.org>
Tested-by: Nam Nguyen <namnguyen@chromium.org>
Diffstat (limited to 'cgpt/cmd_legacy.c')
-rw-r--r-- | cgpt/cmd_legacy.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/cgpt/cmd_legacy.c b/cgpt/cmd_legacy.c index 955ce3f4..5fd742cc 100644 --- a/cgpt/cmd_legacy.c +++ b/cgpt/cmd_legacy.c @@ -15,6 +15,9 @@ static void Usage(void) printf("\nUsage: %s legacy [OPTIONS] DRIVE\n\n" "Switch GPT header signature to \"CHROMEOS\".\n\n" "Options:\n" + " -D NUM Size (in bytes) of the disk where partitions reside\n" + " default 0, meaning partitions and GPT structs are\n" + " both on DRIVE\n" " -e Switch GPT header signature back to \"EFI PART\"\n" "\n", progname); } @@ -24,13 +27,22 @@ int cmd_legacy(int argc, char *argv[]) { memset(¶ms, 0, sizeof(params)); int c; + char* e = 0; int errorcnt = 0; opterr = 0; // quiet, you - while ((c=getopt(argc, argv, ":he")) != -1) + while ((c=getopt(argc, argv, ":heD:")) != -1) { switch (c) { + case 'D': + params.drive_size = strtoull(optarg, &e, 0); + if (!*optarg || (e && *e)) + { + Error("invalid argument to -%c: \"%s\"\n", c, optarg); + errorcnt++; + } + break; case 'e': params.efipart = 1; break; |