From: Nicolas Dichtel <nicolas.dichtel@6wind•com>
To: David Miller <davem@davemloft•net>
Cc: hannes@stressinduktion•org, netdev@vger•kernel.org,
David.Laight@ACULAB•COM, jiri@resnulli•us, vyasevich@gmail•com,
kuznet@ms2•inr.ac.ru, jmorris@namei•org, yoshfuji@linux-ipv6•org,
kaber@trash•net, thaller@redhat•com, stephen@networkplumber•org
Subject: Re: [RFC PATCH net-next 2/2] udp: add sk opt to allow sending pkt with src 0.0.0.0
Date: Thu, 14 Nov 2013 14:05:00 +0100 [thread overview]
Message-ID: <5284CA7C.7030402@6wind.com> (raw)
In-Reply-To: <20131111.001803.110603937316567461.davem@davemloft.net>
Le 11/11/2013 06:18, David Miller a écrit :
> From: Nicolas Dichtel <nicolas.dichtel@6wind•com>
> Date: Sat, 9 Nov 2013 01:54:34 +0100
>
>> This feature allows to a send packets with address source set to 0.0.0.0 even if
>> an ip address is available on another interface.
>>
>> It's useful for DHCP client, to allow them to use UDP sockets and be compliant
>> with the RFC2131, Section 4.1:
>>
>> 4.1 Constructing and sending DHCP messages
>> ...
>> DHCP messages broadcast by a client prior to that client obtaining
>> its IP address must have the source address field in the IP header
>> set to 0.
>>
>> Based on a previous work from
>> Guillaume Gaudonville <guillaume.gaudonville@6wind•com>.
>>
>> Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind•com>
>
> This requirement of the RFC is inconsistent with a host based
> addressing model, that which Linux employs, it assumes an interface
> based one.
There are some exceptions, for example when user tunes arp_ignore sysctl ;-)
>
> The wording here is also very ambiguous.
>
> This RFC fails to even remotely consider what the right behavior
> should be in a host based addressing environment at all, and anyone
> reading this RFC should just accept that.
I agree that this is ambiguous. And it's a 'must', not a 'MUST', which
is not the same for an RFC ;-)
>
> Furthermore, the fact that you're implementing _addressing_ policy in
> the UDP code makes this change even more unreasonable.
>
I made this choice because using 0.0.0.0, for TCP for example, seems a
non-sense.
But fair enough, let's drop this patch.
Thank you,
Nicolas
next prev parent reply other threads:[~2013-11-14 13:05 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-24 13:45 [patch net-next] ipv6: allow userspace to create address with IFLA_F_TEMPORARY flag Jiri Pirko
2013-10-24 13:48 ` [patch iproute2] allow to create temporary addresses Jiri Pirko
2013-10-24 14:02 ` [patch net-next] ipv6: allow userspace to create address with IFLA_F_TEMPORARY flag Hannes Frederic Sowa
2013-10-24 16:59 ` Jiri Pirko
2013-10-25 10:05 ` Vladislav Yasevich
2013-10-25 20:12 ` Hannes Frederic Sowa
2013-10-25 23:05 ` Vladislav Yasevich
2013-10-27 13:29 ` Jiri Pirko
2013-10-27 16:48 ` Hannes Frederic Sowa
2013-10-28 13:56 ` Vladislav Yasevich
2013-10-28 21:17 ` David Miller
2013-10-28 23:16 ` Dan Williams
2013-10-28 23:23 ` Dan Williams
2013-10-29 0:12 ` David Miller
2013-10-28 23:48 ` Hannes Frederic Sowa
2013-10-29 14:31 ` Dan Williams
2013-10-29 14:38 ` Hannes Frederic Sowa
2013-10-29 17:21 ` Dan Williams
2013-10-29 16:58 ` Vlad Yasevich
2013-10-29 17:15 ` Dan Williams
2013-10-29 0:08 ` David Miller
2013-10-29 0:13 ` Hannes Frederic Sowa
2013-10-29 0:46 ` David Miller
2013-10-28 23:31 ` Hannes Frederic Sowa
2013-10-29 0:43 ` David Miller
2013-10-29 9:37 ` David Laight
2013-10-29 12:40 ` Hannes Frederic Sowa
2013-10-29 13:09 ` Eric Dumazet
2013-10-29 13:11 ` Hannes Frederic Sowa
2013-10-29 19:58 ` David Miller
2013-11-01 21:28 ` Hannes Frederic Sowa
2013-11-05 17:02 ` Nicolas Dichtel
2013-11-05 17:12 ` David Laight
2013-11-05 21:11 ` Hannes Frederic Sowa
2013-11-06 9:23 ` David Laight
2013-11-06 12:03 ` Hannes Frederic Sowa
2013-11-05 20:57 ` Hannes Frederic Sowa
2013-11-06 8:11 ` Nicolas Dichtel
2013-11-09 0:54 ` [RFC PATCH net-next 1/2] ipv4: fix wildcard search with inet_confirm_addr() Nicolas Dichtel
2013-11-09 0:54 ` [RFC PATCH net-next 2/2] udp: add sk opt to allow sending pkt with src 0.0.0.0 Nicolas Dichtel
2013-11-09 14:46 ` Julian Anastasov
2013-11-12 8:59 ` Nicolas Dichtel
2013-11-11 5:18 ` David Miller
2013-11-14 13:05 ` Nicolas Dichtel [this message]
2013-11-14 19:57 ` David Miller
2013-11-18 9:15 ` Nicolas Dichtel
2013-11-14 14:31 ` Hannes Frederic Sowa
2013-11-14 20:00 ` David Miller
2013-10-29 19:44 ` [patch net-next] ipv6: allow userspace to create address with IFLA_F_TEMPORARY flag 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=5284CA7C.7030402@6wind.com \
--to=nicolas.dichtel@6wind$(echo .)com \
--cc=David.Laight@ACULAB$(echo .)COM \
--cc=davem@davemloft$(echo .)net \
--cc=hannes@stressinduktion$(echo .)org \
--cc=jiri@resnulli$(echo .)us \
--cc=jmorris@namei$(echo .)org \
--cc=kaber@trash$(echo .)net \
--cc=kuznet@ms2$(echo .)inr.ac.ru \
--cc=netdev@vger$(echo .)kernel.org \
--cc=stephen@networkplumber$(echo .)org \
--cc=thaller@redhat$(echo .)com \
--cc=vyasevich@gmail$(echo .)com \
--cc=yoshfuji@linux-ipv6$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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