jfs: fix a couple races
authorDave Kleikamp <dave.kleikamp@oracle.com>
Wed, 1 May 2013 16:08:38 +0000 (11:08 -0500)
committerDave Kleikamp <dave.kleikamp@oracle.com>
Wed, 1 May 2013 16:16:59 +0000 (11:16 -0500)
This patch fixes races uncovered by xfstests testcase 068.

One race is the result of jfs_sync() trying to write a sync point to the
journal after it has been frozen (or possibly in the process). Since
freezing sync's the journal, there is no need to write a sync point so
we simply want to return.

The second involves jfs_write_inode() being called on a deleted inode.
It calls jfs_flush_journal which is held up by the jfs_commit thread
doing the final iput on the same deleted inode, which itself is
waiting for the I_SYNC flag to be cleared. jfs_write_inode need not
do anything when i_nlink is zero, which is the easy fix.

Reported-by: Michael L. Semon <mlsemon35@gmail.com>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
fs/jfs/inode.c
fs/jfs/jfs_logmgr.c

index b7dc47ba675efe6bb9619230aece38f1d7984de0..77554b61d1247b2346c4c337d874281504e66f1c 100644 (file)
@@ -125,7 +125,7 @@ int jfs_write_inode(struct inode *inode, struct writeback_control *wbc)
 {
        int wait = wbc->sync_mode == WB_SYNC_ALL;
 
-       if (test_cflag(COMMIT_Nolink, inode))
+       if (inode->i_nlink == 0)
                return 0;
        /*
         * If COMMIT_DIRTY is not set, the inode isn't really dirty.
index 2eb952c41a69da5f23314e6d3a9a3012692b3b76..cbe48ea9318eea1ce44cae4bf3190911edf6c4c4 100644 (file)
@@ -1058,7 +1058,8 @@ static int lmLogSync(struct jfs_log * log, int hard_sync)
  */
 void jfs_syncpt(struct jfs_log *log, int hard_sync)
 {      LOG_LOCK(log);
-       lmLogSync(log, hard_sync);
+       if (!test_bit(log_QUIESCE, &log->flag))
+               lmLogSync(log, hard_sync);
        LOG_UNLOCK(log);
 }