Page MenuHomeFreeBSD
Files F113878764

D48283.diff
No OneTemporary
Actions

D48283.diff
View Options

diff --git a/share/man/man9/intr_event.9 b/share/man/man9/intr_event.9
--- a/share/man/man9/intr_event.9
+++ b/share/man/man9/intr_event.9
@@ -27,7 +27,7 @@
.\" OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
.\" SUCH DAMAGE.
.\"
-.Dd October 30, 2022
+.Dd January 24, 2025
.Dt INTR_EVENT 9
.Os
.Sh NAME
@@ -295,6 +295,14 @@
Presently, the
.Dv INTR_ENTROPY
flag is not valid for software interrupt handlers.
+The
+.Dv INTR_SLEEPABLE
+flag specifies that the interrupt ithread may sleep.
+Presently, the
+.Dv INTR_SLEEPABLE
+flag requires the
+.Dv INTR_EXCL
+flag to be set.
.Ss Handler Callbacks
Each
.Vt struct intr_event
diff --git a/sys/kern/kern_intr.c b/sys/kern/kern_intr.c
--- a/sys/kern/kern_intr.c
+++ b/sys/kern/kern_intr.c
@@ -610,6 +610,12 @@
if (ie == NULL || name == NULL || (handler == NULL && filter == NULL))
return (EINVAL);
+ if ((flags & INTR_SLEEPABLE) != 0 && (flags & INTR_EXCL) == 0) {
+ printf("%s: INTR_SLEEPABLE requires INTR_EXCL to be set\n",
+ __func__);
+ return (EINVAL);
+ }
+
/* Allocate and populate an interrupt handler structure. */
ih = malloc(sizeof(struct intr_handler), M_ITHREAD, M_WAITOK | M_ZERO);
ih->ih_filter = filter;
@@ -627,16 +633,18 @@
if (flags & INTR_TYPE_NET)
ih->ih_flags |= IH_NET;
- /* We can only have one exclusive handler in a event. */
+ /* We can only have one exclusive or sleepable handler in a event. */
mtx_lock(&ie->ie_lock);
if (!CK_SLIST_EMPTY(&ie->ie_handlers)) {
- if ((flags & INTR_EXCL) ||
+ if ((flags & (INTR_EXCL | INTR_SLEEPABLE)) ||
(CK_SLIST_FIRST(&ie->ie_handlers)->ih_flags & IH_EXCLUSIVE)) {
mtx_unlock(&ie->ie_lock);
free(ih, M_ITHREAD);
return (EINVAL);
}
}
+ if (flags & INTR_SLEEPABLE)
+ ie->ie_flags |= IE_SLEEPABLE;
/* Create a thread if we need one. */
while (ie->ie_thread == NULL && handler != NULL) {
@@ -1190,11 +1198,11 @@
ithread_execute_handlers(struct proc *p, struct intr_event *ie)
{
- /* Interrupt handlers should not sleep. */
- if (!(ie->ie_flags & IE_SOFT))
+ /* Only specifically marked sleepable interrupt handlers can sleep. */
+ if (!(ie->ie_flags & (IE_SOFT | IE_SLEEPABLE)))
THREAD_NO_SLEEPING();
intr_event_execute_handlers(p, ie);
- if (!(ie->ie_flags & IE_SOFT))
+ if (!(ie->ie_flags & (IE_SOFT | IE_SLEEPABLE)))
THREAD_SLEEPING_OK();
/*
diff --git a/sys/sys/bus.h b/sys/sys/bus.h
--- a/sys/sys/bus.h
+++ b/sys/sys/bus.h
@@ -277,6 +277,7 @@
INTR_EXCL = 256, /* exclusive interrupt */
INTR_MPSAFE = 512, /* this interrupt is SMP safe */
INTR_ENTROPY = 1024, /* this interrupt provides entropy */
+ INTR_SLEEPABLE = 2048, /* this interrupt handler can sleep */
INTR_MD1 = 4096, /* flag reserved for MD use */
INTR_MD2 = 8192, /* flag reserved for MD use */
INTR_MD3 = 16384, /* flag reserved for MD use */
diff --git a/sys/sys/interrupt.h b/sys/sys/interrupt.h
--- a/sys/sys/interrupt.h
+++ b/sys/sys/interrupt.h
@@ -129,6 +129,7 @@
/* Interrupt event flags kept in ie_flags. */
#define IE_SOFT 0x000001 /* Software interrupt. */
+#define IE_SLEEPABLE 0x000002 /* Sleepable ithread */
#define IE_ADDING_THREAD 0x000004 /* Currently building an ithread. */
/* Flags to pass to swi_sched. */

File Metadata

Mime Type
text/plain
Expires
Sun, Apr 6, 12:09 AM (12 h, 1 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
17398672
Default Alt Text
D48283.diff (3 KB)

Event Timeline