xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH 14/59] sysctl: C99 convert xfs ctl_tables

To: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
Subject: [xfs-masters] Re: [PATCH 14/59] sysctl: C99 convert xfs ctl_tables
From: Kirill Korotaev <dev@xxxxx>
Date: Wed, 17 Jan 2007 20:01:48 +0300
Cc: "<Andrew Morton" <akpm@xxxxxxxx>, James.Bottomley@xxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxx, linux-parport@xxxxxxxxxxxxxxxxxxx, minyard@xxxxxxx, rtc-linux@xxxxxxxxxxxxxxxx, clemens@xxxxxxxxxx, heiko.carstens@xxxxxxxxxx, xfs@xxxxxxxxxxx, linuxppc-dev@xxxxxxxxxx, paulus@xxxxxxxxx, openipmi-developer@xxxxxxxxxxxxxxxxxxxxx, linux-390@xxxxxxxxxxxxx, schwidefsky@xxxxxxxxxx, tim@xxxxxxxxxxxx, codalist@xxxxxxxxxxxxxxxxxxxxxxxx, a.zummo@xxxxxxxxxxxx, tony.luck@xxxxxxxxx, vojtech@xxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, xfs-masters@xxxxxxxxxxx, linux-ntfs-dev@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, aia21@xxxxxxxxxx, aharkes@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, ralf@xxxxxxxxxxxxxx, mark.fasheh@xxxxxxxxxx, coda@xxxxxxxxxx, lethal@xxxxxxxxxxxx, kurt.hackel@xxxxxxxxxx, containers@xxxxxxxxxxxxxx, linux390@xxxxxxxxxx, philb@xxxxxxx, andrea@xxxxxxx, linuxsh-shmedia-dev@xxxxxxxxxxxxxxxxxxxxx, ak@xxxxxxx
In-reply-to: <11689656301563-git-send-email-ebiederm@xxxxxxxxxxxx>
References: <m1ac0jc4no.fsf@xxxxxxxxxxxxxxxxxxxxxxxxx> <11689656301563-git-send-email-ebiederm@xxxxxxxxxxxx>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.13) Gecko/20060417
minor extra space in table below...

Kirill

> From: Eric W. Biederman <ebiederm@xxxxxxxxxxxx> - unquoted
> 
> Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> ---
>  fs/xfs/linux-2.6/xfs_sysctl.c |  258 ++++++++++++++++++++++++++++------------
>  1 files changed, 180 insertions(+), 78 deletions(-)
> 
> diff --git a/fs/xfs/linux-2.6/xfs_sysctl.c b/fs/xfs/linux-2.6/xfs_sysctl.c
> index af777e9..5a0eefc 100644
> --- a/fs/xfs/linux-2.6/xfs_sysctl.c
> +++ b/fs/xfs/linux-2.6/xfs_sysctl.c
> @@ -55,95 +55,197 @@ xfs_stats_clear_proc_handler(
>  #endif /* CONFIG_PROC_FS */
>  
>  STATIC ctl_table xfs_table[] = {
> -     {XFS_RESTRICT_CHOWN, "restrict_chown", &xfs_params.restrict_chown.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.restrict_chown.min, &xfs_params.restrict_chown.max},
> -
> -     {XFS_SGID_INHERIT, "irix_sgid_inherit", &xfs_params.sgid_inherit.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.sgid_inherit.min, &xfs_params.sgid_inherit.max},
> -
> -     {XFS_SYMLINK_MODE, "irix_symlink_mode", &xfs_params.symlink_mode.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.symlink_mode.min, &xfs_params.symlink_mode.max},
> -
> -     {XFS_PANIC_MASK, "panic_mask", &xfs_params.panic_mask.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.panic_mask.min, &xfs_params.panic_mask.max},
> -
> -     {XFS_ERRLEVEL, "error_level", &xfs_params.error_level.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.error_level.min, &xfs_params.error_level.max},
> -
> -     {XFS_SYNCD_TIMER, "xfssyncd_centisecs", &xfs_params.syncd_timer.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.syncd_timer.min, &xfs_params.syncd_timer.max},
> -
> -     {XFS_INHERIT_SYNC, "inherit_sync", &xfs_params.inherit_sync.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.inherit_sync.min, &xfs_params.inherit_sync.max},
> -
> -     {XFS_INHERIT_NODUMP, "inherit_nodump", &xfs_params.inherit_nodump.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.inherit_nodump.min, &xfs_params.inherit_nodump.max},
> -
> -     {XFS_INHERIT_NOATIME, "inherit_noatime", &xfs_params.inherit_noatim.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.inherit_noatim.min, &xfs_params.inherit_noatim.max},
> -
> -     {XFS_BUF_TIMER, "xfsbufd_centisecs", &xfs_params.xfs_buf_timer.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.xfs_buf_timer.min, &xfs_params.xfs_buf_timer.max},
> -
> -     {XFS_BUF_AGE, "age_buffer_centisecs", &xfs_params.xfs_buf_age.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.xfs_buf_age.min, &xfs_params.xfs_buf_age.max},
> -
> -     {XFS_INHERIT_NOSYM, "inherit_nosymlinks", &xfs_params.inherit_nosym.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.inherit_nosym.min, &xfs_params.inherit_nosym.max},
> -
> -     {XFS_ROTORSTEP, "rotorstep", &xfs_params.rotorstep.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.rotorstep.min, &xfs_params.rotorstep.max},
> -
> -     {XFS_INHERIT_NODFRG, "inherit_nodefrag", &xfs_params.inherit_nodfrg.val,
> -     sizeof(int), 0644, NULL, &proc_dointvec_minmax,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.inherit_nodfrg.min, &xfs_params.inherit_nodfrg.max},
> +     {
> +             .ctl_name       = XFS_RESTRICT_CHOWN,
> +             .procname       = "restrict_chown",
> +             .data           = &xfs_params.restrict_chown.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.restrict_chown.min,
> +             .extra2         = &xfs_params.restrict_chown.max
> +     },
> +     {
> +             .ctl_name       = XFS_SGID_INHERIT,
> +             .procname       = "irix_sgid_inherit",
> +             .data           = &xfs_params.sgid_inherit.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.sgid_inherit.min,
> +             .extra2         = &xfs_params.sgid_inherit.max
> +     },
> +     {
> +             .ctl_name       = XFS_SYMLINK_MODE,
> +             .procname       = "irix_symlink_mode",
> +             .data           = &xfs_params.symlink_mode.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.symlink_mode.min,
> +             .extra2         = &xfs_params.symlink_mode.max
> +     },
> +     {
> +             .ctl_name       = XFS_PANIC_MASK,
> +             .procname       = "panic_mask",
> +             .data           = &xfs_params.panic_mask.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   =  &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.panic_mask.min,
> +             .extra2         = &xfs_params.panic_mask.max
> +     },
>  
> +     {
> +             .ctl_name       = XFS_ERRLEVEL,
> +             .procname       = "error_level",
> +             .data           = &xfs_params.error_level.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.error_level.min,
> +             .extra2         = &xfs_params.error_level.max
> +     },
> +     {
> +             .ctl_name       = XFS_SYNCD_TIMER,
> +             .procname       = "xfssyncd_centisecs",
> +             .data           = &xfs_params.syncd_timer.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.syncd_timer.min,
> +             .extra2         = &xfs_params.syncd_timer.max
> +     },
> +     {
> +             .ctl_name       = XFS_INHERIT_SYNC,
> +             .procname       = "inherit_sync",
> +             .data           = &xfs_params.inherit_sync.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.inherit_sync.min,
> +             .extra2         = &xfs_params.inherit_sync.max
> +     },
> +     {
> +             .ctl_name       = XFS_INHERIT_NODUMP,
> +             .procname       = "inherit_nodump",
> +             .data           = &xfs_params.inherit_nodump.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec, NULL,
> +             .extra1         = &xfs_params.inherit_nodump.min,
> +             .extra2         = &xfs_params.inherit_nodump.max
> +     },
> +     {
> +             .ctl_name       = XFS_INHERIT_NOATIME,
> +             .procname       = "inherit_noatime",
> +             .data           = &xfs_params.inherit_noatim.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec, NULL,
> +             .extra1         = &xfs_params.inherit_noatim.min,
> +             .extra2         = &xfs_params.inherit_noatim.max
> +     },
> +     {
> +             .ctl_name       = XFS_BUF_TIMER,
> +             .procname       = "xfsbufd_centisecs",
> +             .data           = &xfs_params.xfs_buf_timer.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.xfs_buf_timer.min,
> +             .extra2         = &xfs_params.xfs_buf_timer.max
> +     },
> +     {
> +             .ctl_name       = XFS_BUF_AGE,
> +             .procname       = "age_buffer_centisecs",
> +             .data           = &xfs_params.xfs_buf_age.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec, NULL,
> +             .extra1         = &xfs_params.xfs_buf_age.min,
> +             .extra2         = &xfs_params.xfs_buf_age.max
> +     },
> +     {
> +             .ctl_name       = XFS_INHERIT_NOSYM,
> +             .procname       = "inherit_nosymlinks",
> +             .data           = &xfs_params.inherit_nosym.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.inherit_nosym.min,
> +             .extra2         = &xfs_params.inherit_nosym.max
> +     },
> +     {
> +             .ctl_name       = XFS_ROTORSTEP,
> +             .procname       = "rotorstep",
> +             .data           = &xfs_params.rotorstep.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.rotorstep.min,
> +             .extra2         = &xfs_params.rotorstep.max
> +     },
> +     {
> +             .ctl_name       = XFS_INHERIT_NODFRG,
> +             .procname       = "inherit_nodefrag",
> +             .data           = &xfs_params.inherit_nodfrg.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   = &proc_dointvec_minmax,
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.inherit_nodfrg.min,
> +             .extra2         = &xfs_params.inherit_nodfrg.max
> +     },
>       /* please keep this the last entry */
>  #ifdef CONFIG_PROC_FS
> -     {XFS_STATS_CLEAR, "stats_clear", &xfs_params.stats_clear.val,
> -     sizeof(int), 0644, NULL, &xfs_stats_clear_proc_handler,
> -     &sysctl_intvec, NULL,
> -     &xfs_params.stats_clear.min, &xfs_params.stats_clear.max},
> +     {
> +             .ctl_name       = XFS_STATS_CLEAR,
> +             .procname       = "stats_clear",
> +             .data           = &xfs_params.stats_clear.val,
> +             .maxlen         = sizeof(int),
> +             .mode           = 0644,
> +             .proc_handler   =  &xfs_stats_clear_proc_handler,
<<< minor. extra space.
> +             .strategy       = &sysctl_intvec,
> +             .extra1         = &xfs_params.stats_clear.min,
> +             .extra2         = &xfs_params.stats_clear.max
> +     },
>  #endif /* CONFIG_PROC_FS */
>  
> -     {0}
> +     {}
>  };
>  
>  STATIC ctl_table xfs_dir_table[] = {
> -     {FS_XFS, "xfs", NULL, 0, 0555, xfs_table},
> -     {0}
> +     {
> +             .ctl_name       = FS_XFS,
> +             .procname       = "xfs",
> +             .mode           = 0555,
> +             .child          = xfs_table
> +     },
> +     {}
>  };
>  
>  STATIC ctl_table xfs_root_table[] = {
> -     {CTL_FS, "fs",  NULL, 0, 0555, xfs_dir_table},
> -     {0}
> +     {
> +             .ctl_name       = CTL_FS,
> +             .procname       = "fs",
> +             .mode           = 0555,
> +             .child          = xfs_dir_table
> +     },
> +     {}
>  };
>  
>  void


<Prev in Thread] Current Thread [Next in Thread>