Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Mon, 3 Nov 2014 19:33:45 +0100 Robert Richter wrote: > On 03.11.14 10:16:51, Stephen Hemminger wrote: > > On Fri, 31 Oct 2014 22:44:11 +0530 > > Sunil Kovvuri wrote: > > > > > On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger > > > wrote: > > > > On Thu, 30 Oct 2014 17:54:34 +0100 > > > > Robert Richter wrote: > > > > > > > >> +#ifdef VNIC_RSS_SUPPORT > > > >> +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | > > > >> RSS_UDP_HASH_ENA; > > > >> +module_param(rss_config, int, S_IRUGO); > > > >> +MODULE_PARM_DESC(rss_config, > > > >> + "RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, > > > >> 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, > > > >> 8:L4 bi-directional)"); > > > >> +#endif > > > > > > > > This should managed be via ethtool ETHTOOL_GRXFH rather than a module > > > > parameter. > > > Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. > > > The idea here is to have a choice of hash while module load (through > > > module params) and if it needs to be changed runtime then > > > via Ethtool. > > > > > > Sunil. > > > > Network developers do not like vendor unique module parameters. > > Anything device specific doesn't work in a generic distro environment. > > Do you accept unique module parameters in parallel to ethtool support > or should this be removed? If there is ethtool support the module parameters are not needed. Unneeded code is to be avoided. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On 03.11.14 10:16:51, Stephen Hemminger wrote: > On Fri, 31 Oct 2014 22:44:11 +0530 > Sunil Kovvuri wrote: > > > On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger > > wrote: > > > On Thu, 30 Oct 2014 17:54:34 +0100 > > > Robert Richter wrote: > > > > > >> +#ifdef VNIC_RSS_SUPPORT > > >> +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | > > >> RSS_UDP_HASH_ENA; > > >> +module_param(rss_config, int, S_IRUGO); > > >> +MODULE_PARM_DESC(rss_config, > > >> + "RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, > > >> 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 > > >> bi-directional)"); > > >> +#endif > > > > > > This should managed be via ethtool ETHTOOL_GRXFH rather than a module > > > parameter. > > Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. > > The idea here is to have a choice of hash while module load (through > > module params) and if it needs to be changed runtime then > > via Ethtool. > > > > Sunil. > > Network developers do not like vendor unique module parameters. > Anything device specific doesn't work in a generic distro environment. Do you accept unique module parameters in parallel to ethtool support or should this be removed? -Robert -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Fri, 31 Oct 2014 22:44:11 +0530 Sunil Kovvuri wrote: > On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger > wrote: > > On Thu, 30 Oct 2014 17:54:34 +0100 > > Robert Richter wrote: > > > >> +#ifdef VNIC_RSS_SUPPORT > >> +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | > >> RSS_UDP_HASH_ENA; > >> +module_param(rss_config, int, S_IRUGO); > >> +MODULE_PARM_DESC(rss_config, > >> + "RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, > >> 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 > >> bi-directional)"); > >> +#endif > > > > This should managed be via ethtool ETHTOOL_GRXFH rather than a module > > parameter. > Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. > The idea here is to have a choice of hash while module load (through > module params) and if it needs to be changed runtime then > via Ethtool. > > Sunil. Network developers do not like vendor unique module parameters. Anything device specific doesn't work in a generic distro environment. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Fri, 31 Oct 2014 22:44:11 +0530 Sunil Kovvuri sunil.kovv...@gmail.com wrote: On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger step...@networkplumber.org wrote: On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: +#ifdef VNIC_RSS_SUPPORT +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | RSS_UDP_HASH_ENA; +module_param(rss_config, int, S_IRUGO); +MODULE_PARM_DESC(rss_config, + RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 bi-directional)); +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. The idea here is to have a choice of hash while module load (through module params) and if it needs to be changed runtime then via Ethtool. Sunil. Network developers do not like vendor unique module parameters. Anything device specific doesn't work in a generic distro environment. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On 03.11.14 10:16:51, Stephen Hemminger wrote: On Fri, 31 Oct 2014 22:44:11 +0530 Sunil Kovvuri sunil.kovv...@gmail.com wrote: On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger step...@networkplumber.org wrote: On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: +#ifdef VNIC_RSS_SUPPORT +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | RSS_UDP_HASH_ENA; +module_param(rss_config, int, S_IRUGO); +MODULE_PARM_DESC(rss_config, + RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 bi-directional)); +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. The idea here is to have a choice of hash while module load (through module params) and if it needs to be changed runtime then via Ethtool. Sunil. Network developers do not like vendor unique module parameters. Anything device specific doesn't work in a generic distro environment. Do you accept unique module parameters in parallel to ethtool support or should this be removed? -Robert -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Mon, 3 Nov 2014 19:33:45 +0100 Robert Richter robert.rich...@caviumnetworks.com wrote: On 03.11.14 10:16:51, Stephen Hemminger wrote: On Fri, 31 Oct 2014 22:44:11 +0530 Sunil Kovvuri sunil.kovv...@gmail.com wrote: On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger step...@networkplumber.org wrote: On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: +#ifdef VNIC_RSS_SUPPORT +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | RSS_UDP_HASH_ENA; +module_param(rss_config, int, S_IRUGO); +MODULE_PARM_DESC(rss_config, + RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 bi-directional)); +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. The idea here is to have a choice of hash while module load (through module params) and if it needs to be changed runtime then via Ethtool. Sunil. Network developers do not like vendor unique module parameters. Anything device specific doesn't work in a generic distro environment. Do you accept unique module parameters in parallel to ethtool support or should this be removed? If there is ethtool support the module parameters are not needed. Unneeded code is to be avoided. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger wrote: > On Thu, 30 Oct 2014 17:54:34 +0100 > Robert Richter wrote: > >> +#ifdef VNIC_RSS_SUPPORT >> +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | >> RSS_UDP_HASH_ENA; >> +module_param(rss_config, int, S_IRUGO); >> +MODULE_PARM_DESC(rss_config, >> + "RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, >> 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 >> bi-directional)"); >> +#endif > > This should managed be via ethtool ETHTOOL_GRXFH rather than a module > parameter. Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. The idea here is to have a choice of hash while module load (through module params) and if it needs to be changed runtime then via Ethtool. Sunil. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On 30.10.14 19:45:13, Stephen Hemminger wrote: > On Thu, 30 Oct 2014 17:54:34 +0100 > Robert Richter wrote: > > > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > > index 1fa99a301817..80bd3336691e 100644 > > --- a/include/linux/pci_ids.h > > +++ b/include/linux/pci_ids.h > > @@ -2324,6 +2324,8 @@ > > #define PCI_DEVICE_ID_ALTIMA_AC91000x03ea > > #define PCI_DEVICE_ID_ALTIMA_AC10030x03eb > > > > +#define PCI_VENDOR_ID_CAVIUM 0x177d > > I don't think PCI folks want this updated with every id anymore. This is just the vendor id, the device id is part of the driver. Since there will be multiple drivers I put the vendor id here. -Robert -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On 30.10.14 19:45:13, Stephen Hemminger wrote: On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index 1fa99a301817..80bd3336691e 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -2324,6 +2324,8 @@ #define PCI_DEVICE_ID_ALTIMA_AC91000x03ea #define PCI_DEVICE_ID_ALTIMA_AC10030x03eb +#define PCI_VENDOR_ID_CAVIUM 0x177d I don't think PCI folks want this updated with every id anymore. This is just the vendor id, the device id is part of the driver. Since there will be multiple drivers I put the vendor id here. -Robert -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Fri, Oct 31, 2014 at 8:24 AM, Stephen Hemminger step...@networkplumber.org wrote: On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: +#ifdef VNIC_RSS_SUPPORT +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | RSS_UDP_HASH_ENA; +module_param(rss_config, int, S_IRUGO); +MODULE_PARM_DESC(rss_config, + RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 bi-directional)); +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. Thanks, i will add setting hash options via ETHTOOL_SRXFH as well. The idea here is to have a choice of hash while module load (through module params) and if it needs to be changed runtime then via Ethtool. Sunil. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter wrote: > +#ifdef VNIC_RSS_SUPPORT > +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | > RSS_UDP_HASH_ENA; > +module_param(rss_config, int, S_IRUGO); > +MODULE_PARM_DESC(rss_config, > + "RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, > 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 > bi-directional)"); > +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter wrote: > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index 1fa99a301817..80bd3336691e 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -2324,6 +2324,8 @@ > #define PCI_DEVICE_ID_ALTIMA_AC9100 0x03ea > #define PCI_DEVICE_ID_ALTIMA_AC1003 0x03eb > > +#define PCI_VENDOR_ID_CAVIUM 0x177d I don't think PCI folks want this updated with every id anymore. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index 1fa99a301817..80bd3336691e 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -2324,6 +2324,8 @@ #define PCI_DEVICE_ID_ALTIMA_AC9100 0x03ea #define PCI_DEVICE_ID_ALTIMA_AC1003 0x03eb +#define PCI_VENDOR_ID_CAVIUM 0x177d I don't think PCI folks want this updated with every id anymore. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] VNIC: Adding support for Cavium ThunderX network controller
On Thu, 30 Oct 2014 17:54:34 +0100 Robert Richter r...@kernel.org wrote: +#ifdef VNIC_RSS_SUPPORT +static int rss_config = RSS_IP_HASH_ENA | RSS_TCP_HASH_ENA | RSS_UDP_HASH_ENA; +module_param(rss_config, int, S_IRUGO); +MODULE_PARM_DESC(rss_config, + RSS hash config [bits 8:0] (Bit0:L2 extended, 1:IP, 2:TCP, 3:TCP SYN, 4:UDP, 5:L4 extended, 6:ROCE 7:L3 bi-directional, 8:L4 bi-directional)); +#endif This should managed be via ethtool ETHTOOL_GRXFH rather than a module parameter. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/