No, that's probably wrong, but was not noticed because the legacy interrupt
path has been unused. The only time I've ever used it was during initial
development
and debug :)  I'm taking the day off, but I'll take a closer look at the
code shortly.

Jack


On Fri, Aug 3, 2012 at 9:19 AM, Vijay Singh <vijju.si...@gmail.com> wrote:

> Folks, the ixgbe_legacy_irq() interrupt handler kicks the taskqueue if
> there is more rx or tx.
>
>
> 1440         if (more_rx || more_tx)
> 1441                 taskqueue_enqueue(que->tq, &que->que_task);
>
> But then it enables interrupts unconditionally:
>
> 1454
> 1455         ixgbe_enable_intr(adapter);
>
> Is this correct?
>
> -vijay
> _______________________________________________
> freebsd-net@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-net
> To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"
>
_______________________________________________
freebsd-net@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"

Reply via email to