[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20080730145136.1c604b6a.akpm@linux-foundation.org>
Date: Wed, 30 Jul 2008 14:51:36 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: David Miller <davem@...emloft.net>
Cc: netdev@...r.kernel.org, thomas.petazzoni@...e-electrons.com,
mpm@...enic.com
Subject: Re: [patch 04/11] net: configure out IGMP support
On Wed, 30 Jul 2008 14:45:21 -0700 (PDT)
David Miller <davem@...emloft.net> wrote:
> From: akpm@...ux-foundation.org
> Date: Wed, 30 Jul 2008 12:37:36 -0700
>
> > From: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
> >
> > tAdd adds the CONFIG_IGMP option which allows to remove support for the
> > Internet Group Management Protocol, used in multicast. Multicast is not
> > necessarly used by applications, particularly on embedded devices. As
> > this is a size-reduction option, it depends on CONFIG_EMBEDDED. It allows
> > to save ~10 kilobytes of kernel code/data:
> >
> > text data bss dec hex filename
> > 1718857 143672 221184 2083713 1fcb81 vmlinux
> > 1708838 143640 221184 2073662 1fa43e vmlinux.new
> > -10019 -32 0 -10051 -2743 +/-
> >
> > This patch has been originally written by Matt Mackall <mpm@...enic.com>,
> > and is part of the Linux Tiny project.
> >
> > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
> > Cc: Matt Mackall <mpm@...enic.com>
> > Cc: "David S. Miller" <davem@...emloft.net>
> > Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
>
> I'm not applying this.
>
> This removes core parts of the BSD socket API from applications.
> Like TCP and UDP, multicast capabilities are something applications
> can always depend upon being available.
That's just a bogus argument. Embedded device developers have 100%
control over 100% of the applications which run on their devices. If
they know that their applications don't use multicast then those devices
won't use multicast. Ever.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists