[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <52F22D8B.4000408@mellanox.com>
Date: Wed, 5 Feb 2014 14:24:43 +0200
From: Amir Vadai <amirv@...lanox.com>
To: Venkata Duvvuru <VenkatKumar.Duvvuru@...lex.Com>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
Ben Hutchings <bhutchings@...arflare.com>
Subject: Re: Patchset - Support for configurable RSS hash key
On 05/02/2014 14:20, Venkata Duvvuru wrote:
>
>
>> -----Original Message-----
>> From: Amir Vadai [mailto:amirv.mellanox@...il.com] On Behalf Of Amir
>> Vadai
>> Sent: Sunday, February 02, 2014 9:40 PM
>> To: Venkata Duvvuru
>> Cc: netdev@...r.kernel.org; Ben Hutchings
>> Subject: Re: Patchset - Support for configurable RSS hash key
>>
>> On 17/01/14 13:02 +0000, Venkata Duvvuru wrote:
>>> NIC drivers that support RSS use either a hard-coded value or a random
>> value for the RSS hash key. Irrespective of the type of the key used, the user
>> would want to change the hash key if he/she is not satisfied with the
>> effectiveness of the default hash-key in spreading the incoming flows evenly
>> across the RSS queues.
>>>
>>> This patch set provides support for configuring the RSS hash-key via the
>> ethtool interface.
>>>
>>> The patch set consists of:
>>> a) ethtool user-land patches
>>> b) ethtool kernel patch
>>> c) be2net patch that implements the ethtool hooks
>>
>> Hi,
>>
>> I would be happy to add support in setting the hash function in addition to
>> the hash key.
>>
>> Mellanox NIC's have support in 2 hash functions: Toplitz and XOR.
>>
>> Let me if you can add it to your patchset or would like me to send a patch on
>> top of yours to add it.
>
> I would prefer you to add the proposed changes on top my patchset. I'm working on the review comments currently and I will post the patchset once the patches are ready.
Ok,
I will send as soon as your patchset be applied.
Thanks,
Amir
--
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