JFFS2 wbuf non-contiguous write bug

Josh Boyer jwboyer at gmail.com
Thu Jun 29 10:46:10 EDT 2006


On 6/29/06, Alexander Belyakov <abelyako at googlemail.com> wrote:
> On 6/29/06, Jörn Engel <joern at wohnheim.fh-wedel.de> wrote:
>
> Anyway I was speaking _not_ about reverting part of your patch but
> about getting back following code for Sibley, since it indeed shows if
> we moved to a new eraseblock.
>
>                 if (((c->wbuf_ofs % c->sector_size) == 0) && !c->wbuf_len) {
>                         c->wbuf_ofs = PAGE_DIV(to);
>                         c->wbuf_len = PAGE_MOD(to);
>                         memset(c->wbuf,0xff,c->wbuf_pagesize);
>                 }
>
> > You must be missing something, but I don't know what.  Maybe Josh has
> > some insight into this problem - he did a thorough review and asked
> > many probing questions about this particular part of the patch.
>
> It would be really nice, since current code seems to be broken somehow.

I remember going over this with Jörn a while back and being convinced
NOR ECC didn't need that check any longer.  I don't recall looking if
Sibley does though.  I'll try to look at it again, but obviously if it
fixes a bug then it should probably go back in.  I don't think it can
hurt anything at the moment.

josh




More information about the linux-mtd mailing list