Qingqing Zhou wrote: > > "Bruce Momjian" <[EMAIL PROTECTED]> wrote > > ----------- > > Have test_fsync honor -f filename argument. > > > > The patch may forget to change the error message (Also, it would be better > to append a '\n' in die() function). Another possible problem of this test > is that it pickups the "/var/tmp". OS may have IO optimization on temporary > directories, which will affect the confidence of the results. > > Example: > --- > $#./test_fsync -f $HOME/test.out > Simple write timing: > write 4.265149 > $#./test_fsync -f /var/tmp/test.out > Simple write timing: > write 0.018404 > $#./test_fsync -f /tmp/test.out > Simple write timing: > write 0.018511 > ---
Thanks. Updated error message and added mention of memory file systems to README. -- Bruce Momjian | http://candle.pha.pa.us [email protected] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073 ---------------------------(end of broadcast)--------------------------- TIP 1: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
