Dan Kenigsberg has uploaded a new change for review. Change subject: Revert "hostdev/sr-iov: don't reattach devices" ......................................................................
Revert "hostdev/sr-iov: don't reattach devices" This reverts commit e80ab7fc74b7715. SR-IOV feature depends on automatic re-binding of VFs to their network driver. Without that, the VFs are no longer recognized as available host NICs after a VM using VFs is taken down, or after hotunplug of a VFs. Change-Id: Ia19d4a761023f2ca9fa3f7274fed97159bed1fae Bug-Url: https://bugzilla.redhat.com/1234328 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm/virt/vm.py 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/93/55993/1 diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py index ffac65c..08a20be 100644 --- a/vdsm/virt/vm.py +++ b/vdsm/virt/vm.py @@ -2337,6 +2337,8 @@ try: self._dom.detachDevice(nicXml) self._waitForDeviceRemoval(nic) + if nic.is_hostdevice: + hostdev.reattach_detachable(nic.hostdev) except HotunplugTimeout as e: self.log.error("%s", e) self._rollback_nic_hotunplug(nicDev, nic) -- To view, visit https://gerrit.ovirt.org/55993 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ia19d4a761023f2ca9fa3f7274fed97159bed1fae Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches