From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Wang Subject: [PATCH 8/9] KVM test: Create the background threads before calling process() Date: Mon, 26 Apr 2010 18:04:26 +0800 Message-ID: <20100426100426.26268.93216.stgit@localhost.localdomain> References: <20100426095656.26268.50549.stgit@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: autotest@test.kernel.org, lmr@redhat.com Return-path: In-Reply-To: <20100426095656.26268.50549.stgit@localhost.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: autotest-bounces@test.kernel.org Errors-To: autotest-bounces@test.kernel.org List-Id: kvm.vger.kernel.org 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 --- 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):