Skip to content

fix: resolve SIGSEGV on HIP builds#3992

Open
victorapm wants to merge 4 commits intodevelopfrom
bugfix/paludettomag1/hip-exec
Open

fix: resolve SIGSEGV on HIP builds#3992
victorapm wants to merge 4 commits intodevelopfrom
bugfix/paludettomag1/hip-exec

Conversation

@victorapm
Copy link
Contributor

@victorapm victorapm commented Mar 5, 2026 •
edited
Loading

  1. Fix SIGSEGV from Comm. tools on HIP builds (fix provided by @wrtobin)
  2. Fix memory info retrieval for HIP builds
  3. Improve hypre default for HIP builds
  4. Minor fixes to version header (in the output log)

@victorapm victorapm self-assigned this Mar 5, 2026
@victorapm victorapm added the ci: run CUDA builds Allows to triggers (costly) CUDA jobs label Mar 5, 2026
@victorapm victorapm added flag: ready for review ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: no rebaseline Does not require rebaseline ci: run code coverage enables running of the code coverage CI jobs labels Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@castelletto1 castelletto1 castelletto1 approved these changes

@rrsettgast rrsettgast Awaiting requested review from rrsettgast rrsettgast is a code owner

@wrtobin wrtobin Awaiting requested review from wrtobin wrtobin is a code owner

@OmarDuran OmarDuran Awaiting requested review from OmarDuran OmarDuran is a code owner

@corbett5 corbett5 Awaiting requested review from corbett5 corbett5 is a code owner

@MelReyCG MelReyCG Awaiting requested review from MelReyCG MelReyCG is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@victorapm victorapm

Labels

ci: run code coverage enables running of the code coverage CI jobs ci: run CUDA builds Allows to triggers (costly) CUDA jobs ci: run integrated tests Allows to run the integrated tests in GEOS CI flag: no rebaseline Does not require rebaseline flag: ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@victorapm @castelletto1