All of lore.kernel.org
 help / color / mirror / Atom feed
From: Auke Kok <auke-jan.h.kok@intel.com>
To: Adam Kropelin <akropel1@rochester.rr.com>
Cc: Auke Kok <auke-jan.h.kok@intel.com>,
	"Eric W. Biederman" <ebiederm@xmission.com>,
	Adrian Bunk <bunk@stusta.de>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	jgarzik@pobox.com, alan@lxorguk.ukuu.org.uk,
	Allen Parker <parker@isohunt.com>,
	jesse.brandeburg@intel.com, gregkh@suse.de,
	linux-pci@atrey.karlin.mff.cuni.cz, netdev@vger.kernel.org
Subject: Re: 2.6.20-rc7: known regressions (v2) (part 1)
Date: Sat, 03 Feb 2007 13:26:12 -0800	[thread overview]
Message-ID: <45C4FDF4.4060002@intel.com> (raw)
In-Reply-To: <093c01c747d6$45f9a2f0$84163e05@kroptech.com>

Adam Kropelin wrote:
> Auke Kok wrote:
>> Adam Kropelin wrote:
>>> I've never had this device work 100% with MSI on any kernel version
>>> I've tested so far. But I'm not the original reporter of the
>>> problem, and I believe for him it was a true regression where a
>>> previous kernel wored correctly.
>> maybe I've been unclear, but here's how e1000 detects link changes:
>>
>> 1) by checking every 2 seconds in the watchdog by reading PHY
>> registers
> 
> That would explain why I see link status changes but 0 interrupt count 
> in /proc/interrupts. However, on >= 2.6.19 the link state never changes. 
> Ever. It's always down. On <= 2.6.18 the link state does change but with 
> 0 interupt count.
> 
>> 2) by receiving an interrupt from the NIC with the LSI bit
>> in the interrupt control register
>>
>> if the link is down to start with, the watchdog will obviously spot a
>> 'link up' change since it doesn't use any interrupts.
> 
> This does not seem to work on 2.6.19+. Unless the watchdog interval is 
> tens of minutes. I've waited at least 5 minutes and link never went up.

that's explained by a driver change that did that. Since at initialization we're 
basically waiting for a link change to tell the stack that we're up, we decided 
to change the order to have the hardware fire an LSI interrupt to trigger a 
watchdog run. So no interrupts would immediately explain why the watchdog never 
runs. That's nothing to worry about for this problem, as soon as interrupts are 
seen in /proc/interrupts this all starts working for e1000.

Cheers,

Auke

  reply	other threads:[~2007-02-03 21:27 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-31  4:28 Linux 2.6.20-rc7 Linus Torvalds
     [not found] ` <45C05150.6000802@agmk.net>
2007-01-31 16:04   ` Linus Torvalds
2007-01-31 23:13     ` Andrew Morton
     [not found]     ` <200702010037.48472.pluto@agmk.net>
2007-02-01  0:14       ` Andrew Morton
2007-02-01  0:19       ` Linus Torvalds
2007-02-01  0:26         ` Andrew Morton
2007-02-01  0:39           ` Linus Torvalds
2007-02-01  0:44         ` Nick Piggin
2007-02-01  0:54           ` Linus Torvalds
2007-02-01  6:48     ` Pekka Enberg
2007-01-31 17:11 ` [PATCH] x86_64: Fix preprocessor condition Josef 'Jeff' Sipek
2007-01-31 17:16   ` Andi Kleen
2007-01-31 17:40     ` Josef Sipek
2007-01-31 18:15 ` Linux 2.6.20-rc7 Sunil Naidu
2007-02-01  2:16 ` S.Çağlar Onur
2007-02-01  2:36   ` Linus Torvalds
2007-02-01  3:01     ` S.Çağlar Onur
2007-02-01  3:31       ` Linus Torvalds
2007-02-01  5:44         ` H. Peter Anvin
2007-02-01  6:00           ` Linus Torvalds
2007-02-01  6:10             ` H. Peter Anvin
2007-02-01  6:38               ` Alexander E. Patrakov
2007-02-01  6:53                 ` H. Peter Anvin
2007-02-02 20:52         ` S.Çağlar Onur
2007-02-01 21:06           ` H. Peter Anvin
2007-02-02  5:49 ` 2.6.20-rc7: known regressions Adrian Bunk
2007-02-02  5:49   ` Adrian Bunk
2007-02-02  5:49   ` Adrian Bunk
2007-02-03  1:55   ` Andrew Morton
2007-02-03  2:03     ` Jeff Garzik
2007-02-03  2:15       ` Andrew Morton
2007-02-03  9:19     ` Frédéric Riss
2007-02-03  9:24       ` Andrew Morton
2007-02-03  9:33         ` Andi Kleen
2007-02-03  9:49           ` Frédéric Riss
2007-02-03  9:58             ` Andi Kleen
2007-02-03 10:47               ` Frédéric Riss
2007-02-03 10:51                 ` Andi Kleen
2007-02-03 10:57                   ` Frédéric Riss
2007-02-03 11:08                     ` Frédéric RISS
2007-02-04 13:13                   ` Frédéric Riss
2007-02-04 14:37                     ` Andi Kleen
2007-02-04 17:34                     ` Linus Torvalds
2007-02-04 18:18                       ` Frédéric Riss
2007-02-04 18:29                         ` Linus Torvalds
2007-02-05  8:26                       ` Andi Kleen
2007-02-05  9:35                         ` Eric W. Biederman
2007-02-03  0:44 ` 2.6.20-rc7: known regressions (v2) (part 1) Adrian Bunk
2007-02-03  0:44   ` Adrian Bunk
2007-02-03  6:06   ` Auke Kok
2007-02-03  7:41     ` Eric W. Biederman
2007-02-03 18:06       ` Adam Kropelin
2007-02-03 20:43         ` Auke Kok
2007-02-03 21:00           ` Adam Kropelin
2007-02-03 21:26             ` Auke Kok [this message]
2007-02-03 22:24               ` Eric W. Biederman
2007-02-03 21:12           ` Eric W. Biederman
2007-02-03 23:20             ` Adam Kropelin
2007-02-04  1:14               ` Eric W. Biederman
2007-02-04  4:44                 ` Adam Kropelin
2007-02-04  5:12                   ` Eric W. Biederman
2007-02-03  0:47 ` 2.6.20-rc7: known regressions (v2) (part 2) Adrian Bunk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45C4FDF4.4060002@intel.com \
    --to=auke-jan.h.kok@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=akropel1@rochester.rr.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bunk@stusta.de \
    --cc=ebiederm@xmission.com \
    --cc=gregkh@suse.de \
    --cc=jesse.brandeburg@intel.com \
    --cc=jgarzik@pobox.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@atrey.karlin.mff.cuni.cz \
    --cc=netdev@vger.kernel.org \
    --cc=parker@isohunt.com \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.