From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Revell Subject: Re: My vote against eepro* removal Date: Fri, 20 Jan 2006 20:30:48 -0500 Message-ID: <1137807048.3241.58.camel@mindpipe> References: <20060120095548.GA16000@2ka.mipt.ru> <1137804050.3241.32.camel@mindpipe> <56a8daef0601201719t448a6177lfebabe3ca38a00c7@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: Evgeniy Polyakov , kus Kusche Klaus , Adrian Bunk , linux-kernel@vger.kernel.org, john.ronciak@intel.com, ganesh.venkatesan@intel.com, jesse.brandeburg@intel.com, netdev@vger.kernel.org, Steven Rostedt Return-path: To: John Ronciak In-Reply-To: <56a8daef0601201719t448a6177lfebabe3ca38a00c7@mail.gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, 2006-01-20 at 17:19 -0800, John Ronciak wrote: > On 1/20/06, Lee Revell wrote: > > Seems like the important question is, why does e100 need a watchdog if > > eepro100 works fine without one? Isn't the point of a watchdog in this > > context to work around other bugs in the driver (or the hardware)? > There are a number of things that the watchdog in e100 does. It > checks link (up, down), reads the hardware stats, adjusts the adaptive > IFS and checks to 3 known hang conditions based on certain types of > the hardware. You might be able to get around without doing the > work-arounds (as long as you don't' see hangs happening with the > hardware being used) but the checking of the link and the stats are > probably needed. Why don't these cause excessive scheduling delays in eepro100 then? Can't we just copy the eepro100 behavior? Lee