<br><font size=2 face="sans-serif">Hi,</font>
<br>
<br><font size=2 face="sans-serif">you are right, this can be removed.
Feel free to post a patch.</font>
<br>
<br><font size=2 face="sans-serif">Regards,</font>
<br><font size=2 face="sans-serif">Jan-Bernd</font>
<br>
<br><tt><font size=2>"Eli Cohen" <eli@mellanox.co.il> wrote
on 24.06.2008 15:42:38:<br>
<br>
> On Tue, 2008-06-24 at 15:37 +0300, Or Gerlitz wrote:<br>
> > Vladimir Sokolovsky wrote:<br>
> > > +static int get_skb_hdr(struct sk_buff *skb, void **iphdr,<br>
> > > +               void
**tcph, u64 *hdr_flags, void *priv)<br>
> > > +{<br>
> > ><br>
> > > ...<br>
> > ><br>
> > > +<br>
> > > +    if (unlikely(skb->ip_summed != CHECKSUM_UNNECESSARY))<br>
> > > +        return -1;<br>
> > ><br>
> > > This will return -1 if HCA is not capable of doing checksum
offload.<br>
> > ><br>
> > Yes, but in that case __lro_proc_skb() would use the value of
 <br>
> > lro_mgr->ip_summed which you set independently of the HCA
capabilities, <br>
> > see below the code at the "out" label.<br>
> > <br>
> You're right about your observation. But I can't find a reason why
we<br>
> have this statement at all:<br>
> <br>
> out:  /* Original SKB has to be posted to stack */<br>
>    skb->ip_summed = lro_mgr->ip_summed;<br>
>    return 1;<br>
> <br>
> It seems that the skb's ip_summed field is not changed in the path
that<br>
> leads to this label so why changing this filed to something that was<br>
> initialized globally?<br>
> <br>
> Maybe one of the authors can comment on that?<br>
> <br>
> <br>
> <br>
> <br>
> <br>
> --<br>
> Eli Cohen<br>
> Mellanox Technologies LTD.<br>
> SW- Linux<br>
> Phone: +972 (4) 909 7200 (ext 276)<br>
> Mobile: +972 (54) 650 1206<br>
> E-mail: eli@mellanox.co.il<br>
> <br>
> ----------------------------------------------------------------------<br>
> Emails belong on computers, trees belong in forests; if you must print<br>
> this, do it on recycled paper.<br>
> http://www.greenpeace.org/international/<br>
> ----------------------------------------------------------------------<br>
> <br>
> <br>
> Disclaimer added by CodeTwo Exchange Rules<br>
> http://www.codetwo.com<br>
</font></tt>