Re: [libvirt] [RFC PATCH v2 1/4] PowerPC : Use Sysfs to gather host topology

On 11/14/2011 07:46 AM, Prerna Saxena wrote: Are we sure that sysfs has this information for all kernels as far back as RHEL 5? Or should this patch be doing conditionals; probing sysfs first but falling back to /proc/cpuinfo otherwise? I tried compiling things, but got this warning: nodeinfo.c: In function 'linuxNodeInfoCPUPopulate': nodeinfo.c:287:5: error: suggest parentheses around assignment used as truth value [-Werror=parentheses] nodeinfo.c:202:36: error: unused parameter 'need_hyperthreads' [-Werror=unused-parameter] and seeing other comments in this thread, I didn't review it very closely (this is more a chance to revive the thread after a week, to see if you are planning on a resubmission before we consider a freeze for 0.9.8). -- Eric Blake eblake(a)redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org