[CFT:PATCH] Remove a seemingly unnecessary cache flush

Mikael Pettersson mikpe at it.uu.se
Mon Dec 14 07:15:29 EST 2009


Russell King - ARM Linux writes:
 > On Mon, Dec 14, 2009 at 12:50:11PM +0100, Mikael Pettersson wrote:
 > > Russell King - ARM Linux writes:
 > >  > On Mon, Dec 07, 2009 at 10:12:10PM +0000, Russell King - ARM Linux wrote:
 > >  > > While looking into the possible msync() issue (which turned out to be a
 > >  > > non-issue - it's covered by clever code), I found that we're flushing
 > >  > > the same page multiple times.
 > >  > 
 > >  > I'm going to queue this patch up today since I believe it is correct,
 > >  > though it would be nice to have a few more people test it on pre-ARMv6
 > >  > hardware.
 > > 
 > > Tested here with gcc and glibc builds and testsuite runs on IXP420,
 > > IOP80219, and Kirkwood (Feroceon?), all ARMv5TE. No signs of problems.
 > 
 > Do the builds appear to be any faster?

I didn't record how long it took because I was mainly interested in
the correctness aspect. I can do a pair of properly measured gcc
builds on the fastest machine next weekend.



More information about the linux-arm-kernel mailing list