Re: [lttng-dev] [PATCH lttng-tools] Fix: consumerd: order of metadata cache vs stream lock

2017-01-09 Thread Jérémie Galarneau
Thanks for getting to the bottom of it! Merged in master, stable-2.8 and stable-2.9. Jérémie On 9 January 2017 at 11:23, Mathieu Desnoyers wrote: > The locking order comment in consumer.h is incorrect. First, its > description of locking order is not in sync with

[lttng-dev] [PATCH lttng-tools] Fix: consumerd: order of metadata cache vs stream lock

2017-01-09 Thread Mathieu Desnoyers
The locking order comment in consumer.h is incorrect. First, its description of locking order is not in sync with the comment found in consumer-metadata-cache.h. The comment in struct consumer_metadata_cache only states that the metadata cache lock nests inside the consumer_data lock, and does not