From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Assmann Subject: Re: [PATCH] igb: restore EEPROM 16kB access limit Date: Tue, 26 Apr 2011 17:07:07 +0200 Message-ID: <4DB6DF9B.90706@kpanic.de> References: <1302269695-27188-1-git-send-email-sassmann@kpanic.de> <4D9F6A44.7060809@kpanic.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: "netdev@vger.kernel.org" , "e1000-devel@lists.sourceforge.net" , "Kirsher, Jeffrey T" , "Pieper, Jeffrey E" , "Ronciak, John" , Andy Gospodarek To: "Wyborny, Carolyn" Return-path: Received: from mail.xlhost.de ([213.202.242.118]:51917 "EHLO mysql1.xlhost.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756095Ab1DZPHM (ORCPT ); Tue, 26 Apr 2011 11:07:12 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: New patch against net-next-2.6. Stefan >>From 67ce9e09e10f05054a26560306aa484ae3acc03f Mon Sep 17 00:00:00 2001 From: Stefan Assmann Date: Mon, 18 Apr 2011 15:22:19 +0200 Subject: [PATCH] igb: default to 32kB for EEPROMs reporting invalid size The check that gracefully handled invalid EEPROM sizes was removed by commit 4322e561a93ec7ee034b603a6c610e7be90d4e8a. Without this check the EEPROM validation fails if the size is invalid and the NIC is not usable by the OS. Observed with a 8086:10c9 NIC. igb 0000:03:00.0: 0 vfs allocated igb 0000:03:00.0: The NVM Checksum Is Not Valid ACPI: PCI interrupt for device 0000:03:00.0 disabled igb: probe of 0000:03:00.0 failed with error -5 Re-introducing the check with an additional dev_err() to report the problem. Signed-off-by: Stefan Assmann --- drivers/net/igb/e1000_82575.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/net/igb/e1000_82575.c b/drivers/net/igb/e1000_82575.c index 0cd41c4..f3bdf29 100644 --- a/drivers/net/igb/e1000_82575.c +++ b/drivers/net/igb/e1000_82575.c @@ -31,9 +31,11 @@ #include #include +#include #include "e1000_mac.h" #include "e1000_82575.h" +#include "igb.h" static s32 igb_get_invariants_82575(struct e1000_hw *); static s32 igb_acquire_phy_82575(struct e1000_hw *); @@ -113,6 +115,7 @@ static s32 igb_get_invariants_82575(struct e1000_hw *hw) struct e1000_nvm_info *nvm = &hw->nvm; struct e1000_mac_info *mac = &hw->mac; struct e1000_dev_spec_82575 * dev_spec = &hw->dev_spec._82575; + struct igb_adapter *adapter = hw->back; u32 eecd; s32 ret_val; u16 size; @@ -244,6 +247,13 @@ static s32 igb_get_invariants_82575(struct e1000_hw *hw) */ size += NVM_WORD_SIZE_BASE_SHIFT; + /* gracefully handle NICs reporting an invalid EEPROM size */ + if (size > 15) { + dev_err(&adapter->pdev->dev, + "NVM size is not valid, defaulting to 32kB\n"); + size = 15; + } + nvm->word_size = 1 << size; if (nvm->word_size == (1 << 15)) nvm->page_size = 128; -- 1.7.4