Date: Fri, 10 Sep 2010 23:49:33 +0000 (UTC) From: Rick Macklem <rmacklem@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/fs/nfsserver nfs_nfsdstate.c Message-ID: <201009102349.o8ANnkJj057538@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
rmacklem 2010-09-10 23:49:33 UTC FreeBSD src repository Modified files: sys/fs/nfsserver nfs_nfsdstate.c Log: SVN rev 212443 on 2010-09-10 23:49:33Z by rmacklem 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. MFC after: 2 weeks Revision Changes Path 1.21 +6 -1 src/sys/fs/nfsserver/nfs_nfsdstate.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201009102349.o8ANnkJj057538>