]> git.hungrycats.org Git - linux/commit
jfs: fix a couple races
authorDave Kleikamp <dave.kleikamp@oracle.com>
Wed, 1 May 2013 16:08:38 +0000 (11:08 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 7 Jun 2013 19:46:48 +0000 (12:46 -0700)
commit3b1a317977a8d16d743e8f85ed9a5a7f668b61d5
tree3f8b26b833d2d3ce5821a2488498d551e9b55805
parent3a6b9138a066588649cf0387bf34d7068f0b19a6
jfs: fix a couple races

commit 73aaa22d5ffb2630456bac2f9a4ed9b81d0d7271 upstream.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/jfs/inode.c
fs/jfs/jfs_logmgr.c