---
 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@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

On Mon, Jul 10, 2017 at 07:54:03 -0400, Mikhail Feoktistov wrote:

Any justification or explanation when this happens?

> ---
>  src/vz/vz_driver.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Error occurs in src/conf/capabilities.c line 1736

int virCapabilitiesInitCaches(virCapsPtr caps)
...
    if (virFileReadValueUint(&bank->id,
"%s/cpu/cpu%zd/cache/%s/id",
                                              SYSFS_SYSTEM_PATH, pos, 
ent->d_name) < 0)
           goto cleanup;
...

file "id" doesn't exist

ls -al /sys/devices/system/cpu/cpu0/cache/index0/
total 0
drwxr-xr-x 2 root root    0 Jun 30 06:03 .
drwxr-xr-x 6 root root    0 Jun 29 04:42 ..
-r--r--r-- 1 root root 4096 Jul 10 06:44 coherency_line_size
-r--r--r-- 1 root root 4096 Jul  5 11:33 level
-r--r--r-- 1 root root 4096 Jul 10 06:44 number_of_sets
-r--r--r-- 1 root root 4096 Jul 10 06:44 physical_line_partition
-r--r--r-- 1 root root 4096 Jul 10 06:44 shared_cpu_list
-r--r--r-- 1 root root 4096 Jul 10 06:44 shared_cpu_map
-r--r--r-- 1 root root 4096 Jul 10 06:44 size
-r--r--r-- 1 root root 4096 Jul 10 06:44 type
-r--r--r-- 1 root root 4096 Jul 10 06:44 ways_of_associativity

On 10.07.2017 15:48, Peter Krempa wrote:
> On Mon, Jul 10, 2017 at 07:54:03 -0400, Mikhail Feoktistov wrote:
>
> Any justification or explanation when this happens?
>
>> ---
>>   src/vz/vz_driver.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

On Tue, Jul 11, 2017 at 10:41:15 +0300, Mikhail Feoktistov wrote:
> Error occurs in src/conf/capabilities.c line 1736
> 
> int virCapabilitiesInitCaches(virCapsPtr caps)
> ...
>    if (virFileReadValueUint(&bank->id,
> "%s/cpu/cpu%zd/cache/%s/id",
>                                              SYSFS_SYSTEM_PATH, pos,
> ent->d_name) < 0)
>           goto cleanup;
> ...
> 
> file "id" doesn't exist
> 
> ls -al /sys/devices/system/cpu/cpu0/cache/index0/
> total 0
> drwxr-xr-x 2 root root    0 Jun 30 06:03 .
> drwxr-xr-x 6 root root    0 Jun 29 04:42 ..
> -r--r--r-- 1 root root 4096 Jul 10 06:44 coherency_line_size
> -r--r--r-- 1 root root 4096 Jul  5 11:33 level
> -r--r--r-- 1 root root 4096 Jul 10 06:44 number_of_sets
> -r--r--r-- 1 root root 4096 Jul 10 06:44 physical_line_partition
> -r--r--r-- 1 root root 4096 Jul 10 06:44 shared_cpu_list
> -r--r--r-- 1 root root 4096 Jul 10 06:44 shared_cpu_map
> -r--r--r-- 1 root root 4096 Jul 10 06:44 size
> -r--r--r-- 1 root root 4096 Jul 10 06:44 type
> -r--r--r-- 1 root root 4096 Jul 10 06:44 ways_of_associativity

So your system (cpu/kernel) does not support reporting the cache data
required to populate the capabilities.

So the fix is correct, since other callers of virCapabilitiesInitCaches
ignore the error as well.

Please put the justification into the commit message as it's usually
done in commits to libvirt.

> On 10.07.2017 15:48, Peter Krempa wrote:
> > On Mon, Jul 10, 2017 at 07:54:03 -0400, Mikhail Feoktistov wrote:
> > 
> > Any justification or explanation when this happens?
> > 
> > > ---
> > >   src/vz/vz_driver.c | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list