search

ros / ros_comm

ROS communications-related packages, including core client libraries (roscpp, rospy, roslisp) and graph introspection tools (rostopic, rosnode, rosservice, rosparam).
http://wiki.ros.org/ros_comm
753 stars 911 forks source link

Thread is stucking on shutdown #2281

Open l1va opened 2 years ago

l1va commented 2 years ago

Fixing the issue with 100% CPU usage after the shutdown https://github.com/ros/ros_comm/issues/2238

peci1 commented 1 year ago

I can't follow the reasoning why this proposed solution would work. It is difficult to dig through all the indirections involved.

What is the difference in calling shutdown with args or without? Does it take a different code path?

This would also deserve a test to show the actual problem and that this PR solves it.