From: Michal Privoznik <mprivozn@redhat.com>
Date: Thu, 7 Jan 2016 10:42:59 +0000 (+0100)
Subject: virt-host-validate-common: Print warning on missing IOMMU
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=6e2770174e8f0eaebd04c35996c1a2ab0c202a7c;p=libvirt.git

virt-host-validate-common: Print warning on missing IOMMU

No only coverity warns about this, but it kind of makes sense
too. We have a test whether host supports IOMMU. Some platforms
don't have it, I know. But in that case we should print a message
that it's unknown whether platform has it or not.

Before:
  (no output)

After:
  QEMU: Checking for device assignment IOMMU support                         : WARN (Unknown if this platform has IOMMU support)

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---

diff --git a/tools/virt-host-validate-common.c b/tools/virt-host-validate-common.c
index dd1c414f96..8ebf73eb1f 100644
--- a/tools/virt-host-validate-common.c
+++ b/tools/virt-host-validate-common.c
@@ -367,9 +367,6 @@ int virHostValidateIOMMU(const char *hvname,
         isIntel = true;
     else if (virHostValidateHasCPUFlag("svm"))
         isAMD = true;
-    else
-        /* XXX PPC/ARM/etc support */
-        return 0;
 
     virHostMsgCheck(hvname, "%s", _("for device assignment IOMMU support"));