On Mon, Sep 27, 2010 at 7:20 AM, Chandra Seetharaman <sekharan@xxxxxxxxxx> wrote: > > [ ... ] > > Mainly, I found 3 issues, > > - tgt_events_list (global) needed protection > - it_nexus_list (in struct target) needed protection > - setting of tcp_conn->pthread to 1 in iscsi_tcp_conn_nexus_init() > need to happen before do_tgt_event_add(). > > [ ... ] (resending as plain text) Hello Chandra, Are you aware that there exist open source tools that detect data races automatically, e.g. DRD and Helgrind ? See also http://www.valgrind.org/docs/manual/manual.html for more information. Disclaimer: I'm the author of the first mentioned tool. Bart. -- To unsubscribe from this list: send the line "unsubscribe stgt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html