Re: [CFT][PATCH] 2.5.47 Athlon/Druon, much faster copy_user function

From: Akira Tsukamoto (at541@columbia.edu)
Date: Sat Nov 16 2002 - 13:22:51 EST


On Sat, 16 Nov 2002 11:56:52 +0100
Andi Kleen <ak@suse.de> mentioned:
>
> You don't seem to save/restore the FPU state, so it will be likely
> corrupted after your copy runs.

This is the main question for me that I was wondering for all week.
My first version was using fsave and frstore, so
just changing three lines will accomplish this.
Is it all I need? Any thing elase needed to consider using fpu register?
>
> Also I'm pretty sure that using movntq (= forcing destination out of
> cache) is not a good strategy for generic copy_from_user(). It may
> be a win for the copies in write ( user space -> page cache ),

Yes, that why I included postfetch in the code because movntq does not leave
them in the L2 cache.
Anygood idea to
> but
> will hurt for all the ioctls and other things that actually need the
> data in cache afterwards. I am afraid it is not enough to do micro benchmarks
> here.

check above?

>
>
> -Andi

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Nov 23 2002 - 22:00:17 EST