DragonFly BSD
DragonFly submit List (threaded) for 2005-02
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: rl(4) TX_UNDERRUN fix


From: Max Laier <max@xxxxxxxxxxxxxx>
Date: Mon, 21 Feb 2005 20:39:22 +0100

On Monday 21 February 2005 19:45, Joerg Sonnenberger wrote:
> Hi all,
> please test http://leaf.dragonflybsd.org/~joerg/if_rl.c.diff
> with any kind of rl(4). This patch is a more aggressive version
> of the recently commited patch in FreeBSD 5/6, the version
> orginaly proposed in kern/61448.
>
> It shouldn't negatively impact performance or stability.

I choose the less aggressive one for a (couple of) reason(s):
 - not all chips can RL_TXCFG_CLRABRT some even forbid it in the specs.
 - the ifp->if_opackets++ just isn't right. You should only increase it if you 
get RL_TXSTAT_TX_OK.
 - removing the reset/init cycle *might* be a good thing, but only if you can 
assure that it doesn't break the older chips.  Some notes in the specs 
suggest different.

-- 
/"\  Best regards,                      | mlaier@xxxxxxxxxxx
\ /  Max Laier                          | ICQ #67774661
 X   http://pf4freebsd.love2party.net/  | mlaier@EFnet
/ \  ASCII Ribbon Campaign              | Against HTML Mail and News

Attachment: pgp00008.pgp
Description: PGP signature



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]