Merge branch 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs

Pull ext3/jbd fixes from Jan Kara:
 "A couple of ext3/jbd fixes"

* 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs:
  jbd: use kmem_cache_zalloc for allocating journal head
  jbd: use kmem_cache_zalloc instead of kmem_cache_alloc/memset
  jbd: don't wait (forever) for stale tid caused by wraparound
  ext3: fix data=journal fast mount/umount hang

1 files changed, 14 insertions, 5 deletions

diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c
index 865c4308acb6..6510d6355729 100644
--- a/fs/jbd/journal.c
+++ b/fs/jbd/journal.c
@@ -562,6 +562,16 @@ int log_wait_commit(journal_t *journal, tid_t tid)
         spin_unlock(&journal->j_state_lock);
 #endif
         spin_lock(&journal->j_state_lock);
+        /*
+         * Not running or committing trans? Must be already committed. This
+         * saves us from waiting for a *long* time when tid overflows.
+         */
+        if (!((journal->j_running_transaction &&
+               journal->j_running_transaction->t_tid == tid) ||
+              (journal->j_committing_transaction &&
+               journal->j_committing_transaction->t_tid == tid)))
+                goto out_unlock;
+
         if (!tid_geq(journal->j_commit_waited, tid))
                 journal->j_commit_waited = tid;
         while (tid_gt(tid, journal->j_commit_sequence)) {
@@ -573,6 +583,7 @@ int log_wait_commit(journal_t *journal, tid_t tid)
                                 !tid_gt(tid, journal->j_commit_sequence));
                 spin_lock(&journal->j_state_lock);
         }
+out_unlock:
         spin_unlock(&journal->j_state_lock);
 
         if (unlikely(is_journal_aborted(journal))) {
@@ -1843,7 +1854,7 @@ static struct journal_head *journal_alloc_journal_head(void)
 #ifdef CONFIG_JBD_DEBUG
         atomic_inc(&nr_journal_heads);
 #endif
-        ret = kmem_cache_alloc(journal_head_cache, GFP_NOFS);
+        ret = kmem_cache_zalloc(journal_head_cache, GFP_NOFS);
         if (ret == NULL) {
                 jbd_debug(1, "out of memory for journal_head\n");
                 printk_ratelimited(KERN_NOTICE "ENOMEM in %s, retrying.\n",
@@ -1851,7 +1862,7 @@ static struct journal_head *journal_alloc_journal_head(void)
 
                 while (ret == NULL) {
                         yield();
-                        ret = kmem_cache_alloc(journal_head_cache, GFP_NOFS);
+                        ret = kmem_cache_zalloc(journal_head_cache, GFP_NOFS);
                 }
         }
         return ret;
@@ -1913,10 +1924,8 @@ struct journal_head *journal_add_journal_head(struct buffer_head *bh)
         struct journal_head *new_jh = NULL;
 
 repeat:
-        if (!buffer_jbd(bh)) {
+        if (!buffer_jbd(bh))
                 new_jh = journal_alloc_journal_head();
-                memset(new_jh, 0, sizeof(*new_jh));
-        }
 
         jbd_lock_bh_journal_head(bh);
         if (buffer_jbd(bh)) {