From 04aa2de8f788654dda15e0b598fc874915b0fc06 Mon Sep 17 00:00:00 2001 From: Ned Bass Date: Wed, 15 Jan 2014 13:52:57 -0800 Subject: [PATCH] vdev_file_io_start() to use taskq_dispatch(TQ_PUSHPAGE) The vdev_file_io_start() function may be processing a zio that the txg_sync thread is waiting on. In this case it is not safe to perform memory allocations that may generate new I/O since this could cause a deadlock. To avoid this, call taskq_dispatch() with TQ_PUSHPAGE instead of TQ_SLEEP. Signed-off-by: Ned Bass Signed-off-by: Brian Behlendorf Issue #1928 --- module/zfs/vdev_file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/module/zfs/vdev_file.c b/module/zfs/vdev_file.c index c0d062b0e..858582aca 100644 --- a/module/zfs/vdev_file.c +++ b/module/zfs/vdev_file.c @@ -185,7 +185,7 @@ vdev_file_io_start(zio_t *zio) } VERIFY3U(taskq_dispatch(system_taskq, vdev_file_io_strategy, zio, - TQ_SLEEP), !=, 0); + TQ_PUSHPAGE), !=, 0); return (ZIO_PIPELINE_STOP); }