diff options
| author | David Gwynne <dlg@cvs.openbsd.org> | 2015-02-09 03:15:42 +0000 |
|---|---|---|
| committer | David Gwynne <dlg@cvs.openbsd.org> | 2015-02-09 03:15:42 +0000 |
| commit | 24ad848078fcf32ba92421894c27fa8abb452209 (patch) | |
| tree | 310b78e8bbf34ba1ceab66cfc2bd1afacecde97b /sys/dev/ic | |
| parent | b0353336ff6ce726e4e5366de313c7cbf62de3c1 (diff) | |
we want to defer work traditionally (in openbsd) handled in an
interrupt context to a taskq running in a thread. however, there
is a concern that if we do that then we allow accidental use of
sleeping APIs in this work, which will make it harder to move the
work back to interrupts in the future.
guenther and kettenis came up with the idea of marking a proc with
CANTSLEEP which the sleep paths can check and panic on.
this builds on that so you create taskqs that run with CANTSLEEP
set except when they need to sleep for more tasks to run.
the taskq_create api is changed to take a flags argument so users
can specify CANTSLEEP. MPSAFE is also passed via this flags field
now. this means archs that defined IPL_MPSAFE to 0 can now create
mpsafe taskqs too.
lots of discussion at s2k15
ok guenther@ miod@ mpi@ tedu@ pelikan@
Diffstat (limited to 'sys/dev/ic')
| -rw-r--r-- | sys/dev/ic/qla.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/ic/qla.c b/sys/dev/ic/qla.c index e603d891f66..1552f330ae0 100644 --- a/sys/dev/ic/qla.c +++ b/sys/dev/ic/qla.c @@ -1,4 +1,4 @@ -/* $OpenBSD: qla.c,v 1.47 2015/01/27 03:17:36 dlg Exp $ */ +/* $OpenBSD: qla.c,v 1.48 2015/02/09 03:15:41 dlg Exp $ */ /* * Copyright (c) 2011 David Gwynne <dlg@openbsd.org> @@ -648,7 +648,7 @@ qla_attach(struct qla_softc *sc) goto free_scratch; } - sc->sc_update_taskq = taskq_create(DEVNAME(sc), 1, IPL_BIO); + sc->sc_update_taskq = taskq_create(DEVNAME(sc), 1, IPL_BIO, 0); task_set(&sc->sc_update_task, qla_do_update, sc); /* wait a bit for link to come up so we can scan and attach devices */ |