Change in vdsm[master]: hostdev: destroy VM with hostdev even without IOMMU support
danken at redhat.com
danken at redhat.com
Thu Jun 4 12:37:52 UTC 2015
Dan Kenigsberg has submitted this change and it was merged.
Change subject: hostdev: destroy VM with hostdev even without IOMMU support
......................................................................
hostdev: destroy VM with hostdev even without IOMMU support
Such VM can occur if it is started via vdsClient. Any attempt to
destroy that VM would fail with exception without destroying the VM,
therefore leaving it hanging in vdsm. This patch logs if reattach of
hostdev without valid IOMMU configuration occurs and allows destroy to
finish.
Change-Id: I857ec18c0932de247469f6fb780d1b28aa3873e8
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/41886
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/hostdev.py
M vdsm/virt/vm.py
2 files changed, 9 insertions(+), 2 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
Martin Polednik: Verified
--
To view, visit https://gerrit.ovirt.org/41886
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I857ec18c0932de247469f6fb780d1b28aa3873e8
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan 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: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the vdsm-patches
mailing list