netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Martin Josefsson <gandalf@wlug.westbo.se>
To: kuznet@ms2.inr.ac.ru
Cc: netdev@oss.sgi.com
Subject: Re: raw ipv6 broken in 2.4.19
Date: 13 Aug 2002 19:14:34 +0200	[thread overview]
Message-ID: <1029258874.772.97.camel@tux> (raw)
In-Reply-To: <200208131358.RAA21125@sex.inr.ac.ru>

On Tue, 2002-08-13 at 15:58, kuznet@ms2.inr.ac.ru wrote: 

> Very strange. We have similar phenomenon reported with TCP, by the way.
> So, I have to assume that checksumming routine is wrong and does some shit
> sort of relying on an uninitialized data.

I've added some debug printk's and found out that it's the call to
csum_fold that fails in skb_copy_and_csum_datagram_iovec.

skb_copy_and_csum_datagram_iovec is called from:

net/ipv6/raw.c:rawv6_recvmsg()
	if (skb->ip_summed==CHECKSUM_UNNECESSARY) {
		err = skb_copy_datagram_iovec(skb, 0, msg->msg_iov, copied);
	} else if (msg->msg_flags&MSG_TRUNC) {
		if ((unsigned short)csum_fold(skb_checksum(skb, 0, skb->len, skb->csum)))
			goto csum_copy_err;
		err = skb_copy_datagram_iovec(skb, 0, msg->msg_iov, copied);
	} else {
		err = skb_copy_and_csum_datagram_iovec(skb, 0, msg->msg_iov);
		      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
		if (err == -EINVAL)
			goto csum_copy_err;
	}

This obviosly works with my old ISA ne2k clone but not with the tulip.
And the tcpdump looked ok

Does anyone else have an idea or suggestion I can try?

Alexey, when you had checksum problems, did you see invalid checksums in
tcpdump?

-- 
/Martin

Never argue with an idiot. They drag you down to their level, then beat
you with experience.

  reply	other threads:[~2002-08-13 17:14 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-12 19:07 raw ipv6 broken in 2.4.19 Martin Josefsson
2002-08-12 19:20 ` kuznet
2002-08-12 21:02   ` Martin Josefsson
2002-08-13  4:04     ` kuznet
2002-08-13 11:32       ` Martin Josefsson
2002-08-13 13:30         ` kuznet
2002-08-13 13:47           ` Martin Josefsson
2002-08-13 13:58             ` kuznet
2002-08-13 17:14               ` Martin Josefsson [this message]
2002-08-13 20:06                 ` kuznet
2002-08-13 21:29                   ` Martin Josefsson
2002-08-13 22:13                     ` kuznet
2002-08-14 12:15                   ` Donald Becker
2002-08-14 13:13                     ` kuznet
2002-08-14  0:15                 ` Julian Anastasov
2002-08-13 22:00                   ` kuznet
2002-08-14  1:12                     ` Julian Anastasov
2002-08-13 22:18                       ` kuznet
2002-08-14  1:38                         ` Julian Anastasov
2002-08-13 23:02                           ` Martin Josefsson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1029258874.772.97.camel@tux \
    --to=gandalf@wlug.westbo.se \
    --cc=kuznet@ms2.inr.ac.ru \
    --cc=netdev@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).