| Message ID | 20170627205226.18734-1-harpreet@nestlabs.com |
|---|---|
| State | Accepted |
| Delegated to: | David Oberhollenzer |
| Headers | show |
Applied to mtd-utils.git. Thanks, David On 06/27/2017 10:52 PM, Harpreet Eli Sangha wrote: > From: "Harpreet \"Eli\" Sangha" <harpreet@nestlabs.com> > > 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> > --- > tests/mtd-tests/nandbiterrs.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > 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"); >
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");