diff options
| author | Andrew Morton <akpm@digeo.com> | 2003-06-17 18:30:52 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@home.transmeta.com> | 2003-06-17 18:30:52 -0700 |
| commit | eba4b4b7ac85e7e37af12d6aa1031325dffc1bc5 (patch) | |
| tree | 3ef2d31e6fb547fe53c560a0c3efbd107a18ef94 | |
| parent | dd71e33fcd7e9e3bc89ca87a02c3676417d697f5 (diff) | |
[PATCH] JBD: buffer freeing non-race comment
Add a comment describing why a race isn't there.
| -rw-r--r-- | fs/jbd/transaction.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/fs/jbd/transaction.c b/fs/jbd/transaction.c index fad662aea660..6ee3a0820f2e 100644 --- a/fs/jbd/transaction.c +++ b/fs/jbd/transaction.c @@ -1643,6 +1643,11 @@ int journal_try_to_free_buffers(journal_t *journal, bh = head; do { jbd_lock_bh_state(bh); + /* + * We don't have to worry about the buffer being pulled off its + * journal_head in here, because __try_to_free_cp_buf runs + * under jbd_lock_bh_state() + */ if (buffer_jbd(bh) && !__journal_try_to_free_buffer(journal, bh)) { jbd_unlock_bh_state(bh); |