Date: Sat, 10 Jan 2004 00:56:44 +0100
From: Stephan von Krawczynski <skraw@ithnet.com>
To: Jonathan Lundell <jlundell@lundell-bros.com>
Cc: linux-kernel@vger.kernel.org, linux-net@vger.kernel.org
Subject: Re: Problem with 2.4.24 e1000 and keepalived
Message-Id: <20040110005644.000e4c37.skraw@ithnet.com>
In-Reply-To: <p0602047abc24a5dc714d@[192.168.0.3]>
References: <20040107200556.0d553c40.skraw@ithnet.com>
	<20040107210255.GA545@alpha.home.local>
	<3FFCC430.4060804@candelatech.com>
	<20040108091441.3ff81b53.skraw@ithnet.com>
	<20040108084758.GB9050@alpha.home.local>
	<p0602042fbc2347a37845@[192.168.0.3]>
	<20040109004525.GB545@alpha.home.local>
	<p0602045cbc23ac7a1ada@[192.168.0.3]>
	<20040109131812.11fc4948.skraw@ithnet.com>
	<p0602047abc24a5dc714d@[192.168.0.3]>
Organization: ith Kommunikationstechnik GmbH
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1749
Lines: 43

On Fri, 9 Jan 2004 10:43:13 -0800
Jonathan Lundell <jlundell@lundell-bros.com> wrote:

> At 1:18pm +0100 1/9/04, Stephan von Krawczynski wrote:
> >On Thu, 8 Jan 2004 17:00:42 -0800
> >Jonathan Lundell <jlundell@lundell-bros.com> wrote:
> >
> >>  At 1:45am +0100 1/9/04, Willy Tarreau wrote:
> >>  >  > It's unfortunate that the two conditions are conflated by most net
> >>  >  > drivers.
> >>  >
> >>  >IMHO, saying "most net drivers" is unfair : tg3, tulip, 3c59x, starfire,
> >>  >realtek, sis900, dl2k, pcnet32, and IIRC sunhme are OK. eepro100 is
> >nearly>  >OK but has this annoying bug, and only older 10 Mbps drivers don't
> >report>  >their status, often because the chip itself doesn't know.
> >>
> >>  I'm sure you're right; I should have said most of the drivers that
> >>  I'm using (including e100 &e1000).
> >
> >Can we find the cause for this obviously buggy behaviour inside the source?
> >Where is the handling of physical up/down events different in tulip 
> >compared to
> >e100(0) ?
> 
> In e1000 5.2.20 (as in earlier versions), the link-state reporters 
> rely on netif_carrier_ok() for the state, which is in turned 
> maintained by the driver's watchdog timer.
> 
> e1000_down() both cancels the watchdog timer and calls 
> netif_carrier_off(), guaranteeing that if the interface is logically 
> down, the link will be reported as down regardless of the actual link 
> state.

That cannot be the cause, as the logical interface state is UP in the problem
case.

Regards,
Stephan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/