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

From: Andi Kleen (ak@suse.de)
Date: Sat Nov 16 2002 - 05:56:52 EST


You don't seem to save/restore the FPU state, so it will be likely
corrupted after your copy runs.

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 ), 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.

-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