Signed-off-by: NXu Han <xuhan@redhat.com>

Clean up imports across the board in tp-qemu. The criteria
followed was:

1) Remove unnecessary imports
2) Turn multi imports from a single entry point to one
   import per line. This increases both patch and
   source code readability.
3) Separate blocks of imports with a line:
   * Python standard library imports
   * Autotest imports
   * virttest imports
   * Internal tp-qemu imports
4) Remove try/except blocks refering autotest libs and
   stick with the virttest.staging copy

This paves the way for the future removal of autotest
references, replacing those references with avocado ones.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

There's no need to have the name of the "run_<name>" method (main
test entry point) depend on the test name. So, for simplicity sake,
let's just name the test entry point "run".

I have decided to not fallback gracefully to the previous naming
convention as I believe this can lead to confusion in the future.
Signed-off-by: NCleber Rosa <crosa@redhat.com>

Sometimes we may test qemu-kvm built by ourself not installed one.
At this time 'rpm -q qemu-kvm' could not return correct kvm userspace version.
Signed-off-by: NFeng Yang <fyang@redhat.com>

Make virt-test to have a docstring style mostly compliant with
the sphinx's [1] format, reStructured text. This means that
we have to trade all '@' tags to be ':', mostly.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

With the help of the PEP8 script, a semi-automated clean
of the code base was made, and effectively changing some
of the coding style rules of our code base:

 * No more 2 lines spacing among functions
 * No more combined module imports

In the end, it was going to be too difficult to take
into consideration E501 (line length > 80) errors and
W601 (.has_key() is deprecated), so these were ignored,
at least for now.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

impove the case in the scripts:
- The result file is named by nfs-perf.RHS
- Put the result into test.resultsdir
- Collect version info in result
- Format the result which can be used by regression.py

Change from v1:
- result file name is be set as a parameter
- replace parameter ver_cmd by kvm_userspace_ver_cmd

Change from v2:
- get_path to get the file path
- format the result which can be used by regression.py

Change from v3:
- modify speed result without unit
Signed-off-by: NXiaomei Gao <xigao@redhat.com>
Acked-by: NYiqiao Pu <ypu@redhat.com>

The original regex string for dd output goes wrong when
the dd speed is slow:
"""
2012-09-12 01:03:45: dd if=/dev/zero of=test_0zSfi bs=4k oflag=direct count=10000
2012-09-12 01:05:09: 10000+0 records in
2012-09-12 01:05:09: 10000+0 records out
2012-09-12 01:05:09: 40960000 bytes (41 MB) copied, 84.0859 s, 487 kB/s
"""

This patch update the regex string, making it can recognize more
speed format.

Changes from V3:
 * Fix incorrect placement of speed result convert line.
Changes from V2:
 * Update this patch against latest nfs_perf test script.
 * Use new utils_misc.standard_value api to convert result data.
Changelog from V1:
 * Convert all results in kB/s to MB/s.

CC: Wenli Quan <wquan@redhat.com>
CC: Xiaomei Gao <xigao@redhat.com>
CC: Yiqiao Pu <ypu@redhat.com>
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>
Acked-by: NWenli Quan <wquan@redhat.com>

Save the write and read speed in database for nfs_perf.
It will be like this in database:
  nfs--$blk_size--[write|read]: $speed
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Changes from v2:
 - Clean up temp files after testing.

Changes from v1:
 - Update mount option.
 - Create a new file for each block size testing.
 - Drop caches before running each test.
 - Re-format test results file.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>