Skip to content

Shmem sirius trace - #2717

Closed
Pawan-Kumar-HPLabs wants to merge 4 commits into
sstsimulator:masterfrom
Pawan-Kumar-HPLabs:shmemSiriusTrace
Closed

Shmem sirius trace#2717
Pawan-Kumar-HPLabs wants to merge 4 commits into
sstsimulator:masterfrom
Pawan-Kumar-HPLabs:shmemSiriusTrace

Conversation

@Pawan-Kumar-HPLabs

Copy link
Copy Markdown
Contributor

Erase this information and put your Pull Request comments here

Instructions for Issuing a Pull Request to sst-elements

1 - Verify that the Pull Request is targeted to the devel branch of sstsimulator/sst-elements

2 - Verify that Source branch is up to date with the devel branch of sst-elements

3 - After submitting your Pull Request:

  • Automatic Testing will commence in a short while
    • Pull Requests will be tested with the devel branches of the sst-core and sst-sqe repositories
      • These branches are syncronized with the devel branch of sst-elements. This is why is it important to keep your source branch up to date.
    • If testing passes, the source branch will be automatically merged (if possible)
      • Pull Requests from forks will not be automatically tested until the code is inspected.
      • Pull Requests from forks will not be automatically merged into the devel branch.
    • If testing fails, You will be notified of the test results.
      • The Pull Request will be retested on a regular basis - Changes to the source branch can be made to correct problems

4 - DO NOT DELETE THE BRANCH (OR FORKED REPO) UNTIL THE PULL REQUEST IS MERGED.

berquist and others added 4 commits July 23, 2026 12:50
- Removed a call to Output object in a destructor that could cause segfaults on shutdown. Fixes sstsimulator#2703
- Updated some printf format flags to use PRI versions
It does not return consistent results
The removed variable "id" is not set in the typical constructor but then used in setup() - we suspect this is causing issues with our internal testing of mordred.  Since this variable appears to have been replaced by "net_id" we have removed it from the nic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants