Date: Sun, 27 Mar 2011 22:36:16 +0000 (UTC) From: Pyun YongHyeon <yongari@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/dev/dc if_dc.c Message-ID: <201103272236.p2RMaNKh005082@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
yongari 2011-03-27 22:36:16 UTC
FreeBSD src repository
Modified files: (Branch: RELENG_8)
sys/dev/dc if_dc.c
Log:
SVN rev 220070 on 2011-03-27 22:36:16Z by yongari
MFC r218830-218831:
r218830:
Send frames only when there is a valid link and driver is running
as well as controller has enough free TX descriptors.
Remove check for number of queued frames before attempting to
transmit. I guess it was added to allow draining queued frames
even if there is no link. I'm under the impression this type of
check should be done in upper layer. No other drivers in tree do
that.
r218831:
Rearrange interrupt handler a bit and remove forever loop.
Previously dc(4) always checked whether there is pending interrupts
and this consumed a lot of CPU cycles in interrupt handler. Limit
the number of processing for TX/RX frames to 16. Also allow sending
frames in the loop not to starve TX under high RX load.
Reading DC_ISR register should be protected with driver lock,
otherwise interrupt handler could be run(e.g. link state change)
before the completion of dc_init_locked().
While I'm here remove unneeded code.
Revision Changes Path
1.201.2.7 +22 -26 src/sys/dev/dc/if_dc.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201103272236.p2RMaNKh005082>