Thanks Brice. What's the timeframe for 1.7.1? I want to see if we can
move to that for our mpich-3.1 release.
On 05/07/2013 09:53 AM US Central Time, Brice Goglin wrote:
> I just pushed another round of commits to trunk, which hopefully address
> everything said earlier (except the LDTL problem waiting for Jeff's review).
> 1) autogen args
> 2) sys/sysctl.h #ifdef fix
> 3) sysctl/sysctlbyname check fix (include additional fix for icc)
> 4) HWLOC_CHECK_DECL improvement
> 5) strtoull check
> (1) and (2) will be backported to v1.7 once regression testing is done.
> (5) should likely be OK too.
> (3) and more importantly (4) touch ugly configury, I am not sure I want
> to backport these. Or maybe later, in case we get more testing before
> v1.7.1 occurs.
> Le 05/05/2013 18:18, Pavan Balaji a écrit :
>> Sorry for starting a new thread. I'm trying to round-up all the issues
>> I've reported for hwloc-1.7 so far into a more manageable format.
>> 1. We had noticed errors with -D_POSIX_SOURCE that I had reported here:
>> The error with POSIX_SOURCE itself was pretty straightforward. I was
>> able to fix it in the mpich version:
>> However, with our complete strict build flags, there were more errors.
>> Here's a summary and the relevant fixes:
>> - hwloc's check for whether an explicitly function declaration is
>> needed (using _HWLOC_CHECK_DECL) was relying on whether a dummy call to
>> the function throws an error. This only works if the function
>> declaration is already present in one of the headers. If such a
>> declaration is not present, the test might fail with "implicit function
>> declaration" with the right CFLAGS. This leads the m4 macro to think
>> that the declaration is already there in one of the headers and an
>> additional declaration is not needed.
>> The below commit fixes this by adding a dummy function declaration,
>> together with the dummy function definition:
>> FWIW, mpich's version of this macro also uses a similar dummy function
>> declaration together with the dummy call to the function:
>> - For sysctl and sysctlbyname, I've updated hwloc/config.m4 to use a
>> full link test instead of just using AC_CHECK_FUNCS, which only checks
>> to see if the symbol exists or not. For example, the prototype of
>> sysctl uses u_int, which on some platforms (such as FreeBSD) is only
>> defined under __BSD_VISIBLE, __USE_BSD or other similar definitions. So
>> while the symbols "sysctl" and "sysctlbyname" might still be available
>> in libc (which autoconf checks for), they might not be actually usable.
>> The below commit fixes this:
>> - A minor error where strings.h was not included for strcasecmp.
>> 2. I had reported an issue with libltdl in embedded mode (also in the
>> above thread). I believe Brice is looking into this, so I didn't
>> investigate it further. I'm using a disgusting, but workable, patch to
>> workaround this error in mpich (see the
>> src/pm/hydra/tools/topo/hwloc/hwloc/src/Makefile.am part of the below
>> I'd appreciate a cleaner fix to this issue.
>> 3. I had reported an issue with the usage of getpagesize() instead of
>> hwloc_getpagesize() on the Mac.
>> I believe Samuel has already incorporated this in hwloc trunk. Here is
>> the fix I used for your reference:
>> 4. I had reported some warnings on the FreeBSD strict build here.
>> I believe Brice and Samuel are looking into it, but I don't have a
>> confirmation on whether this is fixed. I didn't fix them in mpich yet.
>> As you can tell, we are looking into upgrading to hwloc-1.7 for the next
>> major release of mpich (3.1). With the above fixes, it looks like
>> things are working well. Of course, we'll be going through a lot more
>> testing before the final release which would be later this year.
>> -- Pavan