[PATCH] fs-tests: integck: fail loudly on unexpected EROFS

Artem Bityutskiy dedekind1 at gmail.com
Tue Apr 26 03:30:22 EDT 2011


From: Artem Bityutskiy <Artem.Bityutskiy at nokia.com>

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 at nokia.com>
---
 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);
-- 
1.7.2.3




More information about the linux-mtd mailing list