tree ee4aecae66061d3aab250fa32bca4fa730019857
parent 895ef59e7ffa5c57d895bebc842a736a902bf41e
author Jacob Moroni <jmoroni@google.com> 1751511438 +0000
committer Nicolas Morey <nmorey@suse.com> 1754460818 +0200

rping: terminate CM event thread before exiting

[ Upstream commit f3ae6534ad93c4f1aca7374d9a75f61b790fa03c ]

The CM event thread processes events in a loop with no explicit
termination. When the last CM event is received, the main thread
proceeds to clean up and destroy the CM event channel. If this occurs
after the CM event thread has processed the last event, but before it
reaches rdma_get_cm_event again, then the subsequent call to
rdma_get_cm_event will fail and cause the process to exit with a failure
code even though the test was actually successful.

This causes flakiness in test scripts that use rping for basic
functional testing.

Fix this by using an eventfd+poll to explicitly signal the CM event
thread for termination.

Tested by running 4096 parallel rping processes.

Fixes: 6f640ff ("r7019: Introduce event channels.")
Signed-off-by: Jacob Moroni <jmoroni@google.com>
Signed-off-by: Nicolas Morey <nmorey@suse.com>