Re: [PATCH 0/4] Couple of nodeinfo enhancements

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

 



On 06/17/2014 02:48 PM, Michal Privoznik wrote:
> Look at the patches :P
> 

:P

> Michal Privoznik (4):
>   nodeinfo: Introduce @arch to linuxNodeInfoCPUPopulate
>   virNodeParseNode: Propagate host architecture
>   nodeinfotest: Print error if cpuinfo file can't be opened
>   virNodeParseSocket: Take ARM into account
> 
>  cfg.mk                                             |   2 +-
>  src/nodeinfo.c                                     | 199 +++++++++++----------
>  src/nodeinfopriv.h                                 |   1 +
>  .../nodeinfodata/linux-armv6l-raspberrypi.cpuinfo  |  12 ++
>  .../nodeinfodata/linux-armv6l-raspberrypi.expected |   1 +
>  .../linux-raspberrypi/cpu/cpu0/topology/core_id    |   1 +
>  .../cpu/cpu0/topology/core_siblings                |   1 +
>  .../cpu/cpu0/topology/core_siblings_list           |   1 +
>  .../cpu/cpu0/topology/physical_package_id          |   1 +
>  .../cpu/cpu0/topology/thread_siblings              |   1 +
>  .../cpu/cpu0/topology/thread_siblings_list         |   1 +
>  .../linux-raspberrypi/cpu/cpuidle/current_driver   |   1 +
>  .../cpu/cpuidle/current_governor_ro                |   1 +
>  .../nodeinfodata/linux-raspberrypi/cpu/kernel_max  |   1 +
>  tests/nodeinfodata/linux-raspberrypi/cpu/offline   |   1 +
>  tests/nodeinfodata/linux-raspberrypi/cpu/online    |   1 +
>  tests/nodeinfodata/linux-raspberrypi/cpu/possible  |   1 +
>  tests/nodeinfodata/linux-raspberrypi/cpu/present   |   1 +
>  ...86-test1.cpuinfo => linux-x86_64-test1.cpuinfo} |   0
>  ...-test1.expected => linux-x86_64-test1.expected} |   0
>  ...86-test2.cpuinfo => linux-x86_64-test2.cpuinfo} |   0
>  ...-test2.expected => linux-x86_64-test2.expected} |   0
>  ...86-test3.cpuinfo => linux-x86_64-test3.cpuinfo} |   0
>  ...-test3.expected => linux-x86_64-test3.expected} |   0
>  ...86-test4.cpuinfo => linux-x86_64-test4.cpuinfo} |   0
>  ...-test4.expected => linux-x86_64-test4.expected} |   0
>  ...86-test5.cpuinfo => linux-x86_64-test5.cpuinfo} |   0
>  ...-test5.expected => linux-x86_64-test5.expected} |   0
>  ...86-test6.cpuinfo => linux-x86_64-test6.cpuinfo} |   0
>  ...-test6.expected => linux-x86_64-test6.expected} |   0
>  ...86-test7.cpuinfo => linux-x86_64-test7.cpuinfo} |   0
>  ...-test7.expected => linux-x86_64-test7.expected} |   0
>  ...86-test8.cpuinfo => linux-x86_64-test8.cpuinfo} |   0
>  ...-test8.expected => linux-x86_64-test8.expected} |   0
>  tests/nodeinfotest.c                               |  63 ++++---
>  35 files changed, 164 insertions(+), 127 deletions(-)
>  create mode 100644 tests/nodeinfodata/linux-armv6l-raspberrypi.cpuinfo
>  create mode 100644 tests/nodeinfodata/linux-armv6l-raspberrypi.expected
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/core_id
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/core_siblings
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/core_siblings_list
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/physical_package_id
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/thread_siblings
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpu0/topology/thread_siblings_list
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpuidle/current_driver
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/cpuidle/current_governor_ro
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/kernel_max
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/offline
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/online
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/possible
>  create mode 100644 tests/nodeinfodata/linux-raspberrypi/cpu/present
>  rename tests/nodeinfodata/{linux-x86-test1.cpuinfo => linux-x86_64-test1.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test1.expected => linux-x86_64-test1.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test2.cpuinfo => linux-x86_64-test2.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test2.expected => linux-x86_64-test2.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test3.cpuinfo => linux-x86_64-test3.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test3.expected => linux-x86_64-test3.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test4.cpuinfo => linux-x86_64-test4.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test4.expected => linux-x86_64-test4.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test5.cpuinfo => linux-x86_64-test5.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test5.expected => linux-x86_64-test5.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test6.cpuinfo => linux-x86_64-test6.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test6.expected => linux-x86_64-test6.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test7.cpuinfo => linux-x86_64-test7.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test7.expected => linux-x86_64-test7.expected} (100%)
>  rename tests/nodeinfodata/{linux-x86-test8.cpuinfo => linux-x86_64-test8.cpuinfo} (100%)
>  rename tests/nodeinfodata/{linux-x86-test8.expected => linux-x86_64-test8.expected} (100%)
> 

Fails syntax-check for me:
tests/nodeinfodata/linux-armv6l-raspberrypi.cpuinfo:3:Features        : swp
half thumb fastmult vfp edsp java tls
maint.mk: found trailing blank(s)

ACK series with that fixed

Also, defining and using macros like:
# define ARCH_IS_X86(arch)  ((arch) == VIR_ARCH_X86_64 ||\
                             (arch) == VIR_ARCH_I686)

for PPC, S390 and ARM could make the code more readable.

Jan

Attachment: signature.asc
Description: OpenPGP digital signature

--
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]