bitbake: server/process: Disable the flush() call in server logging

We've been chasing bitbake timeouts for a while and it was unclear where things
were blocking on IO. It appears the flush() call in server logging can cause
pauses up to minutes long on systems with slow (spinning) disks that are heavily
loaded with IO.

Since the flush() was added to aid debugging of other timing issues, we shouldn't
need it now and it can be disabled. Leave a comment as a reminder of the pain this
can cause.

(Bitbake rev: afbc169e1490a86d6250969f780062c426eb4682)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Richard Purdie 2023-09-14 17:26:07 +01:00
parent 37c31a5adc
commit 022deeb0ef

View File

@ -43,7 +43,8 @@ def currenttime():
def serverlog(msg):
print(str(os.getpid()) + " " + currenttime() + " " + msg)
sys.stdout.flush()
#Seems a flush here triggers filesytem sync like behaviour and long hangs in the server
#sys.stdout.flush()
#
# When we have lockfile issues, try and find infomation about which process is