Re: [PATCH 2.6] net/sk98lin: correct buggy VPD in ASUS MB

["J. Ryan Earl" <heretic@clanhk.org>]

Jeff Lightfoot wrote:

The following patch to net/sk98lin/skvpd.c was put together by Marc
Bouget, mbouget at club-internet.fr.  He currently doesn't have time to
interface with linux-kernel so I offered to work that front.

This patch works around a corrupt EEPROM (VPD?) in the ASUS K8V Deluxe
SE motherboard ethernet chipset and allows the network driver to work
correctly.  We have written to ASUS and the sk98lin maintainers but have
not heard anything back.

Does this have a chance to be included in mainline or is there a
preferable way to fix these kind of issues?

  -- Jeff Lightfoot

------------------------------------------------------------------------

Just to confirm, I experienced this error.  Kernel log as follows:

sk98lin: Network Device Driver v6.22
(C)Copyright 1999-2004 Marvell(R).
eth0:
      PrefPort:A  RlmtMode:Check Link State
eth0: -- ERROR --
        Class:  internal Software error
        Nr:  0x19e
        Msg:  Vpd: Cannot read VPD keys
eth0: -- ERROR --
        Class:  internal Software error
        Nr:  0x19e
        Msg:  Vpd: Cannot read VPD keys
eth0: -- ERROR --
        Class:  internal Software error
        Nr:  0x19e
        Msg:  Vpd: Cannot read VPD keys
eth0: network connection up using port A
    speed:           100
    autonegotiation: yes
    duplex mode:     full
    flowctrl:        symmetric
    irq moderation:  disabled
    scatter-gather:  enabled

And latter in the log I have:
try[18484]: segfault at 0000003000000008 rip 0000002a95e9dcae rsp 
0000007fbfffe230 error 4
try[24948]: segfault at 0000003000000008 rip 0000002a96033cae rsp 
0000007fbfffe250 error 4

>--- drivers/net/sk98lin/skvpd.c.orig	2004-04-05 17:32:59.000000000 -0700
>+++ drivers/net/sk98lin/skvpd.c	2004-04-05 17:33:26.000000000 -0700
>@@ -468,6 +468,16 @@
> 	
> 	pAC->vpd.vpd_size = vpd_size;
> 
>+	/* Asus K8V Se Deluxe bugfix. Correct VPD content */
>+	/* MBo April 2004 */
>+	if( ((unsigned char)pAC->vpd.vpd_buf[0x3f] == 0x38) &&
>+	    ((unsigned char)pAC->vpd.vpd_buf[0x40] == 0x3c) &&
>+	    ((unsigned char)pAC->vpd.vpd_buf[0x41] == 0x45) ) {
>+		printk("sk98lin : humm... Asus mainboard with buggy VPD ? correcting data.\n");
>+		(unsigned char)pAC->vpd.vpd_buf[0x40] = 0x38;
>+	}
>+
>+
> 	/* find the end tag of the RO area */
> 	if (!(r = vpd_find_para(pAC, VPD_RV, &rp))) {
> 		SK_DBG_MSG(pAC, SK_DBGMOD_VPD, SK_DBGCAT_ERR | SK_DBGCAT_FATAL,
>  
>
With the patch:

sk98lin: Network Device Driver v6.23
(C)Copyright 1999-2004 Marvell(R).
sk98lin : humm... Asus mainboard with buggy VPD ? correcting data.
eth0: Yukon Gigabit Ethernet 10/100/1000Base-T Adapter
      PrefPort:A  RlmtMode:Check Link State
eth0: network connection up using port A
    speed:           100
    autonegotiation: yes
    duplex mode:     full
    flowctrl:        symmetric
    irq moderation:  disabled
    scatter-gather:  enabled

I'm also able poll more information back from the driver with the patch 
applied.  Like ifconfig will return number of packets, etc.

-ryan