Skip to content

Conversation

@h-suzuki-isp
Copy link
Contributor

Please, refer to ros2/rclcpp#2448 .
I have added the following trace points.

  • __rmw_publish_serialized_message()
    • tracepoint : rmw_publish
  • _take_serialized_message()
    • tracepoint : rmw_take

Copy link
Collaborator

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

lgtm

@fujitatomoya
Copy link
Collaborator

@mjcarroll can you review and run CI?

@christophebedard
Copy link
Member

Can you rebase this on rolling? There were some changes (like #746 + ros2/ros2#1530) that require the latest commit.

@h-suzuki-isp h-suzuki-isp force-pushed the add_tracepoint_for_generic branch from 2692248 to 9e5f354 Compare March 27, 2024 05:06
@h-suzuki-isp
Copy link
Contributor Author

Can you rebase this on rolling? There were some changes (like #746 + ros2/ros2#1530) that require the latest commit.

@christophebedard Yes, I rebased.

@fujitatomoya fujitatomoya merged commit c0dab2d into ros2:rolling Apr 1, 2024
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.

4 participants