From: "Qinghua(Kevin) Ye" <[EMAIL PROTECTED]>
Date: Thu, 7 Jul 2005 17:43:06 -0600

> So the tg3_tx() and tg3_start_xmit do not include any code of reprogramming
> the hardware?

Right, they just process the TX ring.

> What kinds of code can be classifed to reprogramming the hardware? Should
> the tw32_t/rx_mbox  and tw32_mailbox operation be classified into this
> catalog?

Anything other than normal packet processing.  The MBOX writes
used to process the packets in the RX ring would not be considered
reprogramming of the chip.

> Another problem is about the Flushing  the Status block to host memory. In
> your original code, this is done by
> tr32(MAILBOX_INTERRUPT_0+TG3_64BIT_REG_LOW).

This readback is necessary to flush out any posted PCI writes
to chip registers in most circumstances.  The one exception is
tg3_restart_ints() and normal interrupt handling.

The current 2.6.13-rcX tg3.c driver is totally revamped wrt.
interrupt processing, PIO flushes, and locking in general.
You may wish to check it out.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to