diff options
| author | Chunwei Chen <[email protected]> | 2014-05-13 10:36:35 +0800 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2014-05-14 16:20:21 -0700 |
| commit | bc25c9325b0e5ced897b9820dad239539d561ec9 (patch) | |
| tree | dd7a9bf76d0f0aa232ce2eaaa61c083a5238edc9 /module/zfs/spa_misc.c | |
| parent | 2c33b9127569ad62b4cfe7dd4f651ceeee3d005c (diff) | |
Use a dedicated taskq for vdev_file
Originally, vdev_file used system_taskq. This would cause a deadlock,
especially on system with few CPUs. The reason is that the prefetcher
threads, which are on system_taskq, will sometimes be blocked waiting
for I/O to finish. If the prefetcher threads consume all the tasks in
system_taskq, the I/O cannot be served and thus results in a deadlock.
We fix this by creating a dedicated vdev_file_taskq for vdev_file I/O.
Signed-off-by: Chunwei Chen <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #2270
Diffstat (limited to 'module/zfs/spa_misc.c')
| -rw-r--r-- | module/zfs/spa_misc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/module/zfs/spa_misc.c b/module/zfs/spa_misc.c index cdbe5a522..935a61796 100644 --- a/module/zfs/spa_misc.c +++ b/module/zfs/spa_misc.c @@ -34,6 +34,7 @@ #include <sys/zap.h> #include <sys/zil.h> #include <sys/vdev_impl.h> +#include <sys/vdev_file.h> #include <sys/metaslab.h> #include <sys/uberblock_impl.h> #include <sys/txg.h> @@ -1660,6 +1661,7 @@ spa_init(int mode) dmu_init(); zil_init(); vdev_cache_stat_init(); + vdev_file_init(); zfs_prop_init(); zpool_prop_init(); zpool_feature_init(); @@ -1674,6 +1676,7 @@ spa_fini(void) spa_evict_all(); + vdev_file_fini(); vdev_cache_stat_fini(); zil_fini(); dmu_fini(); |