[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-ext4
Subject: Re: [PATCH 2/5] jbd2: optimize jbd2_journal_force_commit
From: Theodore Ts'o <tytso () mit ! edu>
Date: 2013-08-28 18:33:40
Message-ID: 20130828183340.GD27079 () thunk ! org
[Download RAW message or body]
On Tue, Apr 23, 2013 at 10:51:40AM +0200, Jan Kara wrote:
> > Do you give me your ACK/Reviewed signature?
> Looking at it with a fresh mind I think there's still one bug:
> read_unlock(&journal->j_state_lock);
> if (need_to_start)
> - jbd2_log_start_commit(journal, tid);
> - jbd2_log_wait_commit(journal, tid);
> - return 1;
> + ret = jbd2_log_start_commit(journal, tid);
> + if (!ret)
> + ret = jbd2_log_wait_commit(journal, tid);
> jbd2_log_start_commit() will return 0 if it didn't wake jbd2 thread and 1
> if it did. In either case we should call jbd2_log_wait_commit()...
>
> + if (!ret && progress)
> + *progress = 1;
I was going through old patch series, and I came across this one.
Dmitry, did you have a chance to look at Jan's comments?
Thanks,
- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic