diff options
| author | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-08-10 11:14:39 -0500 |
|---|---|---|
| committer | Dave Kleikamp <shaggy@austin.ibm.com> | 2005-08-10 11:14:39 -0500 |
| commit | 8a9cd6d676728792aaee31f30015d284acd154a3 (patch) | |
| tree | 7e40b3b1802b1a86603f7ac6bf74906775a2956e /fs/jfs/jfs_txnmgr.c | |
| parent | a5c96cab8f3c4ca9b2177dceb5de5a0edb31418e (diff) | |
| download | op-kernel-dev-8a9cd6d676728792aaee31f30015d284acd154a3.zip op-kernel-dev-8a9cd6d676728792aaee31f30015d284acd154a3.tar.gz | |
JFS: Fix race in txLock
TxAnchor.anon_list is protected by jfsTxnLock (TXN_LOCK), but there was
a place in txLock() that was removing an entry from the list without holding
the spinlock.
Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
Diffstat (limited to 'fs/jfs/jfs_txnmgr.c')
| -rw-r--r-- | fs/jfs/jfs_txnmgr.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/jfs/jfs_txnmgr.c b/fs/jfs/jfs_txnmgr.c index 3555acf..c7a92f9 100644 --- a/fs/jfs/jfs_txnmgr.c +++ b/fs/jfs/jfs_txnmgr.c @@ -657,7 +657,9 @@ struct tlock *txLock(tid_t tid, struct inode *ip, struct metapage * mp, /* only anonymous txn. * Remove from anon_list */ + TXN_LOCK(); list_del_init(&jfs_ip->anon_inode_list); + TXN_UNLOCK(); } jfs_ip->atlhead = tlck->next; } else { |
