To: ebiederm@xmission.com
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/23] Removal of binary sysctl support
From: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
References: <m13a4pjjh9.fsf@fess.ebiederm.org>
In-Reply-To: <m13a4pjjh9.fsf@fess.ebiederm.org>
Message-Id: <200911082215.HGJ57827.SJOVFFOHMOLFQt@I-love.SAKURA.ne.jp>
Date: Sun, 8 Nov 2009 22:15:46 +0900
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1101
Lines: 34

Hello.

Eric W. Biederman wrote:
> 
> This patchset reimplements sys_sysctl as a compatibility wrapper
> around /proc/sys.  After which it removes all of the code to all over
> the kernel that is used today to implement the binary sysctls.
> 
> I am posting this patchset to give everyone a heads up what is in
> flight.
> 
> I intend to carry all of these patches in my sysctl tree.
> 
> If you add new sysctls to other trees please don't set the .ctl_name
> or .strategy fields in struct ctl_table, as setting those fields
> is unnecessary now and are removed by this patchset.

I have two questions.

Is there (or was there) an entry with ->procname == NULL ? (I thought so
because the loop condition is

	for ( ; table->ctl_name || table->procname; table++) {

.)

This patchset removes ->ctl_name on the assumption that ->procname != NULL ?

Regards.
--
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/