Re: [libvirt] [PATCH 0/9] Add sysfs_prefix to nodeinfo.c API's

2015-07-13 Thread Andrea Bolognani
On Fri, 2015-07-10 at 17:05 +0200, Andrea Bolognani wrote: 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. Patch 9 looks good as well, so ACK series with the previously mentioned

Re: [libvirt] [PATCH 0/9] Add sysfs_prefix to nodeinfo.c API's

2015-07-13 Thread John Ferlan
On 07/13/2015 12:42 PM, Andrea Bolognani wrote: On Fri, 2015-07-10 at 17:05 +0200, Andrea Bolognani wrote: 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. Patch 9 looks good as

Re: [libvirt] [PATCH 0/9] Add sysfs_prefix to nodeinfo.c API's

2015-07-10 Thread Andrea Bolognani
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

[libvirt] [PATCH 0/9] Add sysfs_prefix to nodeinfo.c API's

2015-07-07 Thread John Ferlan
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: