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