[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Don't fail qemu driver intialization if we can't determine hugepage size



On Mon, Aug 11, 2014 at 11:17:10AM +0200, Michal Privoznik wrote:
> On 10.08.2014 13:51, Guido Günther wrote:
> >Otherwise we fail like
> >
> >   libvirt version: 1.2.7, package: 6 (root 2014-08-08-16:09:22 bogon)
> >   virAuditOpen:62 : Unable to initialize audit layer: Protocol not supported
> >   virFileGetDefaultHugepageSize:2958 : internal error: Unable to parse /proc/meminfo
> >   virStateInitialize:749 : Initialization of QEMU state driver failed: internal error: Unable to parse /proc/meminfo
> >   daemonRunStateInit:922 : Driver state initialization failed
> >
> >if the data can't be determined.
> >
> >Reference: http://bugs.debian.org/757609
> >---
> >  src/util/virfile.c | 5 +++--
> >  1 file changed, 3 insertions(+), 2 deletions(-)
> >
> >diff --git a/src/util/virfile.c b/src/util/virfile.c
> >index f9efc65..b6f5e3f 100644
> >--- a/src/util/virfile.c
> >+++ b/src/util/virfile.c
> >@@ -2953,8 +2953,9 @@ virFileGetDefaultHugepageSize(unsigned long long *size)
> >          goto cleanup;
> >
> >      if (!(c = strstr(meminfo, HUGEPAGESIZE_STR))) {
> >-        virReportError(VIR_ERR_INTERNAL_ERROR,
> >-                       _("Unable to parse %s"),
> >+        virReportError(VIR_ERR_NO_SUPPORT,
> >+                       _("%s not found in %s"),
> >+                       HUGEPAGESIZE_STR,
> >                         PROC_MEMINFO);
> >          goto cleanup;
> >      }
> >
> 
> This merely changes the error code and error message. But the initialization
> will fail anyway. Well, it would up till d26e81083. But the error message
> you suggests is more verbose and describes the origin of fault more
> accurately.
> 
> ACK

Pushed now. Thanks.
 -- Guido


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]