Change in vdsm[master]: vdsm hostdev: add support for USB devices
fromani at redhat.com
fromani at redhat.com
Mon Jun 23 15:00:29 UTC 2014
Francesco Romani has posted comments on this change.
Change subject: vdsm hostdev: add support for USB devices
......................................................................
Patch Set 1:
(3 comments)
http://gerrit.ovirt.org/#/c/29054/1/vdsm/rpc/vdsmapi-schema.json
File vdsm/rpc/vdsmapi-schema.json:
Line 3160: #
Line 3161: # Possible policies for startup with device
Line 3162: #
Line 3163: # @mandatory: fail if missing for any reason (the default)
Line 3164: #
trailing whitespace
Line 3165: # @requisite: fail if missing on boot up, drop if missing
Line 3166: # on migrate/restore/revert
Line 3167: #
Line 3168: # @optional: drop if missing at any start attempt
Line 3164: #
Line 3165: # @requisite: fail if missing on boot up, drop if missing
Line 3166: # on migrate/restore/revert
Line 3167: #
Line 3168: # @optional: drop if missing at any start attempt
ditto
Line 3169: #
Line 3170: # Since: 4.16.0
Line 3171: ##
Line 3172: {'enum': 'StartupPolicy', 'data': ['mandatory', 'requisite', 'optional']}
http://gerrit.ovirt.org/#/c/29054/1/vdsm/virt/vm.py
File vdsm/virt/vm.py:
Line 1706:
Line 1707: rom.setAttrs(**romAttrs)
Line 1708:
Line 1709: elif self.capability == 'usb_device':
Line 1710: addr = self.getPciAddr()
getUsbAddr maybe?
Line 1711: try:
Line 1712: source.appendChildWithArgs('vendor', None, addr['vendor'])
Line 1713: source.appendChildWithArgs('vendor', None, addr['product'])
Line 1714: except:
--
To view, visit http://gerrit.ovirt.org/29054
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac74e7537d56bcb940ef07a4654d45cbcdbb1fb0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list