[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