From bbcce58e824b2793abf50efa52db158ae16e23e4 Mon Sep 17 00:00:00 2001 From: Richard Purdie Date: Mon, 21 Aug 2017 21:26:52 +0100 Subject: process: Ensure we call select() to know which fds to read There is an interesting bug in the current code where a sync command is not seen until the current async command completes, by which time the UI may have shut down. The reason is that if there are idle commands, we may not end up sleeping in the select call at all, partiularly under heavy load like parsing. Fix this by calling select with a zero timeout so that we see active fds and know to read from them. This fixes various problems toaster was having with the recent server changes. [YOCTO #11898] Signed-off-by: Richard Purdie --- lib/bb/server/process.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/bb/server/process.py b/lib/bb/server/process.py index 2ebfdfc7e..3ab793c77 100644 --- a/lib/bb/server/process.py +++ b/lib/bb/server/process.py @@ -312,7 +312,7 @@ class ProcessServer(multiprocessing.Process): # Ignore EINTR return [] else: - return [] + return select.select(fds,[],[],0)[0] class ServerCommunicator(): -- cgit 1.2.3-korg