Skip to content

Debug oom stitchblobs #111

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
hongyuchen1030 wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Debug oom stitchblobs #111

hongyuchen1030 wants to merge 14 commits into from

Conversation

@hongyuchen1030
Copy link
Collaborator

@hongyuchen1030 hongyuchen1030 commented Sep 24, 2025

Optimize the memory usage of the MPI version of the stitchblobs

@hongyuchen1030 hongyuchen1030 self-assigned this Sep 24, 2025
@hongyuchen1030 hongyuchen1030 mentioned this pull request Sep 24, 2025
Open
@hongyuchen1030
Copy link
Collaborator Author

hongyuchen1030 commented Jan 24, 2026

In commit 55d6388

  • Extract MPI exchange operators (TagCollectiveOP, TagExchangeOP, etc.)
    from StitchBlobs.cpp into dedicated MPI utility headers
  • Isolate MPI debug helpers behind STITCHBLOBS_MPI_DEBUG flag
    and move them into a separate debug header
  • Introduce a cached process-wide MPI_Datatype for Tag to avoid
    repeated datatype creation and commit
  • Improve memory safety in TagExchangeOP by eliminating unnecessary
    deep copies and tightening ownership semantics

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@paullric paullric Awaiting requested review from paullric

Assignees

@hongyuchen1030 hongyuchen1030

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hongyuchen1030