netdev - Re: [net-next 09/15] i40e: use priv flags to control packet split

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJ3xEMiqoLa9uKS5pw4ZhkQuE3b+UoVn3OqLHMudgLKH99fA7A@mail.gmail.com>
Date:	Wed, 9 Dec 2015 01:33:52 +0200
From:	Or Gerlitz <gerlitz.or@...il.com>
To:	Jesse Brandeburg <jesse.brandeburg@...il.com>
Cc:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>,
	Jesse Brandeburg <jesse.brandeburg@...el.com>,
	David Miller <davem@...emloft.net>,
	Linux Netdev List <netdev@...r.kernel.org>,
	"nhorman@...hat.com" <nhorman@...hat.com>,
	"sassmann@...hat.com" <sassmann@...hat.com>,
	"jogreene@...hat.com" <jogreene@...hat.com>
Subject: Re: [net-next 09/15] i40e: use priv flags to control packet split

On Wed, Dec 9, 2015 at 1:24 AM, Jesse Brandeburg
<jesse.brandeburg@...il.com> wrote:
> On Sat, Dec 5, 2015 at 1:25 PM, Or Gerlitz <gerlitz.or@...il.com> wrote:
>> HDS is a generic feature,and even if you think only Intel hardware is
>> the only hardware that supports that, you have multiple drivers..
>> right? why not go the right way of adding new netdev feature and
>> enable/disable it through the conventional ethtool feature on/off
>> directive?
>
> In this case, the return on the investment in the new feature is too
> low, IMO. If your hardware also supports this feature we can discuss
> adding this via ethtool, and I'd be glad to review any patches, and
> implement for Intel devices if necessary.

Wasn't sure to follow on the ROI comment, but anyway, we can renew
this discussion if/when Mellanox and/or more HWs to open that. BTW do
you iSCSI/NFS or alike benchmark of (say) 4KB payload to demo the gain
of the patch? where does your HW does the HDSplit in case of these or
other IP based storage protocols?

Or.

Or.
--
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