Martin Polednik has posted comments on this change.
Change subject: vdsm: add support for pci host device passthrough
......................................................................
Patch Set 20:
(2 comments)
http://gerrit.ovirt.org/#/c/22462/20/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1286: m.setAttrs(model=self.specParams['model'])
Line 1287: return m
Line 1288:
Line 1289:
Line 1290: class AcquiredHostDevice(VmDevice):
Why not just HostDevice? name clash with hostdev?
Kind of, this
is a bit of "devices are actually just XML issue" where hostdev is a bit more
than just XML
Line 1291: __slots__ = ('name', 'startupPolicy',
'_hostdevice', '_vm')
Line 1292:
Line 1293: def __init__(self, *args, **kwargs):
Line 1294: super(AcquiredHostDevice, self).__init__(*args, **kwargs)
Line 1324: </hostdev>
Line 1325: """
Line 1326:
Line 1327: if not self._hostdevice:
Line 1328: return
When could this happen?
if acquire failed and startupPolicy is
optional
Line 1329:
Line 1330: hostdev = self.createXmlElem(self.device, self.type)
Line 1331: hostdev.setAttrs(managed='no', mode='subsystem')
Line 1332: source = hostdev.appendChildWithArgs('source')
--
To view, visit
http://gerrit.ovirt.org/22462
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I363d2622d72ca2db75f60032fe0892c348bab121
Gerrit-PatchSet: 20
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes