netdev
[Top] [All Lists]

Re: RFC: [2.6 patch] disallow modular IPv6

To: David Woodhouse <dwmw2@xxxxxxxxxxxxx>
Subject: Re: RFC: [2.6 patch] disallow modular IPv6
From: "David S. Miller" <davem@xxxxxxxxxx>
Date: Tue, 30 Sep 2003 00:03:02 -0700
Cc: bunk@xxxxxxxxx, acme@xxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, pekkas@xxxxxxxxxx, lksctp-developers@xxxxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <1064903562.6154.160.camel@xxxxxxxxxxxxxxxxxxxx>
References: <20030928225941.GW15338@xxxxxxxxx> <20030928231842.GE1039@xxxxxxxxxxxxxxxx> <20030928232403.GX15338@xxxxxxxxx> <20030929220916.19c9c90d.davem@xxxxxxxxxx> <1064903562.6154.160.camel@xxxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Tue, 30 Sep 2003 07:32:42 +0100
David Woodhouse <dwmw2@xxxxxxxxxxxxx> wrote:

> On Mon, 2003-09-29 at 22:09 -0700, David S. Miller wrote:
> > For things inside the kernel, what ipv6 is doing is completely legal.
> > Changing your config setting in any way in the main kernel tree can
> > change just about anything else in the kernel, including the layout
> > of structures.
> 
> With boolean options that's fair enough. But changing any config option
> from 'n' to 'm' should not change anything in the main kernel. To do so
> is confusing and should be considered broken, as Adrian says.

This conflicts with the other reply you've made to me in this
thread where you say that you agree with me.

So which is it? :-)

I don't see why "enabling to 'y'" and "enabling to 'm'" are in any
way fundamentally different.  You're turning something on, therefore
something is going to change.

And when I see suggestions that we add four options to replace the
single one we have now, with a addendum saying "it's not really
complex, we'll explain it in the documentation", I want to pull my
hair out.

I would rather apply a patch that bloats up the structures than
subscribe to crazy ideas such as this four option one being proposed.

<Prev in Thread] Current Thread [Next in Thread>