> On Tue, 17 Feb 2026 09:33:56 +0100 Lorenzo Bianconi wrote: > > + * In case of success, ``ip_summed`` is set to the RX checksum result. > > Possible > > + * values are: > > + * ``XDP_CHECKSUM_NONE`` > > + * ``XDP_CHECKSUM_UNNECESSARY`` > > + * ``XDP_CHECKSUM_COMPLETE`` > > + * > > + * In case of success, ``cksum_meta`` contains the hw computed checksum > > value > > + * for ``XDP_CHECKSUM_COMPLETE`` or the ``csum_level`` for > > + * ``XDP_CHECKSUM_UNNECESSARY``. It is set to 0 for ``XDP_CHECKSUM_NONE`` > > It's fairly common for NICs to report both csum complete and > unnecessary. Which one should the driver return in that case?
Do you mean what is value for cksum_meta if we do not report csum_level for XDP_CHECKSUM_UNNECESSARY/CHECKSUM_UNNECESSARY use-case? (as suggested by Stanislav). My original idea is: - if the hw reports CHECKSUM_COMPLETE: - ip_summed = XDP_CHECKSUM_COMPLETE - cksum_meta contains the checksum computed by the hw - if the hw reports CHECKSUM_UNNECESSARY - ip_summed = XDP_CHECKSUM_UNNECESSARY - cksum_meta = csum_level <-- Stanislav suggests to drop this one - if the hw reports CHECKSUM_NONE - ip_summed = XDP_CHECKSUM_NONE - cksum_meta = 0 Regards, Lorenzo > What if the user prefers the other one?..
signature.asc
Description: PGP signature
