summaryrefslogtreecommitdiffstats
path: root/fs/jfs/inode.c
diff options
context:
space:
mode:
authorDave Kleikamp <dave.kleikamp@oracle.com>2013-05-01 11:08:38 -0500
committerDave Kleikamp <dave.kleikamp@oracle.com>2013-05-01 11:16:59 -0500
commit73aaa22d5ffb2630456bac2f9a4ed9b81d0d7271 (patch)
tree8c5a187306f27ba618a077c5b5a3c0b2e2f7712f /fs/jfs/inode.c
parent9d48017bce890b19e3bba649850bdbc8a6f95903 (diff)
downloadop-kernel-dev-73aaa22d5ffb2630456bac2f9a4ed9b81d0d7271.zip
op-kernel-dev-73aaa22d5ffb2630456bac2f9a4ed9b81d0d7271.tar.gz
jfs: fix a couple races
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>
Diffstat (limited to 'fs/jfs/inode.c')
-rw-r--r--fs/jfs/inode.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jfs/inode.c b/fs/jfs/inode.c
index b7dc47b..77554b6 100644
--- a/fs/jfs/inode.c
+++ b/fs/jfs/inode.c
@@ -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.
OpenPOWER on IntegriCloud