<div dir="auto">Thanks for your suggestion Stephen, I have already updated the patch with v4 & fixed the typo. Will consider your suggestion in the next version of the patch.</div><div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 2 Aug 2024 at 00:24, Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Thu,  1 Aug 2024 17:27:53 +0000<br>
Tathagat Priyadarshi <<a href="mailto:tathagat.dpdk@gmail.com" target="_blank">tathagat.dpdk@gmail.com</a>> wrote:<br>
<br>
> +             if (ol_flags & GVE_TX_CKSUM_OFFLOAD_MASK_DQO)<br>
> +                     csum = 1;<br>
> +             else<br>
> +                     cusm = 0;<br>
> +<br>
<br>
Obvious typo, did you do a final test build?<br>
<br>
Could also use logical inverse operator instead of if() which will<br>
generate better code sometimes.<br>
<br>
                csum = !!(ol_flags & GVE_TX_CKSUM_OFFLOAD_MASK_DQO);<br>
<br>
</blockquote></div></div>