]> code.ossystems Code Review - openembedded-core.git/commitdiff
oeqa/qemurunner: Improve handling of run_serial for shutdown commands
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Sun, 9 May 2021 09:59:44 +0000 (10:59 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sun, 9 May 2021 22:23:23 +0000 (23:23 +0100)
When running a shutdown command, the serial port can close without the
command returning. This is seen as the socket being readable but having
no data. Change the way this case is handled in the code to avoid
tracebacks.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/lib/oeqa/utils/qemurunner.py

index 899152c7adcdfd43faa28e3168bc35e7785c8f21..4f094e29c73fd54d86e38f398c40a02ca6749e04 100644 (file)
@@ -73,6 +73,8 @@ class QemuRunner:
         self.monitorpipe = None
 
         self.logger = logger
+        # Whether we're expecting an exit and should show related errors
+        self.canexit = False
 
         # Enable testing other OS's
         # Set commands for target communication, and default to Linux ALWAYS
@@ -543,6 +545,7 @@ class QemuRunner:
             self.thread.join()
 
     def allowexit(self):
+        self.canexit = True
         if self.thread:
             self.thread.allowexit()
 
@@ -604,7 +607,9 @@ class QemuRunner:
                     if re.search(self.boot_patterns['search_cmd_finished'], data):
                         break
                 else:
-                    raise Exception("No data on serial console socket")
+                    if self.canexit:
+                        return (1, "")
+                    raise Exception("No data on serial console socket, connection closed?")
 
         if data:
             if raw: