netdev - [PATCH net-next 0/2] bnxt_en: implement tcp-data-split ethtool command

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 for Android: free password hash cracker in your pocket

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

Message-Id: <20240906080750.1068983-1-ap420073@gmail.com>
Date: Fri,  6 Sep 2024 08:07:48 +0000
From: Taehee Yoo <ap420073@...il.com>
To: davem@...emloft.net,
	kuba@...nel.org,
	pabeni@...hat.com,
	edumazet@...gle.com,
	michael.chan@...adcom.com,
	netdev@...r.kernel.org
Cc: ap420073@...il.com
Subject: [PATCH net-next 0/2] bnxt_en: implement tcp-data-split ethtool command

NICs that use the bnxt_en driver support tcp-data-split feature named
HDS(header-data-split).
But there is no implementation for the HDS to enable/disable by ethtool.
Only getting the current HDS status is implemented and the HDS is just
automatically enabled only when either LRO, HW-GRO, or JUMBO is enabled.
The hds_threshold follows the rx-copybreak value but it wasn't
changeable.

To implement, it requires decoupling rx-copybreak and hds_threshold
value.

The first patch implements .{set, get}_tunable() in the bnxt_en.
The bnxt_en driver has been supporting the rx-copybreak feature but is
not configurable, Only the default rx-copybreak value has been working.
So, it changes the bnxt_en driver to be able to configure
the rx-copybreak value.

The second patch adds an implementation of tcp-data-split ethtool
command.
The HDS relies on the Aggregation ring, which is automatically enabled
when either LRO, GRO, or large mtu is configured.
So, if the Aggregation ring is enabled, HDS is automatically enabled by
it.

The approach of this patch is to support the bnxt_en driver setting up
enable/disable HDS explicitly, not rely on LRO/GRO, JUMBO.
In addition, hds_threshold no longer follows rx-copybreak.
By this patch, hds_threshold always be 0.

Taehee Yoo (2):
  bnxt_en: add support for rx-copybreak ethtool command
  bnxt_en: add support for tcp-data-split ethtool command

 drivers/net/ethernet/broadcom/bnxt/bnxt.c     | 30 ++++----
 drivers/net/ethernet/broadcom/bnxt/bnxt.h     | 10 ++-
 .../net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 70 ++++++++++++++++++-
 3 files changed, 91 insertions(+), 19 deletions(-)

-- 
2.34.1