Subject: Re: i guess this is your change. please fix.
To: None <manu@netbsd.org>
From: None <itojun@iijlab.net>
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?
itojun