Change in vdsm[master]: hostdev: add support for reporting pci addresses

mpolednik at redhat.com mpolednik at redhat.com
Thu Jan 22 13:46:55 UTC 2015


Martin Polednik has posted comments on this change.

Change subject: hostdev: add support for reporting pci addresses
......................................................................


Patch Set 10:

(2 comments)

http://gerrit.ovirt.org/#/c/35651/10//COMMIT_MSG
Commit Message:

Line 7: hostdev: add support for reporting pci addresses
Line 8: 
Line 9: This patch adds addr field into hostdev tree that holds map of
Line 10: address fields relevant to the device. This is required in order to
Line 11: avoid parsing the pci name (such as pci_0000_05_fe_1).
> looks OK, can you please point me to the patch on which you'll use this new
http://gerrit.ovirt.org/#/c/22462/24 (but bear in mind that it is outdated quite a bit), the address is required when constructing the xml "source" element)
Line 12: 
Line 13: Change-Id: I8208af6b662550de426d8d015421e0ab2c3c617e


http://gerrit.ovirt.org/#/c/35651/10/tests/hostdevTests.py
File tests/hostdevTests.py:

Line 309:                                         'capability': 'pci',
Line 310:                                         'address': {'slot': '27',
Line 311:                                                     'bus': '0',
Line 312:                                                     'domain': '0',
Line 313:                                                     'function': '0'}},
> It is most a matter of personal taste, so please consider it just an open p
I'm not sure how would namedtuple play along with {XML|JSON}RPC, since this is mostly data reported back to engine (will only be used in device xml generation in vdsm)
Line 314:                   u'scsi_0_0_0_0': {'capability': 'scsi',
Line 315:                                     'parent': 'scsi_target0_0_0'},
Line 316:                   u'pci_0000_00_1a_0': {'product': '6 Series/C200 Series '
Line 317:                                         'Chipset Family USB Enhanced Host '


-- 
To view, visit http://gerrit.ovirt.org/35651
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8208af6b662550de426d8d015421e0ab2c3c617e
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek 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