Re: [PATCH 04/10] sysctl: Fix neighbour table sysctls.

From: Andrew Morton
Date: Thu Aug 09 2007 - 21:56:27 EST


On Fri, 10 Aug 2007 10:47:10 +0900 (JST) YOSHIFUJI Hideaki / åèèæ <yoshfuji@xxxxxxxxxxxxxx> wrote:

> Hello.
>
> In article <m1zm108axi.fsf_-_@xxxxxxxxxxxxxxxxxxxxxxxxx> (at Thu, 09 Aug 2007 18:56:09 -0600), ebiederm@xxxxxxxxxxxx (Eric W. Biederman) says:
>
> >
> > - In ipv6 ndisc_ifinfo_syctl_change so it doesn't depend on binary
> > sysctl names for a function that works with proc.
> >
> > - In neighbour.c reorder the table to put the possibly unused entries
> > at the end so we can remove them by terminating the table early.
> >
> > - In neighbour.c kill the entries with questionable binary sysctl
> > handling behavior.
> >
> > - In neighbour.c if we don't have a strategy routine remove the
> > binary path. So we don't the default sysctl strategy routine
> > on data that is not ready for it.
> >
>
> I disagree. It is bad to remove existing interface.

But it is good to remove bad interfaces, if we possibly can.

It is worth making the attempt. Does anyone know of anything which will
break? I fed NET_NEIGH_ANYCAST_DELAY at random into
http://www.google.com/codesearch and came up with nothing...


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/