lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20100328154730.5261e8cf@nehalam> Date: Sun, 28 Mar 2010 15:47:30 -0700 From: Stephen Hemminger <shemminger@...tta.com> To: David Miller <davem@...emloft.net> Cc: netdev@...r.kernel.org Subject: [PATCH 2/2] ethtool: add RSS command flag Add RSS offload option to ethtool command. Also, fix ethtool header to be pure version from net-next. ethtool-copy.h should be generated by kernel 'make install_headers' to ensure that all fields match exactly. Signed-off-by: Stephen Hemminger <shemminger@...tta.com> --- ethtool-copy.h | 32 +++++++++++++++++++++++++------- ethtool.8 | 4 ++++ ethtool.c | 43 ++++++++++++++++++++++++++++++++++++++----- 3 files changed, 67 insertions(+), 12 deletions(-) diff --git a/ethtool-copy.h b/ethtool-copy.h index 8681f5e..a20fe84 100644 --- a/ethtool-copy.h +++ b/ethtool-copy.h @@ -36,7 +36,7 @@ struct ethtool_cmd { __u32 reserved[2]; }; -static inline void ethtool_cmd_speed_set(struct ethtool_cmd *ep, +static __inline__ void ethtool_cmd_speed_set(struct ethtool_cmd *ep, __u32 speed) { @@ -44,7 +44,7 @@ static inline void ethtool_cmd_speed_set(struct ethtool_cmd *ep, ep->speed_hi = (__u16)(speed >> 16); } -static inline __u32 ethtool_cmd_speed(struct ethtool_cmd *ep) +static __inline__ __u32 ethtool_cmd_speed(struct ethtool_cmd *ep) { return (ep->speed_hi << 16) | ep->speed; } @@ -61,6 +61,13 @@ struct ethtool_drvinfo { /* For PCI devices, use pci_name(pci_dev). */ char reserved1[32]; char reserved2[12]; + /* + * Some struct members below are filled in + * using ops->get_sset_count(). Obtaining + * this info from ethtool_drvinfo is now + * deprecated; Use ETHTOOL_GSSET_INFO + * instead. + */ __u32 n_priv_flags; /* number of flags valid in ETHTOOL_GPFLAGS */ __u32 n_stats; /* number of u64's from ETHTOOL_GSTATS */ __u32 testinfo_len; @@ -253,6 +260,17 @@ struct ethtool_gstrings { __u8 data[0]; }; +struct ethtool_sset_info { + __u32 cmd; /* ETHTOOL_GSSET_INFO */ + __u32 reserved; + __u64 sset_mask; /* input: each bit selects an sset to query */ + /* output: each bit a returned sset */ + __u32 data[0]; /* ETH_SS_xxx count, in order, based on bits + in sset_mask. One bit implies one + __u32, two bits implies two + __u32's, etc. */ +}; + enum ethtool_test_flags { ETH_TEST_FL_OFFLINE = (1 << 0), /* online / offline */ ETH_TEST_FL_FAILED = (1 << 1), /* test passed / failed */ @@ -292,6 +310,7 @@ struct ethtool_perm_addr { enum ethtool_flags { ETH_FLAG_LRO = (1 << 15), /* LRO is enabled */ ETH_FLAG_NTUPLE = (1 << 27), /* N-tuple filters enabled */ + ETH_FLAG_RSS = (1 << 28), }; /* The following structures are for supporting RX network flow @@ -389,8 +408,6 @@ struct ethtool_rx_ntuple_flow_spec { #define ETHTOOL_RXNTUPLE_ACTION_DROP -1 }; -#define ETHTOOL_MAX_NTUPLE_LIST_ENTRY 1024 -#define ETHTOOL_MAX_NTUPLE_STRING_PER_ENTRY 14 struct ethtool_rx_ntuple { __u32 cmd; struct ethtool_rx_ntuple_flow_spec fs; @@ -408,6 +425,7 @@ struct ethtool_flash { char data[ETHTOOL_FLASH_MAX_FILENAME]; }; + /* CMDs currently supported */ #define ETHTOOL_GSET 0x00000001 /* Get settings. */ #define ETHTOOL_SSET 0x00000002 /* Set settings. */ @@ -463,9 +481,9 @@ struct ethtool_flash { #define ETHTOOL_SRXCLSRLINS 0x00000032 /* Insert RX classification rule */ #define ETHTOOL_FLASHDEV 0x00000033 /* Flash firmware to device */ #define ETHTOOL_RESET 0x00000034 /* Reset hardware */ - -#define ETHTOOL_SRXNTUPLE 0x00000035 /* Add an n-tuple filter to device */ -#define ETHTOOL_GRXNTUPLE 0x00000036 /* Get n-tuple filters from device */ +#define ETHTOOL_SRXNTUPLE 0x00000035 /* Add an n-tuple filter to device */ +#define ETHTOOL_GRXNTUPLE 0x00000036 /* Get n-tuple filters from device */ +#define ETHTOOL_GSSET_INFO 0x00000037 /* Get string set info */ /* compatibility with older code */ #define SPARC_ETH_GSET ETHTOOL_GSET diff --git a/ethtool.8 b/ethtool.8 index eb6430b..a5afe13 100644 --- a/ethtool.8 +++ b/ethtool.8 @@ -170,6 +170,7 @@ ethtool \- Display or change ethernet card settings .B2 gso on off .B2 gro on off .B2 lro on off +.B2 rss on off .B ethtool \-p|\-\-identify .I ethX @@ -362,6 +363,9 @@ Specifies whether generic receive offload should be enabled .A2 lro on off Specifies whether large receive offload should be enabled .TP +.A2 rss on off +Specifies whether Receive Side Scaling hashing should be enabled +.TP .B \-p \-\-identify Initiates adapter-specific action intended to enable an operator to easily identify the adapter by sight. Typically this involves diff --git a/ethtool.c b/ethtool.c index fc9e419..8b6cc77 100644 --- a/ethtool.c +++ b/ethtool.c @@ -173,6 +173,8 @@ static struct option { " [ gro on|off ]\n" " [ lro on|off ]\n" " [ ntuple on|off ]\n" + " [ rss on|off ]\n" + }, { "-i", "--driver", MODE_GDRV, "Show driver information" }, { "-d", "--register-dump", MODE_GREGS, "Do a register dump", @@ -257,6 +259,7 @@ static int off_gso_wanted = -1; static int off_lro_wanted = -1; static int off_gro_wanted = -1; static int off_ntuple_wanted = -1; +static int off_rss_wanted = -1; static struct ethtool_pauseparam epause; static int gpause_changed = 0; @@ -382,6 +385,7 @@ static struct cmdline_info cmdline_offload[] = { { "lro", CMDL_BOOL, &off_lro_wanted, NULL }, { "gro", CMDL_BOOL, &off_gro_wanted, NULL }, { "ntuple", CMDL_BOOL, &off_ntuple_wanted, NULL }, + { "rss", CMDL_BOOL, &off_rss_wanted, NULL }, }; static struct cmdline_info cmdline_pause[] = { @@ -1526,7 +1530,7 @@ static int dump_coalesce(void) } static int dump_offload(int rx, int tx, int sg, int tso, int ufo, int gso, - int gro, int lro, int ntuple) + int gro, int lro, int ntuple, int rss) { fprintf(stdout, "rx-checksumming: %s\n" @@ -1537,7 +1541,8 @@ static int dump_offload(int rx, int tx, int sg, int tso, int ufo, int gso, "generic-segmentation-offload: %s\n" "generic-receive-offload: %s\n" "large-receive-offload: %s\n" - "ntuple-filters: %s\n", + "ntuple-filters: %s\n" + "receive-side-scaling: %s\n", rx ? "on" : "off", tx ? "on" : "off", sg ? "on" : "off", @@ -1546,7 +1551,8 @@ static int dump_offload(int rx, int tx, int sg, int tso, int ufo, int gso, gso ? "on" : "off", gro ? "on" : "off", lro ? "on" : "off", - ntuple ? "on" : "off"); + ntuple ? "on" : "off", + rss ? "on" : "off"); return 0; } @@ -1863,7 +1869,7 @@ static int do_goffload(int fd, struct ifreq *ifr) { struct ethtool_value eval; int err, allfail = 1, rx = 0, tx = 0, sg = 0; - int tso = 0, ufo = 0, gso = 0, gro = 0, lro = 0, ntuple = 0; + int tso = 0, ufo = 0, gso = 0, gro = 0, lro = 0, ntuple = 0, rss = 0; fprintf(stdout, "Offload parameters for %s:\n", devname); @@ -1935,6 +1941,7 @@ static int do_goffload(int fd, struct ifreq *ifr) } else { lro = (eval.data & ETH_FLAG_LRO) != 0; ntuple = (eval.data & ETH_FLAG_NTUPLE) != 0; + rss = (eval.data & ETH_FLAG_RSS) != 0; allfail = 0; } @@ -1953,7 +1960,7 @@ static int do_goffload(int fd, struct ifreq *ifr) return 83; } - return dump_offload(rx, tx, sg, tso, ufo, gso, gro, lro, ntuple); + return dump_offload(rx, tx, sg, tso, ufo, gso, gro, lro, ntuple, rss); } static int do_soffload(int fd, struct ifreq *ifr) @@ -2087,6 +2094,29 @@ static int do_soffload(int fd, struct ifreq *ifr) return 93; } } + if (off_rss_wanted >= 0) { + changed = 1; + eval.cmd = ETHTOOL_GFLAGS; + eval.data = 0; + ifr->ifr_data = (caddr_t)&eval; + err = ioctl(fd, SIOCETHTOOL, ifr); + if (err) { + perror("Cannot get device flag settings"); + return 91; + } + + eval.cmd = ETHTOOL_SFLAGS; + if (off_rss_wanted) + eval.data |= ETH_FLAG_RSS; + else + eval.data &= ~ETH_FLAG_RSS; + + err = ioctl(fd, SIOCETHTOOL, ifr); + if (err) { + perror("Cannot set n-tuple filter settings"); + return 93; + } + } if (!changed) { fprintf(stdout, "no offload settings changed\n"); @@ -2655,6 +2685,9 @@ static int do_srxntuple(int fd, struct ifreq *ifr) return 0; } +#define ETHTOOL_MAX_NTUPLE_LIST_ENTRY 1024 +#define ETHTOOL_MAX_NTUPLE_STRING_PER_ENTRY 14 + static int do_grxntuple(int fd, struct ifreq *ifr) { struct ethtool_gstrings *strings; -- 1.6.3.3 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists