Change in vdsm[master]: Revert "hostdev/sr-iov: don't reattach devices"

danken at redhat.com danken at redhat.com
Mon Apr 11 15:48:25 UTC 2016


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 <danken at 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 <danken at redhat.com>


More information about the vdsm-patches mailing list