d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

getblk: Remove a non-sensical LK_NOWAIT | LK_SLEEPFAIL

Browse Source 
No functional change.

LK_SLEEPFAIL implies a behavior that is only possible if the lock operation can
sleep.  LK_NOWAIT prevents the lock operation from sleeping.

Discussed with:	kib
This commit is contained in:
Conrad Meyer 2020-07-31 00:13:40 +00:00
parent 59d13f6154
commit d6a75d39e9
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/kern/vfs_bio.c
View File

@ -3887,10 +3887,8 @@ loop:
* Buffer is in-core. If the buffer is not busy nor managed, * Buffer is in-core. If the buffer is not busy nor managed,
* it must be on a queue. * it must be on a queue.
*/ */
lockflags = LK_EXCLUSIVE | LK_SLEEPFAIL | LK_INTERLOCK; lockflags = LK_EXCLUSIVE | LK_INTERLOCK |
((flags & GB_LOCK_NOWAIT) ? LK_NOWAIT : LK_SLEEPFAIL);
if ((flags & GB_LOCK_NOWAIT) != 0)
lockflags |= LK_NOWAIT;
error = BUF_TIMELOCK(bp, lockflags, error = BUF_TIMELOCK(bp, lockflags,
BO_LOCKPTR(bo), "getblk", slpflag, slptimeo); BO_LOCKPTR(bo), "getblk", slpflag, slptimeo);