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>

Using the program codespell

https://github.com/lucasdemarchi/codespell

We found a number of spelling mistakes, that could be
found and fixed. The result of the automated script
was double checked, of course.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

fix not all disk listed issue, get cdrom drive letter with
utils_misc.get_winutils_vol();
Signed-off-by: NXu Tian <xutian@redhat.com>

As an active NetworkManager may give us problems during
testing, stop the daemon before bonding. add a random wait
time before and after set_link.
Signed-off-by: NYunping Zheng <yunzheng@redhat.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

Add missed test scenario '1 0', AKA "indirect_desc on, event_idx off" test;
Signed-off-by: NXu Tian <xutian@redhat.com>

The bacground process was likely to be killed by SIGIO while setting
port to async mode. This was due of bad workflow, where first the
port was set to async mode, than the the new SIGIO handler was set.

With this version when we set async mode, we fist add the SIGIO
handler and then set the port to async mode.

When we are setting the port back, the order is the opposite, so
first the port is set to sync mode and then we remove the SIGIO
handler.

Additionally the workaround with test_delete_guest_script() was
removed as it's fixed by this patch.

NOTE: You have to remove the previous /tmp/virtio_console_guest.py*
      in order to let virtio_console test to copy this updated one!
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Signed-off-by: NFeng Yang <fyang@redhat.com>

Boot up a guest and malloc memory in host to force it to move
among the nodes of host.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Boot up a guest have the same number of node as host. Each node in guest
only have 1 cpu. Try to malloc inside the guest with taksset. The memory
allocated in host should also in the same node as the cpu thread.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Get host numa topology. Start and bind guest to the cpu in each node
one by one. Then check the memory status for qemu process.
The qemu process should mainly used the memory from the node that it
is binded to.
Signed-off-by: NYiqiao Pu <ypu@redhat.com>

Signed-off-by: NQingtang Zhou <zhouqt@gmail.com>

Signed-off-by: NQingtang Zhou <zhouqt@gmail.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

Signed-off-by: NQingtang Zhou <zhouqt@gmail.com>

Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Add some new variants/rules that will run some tests using
additional cpu_model_flags values. Especially:

 * CPU models that require "+apic" to be run on RHEL-6 hosts on Intel
   CPUs
 * CPU models that require "-abm,-sse4a" on Intel CPUs
 * CPU models that require using "-svm" because we don't enable nesting
   by default
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

We don't have any dumps for nokvm mode, so there's no point in enabling
all those variants that won't run because of missing dump files.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

This will allow us to filter/document cases that require a specific host
CPU vendor. Enable only host_cpu_vendor=unknown by default.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

This way we can make filters that include/exclude all RHEL-6 versions.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Add extra CPU model flags to some CPU models when the full CPU model
can't be tested easily.

Note that the code loads the reference files using the CPU flags as part
of the file name too (so we can be sure we're reading the right dump
file)  This way we can be sure we are comparing the output of "-cpu
Foo,-flag" against a dump that was surely generated using "-cpu
Foo,-flag" as well (not just "-cpu Foo").
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

Use cpu_model_flags when running the test and reading CPUID dumps.

In some cases it will be useful to check if dumps are OK on specific
cases where flags are removed (e.g. removing the "svm" flag when running
without -enable-nesting). This change will allow dumps for those special
cases to be used if cpu_model_flags is configured on qemu_cpu.cfg.
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

Those bits were changed on RHEL-6.1 without any compat code, so we need
to ignore the differences we find.
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

The RHEL-6 qemu-kvm binary prints that message when SVM is not available
and can't be used. Treat this case as TestNAError too, as it means the
test can't be run, not a test failure.
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

The CPUID dump test kernel doesn't need USB devices.

This allows some older QEMU versions to be tested.
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

Simplify the config file a little bit.
Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

Signed-off-by: NEduardo Habkost <ehabkost@raisama.net>

Guest has to be prepared for running with qlx
driver. Drivers have to be properly installed
before starting of the test. Otherwise the test
could FAIL.
tests steps:
    1) starts windows XP guest on first host
    2) try login to guest
    3) starts presubtest steps with
       stepsfile = WinXP-set-VGA-mode.steps
       enable VGA mode on GPU
    4) after presubtest finished starts migration
       to host2
    5) after migration starts postsubtests with
       stepfile = WinXP-reset-VGA-mode.steps
Signed-off-by: NJiří Župka <jzupka@redhat.com>

Allow test start pre_sub_test before migration.
Signed-off-by: NJiří Župka <jzupka@redhat.com>

steps:
  allow start steps test with other VM than main_vm.

multi_host migration:
  makes timeouts little longer due to slow win tests.
  add options to skip logging to vm.
Signed-off-by: NJiří Župka <jzupka@redhat.com>

Signed-off-by: NJiří Župka <jzupka@redhat.com>

Signed-off-by: NXiaoqing Wei <xwei@redhat.com>

This reverts commit c080994ddf01914b0bb0162cbe97df9fef75adf3.

Use Monitor.info_block() which covers all versions of "info block"
command to get the information.

Also update the cases which use this function to provide info_block
output instead the Monitor.info('block') one.

Last but not least add unittest.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>