Date: Mon, 18 Jul 2011 21:04:35 +0000 (UTC) From: Rick Macklem <rmacklem@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/fs/nfsclient nfs_clstate.c Message-ID: <201107182104.p6IL4xZN002005@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
rmacklem 2011-07-18 21:04:35 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) sys/fs/nfsclient nfs_clstate.c Log: SVN rev 224203 on 2011-07-18 21:04:35Z by rmacklem MFC: r223774 The algorithm used by nfscl_getopen() could have resulted in multiple instances of the same lock_owner when a process both inherited an open file descriptor plus opened the same file itself. Since some NFSv4 servers cannot handle multiple instances of the same lock_owner string, this patch changes the algorithm used by nfscl_getopen() in the new NFSv4 client to keep that from happening. The new algorithm is simpler, since there is no longer any need to ascend the process's parentage tree because all NFSv4 Closes for a file are done at VOP_INACTIVE()/VOP_RECLAIM(), making the Opens indistinct w.r.t. use with Lock Ops. This problem was discovered at the recent NFSv4 interoperability Bakeathon. Revision Changes Path 1.7.2.18 +94 -81 src/sys/fs/nfsclient/nfs_clstate.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201107182104.p6IL4xZN002005>