Re: [PATCH 2/3] xfs: create symlink proc/fs/xfs/stat to sys/fs/xfs/stats

["Darrick J. Wong" <darrick.wong@xxxxxxxxxx>]

On Thu, Sep 03, 2015 at 11:36:26AM -0500, billodo wrote:
> As a part of the work to move xfs global stats from procfs to sysfs,
> this patch creates the symlink from proc/fs/xfs/stat to sys/fs/xfs/stats.
> 
> Signed-off-by: Bill O'Donnell <billodo@xxxxxxxxxx>
> ---
>  fs/xfs/xfs_stats.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/xfs/xfs_stats.c b/fs/xfs/xfs_stats.c
> index 856cf57..ad435f1 100644
> --- a/fs/xfs/xfs_stats.c
> +++ b/fs/xfs/xfs_stats.c
> @@ -244,9 +244,13 @@ xfs_init_procfs(void)
>       if (!proc_mkdir("fs/xfs", NULL))
>               goto out;
>  
> -     if (!proc_create("fs/xfs/stat", 0, NULL,
> -                      &xfs_stat_proc_fops))
> +     if (!proc_symlink("fs/xfs/stat", NULL,
> +                       "/sys/fs/xfs/stats/stats"))

Uh.... is it actually guaranteed that sysfs is mounted on /sys now?

I sort of recall gregkh grumbling years ago that sysfs can be mounted anywhere,
and that /proc shouldn't hardcode links to it.  But that's just handwaving on
my part.

--D

> +     {
> +             printk(KERN_INFO "failed to created fs/xfs/stat symlink\n");
>               goto out_remove_xfs_dir;
> +     }
> +
>  #ifdef CONFIG_XFS_QUOTA
>       if (!proc_create("fs/xfs/xqmstat", 0, NULL,
>                        &xqmstat_proc_fops))
> -- 
> 2.4.3
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs