Re: [hwloc-devel] 0.9.1rc2 failures

[Jeff Squyres]

K.  I'll do it.

On Oct 23, 2009, at 9:17 AM, Samuel Thibault wrote:


Jeff Squyres, le Fri 23 Oct 2009 09:07:52 -0400, a écrit :
> On Oct 23, 2009, at 8:46 AM, Samuel Thibault wrote:
> >> PGC-W-0006-Empty translation unit (lstopo-cairo.c: 6)
> >> PGC-W-0006-Empty translation unit (lstopo-xml.c: 6)
> >
> >These are expected.
>
> Do you think we should use AM_CONDITIONAL's to conditionally add  
those

> source files to the SOURCES list rather than using #if's to
> effectively zero out the .c file?

Mmm, IIRC some object formats do not support empty objects, so that  
may

be better.

Samuel
___
hwloc-devel mailing list
hwloc-de...@open-mpi.org
http://www.open-mpi.org/mailman/listinfo.cgi/hwloc-devel




--
Jeff Squyres
jsquy...@cisco.com

Re: [hwloc-devel] 0.9.1rc2 failures

[Samuel Thibault]

Hello,

Thanks for the logs.

Pavan Balaji, le Thu 22 Oct 2009 09:57:30 -0500, a écrit :
> topology.c(74): remark #869: parameter "objs" was never referenced
> topology.c(452): remark #981: operands are evaluated in unspecified order

Such remarks are ok. (Google says 981 is really pedantism)

> topology-linux.c(839): (col. 7) remark: LOOP WAS VECTORIZED.
Such remarks are really ok :)

> traversal.c(24): warning #188: enumerated type mixed with another type

This is because we do not include HWLOC_OBJ_TYPE_MAX and UNKNOWN in
the enumeration. One problem with adding them (MAX, in particular) is
that then the compiler will warn if one doesn't handle these cases in a
switch, even if we know they can never happen. From the application
point of view there should always be a default: case anyway since we may
introduce other types of objects, so maybe we should include them and
add cases that just assert(false);

> topology-linux.c(787): remark #593: variable "proc_oscoreids" was set but 
> never used
> unsigned proc_oscoreids[] = { [0 ... HWLOC_NBMAXCPUS-1] = -1 };

Heh, good catch.  We indeed do not need it any more to manage the
computation. Brice, do you think we could just get rid of it?

> PGC-W-0095-Type cast required for this conversion (topology.c: 627)
> PGC-W-0155-Pointer value created from a nonlong integral type  (topology.c: 
> 627)

Should be fixed now.

> PGC-W-0155-Long value is passed to a nonprototyped function - argument #1 
> (cpuset.c: 453)

I hope r1235/1236 will fix it.

> PGC-W-0006-Empty translation unit (lstopo-cairo.c: 6)
> PGC-W-0006-Empty translation unit (lstopo-xml.c: 6)

These are expected.

> PGC-W-0221-Redefinition of symbol CPU_SET 
> (/radix-homes/software/com/packages/pgi-9.0-4/linux86-64/9.0-4/include-gcc40/sched.h:
>  20)

This is the probable culprit of the final link failure. It's odd that
pgcc provides its own copy of sched.h, isn't it supposed to just use
glibc's?  I'm not surprised bugs may appear, I'd tend to think that the
bug is rather in pgcc.

Samuel

Re: [hwloc-devel] 0.9.1rc2 failures

[Pavan Balaji]

I've attached 6 compilation log files: (icc|suncc|pgcc).(c|m).log

*.c.log is generated as:

% ./configure CC= 2>&1 | tee .c.log

... so it should contain sub-configure outputs as well, if any.

*.m.log is generated as:

% make clean && make 2>&1 | tee .m.log

Compiler information:

% icc -V
Intel(R) C Intel(R) 64 Compiler Professional for applications running on
Intel(R) 64, Version 11.0Build 20090131 Package ID: l_cproc_p_11.0.081
Copyright (C) 1985-2009 Intel Corporation.  All rights reserved.

% pgcc -V
pgcc 9.0-4 64-bit target on x86-64 Linux -tp k8-64e
Copyright 1989-2000, The Portland Group, Inc.  All Rights Reserved.
Copyright 2000-2009, STMicroelectronics, Inc.  All Rights Reserved.

% suncc -V
cc: Sun C 5.9 Linux_i386 Patch 124871-01 2007/07/31

This is from sunstudio 12, though the above version information doesn't
say that.

System information:

% uname -a
Linux bblogin 2.6.24-24-generic #1 SMP Sat Aug 22 00:30:49 UTC 2009
x86_64 GNU/Linux

Attached /proc/cpuinfo as cpuinfo.log. Let me know if you need any more
information.

 -- Pavan

On 10/22/2009 02:08 AM, Samuel Thibault wrote:
> Pavan Balaji, le Wed 21 Oct 2009 20:31:17 -0500, a écrit :
>> With suncc (sunstudio 12):
>> ==
>> "topology-linux.c", line 782: syntax error before or at: ...
> 
> Ah, you are using suncc on linux, I wouldn't have expected such
> combination.  Well that can easily be fixed.
> 
>> With pgcc (9.0-4):
>> ==
>> libtool: link: pgcc -g -o .libs/lstopo lstopo-lstopo.o
>> lstopo-lstopo-color.o lstopo-lstopo-text.o lstopo-lstopo-draw.o
>> lstopo-lstopo-fig.o lstopo-lstopo-cairo.o lstopo-lstopo-xml.o
>> -L/home/balaji/tmp/hwloc/hwloc-0.9.1rc2/src -lm ../src/.libs/libhwloc.so
>> -ltermcap
>> ../src/.libs/libhwloc.so: undefined reference to `__CPU_SET'
>> ../src/.libs/libhwloc.so: undefined reference to `__CPU_ZERO'
> 
> Could you also post the warnings?  These are macros and should be
> defined in /usr/include/bits/sched.h.
> 
> Samuel
> ___
> hwloc-devel mailing list
> hwloc-de...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/hwloc-devel

-- 
Pavan Balaji
http://www.mcs.anl.gov/~balaji
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking target system type... x86_64-unknown-linux-gnu
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking how to create a ustar tar archive... gnutar
checking for hwloc version... 0.9.1rc2
checking if this is a developer build... no (doxygen generation is optional)
checking for doxygen... no
checking for pdflatex... /usr/bin/pdflatex
checking for makeindex... /usr/bin/makeindex
checking for fig2dev... no
checking if can build doxygen docs... no
checking for w3m... /usr/bin/w3m
checking for lynx... no
checking if can build top-level README... yes
checking if will build doxygen docs... no
checking if will install doxygen docs... no
checking for style of include used by make... GNU
checking for gcc... icc
checking for C compiler default output file name... a.out
checking whether the C compiler works... yes
checking whether we are cross compiling... no
checking for suffix of executables... 
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether icc accepts -g... yes
checking for icc option to accept ISO C89... none needed
checking dependency style of icc... gcc3
checking how to run the C preprocessor... icc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
checking whether icc and cc understand -c and -o together... yes
checking for icc option to accept ISO C99... -std=c99
checking whether ln -s works... yes
checking for egrep... (cached) /bin/grep -E
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking whether diff accepts -u... -u
checking size of unsigned long... 8
checking size of unsigned int... 4
checking for strncasecmp... yes
checking whether strncasecmp is declared... yes
checking for wchar_t... yes
checking for putwc... yes
checking locale.h usability... yes
checking locale.h presence... yes
checking for locale.h... yes

Re: [hwloc-devel] 0.9.1rc2 failures

[Samuel Thibault]

Pavan Balaji, le Wed 21 Oct 2009 20:31:17 -0500, a écrit :
> With suncc (sunstudio 12):
> ==
> "topology-linux.c", line 782: syntax error before or at: ...

Ah, you are using suncc on linux, I wouldn't have expected such
combination.  Well that can easily be fixed.

> With pgcc (9.0-4):
> ==
> libtool: link: pgcc -g -o .libs/lstopo lstopo-lstopo.o
> lstopo-lstopo-color.o lstopo-lstopo-text.o lstopo-lstopo-draw.o
> lstopo-lstopo-fig.o lstopo-lstopo-cairo.o lstopo-lstopo-xml.o
> -L/home/balaji/tmp/hwloc/hwloc-0.9.1rc2/src -lm ../src/.libs/libhwloc.so
> -ltermcap
> ../src/.libs/libhwloc.so: undefined reference to `__CPU_SET'
> ../src/.libs/libhwloc.so: undefined reference to `__CPU_ZERO'

Could you also post the warnings?  These are macros and should be
defined in /usr/include/bits/sched.h.

Samuel