Subject: CVS commit: src/sys/dev/ic
To: None <source-changes@NetBSD.org>
From: David Young <dyoung@netbsd.org>
List: source-changes
Date: 04/17/2007 22:01:43
Module Name:	src
Committed By:	dyoung
Date:		Tue Apr 17 22:01:43 UTC 2007

Modified Files:
	src/sys/dev/ic: ath.c

Log Message:
Plug an mbuf leak.  Found by inspection.

Do not clear IFF_OACTIVE and reset the transmit watchdog timer
unless we have actually freed some transmit resources.  This prevents
my wireless routers from needlessly entering ath_start() without
any transmit resources.  If the MAC's transmitter has frozen, also,
then the driver will take the "out of xmit buffers" path in
ath_start(), which sets OACTIVE without activating the transmit
watchdog.  That freezes the transmit path until the driver is reset.

XXX The "out of xmit buffers" path in ath_start() needs more
XXX attention.


To generate a diff of this commit:
cvs rdiff -r1.82 -r1.83 src/sys/dev/ic/ath.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.