sanitycheck: remove usage of qemu for generic handlers
We have now different runners/handlers, so avoid using qemu terminology
for the generic classes and for generic usage.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>
diff --git a/scripts/sanitycheck b/scripts/sanitycheck
index 2d4bbfd..7f71601 100755
--- a/scripts/sanitycheck
+++ b/scripts/sanitycheck
@@ -632,15 +632,14 @@
defconfigs) which is why MakeGoal is a separate class from TestInstance.
"""
- def __init__(self, name, text, qemu, make_log, build_log, run_log,
- qemu_log):
+ def __init__(self, name, text, handler, make_log, build_log, run_log, handler_log):
self.name = name
self.text = text
- self.qemu = qemu
+ self.handler = handler
self.make_log = make_log
self.build_log = build_log
self.run_log = run_log
- self.qemu_log = qemu_log
+ self.handler_log = handler_log
self.make_state = "waiting"
self.failed = False
self.finished = False
@@ -658,8 +657,8 @@
# Failure in sub-make for "make run", qemu probably failed to start
return self.run_log
elif self.make_state == "finished":
- # QEMU finished, but timed out or otherwise wasn't successful
- return self.qemu_log
+ # Handler finished, but timed out or otherwise wasn't successful
+ return self.handler_log
def fail(self, reason):
self.failed = True
@@ -828,8 +827,8 @@
run_logfile = os.path.join(outdir, "run.log")
qemu_logfile = os.path.join(outdir, "qemu.log")
- qemu = QEMUHandler(name, outdir, qemu_logfile, timeout)
- args.append("QEMU_PIPE=%s" % q.get_fifo())
+ qemu_handler = QEMUHandler(name, outdir, qemu_logfile, timeout)
+ args.append("QEMU_PIPE=%s" % qemu_handler.get_fifo())
text = (self._get_rule_header(name) +
self._get_sub_make(name, "building", directory,
outdir, build_logfile, args) +
@@ -837,7 +836,7 @@
outdir, run_logfile,
args, make_args="run") +
self._get_rule_footer(name))
- self.goals[name] = MakeGoal(name, text, qemu, self.logfile, build_logfile,
+ self.goals[name] = MakeGoal(name, text, qemu_handler, self.logfile, build_logfile,
run_logfile, qemu_logfile)
def add_unit_goal(self, name, directory, outdir,
@@ -854,14 +853,13 @@
self._get_sub_make(name, "building", directory,
outdir, build_logfile, args) +
self._get_rule_footer(name))
- unit = UnitHandler(name, directory, outdir, run_logfile,
- valgrind_logfile, timeout)
- self.goals[name] = MakeGoal(name, text, unit, self.logfile, build_logfile,
+ unit_handler = UnitHandler(name, directory, outdir, run_logfile, valgrind_logfile, timeout)
+ self.goals[name] = MakeGoal(name, text, unit_handler, self.logfile, build_logfile,
run_logfile, valgrind_logfile)
- def add_test_instance(
- self, ti, build_only=False, enable_slow=False, coverage=False,
- extra_args=[]):
+
+ def add_test_instance(self, ti, build_only=False, enable_slow=False, coverage=False,
+ extra_args=[]):
"""Add a goal to build/test a TestInstance object
@param ti TestInstance object to build. The status dictionary returned
@@ -933,21 +931,21 @@
# will cause the 'make run' invocation to exit with
# nonzero status.
# Need to distinguish this case from a compilation failure.
- if goal.qemu:
+ if goal.handler:
goal.fail("qemu_crash")
else:
goal.fail("build_error")
else:
if state == "finished":
- if goal.qemu:
- if goal.qemu.unit:
+ if goal.handler:
+ if goal.handler.unit:
# We can't run unit tests with Make
- goal.qemu.handle()
- if goal.qemu.returncode == 2:
- goal.qemu_log = goal.qemu.valgrind_log
- elif goal.qemu.returncode:
- goal.qemu_log = goal.qemu.run_log
- thread_status, metrics = goal.qemu.get_state()
+ goal.handler.handle()
+ if goal.handler.returncode == 2:
+ goal.handler_log = goal.handler.valgrind_log
+ elif goal.handler.returncode:
+ goal.handler_log = goal.handler.run_log
+ thread_status, metrics = goal.handler.get_state()
goal.metrics.update(metrics)
if thread_status == "passed":
goal.success()
@@ -1833,8 +1831,8 @@
i.platform.name, i.test.name):
eleTestsuite.remove(tc)
- if not goal.failed and goal.qemu:
- handler_time = "%s" % (goal.metrics["handler_time"])
+ if not goal.failed and goal.handler:
+ handler_time = "%s" %(goal.metrics["handler_time"])
eleTestcase = ET.SubElement(
eleTestsuite, 'testcase', classname="%s:%s" %
@@ -1885,7 +1883,7 @@
rowdict["status"] = goal.reason
else:
rowdict["passed"] = True
- if goal.qemu:
+ if goal.handler:
rowdict["handler_time"] = goal.metrics["handler_time"]
rowdict["ram_size"] = goal.metrics["ram_size"]
rowdict["rom_size"] = goal.metrics["rom_size"]
