diff options
author | Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> | 2016-08-02 17:05:19 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-08-02 19:35:19 -0400 |
commit | aceb4170bb2ba88c5327cc69b9c91a708c7f7046 (patch) | |
tree | a6e402b2cecae9878f7aebeb1c9006493f4d4e59 | |
parent | 39a9dcca61a3d1375b9440676cbfc541804cd217 (diff) |
nilfs2: do not use yield()
Use cond_resched() instead of yield() in the loop of
nilfs_transaction_lock() since the usage corresponds to the "be nice for
others" case that the comment of yield() says.
This removes the following checkpatch.pl warning:
"WARNING: Using yield() is generally wrong. See yield() kernel-doc
(sched/core.c)"
Link: http://lkml.kernel.org/r/1464875891-5443-8-git-send-email-konishi.ryusuke@lab.ntt.co.jp
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/nilfs2/segment.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nilfs2/segment.c b/fs/nilfs2/segment.c index 7e1864c6035b..5a97282aa074 100644 --- a/fs/nilfs2/segment.c +++ b/fs/nilfs2/segment.c | |||
@@ -373,7 +373,7 @@ static void nilfs_transaction_lock(struct super_block *sb, | |||
373 | nilfs_segctor_do_immediate_flush(sci); | 373 | nilfs_segctor_do_immediate_flush(sci); |
374 | 374 | ||
375 | up_write(&nilfs->ns_segctor_sem); | 375 | up_write(&nilfs->ns_segctor_sem); |
376 | yield(); | 376 | cond_resched(); |
377 | } | 377 | } |
378 | if (gcflag) | 378 | if (gcflag) |
379 | ti->ti_flags |= NILFS_TI_GC; | 379 | ti->ti_flags |= NILFS_TI_GC; |