Re: Valgrind-3.17.0.RC1 is available for testing

will schmidt <will_schmidt-DhXWPJtHtuFWk0Htik3J/[email protected]>
Newsgroups gmane.comp.debugging.valgrind.devel,gmane.comp.debugging.valgrind
Message-ID <[email protected]> 
On Mon, 2021-03-15 at 17:14 +0100, Mark Wielaard wrote:
> Hi Carl,
> 
> On Mon, 2021-03-15 at 09:05 -0700, Carl Love wrote:
> > I am seeing issues on various power platforms.  
> > 
> > 

<snip>

> > 
> > I am seeing compilation issues on Power 8LE
> > 
> >    gcc --version
> >    gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
> > 
> >    NAME="Ubuntu"
> >    VERSION="20.04.1 LTS (Focal Fossa)"
> >    ID=ubuntu
> >    ID_LIKE=debian
> >    PRETTY_NAME="Ubuntu 20.04.1 LTS"
> >    VERSION_ID="20.04"
> >    HOME_URL="https://www.ubuntu.com/"
> >    SUPPORT_URL="https://help.ubuntu.com/"
> >    BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
> >    PRIVACY_POLICY_URL="
> >    https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
> >    VERSION_CODENAME=focal
> >    UBUNTU_CODENAME=focal
> > 
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:1112:24:
> > note: in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >     1112 | #        define MPI_UB
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_UB, MPI_Type_create_resized);
> >          |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:281:19: note: in expansion of macro ‘MPI_UB’
> >      281 |    else if (ty == MPI_UB)             fprintf(f,"UB");
> >          |                   ^~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:1113:24:
> > note: in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >     1113 | #        define MPI_LB
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_LB, MPI_Type_create_resized);
> >          |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:282:19: note: in expansion of macro ‘MPI_LB’
> >      282 |    else if (ty == MPI_LB)             fprintf(f,"LB");
> >          |                   ^~~~~~
> >    libmpiwrap.c: In function ‘showCombiner’:
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:743:46:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      743 | #        define MPI_COMBINER_HVECTOR_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_HVECTOR_INTEGER,
> > MPI_COMBINER_HVECTOR);
> >          |                                              ^~~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:354:12: note: in expansion of macro
> > ‘MPI_COMBINER_HVECTOR_INTEGER’
> >      354 |       case MPI_COMBINER_HVECTOR_INTEGER: fprintf(f,
> > "HVECTOR_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:743:46:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      743 | #        define MPI_COMBINER_HVECTOR_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_HVECTOR_INTEGER,
> > MPI_COMBINER_HVECTOR);
> >          |                                              ^~~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:354:12: note: in expansion of macro
> > ‘MPI_COMBINER_HVECTOR_INTEGER’
> >      354 |       case MPI_COMBINER_HVECTOR_INTEGER: fprintf(f,
> > "HVECTOR_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:354:40: error: expected expression before ‘:’ token
> >      354 |       case MPI_COMBINER_HVECTOR_INTEGER: fprintf(f,
> > "HVECTOR_INTEGER"); break;
> >          |                                        ^
> >    In file included from libmpiwrap.c:116:
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:744:47:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      744 | #        define MPI_COMBINER_HINDEXED_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_HINDEXED_INTEGER,
> > MPI_COMBINER_HINDEXED);
> >          |                                               ^~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:359:12: note: in expansion of macro
> > ‘MPI_COMBINER_HINDEXED_INTEGER’
> >      359 |       case MPI_COMBINER_HINDEXED_INTEGER: fprintf(f,
> > "HINDEXED_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:744:47:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      744 | #        define MPI_COMBINER_HINDEXED_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_HINDEXED_INTEGER,
> > MPI_COMBINER_HINDEXED);
> >          |                                               ^~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:359:12: note: in expansion of macro
> > ‘MPI_COMBINER_HINDEXED_INTEGER’
> >      359 |       case MPI_COMBINER_HINDEXED_INTEGER: fprintf(f,
> > "HINDEXED_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:359:41: error: expected expression before ‘:’ token
> >      359 |       case MPI_COMBINER_HINDEXED_INTEGER: fprintf(f,
> > "HINDEXED_INTEGER"); break;
> >          |                                         ^
> >    In file included from libmpiwrap.c:116:
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:745:45:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      745 | #        define MPI_COMBINER_STRUCT_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_STRUCT_INTEGER,
> > MPI_COMBINER_STRUCT);
> >          |                                             ^~~~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:366:12: note: in expansion of macro
> > ‘MPI_COMBINER_STRUCT_INTEGER’
> >      366 |       case MPI_COMBINER_STRUCT_INTEGER: fprintf(f,
> > "STRUCT_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:745:45:
> > note:
> > in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >      745 | #        define MPI_COMBINER_STRUCT_INTEGER
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_COMBINER_STRUCT_INTEGER,
> > MPI_COMBINER_STRUCT);
> >          |                                             ^~~~~~~~~~~~
> > ~~
> > ~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:366:12: note: in expansion of macro
> > ‘MPI_COMBINER_STRUCT_INTEGER’
> >      366 |       case MPI_COMBINER_STRUCT_INTEGER: fprintf(f,
> > "STRUCT_INTEGER"); break;
> >          |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:366:39: error: expected expression before ‘:’ token
> >      366 |       case MPI_COMBINER_STRUCT_INTEGER: fprintf(f,
> > "STRUCT_INTEGER"); break;
> >          |                                       ^
> >    libmpiwrap.c: In function ‘extentOfTy’:
> >    libmpiwrap.c:462:8: warning: implicit declaration of function
> > ‘PMPI_Type_extent’; did you mean ‘MPI_Type_extent’? [-Wimplicit-
> > function-declaration]
> >      462 |    r = PMPI_Type_extent(ty, &n);
> >          |        ^~~~~~~~~~~~~~~~
> >          |        MPI_Type_extent
> >    In file included from libmpiwrap.c:116:
> >    libmpiwrap.c: In function ‘walk_type’:
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:322:57:
> > error: expected expression before ‘_Static_assert’
> >      322 | #define THIS_SYMBOL_WAS_REMOVED_IN_MPI30(func, newfunc)
> > _Static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc "
> > instead.")
> >          |                                                         
> > ^~
> > ~~~~~~~~~~~~
> >    /usr/lib/powerpc64le-linux-gnu/openmpi/include/mpi.h:1113:24:
> > note: in expansion of macro ‘THIS_SYMBOL_WAS_REMOVED_IN_MPI30’
> >     1113 | #        define MPI_LB
> > THIS_SYMBOL_WAS_REMOVED_IN_MPI30(MPI_LB, MPI_Type_create_resized);
> >          |                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    libmpiwrap.c:736:17: note: in expansion of macro ‘MPI_LB’
> >      736 |       if (ty == MPI_LB || ty == MPI_UB)
> >          |                 ^~~~~~
> >    make[2]: *** [Makefile:716: libmpiwrap_ppc64le_linux_so-
> > libmpiwrap.o] Error 1
> >    make[2]: Leaving directory '/home/carll/Valgrind/valgrind-
> > 3.17.0.RC1/mpi'
> >    make[1]: *** [Makefile:855: check-recursive] Error 1
> >    make[1]: Leaving directory '/home/carll/Valgrind/valgrind-
> > 3.17.0.RC1'
> >    make: *** [Makefile:1149: check] Error 2
> > 
> > I will dig into this a bit more and see if I can find a fix for the
> > error.  I will let you know.
> 
> I just did a build on Fedora, and I am not seeing these issues. It
> might depend on the version of openmpi installed I assume.
> 
> https://koji.fedoraproject.org/koji/buildinfo?buildID=1723501
> 

I think so..  we ran into a similar issue late last year, which I think
we had determined was due to the MPI packages in the environment.   
The patch in comment #3 of this bugzilla helped us at that time.  

https://bugs.kde.org/show_bug.cgi?id=401416



> Cheers,
> 
> Mark



_______________________________________________
Valgrind-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/valgrind-developers
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.