Change in vdsm[master]: hostdev: add hostdev device definition

mpolednik at redhat.com mpolednik at redhat.com
Wed Mar 4 15:13:44 UTC 2015


Martin Polednik has posted comments on this change.

Change subject: hostdev: add hostdev device definition
......................................................................


Patch Set 7: Verified+1

Verified by manual testing:

sent device: {'type': 'hostdev', 'device': 'pci_0000_05_10_1'}
vmList returns: {'device': 'pci_0000_05_10_1', 'alias': 'hostdev0', 'type': 'hostdev', 'address': {'slot': '0x04', 'bus': '0x00', 'domain': '0x0000', 'type': 'pci', 'function': '0x0'}}
dumpxml contains:     <hostdev mode='subsystem' type='pci' managed='no'>
      <driver name='vfio'/>
      <source>
        <address domain='0x0000' bus='0x05' slot='0x10' function='0x1'/>
      </source>
      <alias name='hostdev0'/>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x04' function='0x0'/>
    </hostdev>

vfio/udev rules worked as expected
*while running*
ls /dev/vfio/
22  vfio

ls /etc/udev/rules.d/
12-ovirt-iosched.rules  70-persistent-ipoib.rules  99-vdsm-iommu_group_22.rules

*post run*
ls /dev/vfio/
vfio

ls /etc/udev/rules.d/
12-ovirt-iosched.rules  70-persistent-ipoib.rules

-- 
To view, visit https://gerrit.ovirt.org/37700
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I51f6713cf76914d8626718071598421bc1731722
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak 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: No


More information about the vdsm-patches mailing list