Re: RFC: [2.6 patch] disallow modular IPv6

From: Adrian Bunk (bunk_at_fs.tum.de)
Date: 09/30/03

  • Next message: Jamie Lokier: "Re: [PATCH] Mutilated form of Andi Kleen's AMD prefetch errata patch" 
    Date:	Tue, 30 Sep 2003 15:37:29 +0200
To: "David S. Miller" <davem@redhat.com>

    On Mon, Sep 29, 2003 at 10:11:29PM -0700, David S. Miller wrote:
    > On Sun, 28 Sep 2003 21:32:30 -0300
    > Arnaldo Carvalho de Melo <acme@conectiva.com.br> wrote:
    >
    > > Em Mon, Sep 29, 2003 at 02:14:39AM +0200, Adrian Bunk escreveu:
    > > > On Sun, Sep 28, 2003 at 08:39:10PM -0300, Arnaldo Carvalho de Melo wrote:
    > > > What about the following solution (the names and help texts for the
    > > > config options might not be optimal, I hope you understand the
    > > > intention):
    > > >
    > > > config IPV6_SUPPORT
    > > > bool "IPv6 support"
    > > >
    > > > config IPV6_ENABLE
    > > > tristate "enable IPv6"
    > > > depends on IPV6_SUPPORT
    > > >
    > > > IPV6_SUPPORT changes structs etc. and IPV6_ENABLE is responsible for
    > > > ipv6.o .
    > >
    > > Humm, and the idea is? This seems confusing, could you elaborate on why such
    > > scheme is a good thing?
    >
    > I think the idea is totally broken. At first, Adrian comments that
    > changing the layout of structs based upon a config option is broken,
    > then he proposes a config option that does nothing except change the
    > layout of structures.
    >
    > The current situation is perfectly fine.

    I did perhaps express my opinion not clearly.

    My personal opinions:

    It's OK that setting an option to y changes structs or whatever else in
    the kernel.

    It's not OK if adding a module changes the layout of structs compiled
    into the kernel.

    Modules have many advantages, one advantage is e.g. that they allow
    generic distribution kernels without resulting in huge kernel images.

    Another advantage is that you can later add modules to a running kernel,
    you can compile a module for your kernel and insert it without rebooting
    the machine. This is currently not possible with moduler IPv6.

    That was my personal opinion.

    My opinions seem to be very close to the opinions of David Woodhouse, so
    there's no need to repeat your discussion.

    cu
    Adrian 

    -- 
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
  • Next message: Jamie Lokier: "Re: [PATCH] Mutilated form of Andi Kleen's AMD prefetch errata patch"

    Relevant Pages

    • Re: RFC: [2.6 patch] disallow modular IPv6
      ... >> changing the layout of structs based upon a config option is broken, ... > into the kernel. ... > My opinions seem to be very close to the opinions of David Woodhouse, ... no #ifdefs where there was a ton, ...
      (Linux-Kernel)
    • Re: Linux GPL and binary module exception clause?
      ... > So copies to disk and RAM that are "an essential step in the utilization of ... the kernel, ... my public opinions don't bind anybody else. ... me, people do), they have the right to complain on their own. ...
      (Linux-Kernel)
    • Re: [RFC/PATCH] usb-storage: wait for device scanning before mounting root
      ... Add a new kernel config option CONFIG_LATE_ROOT_MOUNT that makes the kernel ... wait until background scanning of USB mass storage devices is complete before ... solution people currently use to boot from USB mass storage devices. ...
      (Linux-Kernel)
    • =?iso-8859-15?Q?Re:_[PATCH]_x86:_introduce_/dev/mem_restrictions_with_?= =?iso-8859-
      ... Subject: [PATCH] x86: introduce /dev/mem restrictions with a config option ... there's just no valid app that uses /dev/mem on actual memory. ... People who want to use /dev/mem for kernel debugging can enable the config ...
      (Linux-Kernel)
    • Re: secure computing for 2.6.7
      ... > implement safely (even easier than the kernel side). ... > making it a syscall. ... >> whether there should be a config option for it. ... > To unsubscribe from this list: ...
      (Linux-Kernel)