Change in vdsm[master]: caps: Fix access to /sys/class/iommu

dkuznets at redhat.com dkuznets at redhat.com
Thu Mar 12 09:46:08 UTC 2015


Dima Kuznetsov has posted comments on this change.

Change subject: caps: Fix access to /sys/class/iommu
......................................................................


Patch Set 1:

As long as master is built for el6 (and we even have official nightlies) we should not make such assumptions in the code.
I haven't tested it on el7 but even if iommu class is present there is does not solve the problem for el6 which we currently support.

[root at host3 ~]# uname -a
Linux host3 2.6.32-504.8.1.el6.x86_64 #1 SMP Wed Jan 28 21:11:36 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
[root at host3 ~]# ls -a /sys/class/
.          bdi    cpuid  firmware  hwmon        infiniband_cm     iscsi_endpoint  iscsi_session    macvtap   misc  net            powercap      raw        scsi_device  tty     virtio-ports
..         block  dma    graphics  i2c-adapter  input             iscsi_host      iscsi_transport  mdio_bus  msr   pci_bus        power_supply  regulator  scsi_host    usbmon  vtconsole
backlight  bsg    dmi    hidraw    infiniband   iscsi_connection  iscsi_iface     leds             mem       mtd   pcmcia_socket  ppdev         rtc        thermal      vc      watchdog
[root at host3 ~]# ls -a /sys/class/iommu
ls: cannot access /sys/class/iommu: No such file or directory

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie11f10163984a8bb4cf2e27225d2e19d6cb2f9ae
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan 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