Age | Commit message (Collapse) | Author | Files | Lines |
|
cmdline is allocated for usage in parse_generic_cmdline(), but is not
released after its final usage.
Signed-off-by: Yuval Mintz <yuvalmin@broadcom.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
When printing the private flags of the device, align all strings
to have the same length.
Signed-off-by: Yuval Mintz <yuvalmin@broadcom.com>
[bwh: Fix indentation]
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
This patch cleans up the mac type checks by using ethtool_regs.version
provided by the driver. This change eliminates the need to add device IDs
every time they are added to the driver.
Note that when using ethtool with this patch with a version of ixgbe that
does not provide the mac_type in ethtool_regs.version the register dump
may be incomplete. However this issue would've existed previously for
device IDs that were not added to ethtool.
Original patch and description by Emil Tantilov <emil.s.tantilov@intel.com>.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Vasundhara Volam <vasundhara.volam@emulex.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Restore the 20000baseKR2 which was removed by mistake back in June 2012.
Signed-off-by: Yaniv Rosner <yanivr@broadcom.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
There were numerous grammatical errors in these descriptions,
combined with poor formatting. Describe each mode under a
separate heading.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Avoid repeating 'flash' as a verb; that doesn't explain anything.
For consistency, describe both parameters (aside from devname) under
separate headings, formatted as lower-case italic.
Explain how the filename is resolved.
Explain what non-zero values of N would mean, and that they are
driver-dependent.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Some version check statements are lacking a space after the if ()
statement. Add them in.
Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Added a version check for the et131x reg dump, in case the dump format
needs to be changed later.
Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Previously only the lower 32 bits would be shown.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
vSwitch configurations.
Signed-off-by: Yan Burman <yanb@mellanox.com>
Signed-off-by: Amir Vadai <amirv@mellanox.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
This covers kernel changes up to:
commit dc2e57340deb8be1133b1eae2c7d4303133c133c
Author: Yan Burman <yanb@mellanox.com>
Date: Thu Dec 13 05:20:59 2012 +0000
net: ethool: Document struct ethtool_flow_ext
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
The current output of -m has been modified so that everything lines up
correctly.
The --module-info option alias has been added.
Signed-off-by: Aurelien Guillaume <aurelien@iwi.me>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
The et131x register set contains useful details of RX/TX queues for
the device. This dumps them with the -d option.
Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
If we have a pretty-printer matching the driver name for a register
dump, but it doesn't handle the reported dump version, then we
currently fail and print:
Cannot dump registers: Success
Instead, fall back to hex dump. Also, don't use perror() to explain a
failure of dump_regs(). errno is not meaningful in either failure
case.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
When setting EEE parameters with the --set-eee command line option,
ethtool will send the ETHTOOL_SEEE ioctl down to the driver even if
none of the provided parameters are a change from current settings.
Simply ignore it when that happens as done with other ethtool commands.
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
There are bits in this bitfield that we want to leave untouched (PAUSE
and ASYM_PAUSE bits) when changing other bits (speed and duplex bits.)
Previously, these were always overwritten to zero when running commands
like "ethtool -s eth0 speed 10 duplex full autoneg off".
Signed-off-by: Johan Gunnarsson <johangu@axis.com>
[bwh: Fixed spelling and formatting]
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
A bit ago ethtool added support for reading MDI-X state, this
patch finishes the implementation, adding the complementary write
command.
Add support to ethtool for controlling the MDI-X (crossover)
state of a network port. Most adapters correctly negotiate
MDI-X, but some ill-behaved switches have trouble and end up
picking the wrong MDI setting, which results in complete loss of
link. Usually this error condition can be observed when multiple
ethtool -r ethX are required before link is achieved.
This patch allows the user to override the normal "auto" setting
and force the crossover state to on or off.
The set will fail if the driver doesn't support the get, as
suggested by Ben Hutchings.
setting MDI not supported
In addition the do_gset output was changed slightly to report the
value set by the user (when the driver supports the set)
old:
MDI-X: on
new:
MDI-X: on (auto)
or
MDI-X: on (forced)
usage is ethtool -s eth0 mdix [auto|on|off]
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
CC: Ben Hutchings <bhutchings@solarflare.com>
Tested-by: Aaron Brown aaron.f.brown@intel.com
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
This covers kernel changes up to:
commit 6f6bbc186dc8e4e0c628db7decbd1a5e02cb5fd8
Author: Jesse Brandeburg <jesse.brandeburg@intel.com>
Date: Thu Jul 26 02:30:53 2012 +0000
ethtool.h: MDI setting support
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
I tagged v3.5 and pushed out master in the middle of rebasing master
to add the last fix for 3.5.
Conflicts:
NEWS
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
The ChangeLog is ancient history, replaced by the version control
changelog.
INSTALL is redundant in a binary package.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
|
|
1. defined values for KR4, CR4, SR4, LR4 PHY.
Signed-off-by: Parav Pandit <parav.pandit@emulex.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
This covers kernel changes up to:
commit e440cf2ca0a1b075c64016240d46c3aa9d877bbf
Author: parav.pandit@emulex.com <parav.pandit@emulex.com>
Date: Wed Jun 27 03:56:12 2012 +0000
net: added support for 40GbE link.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Some drivers have been seen to fill all bytes of
ethtool_drvinfo::fw_version without including a null terminator, which
effectively concatenates the following bytes to the string. We've
already dealt with a similar problem in dump_stats() (commit
7764430a139e4a089127f5616b0d56f497be1036). Try to cover all the
remaining string fields:
- In dump_drvinfo(), limit the length using printf() modifiers
- Add an option to get_stringset() to null-terminate all strings
- Change all callers except dump_stats() to set that option
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
This patch adds 2 new ethtool commands which can be
used to manipulate network interfaces' support in
EEE.
Output of 'get' has the following form:
EEE Settings for p2p1:
EEE status: enabled - active
Tx LPI: 1000 (us)
Supported EEE link modes: 10000baseT/Full
Advertised EEE link modes: 10000baseT/Full
Link partner advertised EEE link modes: 10000baseT/Full
Thanks goes to Giuseppe Cavallaro for his original patch.
Signed-off-by: Yuval Mintz <yuvalmin@broadcom.com>
Signed-off-by: Eilon Greenstein <eilong@broadcom.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
These kernel versions implement ETHTOOL_GFLAGS but do not include the
flags for VLAN tag offload (and do not implement ETHTOOL_GFEATURES).
Since the VLAN tag offload features were already defined and
implemented by many drivers, we shouldn't assume they are off.
Instead, since these feature flag values were stable, read them from
sysfs.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
We should not be checking for fixed features when we have no
information about which are fixed.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
The ethtool function for getting the rule count was not zeroing out the
data field before passing it to the kernel. As a result the value started
uninitialized and was incorrectly returning a result indicating that
devices supported setting new rule indexes. In order to correct this I am
adding a one line fix that sets data to zero before we pass the command to
the kernel.
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|
|
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
|