Re: [PATCH 7/9] nodeinfo: Add sysfs_prefix to nodeCapsInitNUMA

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

 



On Tue, 2015-07-07 at 20:26 -0400, John Ferlan wrote:
> Add the sysfs_prefix argument to the call to allow for setting the
> path for tests to something other than SYSFS_CPU_PATH which is a
> derivative of SYSFS_SYSTEM_PATH
> 
> Use cpupath for nodeCapsInitNUMAFake and remove SYSFS_CPU_PATH
> 
> Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>

Just one remark below.

> @@ -1810,26 +1811,27 @@ nodeGetMemoryFake(unsigned long long *mem,
>  
>  /* returns 1 on success, 0 if the detection failed and -1 on hard 
> error */
>  static int
> -virNodeCapsFillCPUInfo(int cpu_id ATTRIBUTE_UNUSED,
> +virNodeCapsFillCPUInfo(const char *cpupath,
> +                       int cpu_id ATTRIBUTE_UNUSED,
>                         virCapsHostNUMACellCPUPtr cpu 
> ATTRIBUTE_UNUSED)

cpupath should be marked ATTRIBUTE_UNUSED as well.

Cheers.

-- 
Andrea Bolognani
Software Engineer - Virtualization Team

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]