Commit b3706909 authored by Amir Vadai's avatar Amir Vadai Committed by David S. Miller

net/mlx4_en: Prevent setting invalid RSS hash function

mlx4_en_check_rxfh_func() was checking for hardware support before
setting a known RSS hash function, but didn't do any check before
setting unknown RSS hash function. Need to make it fail on such values.
In this occasion, moved the actual setting of the new value from the
check function into mlx4_en_set_rxfh().

Fixes: 947cbb0a ("net/mlx4_en: Support for configurable RSS hash function")
Signed-off-by: default avatarAmir Vadai <amirv@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 33df10e2
...@@ -1102,20 +1102,21 @@ static int mlx4_en_check_rxfh_func(struct net_device *dev, u8 hfunc) ...@@ -1102,20 +1102,21 @@ static int mlx4_en_check_rxfh_func(struct net_device *dev, u8 hfunc)
struct mlx4_en_priv *priv = netdev_priv(dev); struct mlx4_en_priv *priv = netdev_priv(dev);
/* check if requested function is supported by the device */ /* check if requested function is supported by the device */
if ((hfunc == ETH_RSS_HASH_TOP && if (hfunc == ETH_RSS_HASH_TOP) {
!(priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_RSS_TOP)) || if (!(priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_RSS_TOP))
(hfunc == ETH_RSS_HASH_XOR &&
!(priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_RSS_XOR)))
return -EINVAL; return -EINVAL;
if (!(dev->features & NETIF_F_RXHASH))
priv->rss_hash_fn = hfunc; en_warn(priv, "Toeplitz hash function should be used in conjunction with RX hashing for optimal performance\n");
if (hfunc == ETH_RSS_HASH_TOP && !(dev->features & NETIF_F_RXHASH)) return 0;
en_warn(priv, } else if (hfunc == ETH_RSS_HASH_XOR) {
"Toeplitz hash function should be used in conjunction with RX hashing for optimal performance\n"); if (!(priv->mdev->dev->caps.flags2 & MLX4_DEV_CAP_FLAG2_RSS_XOR))
if (hfunc == ETH_RSS_HASH_XOR && (dev->features & NETIF_F_RXHASH)) return -EINVAL;
en_warn(priv, if (dev->features & NETIF_F_RXHASH)
"Enabling both XOR Hash function and RX Hashing can limit RPS functionality\n"); en_warn(priv, "Enabling both XOR Hash function and RX Hashing can limit RPS functionality\n");
return 0; return 0;
}
return -EINVAL;
} }
static int mlx4_en_get_rxfh(struct net_device *dev, u32 *ring_index, u8 *key, static int mlx4_en_get_rxfh(struct net_device *dev, u32 *ring_index, u8 *key,
...@@ -1189,6 +1190,8 @@ static int mlx4_en_set_rxfh(struct net_device *dev, const u32 *ring_index, ...@@ -1189,6 +1190,8 @@ static int mlx4_en_set_rxfh(struct net_device *dev, const u32 *ring_index,
priv->prof->rss_rings = rss_rings; priv->prof->rss_rings = rss_rings;
if (key) if (key)
memcpy(priv->rss_key, key, MLX4_EN_RSS_KEY_SIZE); memcpy(priv->rss_key, key, MLX4_EN_RSS_KEY_SIZE);
if (hfunc != ETH_RSS_HASH_NO_CHANGE)
priv->rss_hash_fn = hfunc;
if (port_up) { if (port_up) {
err = mlx4_en_start_port(dev); err = mlx4_en_start_port(dev);
......
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