From: Hugo Santos <hsantos@av.it.pt>
To: Masahide NAKAMURA <nakam@linux-ipv6.org>
Cc: davem@davemloft.net, yoshfuji@linux-ipv6.org, anttit@tcs.hut.fi,
vnuorval@tcs.hut.fi, netdev@vger.kernel.org,
usagi-core@linux-ipv6.org
Subject: Re: [RFC] Mobile IPv6 introduction
Date: Wed, 2 Aug 2006 11:47:56 +0100 [thread overview]
Message-ID: <20060802104756.GN8334@innerghost.net> (raw)
In-Reply-To: <44D01AEE.3030606@linux-ipv6.org>
[-- Attachment #1: Type: text/plain, Size: 2161 bytes --]
Hi,
Thanks for the reply, however:
On Wed, Aug 02, 2006 at 12:24:30PM +0900, Masahide NAKAMURA wrote:
> Our patch is similar as you said. Our design is that kernel does nothing
> as possible about validation which can be done by user-space.
> As you mentioned ICMPv6 error is hard to be sent by user-space because it carries
> original packet causing error. MIPv6 RFC says when mobility header length is too short
> ICMPv6 error (parameter problem) is sent. We also discussed about design like your choice.
> but we have not taken it because ICMPv6 sending mechanism is already in kernel
> then it is reasonable to use it. We MIPL developers concluded that kernel should
> know mobility header types and their minimum length at least. I guess when we would
> support NEMO and FMIPv6, we just add their defines at that time.
> (Actually, their implementations based on MIPL2 exists.)
> If somebody would feel that such defines should be removed from kernel we have another
> idea to make new socket interface like ICMP filter to store mobility header type and its
> minimum length to kernel by user-space.
Although the ICMP-filter approach would be better, it is not flexible
enough to handle this situation. We must also send ICMPv6 Parameter
Problems when ip6mh_proto isn't IPPROTO_NONE. I don't think it is too
much of a burthen to handle ICMPv6 in the control daemon because you
should already do so to react to ICMPv6 error messages from peers
concerning MIPv6 signalling. I'm strongly against doing these checks in
the kernel for the simple reason that it is not easily extendable. You
wouldn't be able to deploy a new daemon version over an existing kernel
with these changes if it supported a new control protocol with new
messages. I think we should follow a different path here and i propose
either have a hdrinc=1 mode (for reception only) for protocol raw
sockets, possibly adding with control on reception which specifies the
offset of the UPL header; or have a control message to obtain the
network headers. For instance:
put_cmsg(msg, SOL_IPV6, ..., (skb->h.raw - skb->nh.raw),
skb->nh.raw);
Hugo
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2006-08-02 10:48 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-29 9:23 [RFC] Mobile IPv6 introduction Masahide NAKAMURA
2006-07-29 9:28 ` [PATCH 0/23][XFRM] MIPv6 CN introduction (part A) (Re: [RFC] Mobile IPv6 introduction) Masahide NAKAMURA
2006-07-29 9:37 ` [PATCH 0/20][IPV6/XFRM] MIPv6 CN (part B) Masahide NAKAMURA
2006-08-02 0:30 ` David Miller
2006-08-02 8:26 ` Masahide NAKAMURA
2006-07-29 14:12 ` [RFC] Mobile IPv6 introduction Hugo Santos
2006-08-02 0:35 ` David Miller
2006-08-02 0:58 ` Hugo Santos
2006-08-02 1:04 ` David Miller
2006-08-02 1:52 ` YOSHIFUJI Hideaki / 吉藤英明
2006-08-02 7:58 ` Ville Nuorvala
2006-08-02 11:03 ` Hugo Santos
2006-08-02 3:24 ` Masahide NAKAMURA
2006-08-02 10:47 ` Hugo Santos [this message]
2006-08-02 13:03 ` Masahide NAKAMURA
2006-08-02 21:14 ` 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=20060802104756.GN8334@innerghost.net \
--to=hsantos@av.it.pt \
--cc=anttit@tcs.hut.fi \
--cc=davem@davemloft.net \
--cc=nakam@linux-ipv6.org \
--cc=netdev@vger.kernel.org \
--cc=usagi-core@linux-ipv6.org \
--cc=vnuorval@tcs.hut.fi \
--cc=yoshfuji@linux-ipv6.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).