[libvirt] [PATCH v9 0/5] nodeinfo: Add support for subcores

Andrea Bolognani abologna at redhat.com
Wed Jul 29 09:23:41 UTC 2015


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.

John, this should be finally ready to be merged, so if you'd
like to have another go at reviewing it... :)

Cheers.


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                                     | 156 ++++++++++++++++++++-
 src/nodeinfo.h                                     |   1 +
 tests/Makefile.am                                  |   6 +
 [...]
 tests/nodeinfomock.c                               |  35 +++++
 tests/nodeinfotest.c                               |   8 +-
 1348 files changed, 2137 insertions(+), 6 deletions(-)
 [...]
 create mode 100644 tests/nodeinfomock.c

-- 
2.4.3




More information about the libvir-list mailing list