If the screendump and scrialdump threads are created after the
process(), we may lose the progress tracking of guest shutting
down. So this patch creates them before calling process() in preprocess.

Signed-off-by: Jason Wang <[email protected]>
---
 client/tests/kvm/kvm_preprocessing.py |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/client/tests/kvm/kvm_preprocessing.py 
b/client/tests/kvm/kvm_preprocessing.py
index 50d0e35..73e835a 100644
--- a/client/tests/kvm/kvm_preprocessing.py
+++ b/client/tests/kvm/kvm_preprocessing.py
@@ -257,9 +257,6 @@ def preprocess(test, params, env):
                         int(params.get("pre_command_timeout", "600")),
                         params.get("pre_command_noncritical") == "yes")
 
-    # Preprocess all VMs and images
-    process(test, params, env, preprocess_image, preprocess_vm)
-
     # Start the screendump thread
     if params.get("take_regular_screendumps") == "yes":
         logging.debug("Starting screendump thread")
@@ -278,6 +275,8 @@ def preprocess(test, params, env):
                                               args=(test, params, env))
         _serialdump_thread.start()
 
+    # Preprocess all VMs and images
+    process(test, params, env, preprocess_image, preprocess_vm)
 
 
 def postprocess(test, params, env):

_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to