Bug 499386
Summary: | libvirt's qemu driver interprets the <hostdev> "managed" attribute incorrectly | |||
---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Mark McLoughlin <markmc> | |
Component: | libvirt | Assignee: | Mark McLoughlin <markmc> | |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | |
Severity: | medium | Docs Contact: | ||
Priority: | medium | |||
Version: | rawhide | CC: | berrange, clalance, crobinso, itamar, veillard, virt-maint | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | All | |||
OS: | Linux | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 499577 (view as bug list) | Environment: | ||
Last Closed: | 2009-05-06 21:37:37 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 480594 |
Description
Mark McLoughlin
2009-05-06 13:46:47 UTC
Okay, it turns out "managed='yes'" is supposed to detach/reset/reattach the device, not "managed='no'" The fix should be as simple as: @@ -1215,7 +1215,7 @@ static int qemuPrepareHostDevices(virConnectPtr conn, if (hostdev->source.subsys.type != VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI) continue; - if (!hostdev->managed) { + if (hostdev->managed) { pciDevice *dev = pciGetDevice(conn, hostdev->source.subsys.u.pci.domain, hostdev->source.subsys.u.pci.bus, Patch posted to libvir-list: http://www.redhat.com/archives/libvir-list/2009-May/msg00111.html Being built into libvirt-0.6.2-6.fc11 atm. Tag request: https://fedorahosted.org/rel-eng/ticket/1748 * Wed May 6 2009 Mark McLoughlin <markmc> - 0.6.2-5.fc11 - Fix handling of <hostdev managed='yes'> (bug #499386) Tagged now |