diff mbox series

[4/4] xfs: rename xfs_fc_* back to xfs_fs_*

Message ID 160729627541.1608297.18095324548384560045.stgit@magnolia (mailing list archive)
State Accepted
Headers show
Series xfs: refactor extent validation for 5.11 | expand

Commit Message

Darrick J. Wong Dec. 6, 2020, 11:11 p.m. UTC 
From: Darrick J. Wong <darrick.wong@oracle.com>

Get rid of this one-off namespace since we're done converting things to
fscontext now.

Suggested-by: Dave Chinner <david@fromorbit.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
 fs/xfs/xfs_super.c |   26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

Comments

Dave Chinner Dec. 6, 2020, 11:56 p.m. UTC | #1 
On Sun, Dec 06, 2020 at 03:11:15PM -0800, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
> 
> Get rid of this one-off namespace since we're done converting things to
> fscontext now.
> 
> Suggested-by: Dave Chinner <david@fromorbit.com>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>

Yay!

Reviewed-by: Dave Chinner <dchinner@redhat.com>
Christoph Hellwig Dec. 7, 2020, 2:17 p.m. UTC | #2 
Looks good,

Reviewed-by: Christoph Hellwig <hch@lst.de>
Brian Foster Dec. 7, 2020, 5:46 p.m. UTC | #3 
On Sun, Dec 06, 2020 at 03:11:15PM -0800, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
> 
> Get rid of this one-off namespace since we're done converting things to
> fscontext now.
> 
> Suggested-by: Dave Chinner <david@fromorbit.com>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---

Reviewed-by: Brian Foster <bfoster@redhat.com>

>  fs/xfs/xfs_super.c |   26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> 
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index 36002f460d7c..12d6850aedc6 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -1159,7 +1159,7 @@ suffix_kstrtoint(
>   * NOTE: mp->m_super is NULL here!
>   */
>  static int
> -xfs_fc_parse_param(
> +xfs_fs_parse_param(
>  	struct fs_context	*fc,
>  	struct fs_parameter	*param)
>  {
> @@ -1317,7 +1317,7 @@ xfs_fc_parse_param(
>  }
>  
>  static int
> -xfs_fc_validate_params(
> +xfs_fs_validate_params(
>  	struct xfs_mount	*mp)
>  {
>  	/*
> @@ -1386,7 +1386,7 @@ xfs_fc_validate_params(
>  }
>  
>  static int
> -xfs_fc_fill_super(
> +xfs_fs_fill_super(
>  	struct super_block	*sb,
>  	struct fs_context	*fc)
>  {
> @@ -1396,7 +1396,7 @@ xfs_fc_fill_super(
>  
>  	mp->m_super = sb;
>  
> -	error = xfs_fc_validate_params(mp);
> +	error = xfs_fs_validate_params(mp);
>  	if (error)
>  		goto out_free_names;
>  
> @@ -1660,10 +1660,10 @@ xfs_fc_fill_super(
>  }
>  
>  static int
> -xfs_fc_get_tree(
> +xfs_fs_get_tree(
>  	struct fs_context	*fc)
>  {
> -	return get_tree_bdev(fc, xfs_fc_fill_super);
> +	return get_tree_bdev(fc, xfs_fs_fill_super);
>  }
>  
>  static int
> @@ -1782,7 +1782,7 @@ xfs_remount_ro(
>   * silently ignore all options that we can't actually change.
>   */
>  static int
> -xfs_fc_reconfigure(
> +xfs_fs_reconfigure(
>  	struct fs_context *fc)
>  {
>  	struct xfs_mount	*mp = XFS_M(fc->root->d_sb);
> @@ -1795,7 +1795,7 @@ xfs_fc_reconfigure(
>  	if (XFS_SB_VERSION_NUM(&mp->m_sb) == XFS_SB_VERSION_5)
>  		fc->sb_flags |= SB_I_VERSION;
>  
> -	error = xfs_fc_validate_params(new_mp);
> +	error = xfs_fs_validate_params(new_mp);
>  	if (error)
>  		return error;
>  
> @@ -1832,7 +1832,7 @@ xfs_fc_reconfigure(
>  	return 0;
>  }
>  
> -static void xfs_fc_free(
> +static void xfs_fs_free(
>  	struct fs_context	*fc)
>  {
>  	struct xfs_mount	*mp = fc->s_fs_info;
> @@ -1848,10 +1848,10 @@ static void xfs_fc_free(
>  }
>  
>  static const struct fs_context_operations xfs_context_ops = {
> -	.parse_param = xfs_fc_parse_param,
> -	.get_tree    = xfs_fc_get_tree,
> -	.reconfigure = xfs_fc_reconfigure,
> -	.free        = xfs_fc_free,
> +	.parse_param = xfs_fs_parse_param,
> +	.get_tree    = xfs_fs_get_tree,
> +	.reconfigure = xfs_fs_reconfigure,
> +	.free        = xfs_fs_free,
>  };
>  
>  static int xfs_init_fs_context(
>
diff mbox series

Patch

diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index 36002f460d7c..12d6850aedc6 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -1159,7 +1159,7 @@  suffix_kstrtoint(
  * NOTE: mp->m_super is NULL here!
  */
 static int
-xfs_fc_parse_param(
+xfs_fs_parse_param(
 	struct fs_context	*fc,
 	struct fs_parameter	*param)
 {
@@ -1317,7 +1317,7 @@  xfs_fc_parse_param(
 }
 
 static int
-xfs_fc_validate_params(
+xfs_fs_validate_params(
 	struct xfs_mount	*mp)
 {
 	/*
@@ -1386,7 +1386,7 @@  xfs_fc_validate_params(
 }
 
 static int
-xfs_fc_fill_super(
+xfs_fs_fill_super(
 	struct super_block	*sb,
 	struct fs_context	*fc)
 {
@@ -1396,7 +1396,7 @@  xfs_fc_fill_super(
 
 	mp->m_super = sb;
 
-	error = xfs_fc_validate_params(mp);
+	error = xfs_fs_validate_params(mp);
 	if (error)
 		goto out_free_names;
 
@@ -1660,10 +1660,10 @@  xfs_fc_fill_super(
 }
 
 static int
-xfs_fc_get_tree(
+xfs_fs_get_tree(
 	struct fs_context	*fc)
 {
-	return get_tree_bdev(fc, xfs_fc_fill_super);
+	return get_tree_bdev(fc, xfs_fs_fill_super);
 }
 
 static int
@@ -1782,7 +1782,7 @@  xfs_remount_ro(
  * silently ignore all options that we can't actually change.
  */
 static int
-xfs_fc_reconfigure(
+xfs_fs_reconfigure(
 	struct fs_context *fc)
 {
 	struct xfs_mount	*mp = XFS_M(fc->root->d_sb);
@@ -1795,7 +1795,7 @@  xfs_fc_reconfigure(
 	if (XFS_SB_VERSION_NUM(&mp->m_sb) == XFS_SB_VERSION_5)
 		fc->sb_flags |= SB_I_VERSION;
 
-	error = xfs_fc_validate_params(new_mp);
+	error = xfs_fs_validate_params(new_mp);
 	if (error)
 		return error;
 
@@ -1832,7 +1832,7 @@  xfs_fc_reconfigure(
 	return 0;
 }
 
-static void xfs_fc_free(
+static void xfs_fs_free(
 	struct fs_context	*fc)
 {
 	struct xfs_mount	*mp = fc->s_fs_info;
@@ -1848,10 +1848,10 @@  static void xfs_fc_free(
 }
 
 static const struct fs_context_operations xfs_context_ops = {
-	.parse_param = xfs_fc_parse_param,
-	.get_tree    = xfs_fc_get_tree,
-	.reconfigure = xfs_fc_reconfigure,
-	.free        = xfs_fc_free,
+	.parse_param = xfs_fs_parse_param,
+	.get_tree    = xfs_fs_get_tree,
+	.reconfigure = xfs_fs_reconfigure,
+	.free        = xfs_fs_free,
 };
 
 static int xfs_init_fs_context(