RE: [PATCH 0/9] drivers:mtd:UBI: add bakvol module for MLC NAND paired page issue

[Bean Huo éææ (beanhuo)]

> > > As stated before, using OOB in UBI is not going to happen unless
> > > proven that there is absolutely no other way to solve the paired pages
> problem.
> > >
> > > Nacked-by: Richard Weinberger <richard@xxxxxx>
> > >
> > > Sorry,
> > > //Richard
> >
> > Hi, Richard
> > Thanks for your concern. I am a new patch submitter.
> > Can you tell me Nacked-by means?
> >
> > By the way, Do you review my patches series ? I don't backup duplicated
> >data in OOB .
> 
> That's not what Richard said, he just pointed that you were using the OOB
> area, and you're actually using it to store the information about which page
> you're backuping.
> 
> > Can you specify which sector codes ? so that I can explain it in detail.
> 
> And as answered by Richard, check_original_data() and
> mtd_write_dual_plane_oob() are filling the OOB buf with the original page
> offset, so you're definitely using the OOB area to store metadata.

Yes ,currently I only use 4 bytes OOB area to store backup page address.

> --
> Boris Brezillon, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
N‹§²æ¸›yú²X¬¶ÇvØ–)Þ{.nlj·¥Š{±‘êX§¶›¡Ü}©ž²ÆzÚj:+v‰¨¾«‘êZ+€Êzf£¢·hšˆ§~†­†Ûÿû®w¥¢¸?™¨è&¢)ßf”ùy§m…á«a¶Úÿ0¶ìå