public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mike Waychison <Michael.Waychison@Sun.COM>
To: Paul Larson <plars@linuxtestproject.org>
Cc: "David S. Miller" <davem@davemloft.net>,
	Brian Somers <Brian.Somers@Sun.COM>,
	lkml <linux-kernel@vger.kernel.org>
Subject: Re: TG3 doesn't work in kernel 2.4.27 (David S. Miller)
Date: Fri, 03 Sep 2004 15:19:57 -0400	[thread overview]
Message-ID: <4138C3DD.1060005@sun.com> (raw)
In-Reply-To: <1094238777.9913.278.camel@plars.austin.ibm.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Paul Larson wrote:
> I tried this patch alone on top of 2.6.9-rc1 and tg3 is still broken for
> me on JS20 blades.  Was there another patch I should have applied in
> conjunction with this?
>

Is this with or without autonegotiation enabled on the switch?

> Thanks,
> Paul Larson
>
> On Mon, 2004-08-30 at 18:11, David S. Miller wrote:
>
>>Michael Chan at Broadcom spotted the bug.
>>
>>Things are totally broken if the switch/hub does not support
>>autonegotiation.  Checking for the MAC_STATUS_SIGNAL_DET bit
>>in the tg3 polling timer fixes the problem.
>>
>>This is probably why it worked for you and doesn't with the
>>IBM blades as blades are more likely to be connected to
>>non-autoneg'ing devices.
>>
>>===== drivers/net/tg3.c 1.199 vs edited =====
>>--- 1.199/drivers/net/tg3.c	2004-08-18 19:52:35 -07:00
>>+++ edited/drivers/net/tg3.c	2004-08-30 15:08:07 -07:00
>>@@ -5602,7 +5602,8 @@
>> 				need_setup = 1;
>> 			}
>> 			if (! netif_carrier_ok(tp->dev) &&
>>-			    (mac_stat & MAC_STATUS_PCS_SYNCED)) {
>>+			    (mac_stat & (MAC_STATUS_PCS_SYNCED |
>>+					 MAC_STATUS_SIGNAL_DET))) {
>> 				need_setup = 1;
>> 			}
>> 			if (need_setup) {
>>-
>>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/
>>
>
>


- --
Mike Waychison
Sun Microsystems, Inc.
1 (650) 352-5299 voice
1 (416) 202-8336 voice
http://www.sun.com

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
NOTICE:  The opinions expressed in this email are held by me,
and may not represent the views of Sun Microsystems, Inc.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.5 (GNU/Linux)

iD8DBQFBOMPcdQs4kOxk3/MRAoJiAJoCZV1AKTQcOiOz0jNX1eZq9ZkiYACfaYDc
lWGl0C2xVNRuPuaKqt8/J90=
=mWO4
-----END PGP SIGNATURE-----

  reply	other threads:[~2004-09-03 19:23 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20040816110000.1120.31256.Mailman@lists.us.dell.com>
2004-08-16 11:51 ` TG3 doesn't work in kernel 2.4.27 (David S. Miller) Tetsuo Handa
2004-08-16 21:38   ` David S. Miller
2004-08-25 17:48     ` Mike Waychison
2004-08-25 19:08       ` David S. Miller
2004-08-25 20:04         ` Mike Waychison
2004-08-26  0:58           ` David S. Miller
2004-08-26 10:49             ` Brian Somers
2004-08-26 19:37               ` David S. Miller
2004-08-29  9:56                 ` Pekka Pietikainen
2004-09-10 12:35                 ` Brian Somers
2004-09-10 19:40                   ` Roland Dreier
2004-09-10 20:53                   ` David S. Miller
2004-09-10 21:05                     ` Roland Dreier
2004-09-10 21:45                       ` David S. Miller
2004-09-10 22:14                     ` Brian Somers
2004-08-30 23:11               ` David S. Miller
2004-09-03 19:12                 ` Paul Larson
2004-09-03 19:19                   ` Mike Waychison [this message]
2004-09-03 20:18                     ` Roland Dreier
2004-09-03 20:30                       ` David S. Miller
2004-09-03 20:40                         ` Roland Dreier
2004-09-03 23:24                         ` Roland Dreier
2004-09-07 18:33                           ` Jake Moilanen
2004-09-07 19:52                             ` Roland Dreier
2004-09-08 12:34                               ` Jake Moilanen
2004-09-08 13:07                                 ` Anton Blanchard
2004-09-13 22:48                                   ` David S. Miller
2004-09-14 22:20                                     ` Mike Waychison
2004-09-14 22:36                                       ` David S. Miller
2004-09-14 22:58                                       ` Jake Moilanen
2004-09-15  0:34                                       ` Roland Dreier
2004-09-08 13:55                                 ` Paul Larson
2004-09-10 16:00                                   ` Paul Larson
2004-09-03 20:08                   ` David S. Miller

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=4138C3DD.1060005@sun.com \
    --to=michael.waychison@sun.com \
    --cc=Brian.Somers@Sun.COM \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=plars@linuxtestproject.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