Alex Tomas wrote:

quoted

quoted

quoted

quoted

quoted

Badari Pulavarty (BP) writes:

 BP> Test: writes 10,000 blocks of 64k and does fdatasync().

The patch doesn't apply ;) after a minor correction
I've tested the patch too:

...

UP, after:
[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m21.452s user    0m0.026s sys     0m6.105s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m23.317s user    0m0.027s sys     0m4.206s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m25.190s user    0m0.022s sys     0m4.026s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m25.024s user    0m0.021s sys     0m4.103s


UP, before:
[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m19.138s user    0m0.021s sys     0m5.980s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m23.123s user    0m0.026s sys     0m4.083s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m25.152s user    0m0.017s sys     0m3.836s

[root@zefir root]# time /work/tests/fwrite /test/fff 64 10000
real    0m25.217s user    0m0.026s sys     0m3.827s

Hmm.. On UP, the patch made sys.time little worse ?

- Badari