Messages in this thread | | | From | Bean Huo 霍斌斌 (beanhuo) <> | Subject | RE: [PATCH 0/9] drivers:mtd:UBI: add bakvol module for MLC NAND paired page issue | Date | Thu, 8 Oct 2015 00:49:10 +0000 |
| |
> > > 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@nod.at> > > > > > > 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
| |