On Tue, Jan 14, 2025 at 6:30 AM Taehee Yoo <ap420073@xxxxxxxxx> wrote: > > NICs that uses bnxt_en driver supports tcp-data-split feature by the > name of HDS(header-data-split). > But there is no implementation for the HDS to enable 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 rx-copybreak value. and it was unchangeable. > > This implements `ethtool -G <interface name> tcp-data-split <value>` > command option. > The value can be <on> and <auto>. > The value is <auto> and one of LRO/GRO/JUMBO is enabled, HDS is > automatically enabled and all LRO/GRO/JUMBO are disabled, HDS is > automatically disabled. > > HDS feature relies on the aggregation ring. > So, if HDS is enabled, the bnxt_en driver initializes the aggregation ring. > This is the reason why BNXT_FLAG_AGG_RINGS contains HDS condition. > > Acked-by: Jakub Kicinski <kuba@xxxxxxxxxx> > Tested-by: Stanislav Fomichev <sdf@xxxxxxxxxxx> > Tested-by: Andy Gospodarek <gospo@xxxxxxxxxxxx> > Signed-off-by: Taehee Yoo <ap420073@xxxxxxxxx> Thanks. Reviewed-by: Michael Chan <michael.chan@xxxxxxxxxxxx>
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature
