Re: CVS commit: src/sys/kern

[Andrew Doran <ad%netbsd.org@localhost>]

On Sun, Oct 12, 2008 at 09:26:50AM +0000, Iain Hibbert wrote:

> Module Name:  src
> Committed By: plunky
> Date:         Sun Oct 12 09:26:50 UTC 2008
> 
> Modified Files:
>       src/sys/kern: uipc_socket.c
> 
> Log Message:
> fix problem pointed out by ad where sockopt may end up sleeping
> inappropriately with the socket lock held.
> 
>     sockopt_init() may sleep
> 
>     sockopt_set() will not sleep
> 
>     sockopt_getmbuf() for legacy code will not sleep

Are you sure that enough space is always be available for sockopt_set()?
If not, legitimate operations will be failed if the system is paging.

Thanks,
Andrew