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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



"Randy.Dunlap" <[email protected]> writes:

> On Mon, 10 Jul 2006 16:38:59 -0600 Eric W. Biederman wrote:
>
>> 
>> Since sys_sysctl is deprecated start allow it to be compiled out.
>> This should catch any remaining user space code that cares,
>> and paves the way for further sysctl cleanups.
>
> Where is it documented and users notified that sys_sysctl is
> deprecated?  Sounds like it should be added to
> Documentation/feature-removal-schedule.txt.

The deprecation I believe actually predates feature-remove-schedule.txt

>From include/linux/sysctl.h
 
>  ****************************************************************
>  ****************************************************************
>  **
>  **  The values in this file are exported to user space via 
>  **  the sysctl() binary interface.  However this interface
>  **  is unstable and deprecated and will be removed in the future. 
>  **  For a stable interface use /proc/sys.
>  **
>  ****************************************************************
>  ****************************************************************

And I can't actually find any user space applications that continue
to use sys_sysctl.

So the combination of the two patches and makes the deprecation official,
and makes it a compile time option so we can disable sys_sysctl now,
but still have recourse if some parts of user space need the code.

Currently the we dup the internal helpers which makes maintenance
of the code a nightmare.

Eric



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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux