From: Harpreet Eli Sangha <harpreet@nestlabs.com>
Date: Tue, 27 Jun 2017 20:52:26 +0000 (-0700)
Subject: mtd-utils: tests: Fix nandbiterrs Failure Check
X-Git-Tag: v2.0.1~14
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=b0b1bc51496b7c328fcda3e6164292fa0d1dbc18;p=mtd-utils.git

mtd-utils: tests: Fix nandbiterrs Failure Check

After a page read, the old failure statistics are compared against the
new failure statistics before the new values are actually read.

Signed-off-by: Harpreet "Eli" Sangha <harpreet@nestlabs.com>
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
---

diff --git a/tests/mtd-tests/nandbiterrs.c b/tests/mtd-tests/nandbiterrs.c
index 6acbfee..de9d0dd 100644
--- a/tests/mtd-tests/nandbiterrs.c
+++ b/tests/mtd-tests/nandbiterrs.c
@@ -268,15 +268,15 @@ static int read_page(void)
 		return -1;
 	}
 
+	if (ioctl(fd, ECCGETSTATS, &new) != 0)
+		goto failstats;
+
 	if (new.failed > old.failed) {
 		fprintf(stderr, "Failed to recover %d bitflips\n",
 				new.failed - old.failed);
 		return -1;
 	}
 
-	if (ioctl(fd, ECCGETSTATS, &new) != 0)
-		goto failstats;
-
 	return new.corrected - old.corrected;
 failstats:
 	perror("ECCGETSTATS");