On Fri, Nov 16, 2018 at 12:06 PM Cong Wang <xiyou.wangc...@gmail.com> wrote: > > Hmm, now I see how it works. Actually it uses the differences between > these two check's as the difference between hardware checksum with > skb_checksum(). >
Well... This is true only when there is a skb_checksum_init*() or skb_checksum_validate*() prior to it, it seems not true for nf_ip_checksum() where skb->csum is correctly set to pesudo header checksum but there is no validation of the original skb->csum. So this check should be still inverted there?? Or am I still missing anything here?