Date: Mon, 4 Jul 2011 23:32:09 +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: <201107042332.p64NWIuh019503@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
rmacklem 2011-07-04 23:32:09 UTC
FreeBSD src repository
Modified files:
sys/fs/nfsclient nfs_clstate.c
Log:
SVN rev 223774 on 2011-07-04 23:32:09Z by rmacklem
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.
MFC after: 2 weeks
Revision Changes Path
1.26 +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?201107042332.p64NWIuh019503>