Re: [PATCH 1/3] futex: do not pagefault_disable in futex_atomic_cmpxchg_inatomic()

From: Darren Hart
Date: Sun Mar 13 2011 - 21:15:57 EST


On 03/13/2011 05:55 PM, Darren Hart wrote:
I have to apologize. These landed right as I was leaving for OSTS and
I didn't have time to review them properly. I wondered about the
preempt vs pagefault disable and wondered how exit_robust_list() was
covered in Michel's comments (perhaps somewhere higher up the call
chain). I leaned on the review of others when I should have raised the
questions even if I didn't have the time to dig into them myself.
Linus shouldn't have had to raise those questions, I'll do better at
this in the future.

I noticed that my name is the only one in futex.c with an email
address in the header

Scratch that, the others are just on the Copyright line instead of the feature line. What is the policy for updating email addresses in source files? With git, I suppose they are no longer particularly useful. Should I remove a bad address, or just learn from it and not include it in future work?

Thanks,

--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/