Skip to content
Snippets Groups Projects

ENH: add profiling hooks for Extrae into the OpenFOAM profiling (#3043)

Merged Mark OLESEN requested to merge extrae-profiling into develop

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mark OLESEN changed milestone to %v2312

    changed milestone to %v2312

  • assigned to @andy

  • Author Maintainer

    @josep.pocurull - I have applied your patch with some modifications. Would appreciate verification (compilation?) from your side. I unfortunately didn't yet manage to compile Extrae on my openSUSE system.

    Currently configuring with this:

    sources/extrae/extrae-4.0.6/configure \
      --prefix=/usr/local \
      --without-dyninst \
      --disable-mpi-rpath \
      --with-mpi=$MPI_ARCH_PATH \
      --with-binutils=/usr \
      --with-binutils-headers=/usr/include \
      --with-binutils-libs=/usr/lib64 \
      --with-unwind=/usr \
      --with-papi=/usr \

    Using system values for as many things as possible. Error:

    configure:33124: checking whether libbfd and libiberty work
    configure:33152: gcc -o conftest -g -O2 -fno-optimize-sibling-calls -I/usr/include   conftest.c -L/usr/lib64 -lbfd -L/usr/lib64 -liberty -L/usr/local/lib64   >&5
    /usr/lib64/gcc/x86_64-suse-linux/7/../../../../x86_64-suse-linux/bin/ld: /usr/lib64/libbfd.a(elfxx-x86.o): in function `_bfd_x86_elf_create_sframe_plt':
    /home/abuild/rpmbuild/BUILD/binutils-2.41/build-dir/bfd/../../bfd/elfxx-x86.c:1888:(.text+0xb0b): undefined reference to `sframe_encode'

    It appears that the link to libiberty assumes a dynamic linkage to libsframe, but libiberty is static on openSUSE.

    Edited by Mark OLESEN
  • Regarding the verification, I successfully compiled OpenFOAM with the profiling modifications, and it worked just fine.

  • Andrew Heather approved this merge request

    approved this merge request

  • Andrew Heather added 43 commits

    added 43 commits

    Compare with previous version

  • Andrew Heather mentioned in commit 686f2a39

    mentioned in commit 686f2a39

  • merged

Please register or sign in to reply
Loading