Re: CVS commit: syssrc

[Jason R Thorpe <thorpej%zembu.com@localhost>]

On Wed, Jun 06, 2001 at 08:00:01PM +0300, Jason R Thorpe wrote:

 > Module Name: syssrc
 > Committed By:        thorpej
 > Date:                Wed Jun  6 17:00:01 UTC 2001
 > 
 > Modified Files:
 >      syssrc/sys/kern: kern_descrip.c uipc_usrreq.c
 >      syssrc/sys/sys: filedesc.h
 > 
 > Log Message:
 > Change fdalloc() to return ERESTART if we had to reallocate the
 > descriptor array, which may have blocked.  Change callers of
 > fdalloc() to restart whatever they're doing if this condition
 > happens.  (XXX unp_externalize() needs some work, but that will
 > be tackled later.)
 > 
 > While here, garbage-collect UF_MAPPED -- it is not used anywhere.

Oops, forgot a couple of lines -- should have read:

Change fdalloc() to return ERESTART if we had to reallocate the
descriptor array, which may have blocked.  Change callers of
fdalloc() to restart whatever they're doing if this condition
happens.  (XXX unp_externalize() needs some work, but that will
be tackled later.)
 
Change finishdup() to close the descriptor in the `new' slot if
one exists, and change sys_dup(2) accordingly.

Closes a race condition when using kernel-assisted user threads.

While here, garbage-collect UF_MAPPED -- it is not used anywhere.

It's being fixed in the CVS repository.

-- 
        -- Jason R. Thorpe <thorpej%zembu.com@localhost>