The pci_assignable type name is nic_vf/nic_pf in kvm_utils.py whereas in kvm_vm.py they are vf/pf. Weird that why I tested it pass last week. Hope that it will not bring in any trouble before this patch applied.
Signed-off-by: Yolkfull Chow <yz...@redhat.com> --- client/tests/kvm/kvm_utils.py | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/client/tests/kvm/kvm_utils.py b/client/tests/kvm/kvm_utils.py index 83aff66..df26a77 100644 --- a/client/tests/kvm/kvm_utils.py +++ b/client/tests/kvm/kvm_utils.py @@ -971,12 +971,12 @@ class PciAssignable(object): Request PCI assignable devices on host. It will check whether to request PF (physical Functions) or VF (Virtual Functions). """ - def __init__(self, type="nic_vf", driver=None, driver_option=None, + def __init__(self, type="vf", driver=None, driver_option=None, names=None, devices_requested=None): """ Initialize parameter 'type' which could be: - nic_vf: Virtual Functions - nic_pf: Physical Function (actual hardware) + vf: Virtual Functions + pf: Physical Function (actual hardware) mixed: Both includes VFs and PFs If pass through Physical NIC cards, we need to specify which devices @@ -1087,9 +1087,9 @@ class PciAssignable(object): @param count: count number of PCI devices needed for pass through @return: a list of all devices' PCI IDs """ - if self.type == "nic_vf": + if self.type == "vf": vf_ids = self.get_vf_devs() - elif self.type == "nic_pf": + elif self.type == "pf": vf_ids = self.get_pf_devs() elif self.type == "mixed": vf_ids = self.get_vf_devs() -- 1.5.5.6 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html