Skip to content
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.

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

Enable Iceoryx shared memory support in Cyclone DDS #145

Merged
merged 31 commits into from
Aug 16, 2023

Conversation

gmartin82
Copy link
Contributor

This PR adds support for Iceoryx shared memory within Cyclone DDS.

Shared memory support is enabled through the optional dds_shm feature and currently has the following limitations:

  • Not applicable when the --fwd-discovery option is used.
  • Only data readers of compatible types are able to gain any benefit from enabling the feature.
  • ROS2 types are unable to benefit from shared memory. This is a limitation of Cyclone DDS's RMW implementation and may be solved in future by this PR.

gmartin82 added 28 commits May 18, 2023 12:51
README.md Show resolved Hide resolved
$ cargo build --release -p zenoh-bridge-dds --features dds_shm
```

See [here](https://cyclonedds.io/docs/cyclonedds/latest/shared_memory/shared_memory.html) for more details of shared memory support in Cyclone DDS.
Copy link
Member

Choose a reason for hiding this comment

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

Add a note wrt. iox-roundi (to be started or not - what happens if not started)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Requested information added.

DEFAULT_CONFIG.json5 Show resolved Hide resolved
zenoh-plugin-dds/src/dds_mgt.rs Show resolved Hide resolved
@JEnoch JEnoch merged commit 0bf4792 into eclipse-zenoh:master Aug 16, 2023
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.

2 participants