Re: BUG in shared_policy_replace() ?

[Steve Longerbeam]

Andi Kleen wrote:

> > got it, except that there is no "new2 = NULL;" in 2.6.10-mm2!
> >
> > Looks like it was misplaced, because I do see it now in 2.6.10.
> >    
>
> I double checked 2.6.10 and the code also looks correct me,
> working as described by Hugh.
>
> Optimistic locking can be ugly :)
>  

yeah, 2.6.10 makes sense to me too. But I'm working in -mm2, and
the new2 = NULL line is missing, hence my initial confusion. Trivial
patch to -mm2 attached. Just want to make sure it has been, or will be,
put back in.

Steve
--- mm/mempolicy.c.orig	2005-01-19 09:52:47.153910873 -0800
+++ mm/mempolicy.c	2005-01-19 09:53:21.548999628 -0800
@@ -1041,6 +1041,7 @@
 				}
 				n->end = start;
 				sp_insert(sp, new2);
+				new2 = NULL;
 				break;
 			} else
 				n->end = start;