aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRichard Purdie <richard.purdie@linuxfoundation.org>2023-09-14 17:26:07 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2023-09-18 11:34:31 +0100
commitafbc169e1490a86d6250969f780062c426eb4682 (patch)
treec402dd2ef0ca9d2db52156af25806f4034b7f906
parentda3ec3801bdb80180b3f1ac24edb27a698415ff7 (diff)
downloadbitbake-contrib-afbc169e1490a86d6250969f780062c426eb4682.tar.gz
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. Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r--lib/bb/server/process.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/bb/server/process.py b/lib/bb/server/process.py
index 993ae6627..5654f60f9 100644
--- a/lib/bb/server/process.py
+++ b/lib/bb/server/process.py
@@ -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