[PATCH 2/4] mtd: mxc_nand: limit the size of used oob
Baruch Siach
baruch at tkos.co.il
Mon Apr 27 00:20:57 PDT 2015
Hi Uwe,
On Mon, Apr 27, 2015 at 09:12:38AM +0200, Uwe Kleine-König wrote:
> On Mon, Apr 27, 2015 at 07:39:06AM +0300, Baruch Siach wrote:
> > On Sun, Apr 26, 2015 at 10:07:25PM +0200, Uwe Kleine-König wrote:
> > > On Sun, Apr 26, 2015 at 11:16:49AM +0300, Baruch Siach wrote:
> > > > + /* hardware can only use 218 or 128 oob bytes for ecc */
> > > > + if (mtd->oobsize >= 218)
> > > IMHO this is the wrong check. What if your part (with 224 bytes spare)
> > > is used but the machine only uses the small layout e.g. for booting?
> > > (That would work, wouldn't it?)
> >
> > Yes, but how would I know? I am following here the assumption of get_eccsize()
> > that enables 8 bit ECC when there is enough oobsize.
> Hmm I rechecked the reference manual and found a register to specify the
> size of the spare area (I didn't notice that one before). Did you try
> what happens if you set this to 0x70 for 224 bytes oob?
Which register is that?
> Optimally this would result in the last 6 bytes being written but not
> protected by hardware ecc.
Last 6 bytes of what? AFAIK, hardware ECC does not protect oob at all.
> > > Moreover the comment is misleading as it
> > > only applies to 4K flashes. At least the driver works well with (2ki +
> > > 128) bytes pages (while there are only 64 bytes spare used?? Maybe there
> > > are still more bugs?).
> >
> > I suspect there are more bugs. A simple way to trigger the bug I encountered
> > is by doing a sub-page write, that is:
> >
> > dd if=/dev/zero bs=1024 count=1 of=/dev/mtd2
> > dd if=/dev/zero bs=1024 count=1 seek=1 of=/dev/mtd2
> >
> > and then try reading this page. You may need to dirty the (useless; random)
> > content of ecccalc to see the bug (you can conveniently use
> > mxc_nand_calculate_ecc() for that). What copy_spare() currently do (spread ECC
> > chunks evenly over oob) does not match nandv2_hw_eccoob_largepage.
> I didn't follow, but would be willing to review a fix :-)
OK. I'll try something.
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
More information about the linux-mtd
mailing list