[ltt-dev] [UST PATCH] libust: Fix multiple fd close during fork
Mathieu Desnoyers
compudj at krystal.dyndns.org
Wed Feb 23 10:40:05 EST 2011
* Nils Carlson (nils.carlson at ericsson.com) wrote:
> Yep. fixed. merged.
>
>>>> + /*
>>>> + * FIXME: This could be prettier, we loop over the list twice and
>>>> + * following good locking practice should lock around the loop
>>>> + */
>>>> + cds_list_for_each_entry_safe(trace, trace_tmp, <t_traces.head,
>>>> list) {
>>>> + ltt_trace_destroy(trace->trace_name, 1);
>>>> + }
So what' up with these missing locks ?
Mathieu
>>>>
>>>> /* Clean up the listener socket and epoll, keeping the scoket
>>>> file */
>>>> ustcomm_del_named_sock(listen_sock, 1);
>>>> --
>>>> 1.7.1
>>>>
>>>>
>>
>>
>
> _______________________________________________
> ltt-dev mailing list
> ltt-dev at lists.casi.polymtl.ca
> http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
>
--
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com
More information about the lttng-dev
mailing list