diff options
| author | Kent Overstreet <kent.overstreet@gmail.com> | 2020-11-20 19:27:57 -0500 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@gmail.com> | 2020-12-16 15:29:08 -0500 |
| commit | f4baf579053de2156d2ee3b14347d73b88e35dc4 (patch) | |
| tree | f45d82ee64e571762130ce620e02f53b51236faf | |
| parent | 55d4b077760a2009faaa478ab2ce91edfb550862 (diff) | |
bcachefs: Optimize bch2_journal_flush_seq_async()
Avoid taking the journal lock if we don't have to.
Signed-off-by: Kent Overstreet <kent.overstreet@gmail.com>
| -rw-r--r-- | fs/bcachefs/journal.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/fs/bcachefs/journal.c b/fs/bcachefs/journal.c index 1c4dce654aa0..5874a9ff2204 100644 --- a/fs/bcachefs/journal.c +++ b/fs/bcachefs/journal.c @@ -548,12 +548,20 @@ out: * necessary */ int bch2_journal_flush_seq_async(struct journal *j, u64 seq, - struct closure *parent) + struct closure *parent) { struct journal_buf *buf; int ret = 0; + if (seq <= j->err_seq) + return -EIO; + + if (seq <= j->seq_ondisk) + return 1; + spin_lock(&j->lock); + + /* Recheck under lock: */ if (seq <= j->err_seq) { ret = -EIO; goto out; |