PERFORCE change 115603 for review

[Paolo Pisati]

http://perforce.freebsd.org/chv.cgi?CH=115603

Change 115603 by piso at 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);