On 4/24/06, Herbert Xu <[EMAIL PROTECTED]> wrote:
> On Mon, Apr 24, 2006 at 01:42:12PM +0800, Aubrey wrote:
> >
> >   dev->last_rx = jiffies;
> >   skb->dev = dev;
> >   skb->protocol = eth_type_trans(skb, dev);
> >   skb->ip_summed = CHECKSUM_UNNECESSARY;
> >   netif_rx(skb);
>
> This doesn't make sense.  First of all you're setting ip_summed to
> CHECK_UNNECESSARY unconditionally which is most likely wrong.
>
> What's more, if this was the driver that you were using, then
> checksum_udp couldn't possibly fail since the first thing it does
> is check ip_summed.

Hmm, when I change the code in __netpoll_rx, there is no the following
one line in the driver:
==============================================
skb->ip_summed = CHECKSUM_UNNECESSARY;
==============================================

The above one line code in the driver is just a workaround.

Regards,
-Aubrey
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to