Re: [PATCH] sysctl: Allow /proc/sys without sys_sysctl

From: Eric W. Biederman
Date: Tue Jul 11 2006 - 02:56:40 EST


Andrew Morton <akpm@xxxxxxxx> writes:

> On Mon, 10 Jul 2006 16:38:59 -0600
> ebiederm@xxxxxxxxxxxx (Eric W. Biederman) wrote:
>
>> Since sys_sysctl is deprecated start allow it to be compiled out.
>
> This could be a tough one to get rid of (looks at sys_bdflush() again).
>
> I'd suggest we put a sys_bdflush()-style warning in there, see what that
> flushes out.

Sounds sane. I know I have booted several kernels with it compiled out
but just because you can do without it doesn't mean that something
isn't using it.

Hmm. The question is where do I want the put the warning message?

When the code is compiled out?
When the code is compiled in?

Probably both places at this point, and using the rate limited printk
I think instead of just the 5 printks that sys_bdflush uses...

I will cook up an incremental patch once I get some sleep.

Eric

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