[PATCH net] net/mlx4_en:

Subsystems: mellanox ethernet driver (mlx4_en), mellanox mlx4 core vpi driver, networking drivers, the rest

STALE4053d

5 messages, 3 authors, 2015-09-18 · open the first message on its own page

[PATCH net] net/mlx4_en:

From: Eric Dumazet <hidden>
Date: 2015-09-16 01:26:30

From: Eric Dumazet <redacted>

When changing rss key, we do not want to overwrite user provided key
by the one provided by netdev_rss_key_fill(), which is the host random
key generated at boot time.

Fixes: 947cbb0ac242 ("net/mlx4_en: Support for configurable RSS hash function")
Signed-off-by: Eric Dumazet <redacted>
Cc: Eyal Perry <redacted>
CC: Amir Vadai <redacted>
---
 drivers/net/ethernet/mellanox/mlx4/en_rx.c |    2 --
 1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
index 4402a1e48c9b..0ce6ffe73ca8 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
@@ -1268,8 +1268,6 @@ int mlx4_en_config_rss_steer(struct mlx4_en_priv *priv)
 		rss_context->hash_fn = MLX4_RSS_HASH_TOP;
 		memcpy(rss_context->rss_key, priv->rss_key,
 		       MLX4_EN_RSS_KEY_SIZE);
-		netdev_rss_key_fill(rss_context->rss_key,
-				    MLX4_EN_RSS_KEY_SIZE);
 	} else {
 		en_err(priv, "Unknown RSS hash function requested\n");
 		err = -EINVAL;

Re: [PATCH net] net/mlx4_en:

From: Eric Dumazet <hidden>
Date: 2015-09-16 01:28:09

Arg, patch title was meant to be

net/mlx4_en: really allow to change RSS key

[PATCH v2 net] net/mlx4_en: really allow to change RSS key

From: Eric Dumazet <hidden>
Date: 2015-09-16 01:29:48

From: Eric Dumazet <redacted>

When changing rss key, we do not want to overwrite user provided key
by the one provided by netdev_rss_key_fill(), which is the host random
key generated at boot time.

Fixes: 947cbb0ac242 ("net/mlx4_en: Support for configurable RSS hash function")
Signed-off-by: Eric Dumazet <redacted>
Cc: Eyal Perry <redacted>
CC: Amir Vadai <redacted>
---
 drivers/net/ethernet/mellanox/mlx4/en_rx.c |    2 --
 1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
index 4402a1e48c9b..0ce6ffe73ca8 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
@@ -1268,8 +1268,6 @@ int mlx4_en_config_rss_steer(struct mlx4_en_priv *priv)
 		rss_context->hash_fn = MLX4_RSS_HASH_TOP;
 		memcpy(rss_context->rss_key, priv->rss_key,
 		       MLX4_EN_RSS_KEY_SIZE);
-		netdev_rss_key_fill(rss_context->rss_key,
-				    MLX4_EN_RSS_KEY_SIZE);
 	} else {
 		en_err(priv, "Unknown RSS hash function requested\n");
 		err = -EINVAL;

Re: [PATCH v2 net] net/mlx4_en: really allow to change RSS key

From: Or Gerlitz <hidden>
Date: 2015-09-16 11:05:26

On Wed, Sep 16, 2015 at 4:29 AM, Eric Dumazet [off-list ref] wrote:
From: Eric Dumazet <redacted>

When changing rss key, we do not want to overwrite user provided key
by the one provided by netdev_rss_key_fill(), which is the host random
key generated at boot time.

Fixes: 947cbb0ac242 ("net/mlx4_en: Support for configurable RSS hash function")
Signed-off-by: Eric Dumazet <redacted>
Cc: Eyal Perry <redacted>
CC: Amir Vadai <redacted>
Acked-by: Or Gerlitz <redacted>

Dave, can you please push it to -stable of >= 3.19 ?

Re: [PATCH v2 net] net/mlx4_en: really allow to change RSS key

From: David Miller <davem@davemloft.net>
Date: 2015-09-18 04:04:29

From: Or Gerlitz <redacted>
Date: Wed, 16 Sep 2015 14:05:25 +0300
On Wed, Sep 16, 2015 at 4:29 AM, Eric Dumazet [off-list ref] wrote:
quoted
From: Eric Dumazet <redacted>

When changing rss key, we do not want to overwrite user provided key
by the one provided by netdev_rss_key_fill(), which is the host random
key generated at boot time.

Fixes: 947cbb0ac242 ("net/mlx4_en: Support for configurable RSS hash function")
Signed-off-by: Eric Dumazet <redacted>
Cc: Eyal Perry <redacted>
CC: Amir Vadai <redacted>
Acked-by: Or Gerlitz <redacted>

Dave, can you please push it to -stable of >= 3.19 ?
Applied and queued up for -stable, thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help