mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Abraham vd Merwe <abraham@2d3d.co.za>
To: Tim Hockin <thockin@hockin.org>
Cc: Jeff Garzik <jgarzik@mandrakesoft.com>,
	Linux Kernel Development <linux-kernel@vger.kernel.org>
Subject: Re: ethtool documentation
Date: Tue, 6 Aug 2002 12:17:51 +0200	[thread overview]
Message-ID: <20020806121751.A26908@crystal.2d3d.co.za> (raw)
In-Reply-To: <200208051906.g75J6d122986@www.hockin.org>; from thockin@hockin.org on Mon, Aug 05, 2002 at 12:06:39 -0700

[-- Attachment #1: Type: text/plain, Size: 3104 bytes --]

Hi Tim!

> These are the valid parameters to the SIOCETHTOOL ioctl().  Network drivers
> should support these as much as possible.
> 
> ETHTOOL_GSET
> ETHTOOL_SSET
> 
>   Get/set NIC settings.  These commands expect a 'struct ethtool_cmd *'
>   argument.  This struct includes fields for supported features (speed,
>   duplex, transceiver), advertised features, speed, duplex, port,
>   transceiver, and autonegotiation.  If the caller attempts to set an
>   invalid value for any field, return -EINVAL.

What is the difference between the supported and advertising fields?
What is MII? (as in the SUPPORTED_MII feature?).

If you can't control the # of ints before Tx/Rx, I take it you can just
set those fields to 0?

> ETHTOOL_GDRVINFO
> 
>   Get driver information.  This command expects a 'struct ethtool_drvinfo *'
>   argument.  This struct includes the driver identifier as a string, the
>   driver version as a string, bus information for the interface, and length
>   information for other ETHTOOL_* commands.

What do you set the bus_info field to if there is no bus? Where are all
these bus types defined? The header file isn't very clear about anything
other than pci bus types.

I take it you can just set fw_version to zero length string if it is unknown?

> ETHTOOL_GEEPROM
> ETHTOOL_SEEPROM
> 
>   Get/set EEPROM data.  These commands expect a 'struct ethtool_eeprom *'
>   argument.  This struct has a magic number, an offset and length pair, and a
>   data field.  If the offset+length are longer than the maximum size, the
>   extra is silently ignored.

Wouldn't it have been better to make this 'n character device which can be
read from / written to just like a normal file (/dev/nvram-like interface) -
that way applications can actually use unused eeprom space.

> ETHTOOL_GCOALESCE
> ETHTOOL_SCOALESCE
> 
>   Get/set coalescing parameters.  These commands expect a 'struct
>   ethtool_coalesce *' argument.  This struct has several fields for
>   configuring coalescing - see ethtool.h for details.  If the caller
>   attempts to set an invalid value, return -EINVAL.

Wtf is coalescing parameters? These commands aren't even defined in the
2.4.18 kernel headers. Is this 2.5.xx additions?

Same goes for the following commands:

> ETHTOOL_GRINGPARAM
> ETHTOOL_SRINGPARAM
> ETHTOOL_GPAUSEPARAM
> ETHTOOL_SPAUSEPARAM
> ETHTOOL_GRXCSUM
> ETHTOOL_SRXCSUM
> ETHTOOL_GTXCSUM
> ETHTOOL_STXCSUM
> ETHTOOL_GSG
> ETHTOOL_SSG
> ETHTOOL_TEST
> ETHTOOL_GSTRINGS
> ETHTOOL_PHYS_ID
> ETHTOOL_GSTATS

-- 

Regards
 Abraham

Lying is an indispensable part of making life tolerable.
		-- Bergan Evans

__________________________________________________________
 Abraham vd Merwe - 2d3D, Inc.

 Device Driver Development, Outsourcing, Embedded Systems

  Cell: +27 82 565 4451         Snailmail:
   Tel: +27 21 761 7549            Block C, Aintree Park
   Fax: +27 21 761 7648            Doncaster Road
 Email: abraham@2d3d.co.za         Kenilworth, 7700
  Http: http://www.2d3d.com        South Africa


[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]

  parent reply	other threads:[~2002-08-06 10:10 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-05 11:46 Abraham vd Merwe
2002-08-05 15:42 ` Jeff Garzik
2002-08-05 19:06   ` Tim Hockin
2002-08-05 19:16     ` Jeff Garzik
2002-08-05 19:30       ` Jonathan Lundell
2002-08-05 20:20         ` Jeff Garzik
2002-08-06 10:17     ` Abraham vd Merwe [this message]
2002-08-06 10:07       ` David S. Miller
2002-08-06 15:41 Randy.Dunlap
2002-08-06 19:24 ` Richard B. Johnson
2002-08-06 19:40   ` Chris Friesen
2002-08-06 19:44     ` Randy.Dunlap
2002-08-06 20:03     ` Richard B. Johnson
2002-08-06 20:04       ` Randy.Dunlap
2002-08-06 20:36         ` Ben Greear
2002-08-06 21:15       ` Tim Hockin
2002-08-07  1:42         ` Richard B. Johnson
2002-08-07  3:33           ` Tim Hockin
2002-08-06 21:58       ` Alan Cox
2002-08-06 20:42         ` Randy.Dunlap
2002-08-06 22:27           ` Alan Cox
2002-08-19 19:00             ` Mark H. Wood
2002-08-07  1:57         ` Richard B. Johnson
2002-08-07 11:47           ` Alan Cox
2002-08-07  2:54   ` Ryan Anderson
2002-08-07  5:32   ` Dax Kelson
2002-08-07  6:30     ` Jonathan Lundell
2002-08-07 11:47       ` Alan Cox
2002-08-08  1:07         ` Peter Samuelson
2002-08-07 11:18     ` Richard B. Johnson
2002-08-07 12:05       ` Abraham vd Merwe
2002-08-07 18:44       ` Bernd Eckenfels
2002-08-07 20:16       ` erik
2002-08-07  8:02 ` Abraham vd Merwe
2002-08-06 20:57 Leif Sawyer
2002-08-07  1:51 ` Richard B. Johnson
2002-08-07 10:53   ` Jamie Lokier
2002-08-07 18:16 Leif Sawyer
2002-08-08  9:18 Helge Hafting

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=20020806121751.A26908@crystal.2d3d.co.za \
    --to=abraham@2d3d.co.za \
    --cc=jgarzik@mandrakesoft.com \
    --cc=linux-kernel@vger.kernel.org \
    --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

all inboxes | Powered by JetHome®