Subject: Re: i guess this is your change. please fix.
To: None <>
From: None <>
List: current-users
Date: 03/23/2001 00:51:17
>>	/* Set the process idea of the break to the real value */	<- added
>>	((struct linux_emuldata*)(p->p_emuldata))->p_break = 		<-
>>	    p->p_vmspace->vm_daddr + ctob(p->p_vmspace->vm_dsize);	<-
>	oops, the analysis may not be correct, but the function is defintely
>	doing something bad.
>Well, this touches p_emuldata, but it has been allocated just before if it
>was not already allocated... 
>If something bad happens, then it's because p_vmspace on vm_addr, or vm_dsize
>is stale, I don't see anything else that could be wrong.

	yes, i was confused.  lockup (kernel NULL pointer dereference)
	happened because p->p_vmspace was not initialized in some cases.
	could you please check the codepath again?