[PATCH 03/10] nandwrite: consolidate buffer usage

Brian Norris computersforpeace at gmail.com
Mon Sep 19 14:50:14 EDT 2011


On Sat, Sep 17, 2011 at 7:53 PM, Mike Frysinger <vapier.adi at gmail.com> wrote:
> if the useless memcpy is removed by way of future patches, then this is fine

It's not completely removed. Please see patch 07.

I'll summarize: the memcpy is moved out of the main code path so that
if ioctl(MEMWRITE) is supported in your kernel, you have no extra
memcpy's. However, I didn't optimize the original code, which uses the
standard write+ioctl(MEMWRITEOOB[64]); it still has the useless memcpy
that we are discussing.

I'm not sure if/how your solution can work embedded at that level of
libmtd. Perhaps some functions I moved around need to be restructured
a bit.

Brian



More information about the linux-mtd mailing list