From: Marc Herbert <marc.herbert@free.fr>
To: Tim Hockin <thockin@hockin.org>
Cc: netdev@oss.sgi.com, linux-kernel@vger.kernel.org
Subject: Re: [RFC] ethtool semantics
Date: Mon, 14 Jun 2004 21:32:42 +0200 (CEST) [thread overview]
Message-ID: <Pine.LNX.4.58.0406142044060.1607@fcat> (raw)
In-Reply-To: <20040614170138.GA32594@hockin.org>
On Mon, 14 Jun 2004, Tim Hockin wrote:
> > This is precisely the reason why I am concerned about having "rich"
> > ethtool semantics. A unified, standard interface is great,... as long
> > it does not leave behind some features, like setting the advertised
> > values in autoneg. As a user of these features, I hope driver
> > developers will NOT remove those module_param features that cannot
> > migrated to ethtool.
>
> So propose a sane semantic that handles all three cases:
> * autoneg on
> * autoneg off
> * autoneg on but limited
Looking at the examples I mentioned earlier in the thread, one can
draw the following two simple solutions:
1. "Max speed advertised" solution
autoneg | on off
speed |
--------------|-----------------------------
|
<empty> | advertise all force 10
10 | adv. 10 force 10
100 | adv. 10|100 frc. 100
1000 | adv. 10|100|1000 frc. 1000
2. "Fixed speed advertised" solution
autoneg | on off
speed |
--------------|-----------------------------
|
<empty> | advertise all force 10
10 | adv. 10 force 10
100 | adv. 100 frc. 100
1000 | adv. 1000 frc. 1000
You can easily figure out similar and shorter tables for half/full
duplex (considering that duplex > half).
A 3rd solution which kind of avoids the dilemma between 1. and 2. is
to give the user full control on advertised bits, as does (did?) the
e1000 driver and its "AutoNeg" module_param. This third solution is
often less user friendly and probably not very useful. And it would
require a new argument to ethtool, whereas the first two solutions do
not.
If given the choice, I would vote for solution 1., but it probably
does not make much difference with solution number 2 in practice.
Auto negociation of flow control is unfortunately more complex, as you
can see in this discussion with Rich Seifert in
comp.dcom.lans.ethernet for those interested
http://groups.google.com/groups?threadm=87hdvnd0x3.fsf%40free.fr
But I believe flow control issues do not have any influence on the
above, so... first things first: no need to dive into this at this
point.
Obviously this message ignores legacy code, hardware bugs and others
"small matters of implementation". I suspect Roger Luethi has both a
knowledge of the related code and an opinion on this issue.
next prev parent reply other threads:[~2004-06-14 19:32 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-06-07 21:28 [RFC] ethtool semantics Roger Luethi
2004-06-07 21:57 ` David S. Miller
2004-06-07 23:43 ` Marc Herbert
2004-06-08 21:08 ` Roger Luethi
2004-06-09 21:09 ` Bill Davidsen
2004-06-09 21:38 ` Roger Luethi
2004-06-09 22:12 ` David S. Miller
2004-06-14 13:11 ` Marc Herbert
2004-06-14 17:01 ` Tim Hockin
2004-06-14 19:32 ` Marc Herbert [this message]
2004-06-14 19:42 ` Roger Luethi
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=Pine.LNX.4.58.0406142044060.1607@fcat \
--to=marc.herbert@free.fr \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@oss.sgi.com \
--cc=thockin@hockin.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