summaryrefslogtreecommitdiff
path: root/tests/mtd-tests
diff options
context:
space:
mode:
authorMiquel Raynal <miquel.raynal@free-electrons.com>2017-08-24 12:59:46 +0200
committerDavid Oberhollenzer <david.oberhollenzer@sigma-star.at>2017-08-24 13:00:58 +0200
commit4458ad6481f60d9884925d5bc62a7954880d181b (patch)
tree41b28b90221bc6ddb5be3357e0f35eb158ea0818 /tests/mtd-tests
parent76a55b190c3b15f512c1daf5976249b38db916d6 (diff)
mtd-utils: tests: Avoid using less than two blocks in nandpagetest
Forbid the use of less than 2 eraseblocks in nandpagetest. It is obvious that the test cannot run on zero block, but it cannot run on only one block neither. The reason is: get_first_and_last_block() will return the same id for both the first and the last blocks. In erasecrosstest(), the logic is: - erase/write/read/verify first block - erase/write again first block - erase *last* block - read/verify first block When using only one block, 'first' refers to the same block as 'last', leading to erasing the block before reading it. Hence, the test would fail with no actual reason. Signed-off-by: Miquel Raynal <miquel.raynal@free-electrons.com> Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'tests/mtd-tests')
-rw-r--r--tests/mtd-tests/nandpagetest.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/mtd-tests/nandpagetest.c b/tests/mtd-tests/nandpagetest.c
index 4145ef7..86c9005 100644
--- a/tests/mtd-tests/nandpagetest.c
+++ b/tests/mtd-tests/nandpagetest.c
@@ -65,7 +65,7 @@ static void usage(int status)
"Options:\n"
" -h, --help Display this help output\n"
" -b, --peb <num> Index of the first erase block to use\n"
- " -c, --count <num> Number of erase blocks to use (default all)\n"
+ " -c, --count <num> Number of erase blocks to use (at least 2, default all)\n"
" -s, --skip <num> Number of erase blocks to skip\n"
" -S, --seed <num> Seed for pseudor random number generator\n"
" -k, --keep Restore existing contents after test\n",
@@ -143,6 +143,8 @@ static void process_options(int argc, char **argv)
if (optind < argc)
usage(EXIT_FAILURE);
+ if (ebcnt < 2)
+ errmsg_die("Cannot run with less than two blocks.");
if (peb < 0)
peb = 0;
if (!(flags & SEED_SET))