On 09/12/2013 05:17 PM, Xu Tian wrote:
>
> In qemu/tests/pci_hotplug.py:
>
> > @@ -313,3 +313,8 @@ def _device_removed():
> >                  error.context(context_msg % (sub_type, "after hotunplug"),
> >                                logging.info)
> >                  utils_test.run_virt_sub_test(test, params, env, sub_type)
> > +
> > +    if params.get("reboot_vm", "no") == "yes":
> > +        vm = env.get_vm(params["main_vm"])
> > +        vm.verify_alive()
>
> @ypu <https://github.com/ypu> for step info, vm.reboot always print vm 
> reboot info to terminal and logging file, so do we still need to 
> re-recording it?
>

Add steps to the docsting in the file.

def run_pci_hotplug(test, params, env):
     """
     Test hotplug of PCI devices.

     (Elements between [] are configurable test parameters)
     1) PCI add one/multi device (NIC / block) with(or without) repeat
     2) Compare output of monitor command 'info pci'.
     3) Compare output of guest command [reference_cmd].
     4) Verify whether pci_model is shown in [pci_find_cmd].
     5) Check whether the newly added PCI device works fine.
     6) PCI delete the device, verify whether could remove the PCI device.


> Thanks,
> Xu
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/autotest/virt-test/pull/842/files#r6316971>.
>