From: Mathieu Desnoyers Date: Wed, 1 May 2013 20:52:59 +0000 (-0400) Subject: Fix: consumerd metadata channel/cache/timer races X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=ea88ca2a0b3fa898e245893c43ae2185da0acc44;hp=ea88ca2a0b3fa898e245893c43ae2185da0acc44 Fix: consumerd metadata channel/cache/timer races There were various races with the way metadata was handled for UST. Fix this by: - Ensuring that metadata cache is destroyed only when the metadata channel is destroyed (it is already protected by refcounting). - Ensuring that close_metadata() command (issued by sessiond) just closes the stream wakeup_fd, and let the metadata management thread (which has ownership of the metadata stream) perform the teardown. - Ensure that switch timer, for metadata, is always stopped before any of the metadata stream or channel teardown is done. Signed-off-by: Mathieu Desnoyers Signed-off-by: David Goulet ---