diff options
| author | kmacy <kmacy@FreeBSD.org> | 2009-04-13 23:09:44 +0000 |
|---|---|---|
| committer | kmacy <kmacy@FreeBSD.org> | 2009-04-13 23:09:44 +0000 |
| commit | de9c351c80ae5cad56bd50b8f9bc334fa036ce81 (patch) | |
| tree | f975a852b9dfdda34880397a264222cdf2d7ba33 /sys/kern/vfs_vnops.c | |
| parent | a72478d02df097c514b846cc884a22e2856071e6 (diff) | |
| download | FreeBSD-src-de9c351c80ae5cad56bd50b8f9bc334fa036ce81.zip FreeBSD-src-de9c351c80ae5cad56bd50b8f9bc334fa036ce81.tar.gz | |
- use a shared lock for reads
- remove stale comment
Reviewed by: jeffr
Diffstat (limited to 'sys/kern/vfs_vnops.c')
| -rw-r--r-- | sys/kern/vfs_vnops.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/sys/kern/vfs_vnops.c b/sys/kern/vfs_vnops.c index 48a74ac..2f085d9 100644 --- a/sys/kern/vfs_vnops.c +++ b/sys/kern/vfs_vnops.c @@ -379,14 +379,8 @@ vn_rdwr(rw, vp, base, len, offset, segflg, ioflg, active_cred, file_cred, != 0) return (error); vn_lock(vp, LK_EXCLUSIVE | LK_RETRY); - } else { - /* - * XXX This should be LK_SHARED but I don't trust VFS - * enough to leave it like that until it has been - * reviewed further. - */ - vn_lock(vp, LK_EXCLUSIVE | LK_RETRY); - } + } else + vn_lock(vp, LK_SHARED | LK_RETRY); } ASSERT_VOP_LOCKED(vp, "IO_NODELOCKED with no vp lock held"); |
