lkml.org 
[lkml]   [2001]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: rtl8139too in 2.4.5
Andries.Brouwer@cwi.nl wrote:
>
> My RTL8139 (Identified 8139 chip type 'RTL-8139A')
> was fine in 2.4.3 and doesnt work in 2.4.5.
> Copying the 2.4.3 version of 8139too.c makes things work again.
>
> Since lots of people complained about this, I have not tried to
> debug - maybe a fixed version already exists?
>
> One remark:
> 2.4.5 says "eth1: media is unconnected, link down, or incompatible connection"
> coming from 8139too.c line 1367. The code there is
>
> if (mii_reg5) {
> printk(KERN_INFO"%s: Setting %s%s-duplex based on"
> " auto-negotiated partner ability %4.4x.\n", dev->name,
> mii_reg5 == 0 ? "" :
> (mii_reg5 & 0x0180) ? "100mbps " : "10mbps ",
> tp->full_duplex ? "full" : "half", mii_reg5);
> } else {
> printk(KERN_INFO"%s: media is unconnected, link down, "
> "or incompatible connection\n",
> dev->name);
> }
>
> where mii_reg5 is tested against zero inside a conditional
> where we know that it is nonzero.
> Probably the outer test is wrong.

good spotting, though, it's the inner test that's wrong, and surrounding
logic. If the link partner is not advertising anything at all, mii_reg5
will be zero. But, if we are forcing media type, mii_reg5 might be zero
and we don't care, so people get a misinformative message. Not the core
problem (working on it) but a bug nonetheless.

Jeff


--
Jeff Garzik | Echelon words of the day, from The Register:
Building 1024 | FRU Lebed HALO Spetznaz Al Amn al-Askari Glock 26
MandrakeSoft | Steak Knife Kill the President anarchy echelon
| nuclear assassinate Roswell Waco World Trade Center
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:54    [W:0.053 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site