Author: mm
Date: Fri Jan 7 14:17:29 2011
New Revision: 217096
URL: http://svn.freebsd.org/changeset/base/217096
Log:
MFC r216919:
MFp4 186485, 186859:
Fix a race by defining two tasks in the zio structure
as we can still be returning from issue task when interrupt task is used.
Reviewed by: pjd
Approved by: re (kib), pjd
Modified:
releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h
releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio.c
Directory Properties:
releng/8.2/sys/ (props changed)
releng/8.2/sys/amd64/include/xen/ (props changed)
releng/8.2/sys/cddl/contrib/opensolaris/ (props changed)
releng/8.2/sys/contrib/dev/acpica/ (props changed)
releng/8.2/sys/contrib/pf/ (props changed)
Modified: releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h
==============================================================================
--- releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h Fri Jan
7 14:03:28 2011 (r217095)
+++ releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h Fri Jan
7 14:17:29 2011 (r217096)
@@ -340,7 +340,8 @@ struct zio {
#ifdef _KERNEL
/* FreeBSD only. */
- struct ostask io_task;
+ struct ostask io_task_issue;
+ struct ostask io_task_interrupt;
#endif
};
Modified: releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio.c
==============================================================================
--- releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio.c Fri Jan
7 14:03:28 2011 (r217095)
+++ releng/8.2/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zio.c Fri Jan
7 14:17:29 2011 (r217096)
@@ -947,6 +947,18 @@ zio_taskq_dispatch(zio_t *zio, enum zio_
{
spa_t *spa = zio->io_spa;
zio_type_t t = zio->io_type;
+#ifdef _KERNEL
+ struct ostask *task;
+#endif
+
+ ASSERT(q == ZIO_TASKQ_ISSUE || q == ZIO_TASKQ_INTERRUPT);
+
+#ifdef _KERNEL
+ if (q == ZIO_TASKQ_ISSUE)
+ task = &zio->io_task_issue;
+ else /* if (q == ZIO_TASKQ_INTERRUPT) */
+ task = &zio->io_task_interrupt;
+#endif
/*
* If we're a config writer or a probe, the normal issue and
@@ -970,8 +982,13 @@ zio_taskq_dispatch(zio_t *zio, enum zio_
q++;
ASSERT3U(q, <, ZIO_TASKQ_TYPES);
+#ifdef _KERNEL
(void) taskq_dispatch_safe(spa->spa_zio_taskq[t][q],
- (task_func_t *)zio_execute, zio, &zio->io_task);
+ (task_func_t *)zio_execute, zio, task);
+#else
+ (void) taskq_dispatch(spa->spa_zio_taskq[t][q],
+ (task_func_t *)zio_execute, zio, TQ_SLEEP);
+#endif
}
static boolean_t
@@ -2300,9 +2317,16 @@ zio_done(zio_t *zio)
* Reexecution is potentially a huge amount of work.
* Hand it off to the otherwise-unused claim taskq.
*/
+#ifdef _KERNEL
(void) taskq_dispatch_safe(
spa->spa_zio_taskq[ZIO_TYPE_CLAIM][ZIO_TASKQ_ISSUE],
- (task_func_t *)zio_reexecute, zio, &zio->io_task);
+ (task_func_t *)zio_reexecute, zio,
+ &zio->io_task_issue);
+#else
+ (void) taskq_dispatch(
+ spa->spa_zio_taskq[ZIO_TYPE_CLAIM][ZIO_TASKQ_ISSUE],
+ (task_func_t *)zio_reexecute, zio, TQ_SLEEP);
+#endif
}
return (ZIO_PIPELINE_STOP);
}
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
