Re: [PATCH v11 0/5] nodeinfo: Add support for subcores

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

 




On 07/30/2015 05:37 AM, Andrea Bolognani wrote:
> Only patch 1/5 has been updated, all the other patches
> are the same as v8:
> 
>   2/5 https://www.redhat.com/archives/libvir-list/2015-July/msg01045.html
>   3/5 https://www.redhat.com/archives/libvir-list/2015-July/msg01048.html
>   4/5 https://www.redhat.com/archives/libvir-list/2015-July/msg01049.html
>   5/5 https://www.redhat.com/archives/libvir-list/2015-July/msg01050.html
> 
> I'm including the full history below.
> 
> Cheers.
> 
> 
> Changes in v11:
> 
>   * don't declare variables only used inside a code block
>     guarded by #ifdef outside of said code block
> 
>   * use virBitmapNextSetBit() instead of going through
>     all possible index values and calling
>     virBitmapIsBitSet() for each one
> 
> Changes in v10:
> 
>   * don't attempt to close a file that wasn't opened
> 
>   * report a detailed error to help with diagnosis
> 
> Changes in v9:
> 
>   * take into account the fact that kvm might not be loaded
>     or even installed
> 
> Changes in v8:
> 
>   * shortened test names so that make dist doesn't
>     stop working again
> 
> Changes in v7:
> 
>   * rebased on top of master now that the series this one
>     builds on have been merged
> 
> Changes in v6:
> 
>   * updated to work on top of
> 
>       [PATCH v2 00/10] nodeinfo: Various cleanups
> 
> Changes in v5:
> 
>   * streamlined the logic used to decide whether the subcore
>     configuration is valid and moved it to a separate function
> 
>   * split the tests into separate commits for easier review and
>     to hopefully avoid having trouble with the list due to the
>     message size
> 
> Changes in v4:
> 
>   * removed a printf() statement
> 
>   * fixed typo in a commit message
> 
> Changes in v3:
> 
>   * the function to get the number of threads per subcore
>     has been moved to the from virarch.c, which deals with
>     architecture names only and is therefore not the right
>     place to read host configuration, to nodeinfo.c where
>     the rest of this stuff lives
> 
>   * said function has also been given a shorter name
> 
>   * the "valid subcore mode" boolean has been removed:
>     threads_per_subcore will be a positive number if
>     subcores should be taken into account, and if that's
>     not the case (x86 host, tainted configuration) it
>     will simply be zero, so now the code needs to keep
>     track of a single variable instead of two
> 
>   * the test case has been renamed to be more
>     descriptive
> 
>   * the test data has been cleaned up by removing all
>     cpu/cpu*/node* links, which prevented 'make dist'
>     from working due to recursive linking
> 
> 
> Andrea Bolognani (3):
>   tests: Add subcores1 nodeinfo test
>   tests: Add subcores2 nodeinfo test
>   tests: Add subcores3 nodeinfo test
> 
> Shivaprasad G Bhat (2):
>   nodeinfo: Fix output on PPC64 KVM hosts
>   tests: Prepare for subcore tests
> 
>  src/libvirt_private.syms                           |   1 +
>  src/nodeinfo.c                                     | 153 ++++++++++++++++++++-
>  src/nodeinfo.h                                     |   1 +
>  tests/Makefile.am                                  |   6 +
>  [...]
>  tests/nodeinfomock.c                               |  35 +++++
>  tests/nodeinfotest.c                               |   8 +-
>  1348 files changed, 2134 insertions(+), 6 deletions(-)
>  [...]
>  create mode 100644 tests/nodeinfomock.c
> 

Pulled in 2-5 from v8 series, rebuilt, checked, and pushed.

Tks,

John

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