diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-04-30 20:27:39 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2009-09-14 01:18:13 -0400 |
commit | 6e3647acb4f200add1d8e0203514f7ac925ae463 (patch) | |
tree | 95a779d1f59c04ca58956a49340b88497b476f13 | |
parent | 4c5eface5d0e4eb7f77be346193c2850e7e3b983 (diff) |
kill-the-BKL/reiserfs: release the write lock on flush_commit_list()
flush_commit_list() uses ll_rw_block() to commit the pending log blocks.
ll_rw_block() might sleep, and the bkl was released at this point. Then
we can also relax the write lock at this point.
[ Impact: release the reiserfs write lock when it is not needed ]
Cc: Jeff Mahoney <jeffm@suse.com>
Cc: Chris Mason <chris.mason@oracle.com>
Cc: Alexander Beregalov <a.beregalov@gmail.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
-rw-r--r-- | fs/reiserfs/journal.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/reiserfs/journal.c b/fs/reiserfs/journal.c index 5cd600baf8cf..ffb7f50abc2f 100644 --- a/fs/reiserfs/journal.c +++ b/fs/reiserfs/journal.c | |||
@@ -1120,8 +1120,11 @@ static int flush_commit_list(struct super_block *s, | |||
1120 | SB_ONDISK_JOURNAL_SIZE(s); | 1120 | SB_ONDISK_JOURNAL_SIZE(s); |
1121 | tbh = journal_find_get_block(s, bn); | 1121 | tbh = journal_find_get_block(s, bn); |
1122 | if (tbh) { | 1122 | if (tbh) { |
1123 | if (buffer_dirty(tbh)) | 1123 | if (buffer_dirty(tbh)) { |
1124 | ll_rw_block(WRITE, 1, &tbh) ; | 1124 | reiserfs_write_unlock(s); |
1125 | ll_rw_block(WRITE, 1, &tbh); | ||
1126 | reiserfs_write_lock(s); | ||
1127 | } | ||
1125 | put_bh(tbh) ; | 1128 | put_bh(tbh) ; |
1126 | } | 1129 | } |
1127 | } | 1130 | } |