We compute rpkt_per_exit by tx_pkts, and compute tpkt_per_exit
by rx_pkts. It's wrong.
Signed-off-by: NAmos Kong <akong@redhat.com>

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

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

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

qemu.tests: correct test step

Fix for next branch - Update5

for this test we need to compare images when vm is in paused status;
Signed-off-by: NXu Tian <xutian@redhat.com>

Add a useless parameter in device_add_block().

These two function called by following code:
        try:
            # get function for adding device.
            add_fuction = local_functions["%s_%s" % (cmd_type, pci_type)]
        except Exception:
            raise error.TestError("No function for adding '%s' dev with '%s'" %
                                  (pci_type, cmd_type))
        after_add = None
        if add_fuction:
            # Do add pci device.
            after_add = add_fuction(pci_num, queues)

So they must have same parameters.
Now we meet error:
TypeError: device_add_block() takes exactly 1 argument (2 given)
Signed-off-by: NFeng Yang <fyang@redhat.com>

After some updating, current code could not get correct qemu binary.
So Case error with message "TestNAError: Host qemu does not support qmp".
Signed-off-by: NFeng Yang <fyang@redhat.com>

We had a bunch of pull requests that were generated incorrectly
against master, and were pushed, since it is way too easy to
miss that fact when pressing the nice little green buttons that
github offers. So we have to do this to avoid non ff updates to
next and even more confusion.
Signed-off-by: NLucas Meneghel Rodrigues <lmr@redhat.com>

libvirt-test: Backup virsh_change_media xml and fix virsh_edit.

virttest.virsh: Add virsh migrate setspeed/getspeed functions

Svirt install

Signed-off-by: NDongsheng Yang <yangds.fnst@cn.fujitsu.com>

Signed-off-by: Nyangdongsheng <yangds.fnst@cn.fujitsu.com>

Signed-off-by: NDongsheng Yang <yangds.fnst@cn.fujitsu.com>

qemu.test.ipi_x2apic: Typo fix str -> check_string.

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

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

migrate_setspeed(domain, bandwidth, extra=None, **dargs)
migrate_getspeed(domain, **dargs)

* In RHEL-6, vcpucount --config option must be used with --maximum
  or --active option, using only --config option with cause error:

  > virsh vcpucount virt-tests-vm1 --config
  error: when using --config, either --maximum or --active must
  be specified

* The vm need to backup with '--inactive' option to avoid xml
  inconsistence.

* No need to disable pylint E1111

* Fix some pylint/pep8 problem.
Signed-off-by: NHao Liu <hliu@redhat.com>

* In new_from_dumpxml(), the used keyword argument 'options'
  doesn't exist in virsh.dumpxml(). This must be a typo
  s/options/extra.

* Fix a raise typo and some pylint/pep8 issues.

* The original xml of the vm need to be backed up before test
  and restored after it to keep the xml unchanged.

* When backing up, the xml need to be dumped using '--inactive'
  option. Otherwise the xml backup is slightly different from
  the original one like below. I'm not sure this will cause
  any problem, but it's annoying.

---
+++
@@ -4,6 +4,9 @@
   <memory unit='KiB'>1048576</memory>
   <currentMemory unit='KiB'>1048576</currentMemory>
   <vcpu placement='static'>2</vcpu>
+  <resource>
+    <partition>/machine</partition>
+  </resource>
   <os>
     <type arch='x86_64' machine='pc-i440fx-rhel7.0.0'>hvm</type>
     <boot dev='hd'/>
@@ -43,7 +46,9 @@
     </console>
     <input type='tablet' bus='usb'/>
     <input type='mouse' bus='ps2'/>
-    <graphics type='vnc' port='-1' autoport='yes'/>
+    <graphics type='vnc' port='-1' autoport='yes'
listen='127.0.0.1'>
+      <listen type='address' address='127.0.0.1'/>
+    </graphics>
     <video>
       <model type='cirrus' vram='9216' heads='1'/>
       <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/>
@@ -52,4 +57,5 @@
       <address type='pci' domain='0x0000' bus='0x00' slot='0x05' function='0x0'/>
     </memballoon>
   </devices>
+  <seclabel type='dynamic' model='selinux' relabel='yes'/>
 </domain>
Signed-off-by: NHao Liu <hliu@redhat.com>

qemu.tests: new case to check guest clock offset

Add lvm support for qemu tests

virttest.qemu_vm: Log information about the parent PID and the monitor n...

When starting the qemu process, there is not much information about
the running parent PID (the autotest shell script that starts qemu) or
the monitor name that we are trying to connect (for example, hmp1).

So with this commit we're logging this information when qemu is started,
in order to provide information for debugging. Example:

When running  ./run -t qemu --tests=uptime --qemu_sandbox=on

From debug.log:

11:43:17 INFO | Created qemu process with parent PID 15592
11:43:17 INFO | Connecting to monitor 'hmp1'
11:44:17 WARNI| Could not find (qemu) prompt after connecting to monitor. Output so far: ''
11:44:18 ERROR| Could not connect to monitor 'hmp1'
11:44:18 DEBUG| Destroying VM virt-tests-vm1 (PID 15594)

Parent PID is 15592, now we are trying to find what is wrong...

ps -ef | grep 15592
rmoura   15592 15591  0 11:43 pts/3    00:00:00 /bin/bash -c source /tmp/autotestbMRLRL.sh
rmoura   15594 15592  0 11:43 pts/3    00:00:00 [qemu-system-x86] <defunct>

In this situation, it's clear that qemu-system-x86_64 goes bad.
Signed-off-by: NRuda Moura <rmoura@redhat.com>

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

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

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

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

register lvm object to env file when use lvm storage, and restore it at
postprocess to clean up new volumes when "force_remove_image = yes";

CPUID config file updates

The SVM feature leaf is known to change between versions because it was
(incorrectly) non-zero even if SVM was disabled.
Signed-off-by: NEduardo Habkost <ehabkost@redhat.com>

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

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

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

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

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

check guest clock drift less that $acceptable_offset after online/offline
hotpluged vcpus;
Signed-off-by: NXu Tian <xutian@redhat.com>

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