From owner-cvs-src@FreeBSD.ORG Mon Sep 22 22:29:30 2008 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 85CC0106564A; Mon, 22 Sep 2008 22:29:30 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from server.baldwin.cx (bigknife-pt.tunnel.tserv9.chi1.ipv6.he.net [IPv6:2001:470:1f10:75::2]) by mx1.freebsd.org (Postfix) with ESMTP id 2730D8FC23; Mon, 22 Sep 2008 22:29:30 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from localhost.corp.yahoo.com (john@localhost [IPv6:::1]) (authenticated bits=0) by server.baldwin.cx (8.14.2/8.14.2) with ESMTP id m8MMTNqA064204; Mon, 22 Sep 2008 18:29:24 -0400 (EDT) (envelope-from jhb@freebsd.org) From: John Baldwin To: src-committers@freebsd.org Date: Mon, 22 Sep 2008 17:12:28 -0400 User-Agent: KMail/1.9.7 References: <200809222053.m8MKrblQ032408@repoman.freebsd.org> In-Reply-To: <200809222053.m8MKrblQ032408@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200809221712.28256.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH authentication, not delayed by milter-greylist-2.0.2 (server.baldwin.cx [IPv6:::1]); Mon, 22 Sep 2008 18:29:24 -0400 (EDT) X-Virus-Scanned: ClamAV 0.93.1/8310/Mon Sep 22 14:58:13 2008 on server.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-2.5 required=4.2 tests=AWL,BAYES_00,NO_RELAYS autolearn=ham version=3.1.3 X-Spam-Checker-Version: SpamAssassin 3.1.3 (2006-06-01) on server.baldwin.cx Cc: cvs-src@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/ufs/ufs ufs_dirhash.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 22 Sep 2008 22:29:30 -0000 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