Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > - int ret; Do you need to put 'ret' back as it's now used a little further on: > if (clone_flags & CLONE_THREAD) { > - atomic_inc(¤t->signal->count); > - atomic_inc(¤t->signal->live); > - return 0; > + ret = thread_group_cputime_clone_thread(current); > + if (likely(!ret)) { > + atomic_inc(¤t->signal->count); > + atomic_inc(¤t->signal->live); > + } > + return ret; The next bit is fine: > + thread_group_cputime_free(sig); > - exit_thread_group_keys(sig); David -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html