From patchwork Tue Aug 2 03:39:02 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Amos Kong X-Patchwork-Id: 1027862 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter1.kernel.org (8.14.4/8.14.4) with ESMTP id p723Z7Q6000723 for ; Tue, 2 Aug 2011 03:35:08 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753196Ab1HBDeq (ORCPT ); Mon, 1 Aug 2011 23:34:46 -0400 Received: from mx1.redhat.com ([209.132.183.28]:50801 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753027Ab1HBDeq (ORCPT ); Mon, 1 Aug 2011 23:34:46 -0400 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p723YiNv024006 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Mon, 1 Aug 2011 23:34:44 -0400 Received: from [127.0.0.1] (dhcp-8-167.nay.redhat.com [10.66.8.167]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id p723YgsK027977; Mon, 1 Aug 2011 23:34:42 -0400 Subject: [PATCH] KVM-test: Add hdparm subtest To: autotest@test.kernel.org, lmr@redhat.com From: Amos Kong Cc: kvm@vger.kernel.org Date: Tue, 02 Aug 2011 11:39:02 +0800 Message-ID: <20110802033902.422.82801.stgit@t> User-Agent: StGit/0.15 MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.67 on 10.5.11.12 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter1.kernel.org [140.211.167.41]); Tue, 02 Aug 2011 03:35:21 +0000 (UTC) This test uses 'hdparm' to set disk device to low/high performance status, and compare the reading speed. The emulated device should pass all the tests. Signed-off-by: Feng Yang Signed-off-by: Amos Kong --- client/tests/kvm/tests/hdparm.py | 84 ++++++++++++++++++++++++++++++++ client/tests/kvm/tests_base.cfg.sample | 13 +++++ 2 files changed, 97 insertions(+), 0 deletions(-) create mode 100644 client/tests/kvm/tests/hdparm.py -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/client/tests/kvm/tests/hdparm.py b/client/tests/kvm/tests/hdparm.py new file mode 100644 index 0000000..79ce5db --- /dev/null +++ b/client/tests/kvm/tests/hdparm.py @@ -0,0 +1,84 @@ +import re, logging +from autotest_lib.client.common_lib import error + + +@error.context_aware +def run_hdparm(test, params, env): + """ + Test hdparm setting on linux guest os, this case will: + 1) Set/record parameters value of hard disk to low performance status. + 2) Perform device/cache read timings then record the results. + 3) Set/record parameters value of hard disk to high performance status. + 4) Perform device/cache read timings then compare two results. + + @param test: kvm test object + @param params: Dictionary with the test parameters + @param env: Dictionary with test environmen. + """ + + def check_setting_result(set_cmd, timeout): + params = re.findall("(-[a-zA-Z])([0-9]*)", set_cmd) + disk = re.findall("(\/+[a-z]*\/[a-z]*$)", set_cmd)[0] + for (param, value) in params: + cmd = "hdparm %s %s" % (param, disk) + (s, output) = session.cmd_status_output(cmd, timeout) + if s != 0: + raise error.TestError("Fail to get %s parameter value\n" + "Output is: %s" % (param, output)) + if value not in output: + raise error.TestFail("Fail to set %s parameter to value: %s" + % (param, value)) + + + def perform_read_timing(disk, timeout, num=5): + results = 0 + for i in range(num): + cmd = params.get("device_cache_read_cmd") % disk + (s, output) = session.cmd_status_output(cmd, timeout) + if s != 0: + raise error.TestFail("Fail to perform device/cache read" + " timings \nOutput is: %s\n" % output) + logging.info("Output of device/cache read timing check(%s of %s):" + " %s" % (i + 1, num, output)) + (result, post) = re.findall("= *([0-9]*.+[0-9]*) ([a-zA-Z]*)", + output)[1] + if post == "kB": + result = float(result)/1024.0 + results += float(result) + return results/num + + vm = env.get_vm(params["main_vm"]) + vm.create() + session = vm.wait_for_login(timeout=int(params.get("login_timeout", 360))) + try: + timeout = float(params.get("cmd_timeout", 60)) + cmd = params.get("get_disk_cmd") + (s, output) = session.cmd_status_output(cmd) + disk = output.strip() + + error.context("Setting hard disk to lower performance") + cmd = params.get("low_status_cmd") % disk + session.cmd(cmd, timeout) + + error.context("Checking hard disk keyval under lower performance") + check_setting_result(cmd, timeout) + low_result = perform_read_timing(disk, timeout) + logging.info("Buffered disk read speed under low performance" + " configuration: %s" % low_result) + error.context("Setting hard disk to higher performance") + cmd = params.get("high_status_cmd") % disk + session.cmd(cmd, timeout) + + error.context("Checking hard disk keyval under higher performance") + check_setting_result(cmd, timeout) + high_result = perform_read_timing(disk, timeout) + logging.info("Buffered disk read speed under high performance" + " configuration: %s" % high_result) + if not float(high_result) > float(low_result): + raise error.TestFail("High performance setting does not " + "increase read speed\n") + logging.debug("High performance setting increased read speed!") + + finally: + if session: + session.close() diff --git a/client/tests/kvm/tests_base.cfg.sample b/client/tests/kvm/tests_base.cfg.sample index d597b52..5491630 100644 --- a/client/tests/kvm/tests_base.cfg.sample +++ b/client/tests/kvm/tests_base.cfg.sample @@ -1115,6 +1115,19 @@ variants: image_snapshot = yes only Linux + - hdparm: + type = hdparm + get_disk_cmd = \ls /dev/[vhs]da + low_status_cmd = hdparm -a64 -d0 -u0 %s + device_cache_read_cmd = hdparm -tT %s + high_status_cmd = hdparm -a256 -d1 -u1 %s + cmd_timeout = 540 + only Linux + virtio_blk: + get_disk_cmd = \ls /dev/vda + low_status_cmd = hdparm -a32 -r0 %s + high_status_cmd = hdparm -a256 -r1 %s + # NICs variants: