search

eclipse-iceoryx / iceoryx

Eclipse iceoryx™ - true zero-copy inter-process-communication
https://iceoryx.io
Apache License 2.0
1.65k stars 384 forks source link

iox-#2301 32 bit MVP #2335

Closed elBoberido closed 1 month ago

elBoberido commented 1 month ago

Notes for Reviewer

This PR add initial 32 bit support for iceoryx. Mixed mode with 64 bit application does not yet work but will be part of the next PR.

Pre-Review Checklist for the PR Author

  1. [x] Code follows the coding style of CONTRIBUTING.md
  2. [x] Tests follow the best practice for testing
  3. [x] Changelog updated in the unreleased section including API breaking changes
  4. [x] Branch follows the naming format (iox-123-this-is-a-branch)
  5. [x] Commits messages are according to this guideline
  6. [x] Update the PR title
    • Follow the same conventions as for commit messages
    • Link to the relevant issue
  7. [x] Relevant issues are linked
  8. [x] Add sensible notes for the reviewer
  9. [x] All checks have passed (except task-list-completed)
  10. [x] Assign PR to reviewer

Checklist for the PR Reviewer

Post-review Checklist for the PR Author

  1. [x] All open points are addressed and tracked via issues

References

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 78.75000% with 17 lines in your changes missing coverage. Please review.

Project coverage is 78.23%. Comparing base (5bca9a4) to head (72a8dfd). Report is 11 commits behind head on main.

Files Patch % Lines
tools/introspection/source/introspection_app.cpp 0.00% 10 Missing :warning:
iceoryx_posh/experimental/source/node.cpp 0.00% 2 Missing :warning:
...hoofs/memory/include/iox/detail/static_storage.inl 0.00% 0 Missing and 1 partial :warning:
...ofs/posix/ipc/include/iox/detail/message_queue.inl 88.88% 1 Missing :warning:
...rnal/roudi/introspection/mempool_introspection.inl 0.00% 1 Missing :warning:
iceoryx_posh/source/roudi/roudi.cpp 0.00% 1 Missing :warning:
iceoryx_posh/source/runtime/posh_runtime.cpp 0.00% 1 Missing :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335/graphs/tree.svg?width=650&height=150&src=pr&token=KWu8wdCc1S&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx)](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) ```diff @@ Coverage Diff @@ ## main #2335 +/- ## ======================================= Coverage 78.22% 78.23% ======================================= Files 433 433 Lines 16034 16041 +7 Branches 2302 2301 -1 ======================================= + Hits 12543 12549 +6 - Misses 2644 2647 +3 + Partials 847 845 -2 ``` | [Flag](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) | `78.05% <78.75%> (+<0.01%)` | :arrow_up: | | [unittests_timing](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) | `15.04% <23.75%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) | Coverage Δ | | |---|---|---| | [iceoryx\_binding\_c/source/c\_runtime.cpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_binding_c%2Fsource%2Fc_runtime.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9iaW5kaW5nX2Mvc291cmNlL2NfcnVudGltZS5jcHA=) | `84.61% <100.00%> (ø)` | | | [iceoryx\_binding\_c/source/c\_subscriber.cpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_binding_c%2Fsource%2Fc_subscriber.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9iaW5kaW5nX2Mvc291cmNlL2Nfc3Vic2NyaWJlci5jcHA=) | `92.40% <100.00%> (+1.49%)` | :arrow_up: | | [iceoryx\_hoofs/cli/source/command\_line\_parser.cpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fcli%2Fsource%2Fcommand_line_parser.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9jbGkvc291cmNlL2NvbW1hbmRfbGluZV9wYXJzZXIuY3Bw) | `95.34% <100.00%> (ø)` | | | [...oryx\_hoofs/container/include/iox/detail/vector.inl](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fcontainer%2Finclude%2Fiox%2Fdetail%2Fvector.inl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9jb250YWluZXIvaW5jbHVkZS9pb3gvZGV0YWlsL3ZlY3Rvci5pbmw=) | `96.31% <100.00%> (ø)` | | | [...ofs/memory/include/iox/detail/relative\_pointer.inl](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fmemory%2Finclude%2Fiox%2Fdetail%2Frelative_pointer.inl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9tZW1vcnkvaW5jbHVkZS9pb3gvZGV0YWlsL3JlbGF0aXZlX3BvaW50ZXIuaW5s) | `96.38% <100.00%> (ø)` | | | [iceoryx\_hoofs/memory/include/iox/memory.hpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fmemory%2Finclude%2Fiox%2Fmemory.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9tZW1vcnkvaW5jbHVkZS9pb3gvbWVtb3J5LmhwcA==) | `100.00% <ø> (ø)` | | | [...oryx\_hoofs/memory/include/iox/relative\_pointer.hpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fmemory%2Finclude%2Fiox%2Frelative_pointer.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9tZW1vcnkvaW5jbHVkZS9pb3gvcmVsYXRpdmVfcG9pbnRlci5ocHA=) | `100.00% <ø> (ø)` | | | [iceoryx\_hoofs/memory/source/memory.cpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fmemory%2Fsource%2Fmemory.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9tZW1vcnkvc291cmNlL21lbW9yeS5jcHA=) | `63.63% <100.00%> (ø)` | | | [iceoryx\_hoofs/posix/filesystem/source/file.cpp](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fposix%2Ffilesystem%2Fsource%2Ffile.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9wb3NpeC9maWxlc3lzdGVtL3NvdXJjZS9maWxlLmNwcA==) | `33.33% <100.00%> (+1.04%)` | :arrow_up: | | [...osix/ipc/include/iox/detail/unix\_domain\_socket.inl](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree&filepath=iceoryx_hoofs%2Fposix%2Fipc%2Finclude%2Fiox%2Fdetail%2Funix_domain_socket.inl&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx#diff-aWNlb3J5eF9ob29mcy9wb3NpeC9pcGMvaW5jbHVkZS9pb3gvZGV0YWlsL3VuaXhfZG9tYWluX3NvY2tldC5pbmw=) | `83.05% <100.00%> (ø)` | | | ... and [18 more](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx) | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/eclipse-iceoryx/iceoryx/pull/2335/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse-iceoryx)