Commit 757bebb3 authored by Or Gerlitz's avatar Or Gerlitz Committed by Roland Dreier

IPoIB: Remove TX moderation settings from ethtool support

As of commit f56bcd80 ("IPoIB: Use separate CQ for UD send
completions"), there are no TX interrupts.  Change the ethtool code
not to report TX moderation settings, so users will not be misled to
think they can control TX interrupt moderation.  Pointed out by Alex
Vainman <alexv@voltaire.com>
Signed-off-by: default avatarOr Gerlitz <ogerlitz@voltaire.com>
Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
parent 676ad585
...@@ -55,9 +55,7 @@ static int ipoib_get_coalesce(struct net_device *dev, ...@@ -55,9 +55,7 @@ static int ipoib_get_coalesce(struct net_device *dev,
struct ipoib_dev_priv *priv = netdev_priv(dev); struct ipoib_dev_priv *priv = netdev_priv(dev);
coal->rx_coalesce_usecs = priv->ethtool.coalesce_usecs; coal->rx_coalesce_usecs = priv->ethtool.coalesce_usecs;
coal->tx_coalesce_usecs = priv->ethtool.coalesce_usecs;
coal->rx_max_coalesced_frames = priv->ethtool.max_coalesced_frames; coal->rx_max_coalesced_frames = priv->ethtool.max_coalesced_frames;
coal->tx_max_coalesced_frames = priv->ethtool.max_coalesced_frames;
return 0; return 0;
} }
...@@ -69,10 +67,8 @@ static int ipoib_set_coalesce(struct net_device *dev, ...@@ -69,10 +67,8 @@ static int ipoib_set_coalesce(struct net_device *dev,
int ret; int ret;
/* /*
* Since IPoIB uses a single CQ for both rx and tx, we assume * These values are saved in the private data and returned
* that rx params dictate the configuration. These values are * when ipoib_get_coalesce() is called
* saved in the private data and returned when ipoib_get_coalesce()
* is called.
*/ */
if (coal->rx_coalesce_usecs > 0xffff || if (coal->rx_coalesce_usecs > 0xffff ||
coal->rx_max_coalesced_frames > 0xffff) coal->rx_max_coalesced_frames > 0xffff)
...@@ -85,8 +81,6 @@ static int ipoib_set_coalesce(struct net_device *dev, ...@@ -85,8 +81,6 @@ static int ipoib_set_coalesce(struct net_device *dev,
return ret; return ret;
} }
coal->tx_coalesce_usecs = coal->rx_coalesce_usecs;
coal->tx_max_coalesced_frames = coal->rx_max_coalesced_frames;
priv->ethtool.coalesce_usecs = coal->rx_coalesce_usecs; priv->ethtool.coalesce_usecs = coal->rx_coalesce_usecs;
priv->ethtool.max_coalesced_frames = coal->rx_max_coalesced_frames; priv->ethtool.max_coalesced_frames = coal->rx_max_coalesced_frames;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment