Skip to content

feat(shared_memory): add SetInterVMMemoryProvider for inter-VM shm suppprt - #398

Open
Rahul-Sutariya wants to merge 1 commit into
eclipse-score:mainfrom
Rahul-Sutariya:rasu_enable_custom_typemem_provider_injection
Open

feat(shared_memory): add SetInterVMMemoryProvider for inter-VM shm suppprt#398
Rahul-Sutariya wants to merge 1 commit into
eclipse-score:mainfrom
Rahul-Sutariya:rasu_enable_custom_typemem_provider_injection

Conversation

@Rahul-Sutariya

Copy link
Copy Markdown
Contributor

Add a separate InterVM memory provider that is used when the shm path has the inter-VM prefix (/intervm-shared-shmem/). This replaces the previous hard rejection of inter-VM paths in Create/Open/CreateOrOpen with actual provider-backed allocation.

@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@Rahul-Sutariya
Rahul-Sutariya force-pushed the rasu_enable_custom_typemem_provider_injection branch from 3bb7470 to 5e31bca Compare July 22, 2026 06:34
…pport

Add a separate InterVM memory provider that is used when the shm path
has the inter-VM prefix (/intervm-shared-shmem/). This replaces the
previous hard rejection of inter-VM paths in Create/Open/CreateOrOpen
with actual provider-backed allocation.
@Rahul-Sutariya
Rahul-Sutariya force-pushed the rasu_enable_custom_typemem_provider_injection branch from 5e31bca to 59f08f9 Compare July 22, 2026 12:03

@4og 4og left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The memory/shared is currently in transition to the communication repo. An agreement from @LittleHuba is needed if it's fine to merge it or not.

@Rahul-Sutariya

Copy link
Copy Markdown
Contributor Author

The memory/shared is currently in transition to the communication repo. An agreement from @LittleHuba is needed if it's fine to merge it or not.

Hi @4og, yes, Ulrich had approved this pr merge, Thanks!

@LittleHuba

Copy link
Copy Markdown
Contributor

@Rahul-Sutariya that approval is over a week old. Things moved since then.
Unfortunately, we are past the stage where I'm comfortable accepting any changes anymore.

@Rahul-Sutariya

Copy link
Copy Markdown
Contributor Author

@Rahul-Sutariya that approval is over a week old. Things moved since then. Unfortunately, we are past the stage where I'm comfortable accepting any changes anymore.

Thanks @LittleHuba for confirmation, i will migrate pr to S-Core/Com, once lib/mem migrated to it.

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

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants