netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Russell King <rmk+lkml@arm.linux.org.uk>
To: linux-kernel@vger.kernel.org, netdev@oss.sgi.com, akpm@osdl.org,
	jgarzik@pobox.com, marcelo.tosatti@cyclades.com
Subject: Re: [patch 2.4.28-pre3] 3c59x: resync with 2.6
Date: Fri, 8 Oct 2004 19:13:24 +0100	[thread overview]
Message-ID: <20041008191324.J17999@flint.arm.linux.org.uk> (raw)
In-Reply-To: <20041008121307.C14378@tuxdriver.com>; from linville@tuxdriver.com on Fri, Oct 08, 2004 at 12:13:07PM -0400

On Fri, Oct 08, 2004 at 12:13:07PM -0400, John W. Linville wrote:
> Backport of current 3c59x driver (minus EISA/sysfs stuff) from 2.6 to
> 2.4.  This should ease further maintenance in 2.4.
> ---
> I've been chasing some 3c59x driver problems on both 2.4.x and 2.6.x
> kernels.  The 3c59x driver was pretty far out of sync between the two
> trees, so I thought it made sense to sync them back up.

Ah, if someone's looking at the 3c59x driver then please look into the
NWAY autonegotiation code - even maybe update it to use mii.c.

Currently 3c59x is rather buggy - it's a fairly simple bug.  Consider
this:

# mii-tool -v
eth0: negotiated 100baseTx-HD, link ok
  product info: TDK 78Q2120 rev 11
  basic mode:   autonegotiation enabled
  basic status: autonegotiation complete, link ok
  capabilities: 100baseTx-HD 10baseT-HD
  advertising:  100baseTx-HD 10baseT-HD
  link partner: 100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD flow-control

and then wonder why we end up like this:

eth0: Setting full-duplex based on MII #0 link partner capability of 45e1.

Obviously completely bogus.  Luckily, there isn't that much traffic
to this card _and_ it's connected to a switch so it doesn't interfere
much with other network traffic.  However, it is _dog_ slow when doing
large transfers to/from it.

FYI, it's:

04:00.0 Ethernet controller: 3Com Corporation 3c575 [Megahertz] 10/100 LAN CardBus (rev 01)

aka 3CCFE575BT.  The BT version only has a HD-capable MII transceiver,
whereas the CT version has a FD-capable MII transceiver fitted.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:  2.6 PCMCIA      - http://pcmcia.arm.linux.org.uk/
                 2.6 Serial core

  parent reply	other threads:[~2004-10-08 18:13 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-08 16:13 [patch 2.4.28-pre3] 3c59x: resync with 2.6 John W. Linville
2004-10-08 16:16 ` John W. Linville
2004-10-08 18:13 ` Russell King [this message]
2004-10-08 17:26   ` John W. Linville
2004-10-08 18:32   ` Jeff Garzik
2004-10-08 19:05 ` Jeff Garzik
2004-10-08 18:29   ` John W. Linville
2004-10-08 18:56   ` John W. Linville
2004-10-08 19:11   ` [patch 2.6.9-rc3] 3c59x: style change in vortex_ethtool_ops declaration John W. Linville

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=20041008191324.J17999@flint.arm.linux.org.uk \
    --to=rmk+lkml@arm.linux.org.uk \
    --cc=akpm@osdl.org \
    --cc=jgarzik@pobox.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo.tosatti@cyclades.com \
    --cc=netdev@oss.sgi.com \
    /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;
as well as URLs for NNTP newsgroup(s).