Re: cvs commit: src/sys/kern kern_fork.c kern_proc.c src/sys/sys proc.h src/sys/vfs/procfs procfs_vnops.c

[joerg@xxxxxxxxxxxxxxxxx]

On Wed, May 24, 2006 at 12:16:50PM -0700, Matthew Dillon wrote:
> 
> :
> :On Wed, May 24, 2006 at 11:59:51AM -0700, Matthew Dillon wrote:
> :>   Integrate pid generation for the new process into proc_add_allproc(), move
> :>   all related code from kern_fork.c to kern_proc.c.
> :
> :Can you make it a separate function instead? We might want to use a 
> :shared namespace for thread ids and process ids in the long run, I'm not
> :sure yet. That would make it easier to deal with.
> :
> :Joerg
> 
>     The allproc and zombproc lists are used to determine whether a pid is
>     in use or not.  So the new pid generation has to be integrated with
>     adding the process to the allproc list to guarentee that a process with
>     a duplicate pid is not created.

Yes and I want to change that behaviour to decouple them at one point or
another. I have no problem with keeping them under the same lock, but
I'd like to see the isolation :-)

Joerg