exec: remove warning about mempath and hugetlbfs
authorDaniel P. Berrange <berrange@redhat.com>
Wed, 25 Nov 2015 10:52:29 +0000 (10:52 +0000)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 26 Nov 2015 15:47:44 +0000 (16:47 +0100)
The gethugepagesize() method in exec.c printed a warning if
the file path for "-mem-path" or "-object memory-backend-file"
was not on a hugetlbfs filesystem. This warning is bogus, because
QEMU functions perfectly well with the path on a regular tmpfs
filesystem. Use of hugetlbfs vs tmpfs is a choice for the management
application or end user to make as best fits their needs. As such it
is inappropriate for QEMU to have an opinion on whether the user's
choice is right or wrong in this case.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Message-Id: <1448448749-1332-3-git-send-email-berrange@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
exec.c

diff --git a/exec.c b/exec.c
index b09f18b2a4d9e063929486cc700fad8075784d25..de1cf191549fbf91cadc5703d1bfff31529c681f 100644 (file)
--- a/exec.c
+++ b/exec.c
@@ -1196,9 +1196,6 @@ static long gethugepagesize(const char *path, Error **errp)
         return 0;
     }
 
-    if (fs.f_type != HUGETLBFS_MAGIC)
-        fprintf(stderr, "Warning: path not on HugeTLBFS: %s\n", path);
-
     return fs.f_bsize;
 }