Le 24/07/2012 13:39, Brice Goglin a écrit :
> Le 23/07/2012 18:40, Pavan Balaji a écrit :
>>>> 3. The changes to config/hwloc.m4 and include/private/private.h are
>>>> essentially a warning squash when getpagesize() requires an explicit
>>>> prototype declaration. But it's not clear how easy it is for you to
>>>> absorb it as it uses an MPICH2 internal m4 macro. Maybe there's a
>>>> cleaner way to integrate this patch.
>>> I will look at this. On which platform/system did you need this patch?
>> On regular x86_64 machines. We use strict builds in mpich2. I can
>> figure out the exact flags that trigger it, but my guess is that -Wall
>> would.
> Looking deeper into this, it looks like the right solution is to use
> sysconf(_SC_PAGESIZE) instead of getpagesize(). getpagesize() looks
> deprecated, that's why it's getting disabled when modern features are
> enabled. I'll send a patch to use sysconf when supported and fallback to
> the old getpagesize() otherwise.
>
Can you try this patch of some of your platforms?
I think I couldn't reproduce your problem because hwloc's
AC_USE_SYSTEM_EXTENSIONS defines _GNU_SOURCE, which defines _BSD_SOURCE,
which defines __USE_BSD, which makes getpagesize() available without
looking at other XOPEN/POSIX defines.
Brice
Brice
|
![[Flag]](../../../../../community/mirrors/flag-us.gif){kind=small}