diff options
| author | Lu Fengqi <lufq.fnst@cn.fujitsu.com> | 2017-02-21 16:34:37 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2017-03-08 13:00:49 +0100 |
| commit | a0e44021abd7fa612f3060d6b9fe7a50211aafcf (patch) | |
| tree | e21899a443c153989439c20d10800a5a0668ba36 /tests/common.local | |
| parent | cbaa289d3d1350a7aad8f3f96df2b66ffeaa9155 (diff) | |
btrfs-progs: check: Fix lowmem mode override to allow it skip repair work
Current common.local doesn't handle lowmem mode well.
It passes "--mode=lowmem" alone with "--repair", making it unable to
check lowmem mode.
It's caused by the following bugs:
1) Wrong variable in test/common.local
We should check TEST_ARGS_CHECK, not TEST_CHECK, which is not defined
so we never return 1.
2) Wrong parameter passed to _cmd_spec() in test/common
This prevents us from grepping the correct parameters.
Fix it.
Signed-off-by: Lu Fengqi <lufq.fnst@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'tests/common.local')
| -rw-r--r-- | tests/common.local | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/common.local b/tests/common.local index 9f567c27..4f56bb08 100644 --- a/tests/common.local +++ b/tests/common.local @@ -17,7 +17,7 @@ TEST_ARGS_CHECK=--mode=lowmem # break tests _skip_spec() { - if echo "$TEST_CHECK" | grep -q 'mode=lowmem' && + if echo "$TEST_ARGS_CHECK" | grep -q 'mode=lowmem' && echo "$@" | grep -q -- '--repair'; then return 0 fi |