Branch: refs/heads/main
Revision: 7d5d19f35b79d433dc57a995251d87de58ca7d7c
Author: jabraham17
Link: Patch bundled gasnet to pull in bugfix for IBV by jabraham17 · Pull Request #24899 · chapel-lang/chapel · GitHub
Log Message:
Patch bundled gasnet to pull in bugfix for IBV (#24899)
This PR patches the bundled gasnet to pull in a fix for gasnet using the
ibv-conduit with clang16+.
Gasnet bug: Bug 4617 – SEGV in remote firehose, NDEBUG and clang-16.0.0+
Gasnet patch: https://bitbucket.org/berkeleylab/gasnet/pull-requests/625
Resolves [Bug]: Segfault when running applications with LLVM 16 or 17 + Gasnet IBV · Issue #24779 · chapel-lang/chapel · GitHub
Resolves Using clang 16.0.0 results in SIGSEGV when using GASNet with segment=large · Issue #22055 · chapel-lang/chapel · GitHub
Tested by building with CHPL_COMM=gasnet
and CHPL_COMM_SUBSTRATE=ibv
and confirming that the correct codepath for the conditional compilation
is taken
I was also able to reproduce the issue on an InfiniBand system without
the patch, and confirmed that applying the patch results in correct
execution.
> chpl examples/hello6-taskpar-dist.chpl
> ./hello6-taskpar-dist.chpl -nl 4
SEGFAULT
> ...apply patch....
> chpl examples/hello6-taskpar-dist.chpl
> ./hello6-taskpar-dist.chpl -nl 4
Hello, world! .....
......
[Reviewed by @jhh67]
Diff:
M third-party/gasnet/gasnet-src/other/firehose/firehose_region.c
https://github.com/chapel-lang/chapel/pull/24899.diff