Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 22 Sep 2008 17:12:28 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        src-committers@freebsd.org
Cc:        cvs-src@freebsd.org, cvs-all@freebsd.org
Subject:   Re: cvs commit: src/sys/ufs/ufs ufs_dirhash.c
Message-ID:  <200809221712.28256.jhb@freebsd.org>
In-Reply-To: <200809222053.m8MKrblQ032408@repoman.freebsd.org>
References:  <200809222053.m8MKrblQ032408@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Monday 22 September 2008 04:53:22 pm John Baldwin wrote:
> jhb         2008-09-22 20:53:22 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/ufs/ufs          ufs_dirhash.c 
>   Log:
>   SVN rev 183280 on 2008-09-22 20:53:22Z by jhb
>   
>   Close a race between concurrent calls to ufsdirhash_recycle() and
>   ufsdirhash_free() introduced in my last commit by removing the dirhash
>   about to be free'd in ufsdirhash_free() from the global dirhash list
>   before dropping the sx lock.
>   
>   Tested by:      kris

Aside from some pending patches to better handle errors during a umount -f 
case (albeit an edge case), I think UFS is ok to do shared lookups now.

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200809221712.28256.jhb>