[libvirt] [PATCH v2] vz: allow to start vz driver without host cache info

Martin Kletzander mkletzan at redhat.com
Tue Jul 11 09:22:19 UTC 2017


On Tue, Jul 11, 2017 at 04:59:05AM -0400, Mikhail Feoktistov wrote:
>Show warning message instead of fail operation.
>It happens if kernel or cpu doesn't support reporting cpu cache info.
>In case of Virtuozzo file "id" doesn't exist.

What is your kernel version?

Is there another way of getting the information about the cache ID?
Maybe we need to parse the name of the cache directory 'index2' would be
id 2 maybe?  If there is no other way, then this fix is fine (as most
drivers do the same thing), but I would rather fix it if that's
possible.  Unfortunately the cache information is structured stupidly
compared to other kernel-provided topology-related information.

>---
> src/vz/vz_driver.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
>index 6f4aee3..eb97e54 100644
>--- a/src/vz/vz_driver.c
>+++ b/src/vz/vz_driver.c
>@@ -119,7 +119,7 @@ vzBuildCapabilities(void)
>         goto error;
>
>     if (virCapabilitiesInitCaches(caps) < 0)
>-        goto error;
>+        VIR_WARN("Failed to get host CPU cache info");
>
>     verify(ARRAY_CARDINALITY(archs) == ARRAY_CARDINALITY(emulators));
>
>--
>1.8.3.1
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170711/2a82e901/attachment-0001.sig>


More information about the libvir-list mailing list