Re: [patch] user-vm-unlock-2.5.31-A2

[Ingo Molnar]

On Thu, 15 Aug 2002, Jamie Lokier wrote:

> Is this correct?  I would have expected this, given that stacks are
> pre-decrement, and given that the value of `esp' is typically just after
> the end of an mmaped region:
> 
> +		childregs->esp -= sizeof(0UL);
> +		p->user_vm_lock = (long *) childregs->esp;

you are right. Fix against BK-curr attached.

	Ingo

--- linux/arch/i386/kernel/process.c.orig	Thu Aug 15 08:37:52 2002
+++ linux/arch/i386/kernel/process.c	Thu Aug 15 08:37:59 2002
@@ -627,7 +627,7 @@
 	 */
 	if (clone_flags & CLONE_RELEASE_VM) {
 		childregs->esp -= sizeof(0UL);
-		p->user_vm_lock = (long *) esp;
+		p->user_vm_lock = (long *) childregs->esp;
 	}
 	return 0;
 }