From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: David Miller <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Subject: Re: [PATCH] sungem: PHY updates & pause fixes
Date: Wed, 03 Jan 2007 16:40:02 +1100 [thread overview]
Message-ID: <1167802802.6165.129.camel@localhost.localdomain> (raw)
In-Reply-To: <20070102.212605.15263907.davem@davemloft.net>
On Tue, 2007-01-02 at 21:26 -0800, David Miller wrote:
> From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Date: Wed, 03 Jan 2007 16:20:14 +1100
>
> > Now, I cross-over with a TG3 and I get:
> >
> > - I advertise 0x5e1 (hopefully same value)
> > - I read in LPA 0xc5e1 from the TG3
> > (and that other register tells me Rx and Tx pause can be enabled).
>
> Does flow control get enabled with the TG3 on these
> switches? Just curious.
On the quad G5 which has tg3, I get "Flow control is on for TX and on
for RX". Let me double check if I'm on the same switch...
Heh ! It's not :-)
If I switch the cables, then pause is enabled on the sungem box and
disabled (both Tx and Rx) on tg3, so at least the behaviour is
consistent and follow the switches. Let me check the switch models...
The one with only asym. support is a big Cisco Catalyst 3350 (well.. big
but not that many ports :-)
The one with "classic" pause support is a tiny Netgear GS116
It's possible that the cisco is just misconfigured though, I'll have to
ask Keith when he's around tomorrow to take a peek at the mgmnt
interface.
> > > You should try to use flow control, even slower than 1000Mbit links.
> >
> > Sorry, not sure I parse ;-) You mean allowing pause on 10 and 100 as
> > well ? I sure can, it's easy to fix.
>
> That's correct.
On it's way.
Ben.
next prev parent reply other threads:[~2007-01-03 5:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-03 4:58 [PATCH] sungem: PHY updates & pause fixes Benjamin Herrenschmidt
2007-01-03 5:11 ` David Miller
2007-01-03 5:20 ` Benjamin Herrenschmidt
2007-01-03 5:26 ` David Miller
2007-01-03 5:40 ` Benjamin Herrenschmidt [this message]
2007-01-03 6:20 ` Benjamin Herrenschmidt
2007-01-04 2:55 ` David Miller
2007-01-04 20:06 ` Eric Lemoine
2007-01-04 20:27 ` David Miller
2007-01-04 20:57 ` Eric Lemoine
2007-01-04 21:15 ` Benjamin Herrenschmidt
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=1167802802.6165.129.camel@localhost.localdomain \
--to=benh@kernel.crashing.org \
--cc=davem@davemloft.net \
--cc=netdev@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;
as well as URLs for NNTP newsgroup(s).