Skip to content

Conversation

GuanLuo
Copy link
Contributor

@GuanLuo GuanLuo commented Aug 19, 2025

Thanks for submitting a PR to Triton!
Please go the the Preview tab above this description box and select the appropriate sub-template:

If you already created the PR, please replace this message with one of

and fill it out.

shm_ip1_handle = shm.create_shared_memory_region(
shm_region_names[1] + "_data", "/" + shm_region_names[1], input1_byte_size
)
with CREATION_LOCK:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have issues with system shared memory. So probably we can remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lock for any creation activity for simplicity

Copy link
Contributor

@whoisj whoisj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@whoisj whoisj added the PR: fix A bug fix label Sep 17, 2025
@yinggeh
Copy link
Contributor

yinggeh commented Sep 17, 2025

@GuanLuo You can close this PR since it was merged to r25.08, which was merged back to main already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: fix A bug fix
Development

Successfully merging this pull request may close these issues.

3 participants