Re: Realtek RT8139 (onboard) - failed to receive packet in loopback mode

On 06/17/04 20:25:59 +0000 Bill Paul wrote:

>> re0: <RealTek 8139C+ 10/100BaseTX> port 0xdc00-0xdcff mem 0xe3400000-0xe34000ff irq 11 at device 9.0 on pci0
>> re0: Ethernet address: 00:90:fb:04:1b:31
>> re0: diagnostic failed, failed to receive packet in loopback mode
>> re0: attach aborted due to hardware diag failure
>> device_attach: re0 attach returned 5
>> 
>> 
>> It seems that the 8169-specific hardware diagnostic (re_diag(sc))
>> should only carried out for chips being identified as 8169 based.
> 
> No, it works with the 8139C+ too (though it's not strictly necessary:
> the hardware bug that the re_diag() routine was meant to find is only
> present on some 8169 boards, but running the DMA test on all boards
> doesn't hurt).

ok. 

> [...]
> 
>> I see no ill effects so far.

I didn't notice an additional problem at first sight: ifconfig 
doesn't show the media types.

gate# ifconfig re0
re0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> mtu 1500
        options=1b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING>
        inet 192.168.222.222 netmask 0xffffff00 broadcast 192.168.222.255
        ether 00:90:fb:04:1b:31
        media: Ethernet autoselect (none)

The PHY is recognized as follows:

rlphy0: <RealTek internal media interface> on miibus0
rlphy0:  10baseT, 10baseT-FDX, 100baseTX, 100baseTX-FDX, auto


 
> I discovered some problems with the diagnostic routine on the C+
> chip recently, but it only manifested on this one really old and
> crufty P166 machine

My systems are equipped with VIA C3/533 CPUs. 

> that I have (and then only intermittently).
> It turns out that the link state has some effect on the behavior
> of loopback mode (even though logically, it shouldn't). That is,
> sometimes the test will fail if the network cable is unplugged.

That might have been the cause in my case, too. The ports were not
connected to the switch at probe time. I just double-checked this
and the diagnostic fails with the port connected to the switch, too.


> I made some changes to re_diag() to fix this, 

I would test the changes gladly.


> which I can send you once I get home from the office.

Thanks.

-Andreas