Call sv_onexec handler after the process VA is created.

[dchagin]

Summary

For future use in the Linux emulation layer call sv_onexec handler
right after the new process address space is created.
It's safe, as sv_onexec used only by Linux abi and linux_on_exec()
does not depend on a state of process VA.

[kib]

Does the move after process event handler breaks existing linux_on_exec()?

sys/kern/kern_exec.c

1116

This is weird place to put the call. The vmspace is indeed constructed, but not everything yet mapped into it. It is not quite logical to have e.g. shared page mapped, but stack not, when claiming that you put the callback after the vmspace is constructed.

[dchagin]

In D30899#695137, @kib wrote:

Does the move after process event handler breaks existing linux_on_exec()?

no, I even tested it, linux_on_exec() does not depend on process vm space noe eventhandler

sys/kern/kern_exec.c

1116

well, I can put it right after the stack creation or change clarification, for linux_on_exec() it doesn't matter

[dchagin]

move linux_on_exec() call below, after the stack is created

[kib]

sys/kern/kern_exec.c

1116

Then move it to the very end of the function, right before return.

1157

Move the call right before this return.

[dchagin]

fixed, thank you,
one question, in D30900 I'll change this, sv_onexec() will return an error,
should I call vm_object_deallocate(obj) or somthing other. seems that
on if vm_map_stack() return error no resources are released

[kib]

In D30899#695202, @dchagin wrote:

fixed, thank you,
one question, in D30900 I'll change this, sv_onexec() will return an error,
should I call vm_object_deallocate(obj) or somthing other. seems that
on if vm_map_stack() return error no resources are released

Everything done before the call is recorded as the state of the new vmspace. In case of error return, we eventually call exit1() terminate the process, and this destroys current vmspace, which holds references to the shared_page object etc.

I suggest you to make the dedicated change for making sv_onexec() possibly return an error.