siggen: Remove fsync that is causing performance issues
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Sat, 18 Jan 2014 23:23:02 +0000 (23:23 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sat, 18 Jan 2014 23:23:05 +0000 (23:23 +0000)
commit46fd841319479f6079d850b3813e64bd8c2680a3
treefac11dafafa70035ffe4462cd1b6c262f33dc223
parent2ca15117e4bbda38cda07511d0ff317273f91528
siggen: Remove fsync that is causing performance issues

This fsync was added for belt and braces protection for things like sstate
on NFS. To be honest, it probably doesn't buy much, if the rename isn't
atomic, all bets are off anyway and there are bigger issues to worry about.

The issue is that at the end of every task, the dump_sig() code is triggered
to save out information about the task and this was triggering an fsync(fd).

Whilst it may select the file descriptor, on file systems like ext4, it will
require large parts of the journal to be written out so it can have
significant impact. latencytop showed an average fsync() call overhead of about
2s and if that happens for 5000 tasks, the time mounts up. This blocks the next
task execution by that time.

We therefore drop the fsync since in reality its causing problems and is unlikely
to buy much.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
lib/bb/siggen.py