PERFORCE change 101845 for review
Scott Long
scottl at samsco.org
Tue Jul 18 16:23:36 UTC 2006
Paolo Pisati wrote:
> http://perforce.freebsd.org/chv.cgi?CH=101845
>
> Change 101845 by piso at piso_newluxor on 2006/07/18 11:07:29
>
> Defining an interrupt handler as INTR_FAST means
> the interrupt thread associated to this interrupt event
> won't be created: actually with filter+ithread, we need
> that thread.
>
> Affected files ...
>
> .. //depot/projects/soc2006/intr_filter/dev/em/if_em.c#7 edit
>
> Differences ...
>
> ==== //depot/projects/soc2006/intr_filter/dev/em/if_em.c#7 (text+ko) ====
>
> @@ -1986,7 +1986,7 @@
> */
> TASK_INIT(&sc->link_task, 0, em_handle_link, sc);
> if ((error = bus_setup_intr(dev, sc->res_interrupt,
> - INTR_TYPE_NET | INTR_FAST, em_intr_fast, em_handle_rxtx, sc,
> + INTR_TYPE_NET, em_intr_fast, em_handle_rxtx, sc,
> &sc->int_handler_tag)) != 0) {
> device_printf(dev, "Failed to register fast interrupt "
> "handler: %d\n", error);
Now you've made if_em no longer be MPSAFE.
Scott
More information about the p4-projects
mailing list