[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, &ltt_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