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

Bean Huo 霍斌斌 (beanhuo)
2015-10-08T02:11:53Z

[-- Attachment #1: Type: text/plain; charset="gb2312", Size: 2011 bytes --]

> Am 30.09.2015 um 08:55 schrieb Richard Weinberger:
> >> By the way, Do you review my patches series ? I don't backup duplicated
> data in OOB .
> >> Can you specify which sector codes ? so that I can explain it in detail.
> >
> > Okay. Maybe both Boris and I misread your code, can you please explain
> > in detail what functions like check_original_data() or
> mtd_write_dual_plane_oob() do?
>
> Care to explain?
> Maybe we can find a solution without OOB.
> Boris proposed already some very promising ideas:
> http://comments.gmane.org/gmane.linux.drivers.mtd/61658

> BTW: Will you be at Embedded Linuxcon Europe in Dublin?
> Many MTD guys, including me, will be there.
> Maybe we can discuss the problem in a nice pub, first round is on me. :-)

Richard, thanks for your care about this patch. We are all very happy to do this job.
I think ,any comments or suggests welcome. we now in China, so maybe it is not convenient for us
to discuss the problem in a nice pub. can we create a topic or main project about this issue, then we can
work on it together. It is easy to trace and review.

This year, we ever attended Embedded Linux conference that held at San Jose Marriott,
and met Brain, Brain. We had a short discussion about paired page issue.
One of concerns is patent, you know , there are so many patents on how to solve paired page issue.
Linux seems that not very like to one accept patch based on certain patent.

I want to say, my this series patch is also based on a Micron patent, this is also one of my key concerns.
I don't know, how do you think about this.

For my this patch, another key concern is that memory overhead, maybe you have found that
Reserved 20 PEBs for backup log volume, but these 20 PEBs will be soon quickly used up, because of dual plane program
Page address special requirement.

> Thanks,
> //richard