netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David Schwartz" <davids@webmaster.com>
To: "Linux-Kernel@Vger. Kernel. Org" <linux-kernel@vger.kernel.org>
Cc: <netdev@vger.kernel.org>
Subject: RE: [RFC/PATCH] SO_NO_CHECK for IPv6
Date: Sun, 25 Nov 2007 07:41:52 -0800	[thread overview]
Message-ID: <MDEHLPKNGKAHNMBLJOLKCENAIGAC.davids@webmaster.com> (raw)
In-Reply-To: <E1Ivo4H-0006Z2-00@gondolin.me.apana.org.au>


> David Schwartz <davids@webmaster.com> wrote:

> >> Regardless of whatever verifications your application is doing
> >> on the data, it is not checksumming the ports and that's what
> >> the pseudo-header is helping with.

> > So what? We are in the case where the data has already gotten
> > to him. If it
> > got to him in error, he'll reject it anyway. The receive
> > checksum check will
> > only reject packets that he would reject anyway. That makes it needless.

> What if it goes to the wrong recipient who doesn't have the upper-
> level checksums?

Since that's not him, he has no control over its policy and thus no ability
to harm it or help it.

> This is the whole point, IPv6 unlike IPv4 does not have IP header
> checksums so the high-level needs to protect it by checksumming
> the pseudo-header.

Exactly. But *he* doesn't need to check that checksum, given that he already
got the packet, since he has an upper-level checksum. He is not saying that
his reasoning applies to everyone, just that it applies to him. He is not
talking about disabling the send checksum, but the receive checksum. He
knows that he does not need it.

DS



  reply	other threads:[~2007-11-25 15:41 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-21 12:45 [RFC/PATCH] SO_NO_CHECK for IPv6 Jeff Garzik
2007-11-21 13:20 ` YOSHIFUJI Hideaki / 吉藤英明
2007-11-21 13:42   ` Herbert Xu
2007-11-22  0:17   ` Jeff Garzik
2007-11-22  2:34     ` Herbert Xu
2007-11-22  2:46       ` YOSHIFUJI Hideaki / 吉藤英明
2007-11-23  1:13     ` David Miller
2007-11-23  6:06       ` David Schwartz
2007-11-24  6:00         ` Herbert Xu
2007-11-25 15:41           ` David Schwartz [this message]
2007-11-26  2:47             ` Herbert Xu
2007-11-21 18:35 ` David Miller

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=MDEHLPKNGKAHNMBLJOLKCENAIGAC.davids@webmaster.com \
    --to=davids@webmaster.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    /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).