diff mbox

myri10ge: use src+dst for rss hashing

Message ID 4B196A4F.5040408@myri.com
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Brice Goglin Dec. 4, 2009, 8 p.m. UTC
Use a more effective rss hash by default (src + dst, rather than just src).

Signed-off-by: Brice Goglin <brice@myri.com>



--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

David Miller Dec. 9, 2009, 4:25 a.m. UTC | #1
From: Brice Goglin <brice@myri.com>
Date: Fri, 04 Dec 2009 21:00:15 +0100

> Use a more effective rss hash by default (src + dst, rather than just src).
> 
> Signed-off-by: Brice Goglin <brice@myri.com>

Applied, but please get rid of this module parameter.

We have ethtool operations to configure this stuff, make
use of it instead of adding driver specific hacks.

Thank you.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

--- net-next-2.6/drivers/net/myri10ge/myri10ge.c	2009-12-04 20:10:48.000000000 +0100
+++ linux-tmp/drivers/net/myri10ge/myri10ge.c	2009-12-04 20:57:15.000000000 +0100
@@ -75,7 +75,7 @@ 
 #include "myri10ge_mcp.h"
 #include "myri10ge_mcp_gen_header.h"
 
-#define MYRI10GE_VERSION_STR "1.5.1-1.451"
+#define MYRI10GE_VERSION_STR "1.5.1-1.453"
 
 MODULE_DESCRIPTION("Myricom 10G driver (10GbE)");
 MODULE_AUTHOR("Maintainer: help@myri.com");
@@ -347,7 +347,7 @@ 
 module_param(myri10ge_max_slices, int, S_IRUGO);
 MODULE_PARM_DESC(myri10ge_max_slices, "Max tx/rx queues");
 
-static int myri10ge_rss_hash = MXGEFW_RSS_HASH_TYPE_SRC_PORT;
+static int myri10ge_rss_hash = MXGEFW_RSS_HASH_TYPE_SRC_DST_PORT;
 module_param(myri10ge_rss_hash, int, S_IRUGO);
 MODULE_PARM_DESC(myri10ge_rss_hash, "Type of RSS hashing to do");