sanitycheck: Fix incorrect QEMUHandler status reporting.

The current QEMUHandler implementation in sanitycheck does not check
for the process exit code and reports "PASS" even when either the QEMU
executable cannot be launched or exited immediately due to an error
(e.g. unsupported machine type, missing file, ...).

This commit adds QEMU process exit code check and error reporting when
the exit code is a non-zero value.

Signed-off-by: Stephanos Ioannidis <[email protected]>

Author: stephanosio

scripts/sanitycheck

@@ -2003,7 +2003,7 @@ class ProjectBuilder(FilterBuilder):
                 "test": self.instance,
                 "state": "executed",
                 "status": self.instance.status,
-                "reason": self.instance.status}
+                "reason": self.instance.reason}
                 )
 
         # Report results and output progress to screen
@@ -2126,6 +2126,10 @@ class ProjectBuilder(FilterBuilder):
                 proc.wait()
                 self.returncode = proc.returncode
 
+            if self.returncode != 0:
+                instance.handler.set_state("failed", 0)
+                instance.reason = "Exited with {}".format(self.returncode)
+
         sys.stdout.flush()