Re: Replace lockmgr for vnodes

[Antti Kantee <pooka%cs.hut.fi@localhost>]

On Tue Jan 29 2008 at 03:48:57 +0000, David Holland wrote:
> On Mon, Jan 28, 2008 at 06:14:27PM -0800, Bill Stouder-Studenmund wrote:
>  > Do all of the places that try to lock a kmutex always check return values? 
>  
>  > The older locking primitives, such as vnode locks, often were called in
>  > places that didn't cope with getting a return error code. Many times that
>  > means they didn't bother to check. :-)
> 
> No, they don't. In the long run I'm intending to fix at least the
> vnode locks, because IMO they should be interruptible, but that'll
> likely be a slow process.

While this is theoretically a good idea, I am not sure it won't introduce
more bugs than it works around.  Every lock call would demand an error
path, and error paths are usually good at one thing: not being tested
at all.  Increased code clutter is another downside.

-- 
Antti Kantee <pooka%iki.fi@localhost>                     Of course he runs 
NetBSD
http://www.iki.fi/pooka/                          http://www.NetBSD.org/
    "la qualité la plus indispensable du cuisinier est l'exactitude"