[PATCH] JFFS2: Fix writebuffer recovery in the first page of a block
Thomas Gleixner
tglx at linutronix.de
Mon Feb 26 11:09:09 EST 2007
From: Vitaly Wool <vwool at ru.mvista.com>
For the case when nand_write_page fail with -EIO for the first page in an
eraseblock, jffs2_wbuf_recover ends up producing a BUG in jffs2_block_refile
as jeb->first_node is not yet set up (it's set up later in jffs2_wbuf_recover).
This BUG is not really a bug; it's just jffs2_wbuf_recover calling
jffs2_block_refile with the wrong second parameter.
This patch takes care of this situation.
fs/jffs2/wbuf.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletion(-)
Signed-off-by: Vitaly Wool <vwool at ru.mvista.com>
Signed-off-by: Thomas Gleixner <tglx at linutronix.de>
Index: linux-2.6/fs/jffs2/wbuf.c
===================================================================
--- linux-2.6.orig/fs/jffs2/wbuf.c 2006-11-28 17:08:25.000000000 +0300
+++ linux-2.6/fs/jffs2/wbuf.c 2007-02-26 18:29:58.000000000 +0300
@@ -237,7 +237,10 @@
jeb = &c->blocks[c->wbuf_ofs / c->sector_size];
spin_lock(&c->erase_completion_lock);
- jffs2_block_refile(c, jeb, REFILE_NOTEMPTY);
+ if (c->wbuf_ofs % c->mtd->erasesize)
+ jffs2_block_refile(c, jeb, REFILE_NOTEMPTY);
+ else
+ jffs2_block_refile(c, jeb, REFILE_ANYWAY);
spin_unlock(&c->erase_completion_lock);
BUG_ON(!ref_obsolete(jeb->last_node));
More information about the linux-mtd
mailing list