From 7dce0fb208cd26be903428908186ea16d12d00bb Mon Sep 17 00:00:00 2001
From: Marc Bonnici <marc.bonnici@arm.com>
Date: Fri, 12 Oct 2018 13:07:47 +0100
Subject: [PATCH] workloads/jankbench: Ensure logcat monitor thread is
 terminated

Previously the LogcatRunMonitor left the logcat process running in the
background causing issues with concurrent accesses. Now ensure the thread
terminates correctly.
---
 wa/workloads/jankbench/__init__.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/wa/workloads/jankbench/__init__.py b/wa/workloads/jankbench/__init__.py
index d4c60cef..16eeb187 100644
--- a/wa/workloads/jankbench/__init__.py
+++ b/wa/workloads/jankbench/__init__.py
@@ -227,6 +227,7 @@ class JankbenchRunMonitor(threading.Thread):
                         line = line.decode(sys.stdout.encoding, 'replace')
                     if self.regex.search(line):
                         self.run_ended.set()
+        proc.terminate()
 
     def stop(self):
         self.stop_event.set()