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>

use live_snapshot function function in test framework, drop useless
module and update test configuration;
Signed-off-by: NXu Tian <xutian@redhat.com>

We do not want to fail the case for exception in clean up command.
Signed-off-by: NFeng Yang <fyang@redhat.com>

Add catch_serial_cmd for virtio_scsi, ide block device;
Signed-off-by: NXu Tian <xutian@redhat.com>

In some places of our code, we're calling utils.system_output
with a parameter verbose, introduced in autotest 0.15. Therefore,
for people using the Fedora/RHEL shipped RPMs, a bug will
happen since the API on their RPMs doesn't know about the verbose
param.

Therefore, remove such calls and get the equivalent construction
using the utils.run API, that does support the parameter.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

1). update pre_command: used scsi_debug parm to add_host instead write
sys filesystem to avoid device create delay;

2). fix passthrough scsi device cache to writethrough to avoid fail to
open /dev/sg* when drive_cache=none; do this because /dev/sg* is
emulated by scsI_debug module, backend of these device is memery, open
with O_DRIECT is not supported;

3). removed useless import module 'qemu_monitor'
Signed-off-by: NXu Tian <xutian@redhat.com>

I don't know why 486 was being excluded, as the CPUID instruction really
works, and QEMU has CPUID data being set for 486, too. The tests keep
working after removing the "no 486" line from the config file.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>