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