10/07/2020 16:40, Olivier Matz: > > > > > > > On Fri, Jul 10, 2020 at 02:55:51PM +0800, Hongzhi Guo wrote: > > > > > > > > Per RFC768: > > > > > > > > If the computed checksum is zero, it is transmitted as all > > > ones. > > > > > > > > An all zero transmitted checksum value means that the > > > transmitter > > > > > > > > generated no checksum. > > > > > > > > > > > > > > > > RFC793 for TCP has no such special treatment for the checksum > > > of > > > > > > > zero. > > > > > > > > > > > > > > > > Fixes: 6006818cfb26 ("net: new checksum functions") > > > > > > > > Cc: sta...@dpdk.org > > > > > > > > > > > > > > > > Signed-off-by: Hongzhi Guo <guohongz...@huawei.com> [...] > After reading again your arguments, I think I prefer your first > proposal, which was also Hongzhi's initial submission: > > - * The complemented checksum to set in the IP packet > - * or 0 on error > + * The complemented checksum to set in the IP packet. > > Thomas, do you want to to resubmit with this change?
Applied with all changes in comments.