On Tue, 2015-07-07 at 20:26 -0400, John Ferlan wrote:
This series adds/processes a sysfs_prefix for the nodeinfo.c
API's.
Although the nodeinfotest.c passes a local test directory path, it
was
never used.
This was all brought to light by patch 9 in this series which is
essentially Kothapally Madhu Pavan's v3 patch:
http://www.redhat.com/archives/libvir-list/2015-June/msg00395.html
With the adjustment to call nodeGetPresentCPUBitmap and
virNodeParseNode
with the sysfs prefix.
Without the first 5 patches, the patch as posted caused nodeinfotest
failure in my test environment which doesn't have the "larger"
environments
that the test tried to set up because the test environment used my
present mask file.
Patches 1-8 look good to me. Great job splitting the changes in
such a nice way! I've commented on patch 7 in a separate mail.
I'll look at patch 9 on Monday.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team