Bug 46647

Summary: Failure to initialize MII on 3Com NIC results in panic
Product: Base System Reporter: Joerg Wallerich <jw>
Component: kernAssignee: silby
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Unspecified   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Joerg Wallerich 2002-12-31 10:20:01 UTC
	System fails to initialize NIC Phys (xlphy0) and doesn't find
        any supported media types. Any attempt to configure the
        interface results in a kernel panic.

        dmesg output:
xl0: <3Com 3cSOHO100-TX OfficeConnect> port 0xb000-0xb07f mem 0xd5000000-0xd500007f irq 10 at device 11.0 on pci0
xl0: Ethernet address: 00:04:76:8c:fb:40
miibus0: <MII bus> on xl0
xlphy0: <3Com internal media interface> on miibus0
xlphy0:  

This appears since revision 1.107 of if_xl.c

Fix: As workaround I added some of the DELAY(1) lines removed
        from 1.106 back to 1.111 of if_xl.c (see patch). This
        seems to fix the problem.
How-To-Repeat: 	Boot, configure interface using 'ifconfig up'
Comment 1 Tilman Keskinoz freebsd_committer freebsd_triage 2003-01-21 16:38:03 UTC
State Changed
From-To: open->analyzed

I have tested this patch, it fixes my panic described in PR: 47251.  


Comment 2 Tilman Keskinoz freebsd_committer freebsd_triage 2003-01-21 16:38:03 UTC
Responsible Changed
From-To: freebsd-bugs->silby

Hello silby,  
Can you please commit this patch? 
It is a patch to a commit from you 4 months ago.  
Although this delays may not be required by the MII specification,  
my 3c905B requires them.  

regards  
tilman
Comment 3 tedm 2007-11-25 07:51:11 UTC
It appears this PR should be closed pending feedback from
the original submitter.  The DELAYS added by
the patch included from the submitter were added into the
driver in revision 1.116 as two more MII_SETS along with a
note in the code, perhaps feedback weas sent to the
submitter who didn't respond?
Comment 4 Eitan Adler freebsd_committer freebsd_triage 2011-04-03 16:55:46 UTC
State Changed
From-To: analyzed->closed

per Ted Mittelstaedt