Change in vdsm[master]: hostdev: fix creation and deletion of multiple udev rules

danken at redhat.com danken at redhat.com
Thu Mar 12 10:36:43 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: hostdev: fix creation and deletion of multiple udev rules
......................................................................


hostdev: fix creation and deletion of multiple udev rules

When attaching multiple devices from the same group, each of the
devices needs to be detached and then later reattached to the host.
The udev rule creation is hooked into this routine to give qemu access to
the /dev/vfio endpoint. This patch corrects behaviour of udev if the file
already exists or does not exist due to creation or deletion of VM.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1099760
Change-Id: I83fffb2f6e42cc1746f30ca2b63c5d594de014e5
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/38208
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/hostdev.py
M vdsm/supervdsmServer
2 files changed, 39 insertions(+), 14 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Martin Polednik: Verified



-- 
To view, visit https://gerrit.ovirt.org/38208
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I83fffb2f6e42cc1746f30ca2b63c5d594de014e5
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: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak 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