Re: Possible regression with msk driver

On Wed, Nov 11, 2009 at 11:11:51PM +0100, Mel Flynn wrote:
> Hi,
> 
> I just booted a box from 7.2-p4 to FreeBSD 8.0-PRERELEASE #0 r199185. It 
> didn't ping, even though the interface was marked up and active. No traffic at 
> all.
> 
> The fix was to ifconfig msk0 down; ifconfig msk0 up. From then on, everything 
> worked and still is:
> mskc0: <Marvell Yukon 88E8056 Gigabit Ethernet> port 0xe800-0xe8ff mem 
> 0xf9efc000-0xf9efffff irq 16 at device 0.0 on pci3
> msk0: <Marvell Technology Group Ltd. Yukon EC Ultra Id 0xb4 Rev 0x03> on mskc0
> msk0: Ethernet address: 00:1b:fc:e3:9b:6a
> miibus0: <MII bus> on msk0
> e1000phy0: <Marvell 88E1149 Gigabit PHY> PHY 0 on miibus0
> e1000phy0:  10baseT, 10baseT-FDX, 100baseTX, 100baseTX-FDX, 1000baseT, 
> 1000baseT-FDX, auto
> mskc0: [FILTER]
> 
> mskc0_at_pci0:3:0:0:       class=0x020000 card=0x826e1043 chip=0x436411ab 
> rev=0x12 hdr=0x00
>     vendor     = 'Marvell Semiconductor (Was: Galileo Technology Ltd)'
>     device     = 'Yukon PCI-E Gigabit Ethernet Controller (88E8056)'
>     class      = network
>     subclass   = ethernet
> 
> msk0: flags=8843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST> metric 0 mtu 1500
>         options=11a<TXCSUM,VLAN_MTU,VLAN_HWTAGGING,TSO4>
>         ether 00:1b:fc:xx:xx:xx
>         inet 192.168.xxx.xxx netmask 0xffffff00 broadcast 192.168.xxx.255
>         media: Ethernet autoselect (1000baseT <full-duplex,flag0,flag1,flag2>)
>         status: active
> 
> Now - this machine still had 7.x /etc/rc.d, but since ifconfig down/up fixed 
> the problem I don't know if this is a likely suspect.
> 

Most likely it lost link state changes so I guess msk(4) still
think it has no established link.
Because there had been several problems with EC Ultra + 88E1149 I'm
not sure it's newly introduced regression though. There are some
changes in CURRENT. Would you try latest msk(4)/e1000phy(4) in
CURRENT? I think you can download the following files from CURRENT.
sys/dev/msk/if_msk.c
sys/dev/msk/if_mskreg.h
sys/dev/mii/e1000phy.c
sys/dev/mii/e1000phyreg.h

If that does not work I'll send you a possible patch.

> My onsite contact is unavailable right now, but if she gets back I will reboot 
> the machine with the mergemaster'd rc.d to see if it's reproducible.

I don't think it's related with rc.d scripts.

> The machine is a dual core 64-bit running i386 SMP kernel with 4G memory and a 
> few ACPI problems:
> ACPI APIC Table: <A M I  OEMAPIC >
> ACPI Warning: \\_SB_.PCI0.SBRG.FDC_._FDE: Return type mismatch - found 
> Package, expected Buffer 20090521 nspredef-1051
> ACPI Warning: Incorrect checksum in table [OEMB] - 4E, should be 43 20090521 
> tbutils-275
> 
> With release around the door, I figured I'd get this out there, at the risk of 
> posting  a red herring.
> -- 
> Mel