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

Multithread and shared memory #216

Merged
merged 4 commits into from Apr 30, 2021
Merged

Multithread and shared memory #216

merged 4 commits into from Apr 30, 2021

Conversation

pablogs9
Copy link
Member

@pablogs9 pablogs9 commented Mar 26, 2021

Before merge:

TODO :

  • Rename all interprocess to shared memory
  • Implement req/rep
  • Add tests

@pablogs9 pablogs9 marked this pull request as draft March 26, 2021 10:31
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@Acuadros95
Copy link
Contributor

Request reply test failing with the following output:

/home/jenkins/workspace/Micro-XRCE-DDS-Client Manual Linux/test/shared_memory/SharedMemory.cpp:718: Failure
Expected equality of these values:
    object_id
        Which is: 4-byte object <02-00 07-00>
    requester_id_1
        Which is: 4-byte object <01-00 07-00>

The replier is sending the response to all the matched requesters, in this case only the requester_id_1 should get the reply.

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@Acuadros95 Acuadros95 marked this pull request as ready for review April 6, 2021 10:12
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@pablogs9 pablogs9 force-pushed the feature/multithread branch 2 times, most recently from 3a5dce4 to cca23dd Compare April 30, 2021 07:20
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

Update


Revert "Initial"

This reverts commit bbef97f3599786be2febc15f8e8b474a4006234d.

Initial


Update


Update


Initial


Update


Revert example


Uncrustify fix


Add include


Fix header


Fix headers


Fix


Uncrus


Update to const char


Update


Update


Revert "Update"

This reverts commit 3a05e40.

Update


Multithread initial


Interprocess initial


Matching initial


Fix interprocess indexing


Update multithread public API


Initial DLL


Double linked list for ucdrBuffers


renaming


Update


Update


Indent


Deinit


Add shared memory tests (#221)

* Initial

* Update

* Update

* Update

* Update

* Fix leak
Update on request reply shared mem

Initial changes req/res


Update


Initial simple API


Working


Renaming


Fix example pthread header

Create shared memory profile folder


Fix warning


Fix warnings


Fix warnings


Multithread test


Update


Add tests


Style


Uncrustify


Uncrustify examples


Fix


Update


Fix


Add test multithread request reply

Update SharedMemory.cpp

Fix build warning
Include requester info on reply SampleIdentity info

Uncrustify

Fix warnings

Add request reply fragmentation test

Update


Update shared memory engine to bin


Update


Uncrustify


Fix
@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Windows Build Status

@pablogs9 pablogs9 merged commit 527b4dd into develop Apr 30, 2021
@pablogs9 pablogs9 deleted the feature/multithread branch April 30, 2021 10:03
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.

None yet

3 participants