[PATCH v2 3/4] mtd: mxc_nand: fix truncate of unaligned oob copying

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Wed May 13 00:18:28 PDT 2015


On Wed, May 13, 2015 at 10:12:01AM +0300, Baruch Siach wrote:
> Hi Uwe,
> 
> On Wed, May 13, 2015 at 09:01:40AM +0200, Uwe Kleine-König wrote:
> > On Wed, May 13, 2015 at 09:59:22AM +0300, Baruch Siach wrote:
> > > On Wed, May 13, 2015 at 08:47:34AM +0200, Uwe Kleine-König wrote:
> > > > On Wed, May 13, 2015 at 09:44:04AM +0300, Baruch Siach wrote:
> > > > > > I thought we could get rid of the memcpy32 variants. Where do we need
> > > > > > memcpy32_* where memcpy16 wouldn't work?
> > > > > 
> > > > > memcpy16 should work, but would take twice as much IO/memory accesses. That 
> > > > > would definitely affect performance, as this is the flash data read/write hot 
> > > > > path. I didn't test, though.
> > > > > 
> > > > > Are you sure we want to do that?
> > > > no, I'm not sure. But I think it's worth to test how much performance
> > > > degrades.
> > > 
> > > That will have to wait a few weeks as I don't have the hardware handy at the 
> > > moment.
> > In the hope the test will not be forgotten I consider it ok to take this
> > series without the test (with keeping the memcpy32 that is).
> 
> Agreed.
> 
> I'll respin this series with the ecc_8bit_layout name change as you suggested.
> 
> May I have your Reviewed-by/Acked-by for the patches you are not the author 
> of?
Sure, you can add an Acked-by for patches 2 and 3 for your resent.  Note
that I prefer to have spaces around operators, too. That shouldn't be a
show stopper though.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-mtd mailing list