summaryrefslogtreecommitdiffstats
path: root/fs/jfs/jfs_txnmgr.c
diff options
context:
space:
mode:
authorDave Kleikamp <shaggy@linux.vnet.ibm.com>2007-01-17 21:18:35 -0600
committerDave Kleikamp <shaggy@linux.vnet.ibm.com>2007-01-17 21:18:35 -0600
commit4aa0d230c2cfc1ac4bcf7c5466f9943cf14233a9 (patch)
treeb873dce6146f4880c6c48ab53c0079566f52a60b /fs/jfs/jfs_txnmgr.c
parent82d5b9a7c63054a9a2cd838ffd177697f86e7e34 (diff)
downloadop-kernel-dev-4aa0d230c2cfc1ac4bcf7c5466f9943cf14233a9.zip
op-kernel-dev-4aa0d230c2cfc1ac4bcf7c5466f9943cf14233a9.tar.gz
JFS: call io_schedule() instead of schedule() to avoid deadlock
The introduction of Jens Axboe's explicit i/o plugging patches introduced a deadlock in jfs. This was caused by the process initiating I/O not unplugging the queue before waiting on the commit thread. The commit thread itself was waiting for that I/O to complete. Calling io_schedule() rather than schedule() unplugs the I/O queue avoiding the deadlock, and it appears to be the right function to call in any case. Signed-off-by: Dave Kleikamp <shaggy@austin.ibm.com>
Diffstat (limited to 'fs/jfs/jfs_txnmgr.c')
-rw-r--r--fs/jfs/jfs_txnmgr.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/jfs/jfs_txnmgr.c b/fs/jfs/jfs_txnmgr.c
index d558e51..6988a10 100644
--- a/fs/jfs/jfs_txnmgr.c
+++ b/fs/jfs/jfs_txnmgr.c
@@ -135,7 +135,7 @@ static inline void TXN_SLEEP_DROP_LOCK(wait_queue_head_t * event)
add_wait_queue(event, &wait);
set_current_state(TASK_UNINTERRUPTIBLE);
TXN_UNLOCK();
- schedule();
+ io_schedule();
current->state = TASK_RUNNING;
remove_wait_queue(event, &wait);
}
OpenPOWER on IntegriCloud