On Fri, 20 Apr 2007 12:25:35 +0200 Miklos Szeredi <[EMAIL PROTECTED]> wrote:

> Add sysctl variables for accounting and limiting the number of user
> mounts.
> 
> The maximum number of user mounts is set to 1024 by default.  This
> won't in itself enable user mounts, setting a mount to be owned by a
> user is first needed
> 
> Signed-off-by: Miklos Szeredi <[EMAIL PROTECTED]>
> ---
> 
> Index: linux/include/linux/sysctl.h
> ===================================================================
> --- linux.orig/include/linux/sysctl.h 2007-04-20 11:55:02.000000000 +0200
> +++ linux/include/linux/sysctl.h      2007-04-20 11:55:07.000000000 +0200
> @@ -818,6 +818,8 @@ enum
>       FS_AIO_NR=18,   /* current system-wide number of aio requests */
>       FS_AIO_MAX_NR=19,       /* system-wide maximum number of aio requests */
>       FS_INOTIFY=20,  /* inotify submenu */
> +     FS_NR_USER_MOUNTS=21,   /* int:current number of user mounts */
> +     FS_MAX_USER_MOUNTS=22,  /* int:maximum number of user mounts */
>       FS_OCFS2=988,   /* ocfs2 */

Is there a special reason why the enumerated sysctls are needed?  We're
trying to get away from using them.


diff -puN include/linux/sysctl.h~unprivileged-mounts-account-user-mounts-fix 
include/linux/sysctl.h
--- a/include/linux/sysctl.h~unprivileged-mounts-account-user-mounts-fix
+++ a/include/linux/sysctl.h
@@ -819,8 +819,6 @@ enum
        FS_AIO_NR=18,   /* current system-wide number of aio requests */
        FS_AIO_MAX_NR=19,       /* system-wide maximum number of aio requests */
        FS_INOTIFY=20,  /* inotify submenu */
-       FS_NR_USER_MOUNTS=21,   /* int:current number of user mounts */
-       FS_MAX_USER_MOUNTS=22,  /* int:maximum number of user mounts */
        FS_OCFS2=988,   /* ocfs2 */
 };
 
diff -puN kernel/sysctl.c~unprivileged-mounts-account-user-mounts-fix 
kernel/sysctl.c
--- a/kernel/sysctl.c~unprivileged-mounts-account-user-mounts-fix
+++ a/kernel/sysctl.c
@@ -1028,7 +1028,7 @@ static ctl_table fs_table[] = {
 #endif 
 #endif
        {
-               .ctl_name       = FS_NR_USER_MOUNTS,
+               .ctl_name       = CTL_UNNUMBERED,
                .procname       = "nr_user_mounts",
                .data           = &nr_user_mounts,
                .maxlen         = sizeof(int),
@@ -1036,7 +1036,7 @@ static ctl_table fs_table[] = {
                .proc_handler   = &proc_dointvec,
        },
        {
-               .ctl_name       = FS_MAX_USER_MOUNTS,
+               .ctl_name       = CTL_UNNUMBERED,
                .procname       = "max_user_mounts",
                .data           = &max_user_mounts,
                .maxlen         = sizeof(int),
_

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

Reply via email to