diff options
| author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-04-26 09:59:56 +0300 |
|---|---|---|
| committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-04-29 21:19:58 +0300 |
| commit | 8d723c74983953a5b2fe64164893e3aee84c9d78 (patch) | |
| tree | 588dd86ea3bd0c6427ae9139107803573d999d2a | |
| parent | 45937cbc1578546b6930d0e6acc357f3dadd05e5 (diff) | |
fs-tests: integck: fail loudly on unexpected EROFS
If we are not in power cut testing mode (-p was not used) and we get an
unexpected EROFS, we silently exit. This patch fixes this behavior and make
the test loudly fail instead.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
| -rw-r--r-- | tests/fs-tests/integrity/integck.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/tests/fs-tests/integrity/integck.c b/tests/fs-tests/integrity/integck.c index bbbc7cb..1dd424e 100644 --- a/tests/fs-tests/integrity/integck.c +++ b/tests/fs-tests/integrity/integck.c @@ -69,9 +69,11 @@ check_failed(stringify(cond), __func__, __FILE__, __LINE__); \ } while(0) -#define pcv(fmt, ...) do { \ - if (args.power_cut_mode && args.verbose) \ - normsg(fmt " (line %d)", ##__VA_ARGS__, __LINE__); \ +#define pcv(fmt, ...) do { \ + if (!args.power_cut_mode || (args.power_cut_mode && args.verbose)) \ + normsg(fmt " (line %d)", ##__VA_ARGS__, __LINE__); \ + if (!args.power_cut_mode) \ + CHECK(0); \ } while(0) /* The variables below are set by command line arguments */ @@ -3048,8 +3050,10 @@ int main(int argc, char *argv[]) /* * Iterate forever only in case of power-cut emulation testing. */ - if (!args.power_cut_mode) + if (!args.power_cut_mode) { + CHECK(!ret); break; + } CHECK(ret); CHECK(errno == EROFS); |