diff options
author | David Woodhouse <dwmw2@infradead.org> | 2007-05-05 04:52:49 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2007-05-05 04:52:49 -0400 |
commit | 3fddb6c985e3823c991399840d2d5ef5940e1b60 (patch) | |
tree | d8da4d00e1c66eca503c511913532e6d98ea0ab2 /fs/jffs2 | |
parent | 693ef66d8914f50cb899b5268676ea508d1f3178 (diff) |
[JFFS2] Don't advance c->wbuf_ofs to next eraseblock after wbuf flush
After flushing the last page of an eraseblock, don't leave the
wbuf 'offset' field pointing at the start of the next physical
eraseblock. This was causing a BUG() on NOR-ECC (Sibley) flash, where
we start writing a little further in, after the cleanmarker.
Debugged by Alexander Belyakov <abelyako@googlemail.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Diffstat (limited to 'fs/jffs2')
-rw-r--r-- | fs/jffs2/wbuf.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/jffs2/wbuf.c b/fs/jffs2/wbuf.c index c556e85a565c..91d1d0f1c66c 100644 --- a/fs/jffs2/wbuf.c +++ b/fs/jffs2/wbuf.c | |||
@@ -637,7 +637,10 @@ static int __jffs2_flush_wbuf(struct jffs2_sb_info *c, int pad) | |||
637 | 637 | ||
638 | memset(c->wbuf,0xff,c->wbuf_pagesize); | 638 | memset(c->wbuf,0xff,c->wbuf_pagesize); |
639 | /* adjust write buffer offset, else we get a non contiguous write bug */ | 639 | /* adjust write buffer offset, else we get a non contiguous write bug */ |
640 | c->wbuf_ofs += c->wbuf_pagesize; | 640 | if (SECTOR_ADDR(c->wbuf_ofs) == SECTOR_ADDR(c->wbuf_ofs+c->wbuf_pagesize)) |
641 | c->wbuf_ofs += c->wbuf_pagesize; | ||
642 | else | ||
643 | c->wbuf_ofs = 0xffffffff; | ||
641 | c->wbuf_len = 0; | 644 | c->wbuf_len = 0; |
642 | return 0; | 645 | return 0; |
643 | } | 646 | } |