diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2008-05-11 19:04:48 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-05-11 19:04:48 -0400 |
commit | c3921ab71507b108d51a0f1ee960f80cd668a93d (patch) | |
tree | b1408b898a8b50f15ad4a0cf1f29e17cc0138485 /fs/locks.c | |
parent | 9662369786b9d07fd46d65b0f9e3938a3e01a5d9 (diff) |
Add new 'cond_resched_bkl()' helper function
It acts exactly like a regular 'cond_resched()', but will not get
optimized away when CONFIG_PREEMPT is set.
Normal kernel code is already preemptable in the presense of
CONFIG_PREEMPT, so cond_resched() is optimized away (see commit
02b67cc3ba36bdba351d6c3a00593f4ec550d9d3 "sched: do not do
cond_resched() when CONFIG_PREEMPT").
But when wanting to conditionally reschedule while holding a lock, you
need to use "cond_sched_lock(lock)", and the new function is the BKL
equivalent of that.
Also make fs/locks.c use it.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/locks.c')
-rw-r--r-- | fs/locks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/locks.c b/fs/locks.c index 0ac6b92cb0b6..11dbf08651b7 100644 --- a/fs/locks.c +++ b/fs/locks.c | |||
@@ -773,7 +773,7 @@ static int flock_lock_file(struct file *filp, struct file_lock *request) | |||
773 | * give it the opportunity to lock the file. | 773 | * give it the opportunity to lock the file. |
774 | */ | 774 | */ |
775 | if (found) | 775 | if (found) |
776 | cond_resched(); | 776 | cond_resched_bkl(); |
777 | 777 | ||
778 | find_conflict: | 778 | find_conflict: |
779 | for_each_lock(inode, before) { | 779 | for_each_lock(inode, before) { |