[JFFS2] Correct cleanmarker checks -- we should use only 8 bytes

Linux-MTD Mailing List linux-mtd at lists.infradead.org
Thu Aug 9 05:59:01 EDT 2007


Gitweb:     http://git.infradead.org/?p=mtd-2.6.git;a=commit;h=09b3fba562ce366312b90a6f71d0b727b4d93ba9
Commit:     09b3fba562ce366312b90a6f71d0b727b4d93ba9
Parent:     471f717a48d25afcb9428c9523cd0557738b7115
Author:     David Woodhouse <dwmw2 at infradead.org>
AuthorDate: Thu Aug 9 17:28:20 2007 +0800
Committer:  David Woodhouse <dwmw2 at infradead.org>
CommitDate: Thu Aug 9 17:28:20 2007 +0800

    [JFFS2] Correct cleanmarker checks -- we should use only 8 bytes
    
    Commit a7a6ace1406f95c3edb8365788f85984377f3832 revamped the OOB
    handling but accidentally switched to 12-byte cleanmarkers, which is
    incompatible with what 'flash_eraseall -j' will do. So using
    flash_eraseall -j and then trying to mount the 'empty' flash will fail,
    because the cleanmarkers aren't recognised.
    
    Signed-off-by: David Woodhouse <dwmw2 at infradead.org>
---
 fs/jffs2/wbuf.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/jffs2/wbuf.c b/fs/jffs2/wbuf.c
index ec99c8e..d1d4f27 100644
--- a/fs/jffs2/wbuf.c
+++ b/fs/jffs2/wbuf.c
@@ -1008,8 +1008,8 @@ exit:
 
 #define NR_OOB_SCAN_PAGES 4
 
-/* For historical reasons we use only 12 bytes for OOB clean marker */
-#define OOB_CM_SIZE 12
+/* For historical reasons we use only 8 bytes for OOB clean marker */
+#define OOB_CM_SIZE 8
 
 static const struct jffs2_unknown_node oob_cleanmarker =
 {



More information about the linux-mtd-cvs mailing list