public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Peter Lund <firefly@netgroup.dk>
To: linux-kernel@vger.kernel.org
Cc: ionut@cs.columbia.edu
Subject: Re: [PATCH] eepro100.c, kernel 2.4.1
Date: Fri, 09 Feb 2001 15:38:15 +0100	[thread overview]
Message-ID: <3A8400D7.82A61E6A@netgroup.dk> (raw)

Alan Cox, Thu Feb 08 2001 - 02:42:52 EST:

> > It's the printk that gets it wrong, although that's harmless. 
> > Intel's documentation states that the bug does NOT exist if the 
> > bits 0 and 1 in eeprom[3] are 1. Thus, the workaround is correct, 
> > the printk is wrong. 
> 
> So why does it fix the problem for him. His report and your reply don't 
> make sense viewed together 

Wish I'd seen this patch about a month and a half before.  I had borrowed two
machines from IBM Denmark for evaluation and their motherboard mounted eepro100
cards (forget which exact chip version it was) didn't quite work with the driver
in the standard RH 6.2.

On boot up it said something about the Receiver lock up bug (only one of the two
messages, I think) and then it locked up anyway half an hour and a couple of
hundred ethernet packets later.  I didn't have time to look really closely at
the source code at the time :/

Just another data point indicating that the current receiver lock up enabling
code isn't good enough on newish chips.

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

             reply	other threads:[~2001-02-09 14:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-02-09 14:38 Peter Lund [this message]
  -- strict thread matches above, loose matches on Subject: below --
2001-02-08  5:53 [PATCH] eepro100.c, kernel 2.4.1 Augustin Vidovic
2001-02-08  7:23 ` Ion Badulescu
2001-02-08  7:37   ` Augustin Vidovic
2001-02-08  7:42   ` Alan Cox
2001-02-08  7:59     ` Ion Badulescu
2001-02-08 10:41       ` Augustin Vidovic
2001-02-08 11:00         ` Ion Badulescu
2001-02-08 11:15           ` Augustin Vidovic
2001-02-08 11:26             ` Ion Badulescu
2001-02-08 11:44               ` Augustin Vidovic
2001-02-08 11:53                 ` Ion Badulescu
2001-02-08 12:08                   ` Augustin Vidovic

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=3A8400D7.82A61E6A@netgroup.dk \
    --to=firefly@netgroup.dk \
    --cc=ionut@cs.columbia.edu \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox