Change in vdsm[master]: hostdev: add dynamic udev rule creation for iommu groups

danken at redhat.com danken at redhat.com
Mon Feb 2 18:03:22 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: add dynamic udev rule creation for iommu groups
......................................................................


hostdev: add dynamic udev rule creation for iommu groups

Detaching a device from a host (via detach_detachable) spawns
a new vfio endpoint in /dev/vfio/$name, where $name = iommu group
number. In order to start a VM with this endpoint assigned (= vm
with device passed through) we need rw permissions for QEMU.

This patch adds dynamic udev rule creation/deletion for when device is
detached/reattached from or to the host.

Creating an udev rule will happen one time for each IOMMU group, that
is passed through to the host. That means we're expecting the call to
only happen on VM creation, where the number of such VMs is very limited
(possibly 1 or few).

Change-Id: Ieac8d58e01d7277e535a2101d522961816ea88eb
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/36268
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/hostdev.py
M vdsm/supervdsmServer
2 files changed, 46 insertions(+), 2 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve
  Martin Polednik: Verified



-- 
To view, visit http://gerrit.ovirt.org/36268
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieac8d58e01d7277e535a2101d522961816ea88eb
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list