Without this option, autotest will set a "cpu_model" config option by
itself.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Instead of just defining rhel64 and qemu13 and enabling both by default,
define rhel64, qemu13, and unknown_qemu.

unknown_qemu will be enabled by default, so we don't test for things
that may fail on non-RHEL-6 or non-qemu-1.3 hosts. This should make the
tests work by default on most hosts, until we implement some kind of
host-based autoconfiguration system.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

cpuid.custom will fail if qemu is built without kvm,
due to virt-test tries to start qemu in kvm mode.

To fix it run custom variants in TCG mode, since they do
not have anything kvm specific.
Signed-off-by: NIgor Mammedov <imammedo@redhat.com>

If config has only 'disable-kvm=yes' but not 'enable-kvm=no'
qemu will be started in kvm mode and with kvm disabled at
compile time it fails to launch.

Currently disable/enable-kvm config options and qemu supported
options are intermingled in a bunch of ifs, separate apples
and oranges in different bins and make logic straightforward:

* alias disable-kvm to enable-kvm param
* get right option set to disable/enable kvm on qemu/qemu-kvm
* set appropriate option depending on value of param enable-kvm
Signed-off-by: NIgor Mammedov <imammedo@redhat.com>

Make use of extract_cpu_models() on cpuid test.
One of the patches from ehabkost introducing
get_qemu_cpu_models would have to be rebased
according to the fact they were introduced
earlier by this patchset.

CC: Eduardo Habkost <ehabkost@redhat.com>
CC: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

When auto_cpu_model = yes, we'll try to find the best
CPU model to be used by qemu, the process being:

1) Find the host CPU model
2) Try to pass it to qemu

We used to ignore invalid CPU models passed to qemu
in qemu_vm code, but we had to change that to allow
negative CPU model testing. So the sanity check of
whether the CPU model is valid has to be made way
earlier in the code.

Therefore, introduce get_qemu_best_cpu_model, that
will find the host CPU model and then check this
value against the valid CPU models on qemu. In case
there's no such cpu model in qemu, chose a sensible
default value (default_cpu_model if it is defined
in the params dict, or 'qemu64').

The new functions introduced to utils_misc are:
 * get_host_cpu_models -> Rename of get_cpu_model
 * get_qemu_cpu_models
 * extract_qemu_cpu_models
 * get_qemu_best_cpu_model

Some of the functions were extracted from the
cpuid test. In the next patch, we make the
test to use the new functions.

Changes from v1:
 * Refactor code to get qemu cpu models to its
   own function on utils_misc
 * Make the code actually work, by making
   involved functions to return lists

CC: Eduardo Habkost <ehabkost@redhat.com>
CC: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Last line said that No newline at end of file.
When we create subtests.cfg in libvirt project,
the end of virsh_save will like this:
(omit)
save_libvirtd = "off"    - virsh_setmem:
(omit)

Signed-off-by: Yu Mingfei<yumingfei@cn.fujitsu.com>

Hi Chris,Lucas:
I found that in virsh_managedsave.py, the way to get VM's id or uuid is
not very exacting,
so I changed it to use libvirt_vm's function.
Signed-off-by: NLi Yang <liyang.fnst@cn.fujitsu.com>

The command being run is a shell command, so running it as "-cpu ?"
without any quotes around "?" will make the shell expand "?" to the list
of filenames with one letter in the current directory.
Let's fix this by properly escaping the question mark.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

The existence of additional CPU models is not a bug, and should not
cause a test failure.

This changes the code to generate a INFO message in the log file, but
not cause the test to fail.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

We're having trouble to make both RHEL 6 and Fedora 18
koji/brew based installs work with the same cleanup
settings. So, allow to pass a cartesian param
installer_cleanup = yes/no to override the default
parameters being passed to the install function.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Yet another place that didn't have its location updated
according to the new organization of the symlinks and
data files.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

In order to distinguish better between a legitimate
TestFail (the test did fail) and a TestError
(unexpected condition, test bug or misconfiguration),
print a different ERROR state when error.TestError
is raised.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Autotest upstream renamed one particular API:

global_config -> settings

In order to make it cleaner. As we are still in the process
to make autotest 0.15, let's handle gracefully both
libraries installed in the system. This way, we can resume
regression testing for the next branches of both projects.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

The guest might not get ip address sometimes, thus let's record
tcpdump output as well, so that we can debug network problem.
Signed-off-by: NQingtang Zhou <qzhou@redhat.com>

The list of CPU models is the same on RHEL-6 and QEMU-1.3.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

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

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

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

The duplication of settings makes it very hard to spot mistakes. This is
a new version of the config file that uses nesting and cartesian
products of variants to avoid duplication.

To make it easier to test wheter the changes are working, this generates
exactly the same dictionaries as the original config, in exactly the
same sequence, including all the bugs. Look for the #FIXME comments for
places that will be cleaned up or fixed later.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * s/TestFail/TestNAError/
    * s/custom_model_id/custom.model_id/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * s/TestFail/TestNAError/
    * s/custom_xlevel/custom.xlevel/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
   * s/TestFail/TestNAError/
   * s/custom_stepping/custom.stepping/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
v2:
   * s/TestFail/TestNAError/
   * s/custom_model/custom.model/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * s/TestFail/TestNAError/
    * s/custom_family/custom.family/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * s/TestFail/TestNAError/
    * s/custom_level/custom.level/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * s/TestFail/TestNAError/
    * s/custom_vendor/custom.vendor/ in config

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
v2:
    * remove rhelcpu models form qemu13 test
    * if vendor is missing use  host's vendor
    * remove no needed cpu_models list from kvm test
    * replace cpu_models with shorter ignore_cpu_models in rhel64.kvm.cpuid.default.vendor
    * fix typo
    * s/default_vendor/default.vendor/ in config so we could use
      cpuid.default selector for all default tests

default.vendor fix

... and test utilities to parse its output

CPUID leaves are capped by reported level, xlevel, x2level values.
Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
---
  v3:
    * dump CPUID(EAX=3), CPUID(EAX=4,ECX=3), CPUID(EAX={0xB, 0xC})
    * dump CPUID(EAX=0x80000007) and known paravirt cpuid leaves if
      detected
    * make target name specific to function cpuid_dump_kernel.bin,
      so we could later add another targets for other tests
  v2:
    * log cpuid dump on parsing failure
    * cap by reported level, xlevel, x2level

Signed-off-by: NIgor Mammedov <imammedo@redhat.com>
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>
---
v2:
  * rebased on top of tree where there is no subtests.cfg.example
  * s/kvm/qemu/ in pathnames and test config
  * Change some errors types to a more suitable ones

Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NChris Evich <cevich@redhat.com>

Signed-off-by: NChris Evich <cevich@redhat.com>

* data_dir.SubdirList class returns a flat-list of all non-hidden
  subdirectories beneath a base directory (inclusive).

* data_dir.SubdirGlobList class returns flat-list of all files
  matching a glob pattern, within all non-hidden subdirectories
  beneath a base directory (inclusive).
Signed-off-by: NChris Evich <cevich@redhat.com>

Iterate over all allowed cpu_models and in case none is supported,
force the last one.

This should suit negative testing and also it should be safer when
using auto_cpu_model.
Signed-off-by: NLukáš Doktor <ldoktor@redhat.com>

Currently, it will rase and error when '/dev/sdaa' match the pattern
of the blacklist. This should not be happen as we just want sda sda1
and so on to be in the blacklist.

add one more check to avoid this error.

Signed-off-by: Mike Qiu<qiudayu@linux.vnet.ibm.com>

Currently, autotest use such as /dev/sda.* to get the disk of
the blacklist, but this will raise an error if the disk 'sdaa'
or some name like this, it's not correct as we just want to add
sda or name like sda1-9 to the blacklist.

change the pattern to /dev/sda[\d]* can avoid this.

Signed-off-by: Mike Qiu<qiudayu@linux.vnet.ibm.com>

Currently, function sorted inside the python was not suitable for
storage, for example, if ls -1d /dev/sd* gets result like this
after split('\n'):
['/dev/sda', '/dev/sda1', '/dev/sda2', '/dev/sda3',
 '/dev/sdaa', '/dev/sdab','/dev/sdac', '/dev/sdad', '/dev/sdae',
 '/dev/sdaf', '/dev/sdag', '/dev/sdah', '/dev/sdai', '/dev/sdaj',
 '/dev/sdak', '/dev/sdal', '/dev/sdam', '/dev/sdan', '/dev/sdao',
 '/dev/sdap', '/dev/sdaq', '/dev/sdar', '/dev/sdas', '/dev/sdb',
 '/dev/sdb1', '/dev/sdb2', '/dev/sdb3', '/dev/sdc', '/dev/sdd',
 '/dev/sde', '/dev/sdf', '/dev/sdg', '/dev/sdg1', '/dev/sdg2',
 '/dev/sdg3', '/dev/sdh', '/dev/sdh1', '/dev/sdh2', '/dev/sdh3',
 '/dev/sdi', '/dev/sdj', '/dev/sdk', '/dev/sdl', '/dev/sdm',
 '/dev/sdn', '/dev/sdo', '/dev/sdp', '/dev/sdq', '/dev/sdr',
 '/dev/sds', '/dev/sdt', '/dev/sdu', '/dev/sdv', '/dev/sdw',
 '/dev/sdx', '/dev/sdy', '/dev/sdz']

it will make no any change if just use sorted. As the default,
it will choose last 9 disk:
['/dev/sdr', '/dev/sds', '/dev/sdt', '/dev/sdu', '/dev/sdv', '/dev/sdw',
 '/dev/sdx', '/dev/sdy', '/dev/sdz']

Sometimes this is very dangerous, for lots of them may be the disk contains
lots of important data. Actually, maybe you just use scsi_debug just create:
['/dev/sdak','/dev/sdal', '/dev/sdam', '/dev/sdan', '/dev/sdao',
 '/dev/sdap', '/dev/sdaq', '/dev/sdar', '/dev/sdas']

But select the wrong disks as mentioned above.

To solve this problem, add param cmp to sorted and get the right order.
Signed-off-by: NMike Qiu <qiudayu@linux.vnet.ibm.com>

maxcpus is a sub-option of -smp, so need a Regex to match
Signed-off-by: NXiaoqing Wei <xwei@redhat.com>