Messages in this thread | | | Date | Sun, 13 Mar 2011 18:15:44 -0700 | From | Darren Hart <> | Subject | Re: [PATCH 1/3] futex: do not pagefault_disable in futex_atomic_cmpxchg_inatomic() |
| |
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
| |