Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-17 Thread Andrea Bolognani
On Tue, 2015-07-14 at 11:47 +0200, Andrea Bolognani wrote: FWIW, now that John's series has been merged I'm going to rebase my version of the patch on top of it and post if for review. Shouldn't take long. Well, it took way more than I hoped it would, but that's what happens when you dive

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-14 Thread Shivaprasad bhat
On Fri, Jul 10, 2015 at 7:21 PM, John Ferlan jfer...@redhat.com wrote: On 07/10/2015 09:04 AM, Shivaprasad bhat wrote: Hi John, Thanks for the comments. On Thu, Jul 9, 2015 at 9:16 PM, John Ferlan jfer...@redhat.com wrote: On 07/07/2015 03:25 AM, Andrea Bolognani wrote: Changes from v3

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-14 Thread Andrea Bolognani
On Tue, 2015-07-14 at 14:56 +0530, Shivaprasad bhat wrote: As you could see in the series I referenced - there are a number of nodeinfo.c API's which don't process the sysfs properly, e.g. they assume /sys/devices/system. I haven't been fully convinced that the patch which ends up as

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-10 Thread Shivaprasad bhat
Hi John, Thanks for the comments. On Thu, Jul 9, 2015 at 9:16 PM, John Ferlan jfer...@redhat.com wrote: On 07/07/2015 03:25 AM, Andrea Bolognani wrote: Changes from v3 to v4: * removed a printf() statement; * fixed typo in a commit message. Shivaprasad G Bhat (2): Fix nodeinfo

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-10 Thread Andrea Bolognani
On Thu, 2015-07-09 at 11:46 -0400, John Ferlan wrote: On 07/07/2015 03:25 AM, Andrea Bolognani wrote: Changes from v3 to v4: * removed a printf() statement; * fixed typo in a commit message. Shivaprasad G Bhat (2): Fix nodeinfo output on PPC64 KVM hosts Add testcase

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-10 Thread John Ferlan
On 07/10/2015 09:04 AM, Shivaprasad bhat wrote: Hi John, Thanks for the comments. On Thu, Jul 9, 2015 at 9:16 PM, John Ferlan jfer...@redhat.com wrote: On 07/07/2015 03:25 AM, Andrea Bolognani wrote: Changes from v3 to v4: * removed a printf() statement; * fixed typo in a

Re: [libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-09 Thread John Ferlan
On 07/07/2015 03:25 AM, Andrea Bolognani wrote: Changes from v3 to v4: * removed a printf() statement; * fixed typo in a commit message. Shivaprasad G Bhat (2): Fix nodeinfo output on PPC64 KVM hosts Add testcase for PPC64 kvm host nodeinfo Never saw the v4 2/2 come

[libvirt] [PATCH v4 0/2] Fix nodeinfo output on PPC64 KVM hosts

2015-07-07 Thread Andrea Bolognani
Changes from v3 to v4: * removed a printf() statement; * fixed typo in a commit message. Shivaprasad G Bhat (2): Fix nodeinfo output on PPC64 KVM hosts Add testcase for PPC64 kvm host nodeinfo src/libvirt_private.syms | 1 + src/nodeinfo.c