Subject: Re: new pid allocation code
To: Alan Barrett <apb@cequrux.com>
From: David Laight <david@l8s.co.uk>
List: tech-kern
Date: 03/13/2003 09:55:21
> I see several changes from lists managed by macros to lists managed by
> hand-coded pointer manipulation.  Is this wise?
> 
> For example:
> > Index: sys/proc.h
>   [...]
> > -extern struct proclist deadproc;	/* List of dead processes */
> > +extern struct proc	*deadprocs;	/* List of dead processes */
>   [...]
> > Index: kern/kern_exit.c
>   [...]
> > -	LIST_INSERT_HEAD(&deadproc, p, p_hash);
> > +	p->p_dead = deadprocs;
> > +	deadprocs = p;

Well the deadproc list was using the p_hash member - which no longer exists.
Since processes only sit on the deadprocs list for a very short time,
changing it for FIFO to LIFO isn't an issue.
A LIFO list doesn't need any macros...

For a readability point of view I find the 'hand-coded pointer manipulation'
better than the list macros.

	David

-- 
David Laight: david@l8s.co.uk