diff options
| author | Nam T. Nguyen <namnguyen@chromium.org> | 2014-12-15 11:03:36 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-12-16 01:28:48 +0000 |
| commit | d53b6684a2b451e41d9af84829b08a0c3325792f (patch) | |
| tree | 051b4914cfb15b3a26a34e420dd42a205cc83ec5 /tests/run_cgpt_tests.sh | |
| parent | 3f4d8d05ba4e32990c8584bd47cdf082d4604232 (diff) | |
| download | vboot-d53b6684a2b451e41d9af84829b08a0c3325792f.tar.gz | |
cgpt: Rework number of entries calculation
As pointed out by Daniel in CL:234996, the half_size_sectors should have
been checked in both places.
This CL reworks that part of the code to read easier.
BUG=none
BRANCH=none
TEST=unittest
Change-Id: I8faea3b094c375e4fd1a604a8fe759af88943fdf
Reviewed-on: https://chromium-review.googlesource.com/235792
Reviewed-by: Daniel Ehrenberg <dehrenberg@chromium.org>
Tested-by: Nam Nguyen <namnguyen@chromium.org>
Commit-Queue: Nam Nguyen <namnguyen@chromium.org>
Diffstat (limited to 'tests/run_cgpt_tests.sh')
| -rwxr-xr-x | tests/run_cgpt_tests.sh | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/run_cgpt_tests.sh b/tests/run_cgpt_tests.sh index 7170eef8..5176017a 100755 --- a/tests/run_cgpt_tests.sh +++ b/tests/run_cgpt_tests.sh @@ -291,6 +291,12 @@ $CGPT find $MTD -t kernel ${DEV} >/dev/null # Enable write access again to test boundary in off device storage chmod 600 ${DEV} +# GPT too small +dd if=/dev/zero of=${DEV} bs=5632 count=1 +assert_fail $CGPT create -D 1024 ${DEV} +# GPT is just right for 16 entries (512 + 512 + 16 * 128) * 2 = 6144 +dd if=/dev/zero of=${DEV} bs=6144 count=1 +$CGPT create -D 1024 ${DEV} # Create a small 8K file to simulate Flash NOR section dd if=/dev/zero of=${DEV} bs=8K count=1 # Drive size is not multiple of 512 |