Hi Herbert, Is there any update of this issue?
Regards, - Aubrey On 4/24/06, Aubrey <[EMAIL PROTECTED]> wrote: > 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
