Messages in this thread | | | Date | Tue, 15 Jun 2010 17:25:40 +1000 | From | Paul Mackerras <> | Subject | Re: [PATCH] Fix a race in pid generation that causes pids to be reused immediately. |
| |
On Mon, Jun 14, 2010 at 09:21:50PM -0700, Andrew Morton wrote:
> If that happens then the best fix is for those architectures to get > themselves a cmpxchg64(). Unless for some reason it's simply > unimplementable? Worst case I guess one could use a global spinlock. > Second-worst-case: hashed spinlocks.
Spinlocks won't help as long as you can use cmpxchg64 on any old u64 that is also accessed directly. UP can disable interrupts, of course, but for SMP we would have to restrict cmpxchg64 to some type (atomic64_t, maybe) for which you have to use an accessor function to read it, so we can take the spinlock around the reads.
I suspect it isn't worth the trouble. Note that I'm talking specifically about cmpxchg64 here, not cmpxchg.
Paul.
| |