From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH 01/24] sysctl: net: call unregister_net_sysctl_table where needed Date: Wed, 06 Apr 2011 13:06:30 -0700 (PDT) Message-ID: <20110406.130630.229741188.davem@davemloft.net> References: <05ae3c2fdd2db1f6a2fd3cd900164667e615f4f8.1301711868.git.lucian.grijincu@gmail.com> <20110406.125256.102558618.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: adobriyan@gmail.com, ebiederm@xmission.com, tavi@cs.pub.ro, linux-kernel@vger.kernel.org, netdev@vger.kernel.org To: lucian.grijincu@gmail.com Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org =46rom: Lucian Adrian Grijincu Date: Wed, 6 Apr 2011 22:04:53 +0200 > On Wed, Apr 6, 2011 at 9:52 PM, David Miller wr= ote: >> Oops, hmmm... =A0Can this actually cause any real problems currently= ? >> >> If so I'd like to toss this specific patch into net-2.6 >=20 >=20 > Nope: >=20 > void unregister_net_sysctl_table(struct ctl_table_header *header) > { > unregister_sysctl_table(header); > } > EXPORT_SYMBOL_GPL(unregister_net_sysctl_table); Ok, that's good. > But I thought that in the future, if something special needs to be > done for unregister_net_sysctl_table, it would be better to have thes= e > consistent. I'll do a check on all the tree to see if there are other > cases. >=20 > Another approach would be to remove unregister_net_sysctl_table, as i= t > does nothing different of unregister_sysctl_table and let our future > selves deal with the problem. >=20 > Can you tell me what you'd like? I think the thing to do is to keep this patch as it is, and only apply it along with this cookie patch series, for -next.