Date: Fri, 9 Mar 2007 14:37:37 GMT From: Paolo Pisati <piso@FreeBSD.org> To: Perforce Change Reviews <perforce@freebsd.org> Subject: PERFORCE change 115603 for review Message-ID: <200703091437.l29EbbW5057810@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=115603 Change 115603 by piso@piso_newluxor on 2007/03/09 14:37:29 o Remove the last vestige of interrupt stray handling. o Style and spacing. o Reduce diff against HEAD. Affected files ... .. //depot/projects/soc2006/intr_filter/kern/kern_intr.c#37 edit Differences ... ==== //depot/projects/soc2006/intr_filter/kern/kern_intr.c#37 (text+ko) ==== @@ -58,9 +58,6 @@ #include <ddb/db_sym.h> #endif -/* MD function */ -extern void intr_callout_reset(void); - /* * Describe an interrupt thread. There is one of these per interrupt event. */ @@ -96,11 +93,12 @@ static void intr_event_update(struct intr_event *ie); static struct intr_thread *ithread_create(const char *name, - struct intr_handler *ih); + struct intr_handler *ih); static void ithread_destroy(struct intr_thread *ithread); -static void ithread_execute_handlers(struct proc *p, struct intr_event *ie); +static void ithread_execute_handlers(struct proc *p, + struct intr_event *ie); static void priv_ithread_execute_handler(struct proc *p, - struct intr_handler *ih); + struct intr_handler *ih); static void ithread_loop(void *); static void ithread_update(struct intr_thread *ithd); static void start_softintr(void *); @@ -718,21 +716,21 @@ continue; /* - * For software interrupt threads, we only execute - * handlers that have their need flag set. Hardware - * interrupt threads always invoke all of their handlers. - */ - if (ie->ie_flags & IE_SOFT) { - if (!ih->ih_need) - continue; - else - atomic_store_rel_int(&ih->ih_need, 0); - } + * For software interrupt threads, we only execute + * handlers that have their need flag set. Hardware + * interrupt threads always invoke all of their handlers. + */ + if (ie->ie_flags & IE_SOFT) { + if (!ih->ih_need) + continue; + else + atomic_store_rel_int(&ih->ih_need, 0); + } /* Execute this handler. */ CTR6(KTR_INTR, "%s: pid %d exec %p(%p) for %s flg=%x", - __func__, p->p_pid, (void *)ih->ih_handler, ih->ih_argument, - ih->ih_name, ih->ih_flags); + __func__, p->p_pid, (void *)ih->ih_handler, + ih->ih_argument, ih->ih_name, ih->ih_flags); if (!(ih->ih_flags & IH_MPSAFE)) mtx_lock(&Giant);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200703091437.l29EbbW5057810>