From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-2.6 2/3] be2net: fix a bug in UE detection logic Date: Wed, 25 Aug 2010 21:15:53 -0700 (PDT) Message-ID: <20100825.211553.226786424.davem@davemloft.net> References: <20100825102720.GA14925@serverengines.com> <20100825.142311.115915728.davem@davemloft.net> <20100825.211500.193701446.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org To: ajitk@serverengines.com Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:41796 "EHLO sunset.davemloft.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751116Ab0HZEPh (ORCPT ); Thu, 26 Aug 2010 00:15:37 -0400 In-Reply-To: <20100825.211500.193701446.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: From: David Miller Date: Wed, 25 Aug 2010 21:15:00 -0700 (PDT) > From: David Miller > Date: Wed, 25 Aug 2010 14:23:11 -0700 (PDT) > >> From: Ajit Khaparde >> Date: Wed, 25 Aug 2010 15:57:31 +0530 >> >>> The ONLINE registers can return 0xFFFFFFFF on more than one >>> occassion. On systems that care, reading these registers could >>> lead to problems. >>> >>> So the new code decides that the ASIC has encountered and error >>> by reading the UE_STATUS_LOW/HIGH registers. AND them with >>> the mask values and a non-zero result indicates an error. >>> >>> Signed-off-by: Ajit Khaparde >> >> Applied. > > You didn't even build test what you submitted to me: > > ERROR: "be_dump_ue" [drivers/net/benet/be2net.ko] undefined! > > I'm reverting both patches. Of course I meant "I'm reverting all 3 patches."