netdev - Re: [PATCH] [ETHTOOL]: Add support for large eeproms

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080424190148.GA19511@google.com>
Date:	Thu, 24 Apr 2008 12:01:48 -0700
From:	Mandeep Singh Baines <msb@...gle.com>
To:	Breno Leitao <leitao@...ux.vnet.ibm.com>
Cc:	David Miller <davem@...emloft.net>, thockin@...gle.com,
	netdev@...r.kernel.org, jeff@...zik.org, joe@...ches.com,
	nil@...gle.com, matthew@....cx
Subject: Re: [PATCH] [ETHTOOL]: Add support for large eeproms

Hi Breno,

Comments are inline.

Breno Leitao (leitao@...ux.vnet.ibm.com) wrote:
> I've just applied this patch, and I got something really strange now.  I 
> got the following message "Magic number 0x00000000 does not match 0x669955aa
> " when I try to dump the eeprom from my tg3 card (BCM5780S rev. 03). I 
> suppose that somehow it isn't getting the correct NIC magic number.
> 

You should not be getting this error when trying to dump an EEPROM.

The magic number is only checked when writing to an eeprom. The magic number
prevents accidental writes. The check is implemented inside the driver's
set_eeprom function. In this case, the function would be tg3_set_eeprom().

> Well, without the patch, I just got a "Cannnot allocate enough memory" 
> message when I rung the ethtool command, but if I pass the length 
> argument < 128k, then everything went ok.
> Now, with the patch applied, I can't even dump  even using the length 
> parameter, since I hit the same Magic number error.
> 
> Note that I didn't run the kernel from your tree, I just backport this 
> patch to my current kernel 2.6.16, and tried it over a ppc machine. I'll 
> run the kernel from your tree later and then post the result.
> 

I suspect you may have mis-applied the patch. The patch changes both
set_eeprom and get_eeprom. I think you may have modified get_eeprom to call
ops->set_eeprom instead of ops->get_eeprom. If not, please send me ethtool.c
and tg3.c and I can take a look.

Regards,
Mandeep
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html