My testing is underway...
Will report anything of significance.
-Paul
On Mon, Aug 24, 2015 at 6:58 AM, Ralph Castain <r...@open-mpi.org> wrote:
> Okay - one (hopefully) very last time!!!
>
> rc6 has now been posted, including all these changes
>
>
>
> On Aug 24, 2015, at 1:05 AM, Ralph Castain <r...@open-mpi.org> wrote:
>
> Indeed, you are absolutely correct! I will fix all of them, but wanted
> first to ensure that was the correct fix.
>
> Thanks Paul!
>
> On Aug 23, 2015, at 10:44 PM, Paul Hargrove <phhargr...@lbl.gov> wrote:
>
> Ralph,
>
> As I expected your fixes to ompi/debuggers/Makefile.am only fixed the
> tests in that directory.
> I am still left with
>
> make[4]: Entering directory
> `/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/test/datatype'
> PASS: opal_datatype_test
> FAIL: checksum
> FAIL: position
> FAIL: position_noncontig
> FAIL: ddt_test
> FAIL: ddt_raw
>
> I was able to resolve those by making the "obvious" changes in
> test/datatype/Makefile.am to replace undefined uses of
> $(WRAPPER_EXTRA_LDFLAGS) with $(OMPI_WRAPPER_EXTRA_LDFLAGS).
>
> That leaves 6 more undefined references to $(WRAPPER_EXTRA_LDFLAGS).
> I think somebody should look at those, but I am guessing that all in
> test/datatype/Makefile.am should be $(OMPI_WRAPPER_EXTRA_LDFLAGS) while all
> in test/event/Makefile.am should probably be $(OPAL_WRAPPER_EXTRA_LDFLAGS).
>
> -Paul
>
> On Sun, Aug 23, 2015 at 9:58 PM, Paul Hargrove <phhargr...@lbl.gov> wrote:
>
>> Ralph,
>>
>> I will try that and expect it will fix exactly those two tests.
>>
>> However, that leaves 11 other undefined references in
>> WRAPPER_EXTRA_LDFLAGS.
>> Five of those I already know will cause test failures as shown in my
>> previous email.
>>
>> -Paul
>>
>> On Sun, Aug 23, 2015 at 8:50 PM, Ralph Castain <r...@open-mpi.org> wrote:
>>
>>> Hmm….I’ll bet this is the correct change, then:
>>>
>>> *diff --git a/ompi/debuggers/Makefile.am b/ompi/debuggers/Makefile.am*
>>> *index 3e48af8..93a3046 100644*
>>> *--- a/ompi/debuggers/Makefile.am*
>>> *+++ b/ompi/debuggers/Makefile.am*
>>> @@ -10,6 +10,7 @@
>>> # Copyright (c) 2004-2005 The Regents of the University of California.
>>> # All rights reserved.
>>> # Copyright (c) 2007-2015 Cisco Systems, Inc. All rights reserved.
>>> +# Copyright (c) 2015 Intel, Inc. All rights reserved.
>>> # $COPYRIGHT$
>>> #
>>> # Additional copyrights may follow
>>> @@ -43,13 +44,14 @@ headers = \
>>> # Simple checks to ensure that the DSOs are functional
>>>
>>> dlopen_test_SOURCES = dlopen_test.c
>>> +dlopen_test_LDFLAGS = $(OMPI_WRAPPER_EXTRA_LDFLAGS)
>>> dlopen_test_LDADD = \
>>> $(top_builddir)/ompi/libmpi.la \
>>> $(top_builddir)/opal/libopen-pal.la
>>> dlopen_test_DEPENDENCIES = $(ompi_predefined_LDADD)
>>>
>>> predefined_gap_test_SOURCES = predefined_gap_test.c
>>> -predefined_gap_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> +predefined_gap_test_LDFLAGS = $(OMPI_WRAPPER_EXTRA_LDFLAGS)
>>> predefined_gap_test_LDADD = $(top_builddir)/ompi/libmpi.la
>>> predefined_gap_test_DEPENDENCIES = $(ompi_predefined_LDADD)
>>>
>>>
>>>
>>> On Aug 23, 2015, at 8:41 PM, Paul Hargrove <phhargr...@lbl.gov> wrote:
>>>
>>> Ralph,
>>>
>>> Sorry, but no change - still failing to load libmyriexpress.so and still
>>> no rpath at link:
>>>
>>> /bin/sh ../../libtool --tag=CC --mode=link gcc -std=gnu99
>>> -fno-strict-aliasing -pthread -g -o dlopen_test dlopen_test.o ../../ompi/
>>> libmpi.la ../../opal/libopen-pal.la -lrt -lm -lutil
>>> libtool: link: gcc -std=gnu99 -fno-strict-aliasing -pthread -g -o
>>> dlopen_test dlopen_test.o ../../ompi/.libs/libmpi.a -L/opt/mx2g/lib
>>> -libverbs -lmyriexpress
>>> /home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/orte/.libs/libopen-rte.a
>>> /home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/opal/.libs/libopen-pal.a
>>> ../../opal/.libs/libopen-pal.a -ldl -lrt -lm -lutil -pthread
>>>
>>> I am not surprised, since predefined_gap_test already had those LDFLAGS
>>> and had also failed to locate the library at run time.
>>> Don't misunderstand - I agree the change you made is correct (and
>>> necessary), but it is apparently not sufficient.
>>>
>>> It does look like configure did its job:
>>>
>>> $ grep ,-rpath config.status
>>> S["OMPI_PKG_CONFIG_LDFLAGS"]="-L/opt/mx2g/lib -Wl,-rpath
>>> -Wl,/opt/mx2g/lib -Wl,-rpath -Wl,${libdir} -Wl,--enable-new-dtags"
>>> S["OMPI_WRAPPER_EXTRA_LDFLAGS"]="-L/opt/mx2g/lib -Wl,-rpath
>>> -Wl,/opt/mx2g/lib -Wl,-rpath -Wl,@{libdir} -Wl,--enable-new-dtags"
>>> S["ORTE_PKG_CONFIG_LDFLAGS"]=" -Wl,-rpath -Wl,${libdir}
>>> -Wl,--enable-new-dtags"
>>> S["ORTE_WRAPPER_EXTRA_LDFLAGS"]=" -Wl,-rpath -Wl,@{libdir}
>>> -Wl,--enable-new-dtags"
>>> S["OPAL_PKG_CONFIG_LDFLAGS"]=" -Wl,-rpath -Wl,${libdir}
>>> -Wl,--enable-new-dtags"
>>> S["OPAL_WRAPPER_EXTRA_LDFLAGS"]=" -Wl,-rpath -Wl,@{libdir}
>>> -Wl,--enable-new-dtags"
>>> D["WRAPPER_EXTRA_LDFLAGS"]=" \"-L/opt/mx2g/lib -Wl,-rpath
>>> -Wl,/opt/mx2g/lib -Wl,-rpath -Wl,@{libdir} -Wl,--enable-new-dtags\""
>>>
>>> However, WRAPPER_EXTRA_LDFLAGS isn't defined in any Makefile, despite
>>> being used in several:
>>>
>>> $ find . -name Makefile | xargs grep -w WRAPPER_EXTRA_LDFLAGS
>>> ./test/event/Makefile:signal_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/event/Makefile:time_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/event/Makefile:event_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:ddt_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:ddt_raw_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:ddt_pack_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:checksum_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:position_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:position_noncontig_LDFLAGS =
>>> $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:to_self_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./test/datatype/Makefile:opal_datatype_test_LDFLAGS =
>>> $(WRAPPER_EXTRA_LDFLAGS)
>>> ./ompi/debuggers/Makefile:dlopen_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>> ./ompi/debuggers/Makefile:predefined_gap_test_LDFLAGS =
>>> $(WRAPPER_EXTRA_LDFLAGS)
>>>
>>> So, the following result is probably not too surprising (unless due to
>>> the tests NOT appearing):
>>>
>>> $ make -k check 2>&1 | grep ^FAIL
>>> FAIL: predefined_gap_test
>>> FAIL: dlopen_test
>>> FAIL: checksum
>>> FAIL: position
>>> FAIL: position_noncontig
>>> FAIL: ddt_test
>>> FAIL: ddt_raw
>>>
>>>
>>> -Paul
>>>
>>>
>>> On Sun, Aug 23, 2015 at 7:46 PM, Paul Hargrove <phhargr...@lbl.gov>
>>> wrote:
>>>
>>>> Ralph,
>>>>
>>>> I will try the requested change and let you know.
>>>>
>>>> -Paul
>>>>
>>>> On Sun, Aug 23, 2015 at 7:27 PM, Ralph Castain <r...@open-mpi.org>
>>>> wrote:
>>>>
>>>>> Rather than generating another rc right away, could you please apply
>>>>> the following patch and see if it fixes the problem?
>>>>>
>>>>> *diff --git a/ompi/debuggers/Makefile.am b/ompi/debuggers/Makefile.am*
>>>>> *index 3e48af8..343a0c4 100644*
>>>>> *--- a/ompi/debuggers/Makefile.am*
>>>>> *+++ b/ompi/debuggers/Makefile.am*
>>>>> @@ -10,6 +10,7 @@
>>>>> # Copyright (c) 2004-2005 The Regents of the University of California.
>>>>> # All rights reserved.
>>>>> # Copyright (c) 2007-2015 Cisco Systems, Inc. All rights reserved.
>>>>> +# Copyright (c) 2015 Intel, Inc. All rights reserved.
>>>>> # $COPYRIGHT$
>>>>> #
>>>>> # Additional copyrights may follow
>>>>> @@ -43,6 +44,7 @@ headers = \
>>>>> # Simple checks to ensure that the DSOs are functional
>>>>>
>>>>> dlopen_test_SOURCES = dlopen_test.c
>>>>> +dlopen_test_LDFLAGS = $(WRAPPER_EXTRA_LDFLAGS)
>>>>> dlopen_test_LDADD = \
>>>>> $(top_builddir)/ompi/libmpi.la \
>>>>> $(top_builddir)/opal/libopen-pal.la
>>>>>
>>>>>
>>>>> Thanks
>>>>> Ralph
>>>>>
>>>>>
>>>>> On Aug 23, 2015, at 4:58 PM, Paul Hargrove <phhargr...@lbl.gov> wrote:
>>>>>
>>>>> I regret to say that in my endless search for perfection (which is a
>>>>> journey, not a destination) I believe I found another issue:
>>>>>
>>>>> I had mx2g sources sitting around, which I compiled and installed on
>>>>> two systems (x86 and x86-64).
>>>>> These provide only compile/link tests, since my systems lack the
>>>>> hardware (actually do have some in a box somewhere).
>>>>>
>>>>> Things worked fine on both systems when configure with
>>>>> --with-mx=[path].
>>>>> However, on both systems adding "--enable-static --disable-shared"
>>>>> results in failure from "make check":
>>>>>
>>>>> make[3]: Entering directory
>>>>> `/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/ompi/debuggers'
>>>>> CC predefined_gap_test.o
>>>>> CCLD predefined_gap_test
>>>>> CC dlopen_test.o
>>>>> CCLD dlopen_test
>>>>> make[3]: Leaving directory
>>>>> `/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/ompi/debuggers'
>>>>> make check-TESTS
>>>>> make[3]: Entering directory
>>>>> `/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/ompi/debuggers'
>>>>> ./predefined_gap_test: error while loading shared libraries:
>>>>> libmyriexpress.so: cannot open shared object file: No such file or
>>>>> directory
>>>>> FAIL: predefined_gap_test
>>>>> ./dlopen_test: error while loading shared libraries:
>>>>> libmyriexpress.so: cannot open shared object file: No such file or
>>>>> directory
>>>>> FAIL: dlopen_test
>>>>>
>>>>>
>>>>> The following *does* show mpicc using a correct rpath argument for
>>>>> the mx library:
>>>>>
>>>>> $ ./INST/bin/mpicc --show openmpi-1.10.0rc5/examples/ring_c.c
>>>>> gcc openmpi-1.10.0rc5/examples/ring_c.c
>>>>> -I/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/INST/include
>>>>> -pthread -L/opt/mx2g/lib -Wl,-rpath -Wl,/opt/mx2g/lib -Wl,-rpath
>>>>> -Wl,/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/INST/lib
>>>>> -Wl,--enable-new-dtags
>>>>> -L/home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/INST/lib -lmpi
>>>>> -lopen-rte -lopen-pal -lm -ldl -lrt -lmyriexpress -libverbs -lutil
>>>>> $ ls /opt/mx2g/lib
>>>>> debug libmyriexpress.a libmyriexpress.so
>>>>>
>>>>>
>>>>> However, the *test* is built without an rpath:
>>>>>
>>>>> /bin/sh ../../libtool --tag=CC --mode=link gcc -std=gnu99
>>>>> -fno-strict-aliasing -pthread -g -o dlopen_test dlopen_test.o
>>>>> ../../ompi/
>>>>> libmpi.la ../../opal/libopen-pal.la -lrt -lm -lutil
>>>>> libtool: link: gcc -std=gnu99 -fno-strict-aliasing -pthread -g -o
>>>>> dlopen_test dlopen_test.o ../../ompi/.libs/libmpi.a -L/opt/mx2g/lib
>>>>> -libverbs -lmyriexpress
>>>>> /home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/orte/.libs/libopen-rte.a
>>>>> /home/phargrov/OMPI/openmpi-1.10.0rc5-linux-x86-mx-static/BLD/opal/.libs/libopen-pal.a
>>>>> ../../opal/.libs/libopen-pal.a -ldl -lrt -lm -lutil -pthread
>>>>>
>>>>>
>>>>> I have no reason to suspect this is not also present in 1.8.x, but
>>>>> have not checked.
>>>>> Sometime after 1.10.0 is out to door, I expect to test the v1.8
>>>>> nightly tarball.
>>>>>
>>>>> -Paul [who enjoyed breaking his toys as a child and now breaks
>>>>> software with the same enthusiasm]
>>>>>
>>>>>
>>>>> --
>>>>> Paul H. Hargrove phhargr...@lbl.gov
>>>>> Computer Languages & Systems Software (CLaSS) Group
>>>>> Computer Science Department Tel: +1-510-495-2352
>>>>> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
>>>>> _______________________________________________
>>>>> devel mailing list
>>>>> de...@open-mpi.org
>>>>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>>>> Link to this post:
>>>>> http://www.open-mpi.org/community/lists/devel/2015/08/17803.php
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> devel mailing list
>>>>> de...@open-mpi.org
>>>>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>>>> Link to this post:
>>>>> http://www.open-mpi.org/community/lists/devel/2015/08/17804.php
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Paul H. Hargrove phhargr...@lbl.gov
>>>> Computer Languages & Systems Software (CLaSS) Group
>>>> Computer Science Department Tel: +1-510-495-2352
>>>> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
>>>>
>>>
>>>
>>>
>>> --
>>> Paul H. Hargrove phhargr...@lbl.gov
>>> Computer Languages & Systems Software (CLaSS) Group
>>> Computer Science Department Tel: +1-510-495-2352
>>> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
>>> _______________________________________________
>>> devel mailing list
>>> de...@open-mpi.org
>>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>> Link to this post:
>>> http://www.open-mpi.org/community/lists/devel/2015/08/17807.php
>>>
>>>
>>>
>>> _______________________________________________
>>> devel mailing list
>>> de...@open-mpi.org
>>> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
>>> Link to this post:
>>> http://www.open-mpi.org/community/lists/devel/2015/08/17808.php
>>>
>>
>>
>>
>> --
>> Paul H. Hargrove phhargr...@lbl.gov
>> Computer Languages & Systems Software (CLaSS) Group
>> Computer Science Department Tel: +1-510-495-2352
>> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
>>
>
>
>
> --
> Paul H. Hargrove phhargr...@lbl.gov
> Computer Languages & Systems Software (CLaSS) Group
> Computer Science Department Tel: +1-510-495-2352
> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post:
> http://www.open-mpi.org/community/lists/devel/2015/08/17810.php
>
>
>
>
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post:
> http://www.open-mpi.org/community/lists/devel/2015/08/17817.php
>
--
Paul H. Hargrove phhargr...@lbl.gov
Computer Languages & Systems Software (CLaSS) Group
Computer Science Department Tel: +1-510-495-2352
Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
