From owner-cvs-src-old@FreeBSD.ORG Mon Sep 27 02:31:22 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7DB601065696 for ; Mon, 27 Sep 2010 02:31:22 +0000 (UTC) (envelope-from rmacklem@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 6BC078FC27 for ; Mon, 27 Sep 2010 02:31:22 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id o8R2VMfE029607 for ; Mon, 27 Sep 2010 02:31:22 GMT (envelope-from rmacklem@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id o8R2VMKc029606 for cvs-src-old@freebsd.org; Mon, 27 Sep 2010 02:31:22 GMT (envelope-from rmacklem@repoman.freebsd.org) Message-Id: <201009270231.o8R2VMKc029606@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to rmacklem@repoman.freebsd.org using -f From: Rick Macklem Date: Mon, 27 Sep 2010 02:30:56 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_8 Subject: cvs commit: src/sys/fs/nfsserver nfs_nfsdstate.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Sep 2010 02:31:22 -0000 rmacklem 2010-09-27 02:30:56 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) sys/fs/nfsserver nfs_nfsdstate.c Log: SVN rev 213191 on 2010-09-27 02:30:56Z by rmacklem MFC: r212443 This patch applies one of the two fixes suggested by zack.kirsch at isilon.com for a race between nfsrv_freeopen() and nfsrv_getlockfile() in the experimental NFS server that he found during testing. Although nfsrv_freeopen() holds a sleep lock on the lock file structure when called with cansleep != 0, nfsrv_getlockfile() could still search the list, once it acquired the NFSLOCKSTATE() mutex. I believe that acquiring the mutex in nfsrv_freeopen() fixes the race. Revision Changes Path 1.10.2.12 +6 -1 src/sys/fs/nfsserver/nfs_nfsdstate.c