All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xfs: drop experimental warnings for bigtime and inobtcount
@ 2021-07-07  0:23 Darrick J. Wong
  2021-07-26 10:21 ` Carlos Maiolino
  2021-07-26 20:40 ` Bill O'Donnell
  0 siblings, 2 replies; 3+ messages in thread
From: Darrick J. Wong @ 2021-07-07  0:23 UTC (permalink / raw)
  To: xfs; +Cc: Dave Chinner, Eric Sandeen

From: Darrick J. Wong <djwong@kernel.org>

These two features were merged a year ago, userspace tooling have been
merged, and no serious errors have been reported by the developers.
Drop the experimental tag to encourage wider testing.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
---
 fs/xfs/xfs_super.c |    8 --------
 1 file changed, 8 deletions(-)

diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index cbc1f0157bcd..321e3590c6fe 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -1539,10 +1539,6 @@ xfs_fs_fill_super(
 	if (XFS_SB_VERSION_NUM(&mp->m_sb) == XFS_SB_VERSION_5)
 		sb->s_flags |= SB_I_VERSION;
 
-	if (xfs_sb_version_hasbigtime(&mp->m_sb))
-		xfs_warn(mp,
- "EXPERIMENTAL big timestamp feature in use. Use at your own risk!");
-
 	if (mp->m_flags & XFS_MOUNT_DAX_ALWAYS) {
 		bool rtdev_is_dax = false, datadev_is_dax;
 
@@ -1598,10 +1594,6 @@ xfs_fs_fill_super(
 		goto out_filestream_unmount;
 	}
 
-	if (xfs_sb_version_hasinobtcounts(&mp->m_sb))
-		xfs_warn(mp,
- "EXPERIMENTAL inode btree counters feature in use. Use at your own risk!");
-
 	error = xfs_mountfs(mp);
 	if (error)
 		goto out_filestream_unmount;

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] xfs: drop experimental warnings for bigtime and inobtcount
  2021-07-07  0:23 [PATCH] xfs: drop experimental warnings for bigtime and inobtcount Darrick J. Wong
@ 2021-07-26 10:21 ` Carlos Maiolino
  2021-07-26 20:40 ` Bill O'Donnell
  1 sibling, 0 replies; 3+ messages in thread
From: Carlos Maiolino @ 2021-07-26 10:21 UTC (permalink / raw)
  To: Darrick J. Wong; +Cc: xfs, Dave Chinner, Eric Sandeen

On Tue, Jul 06, 2021 at 05:23:13PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
> 
> These two features were merged a year ago, userspace tooling have been
> merged, and no serious errors have been reported by the developers.
> Drop the experimental tag to encourage wider testing.

And grub already supports it :) So.

Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>


> 
> Signed-off-by: Darrick J. Wong <djwong@kernel.org>
> ---
>  fs/xfs/xfs_super.c |    8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index cbc1f0157bcd..321e3590c6fe 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -1539,10 +1539,6 @@ xfs_fs_fill_super(
>  	if (XFS_SB_VERSION_NUM(&mp->m_sb) == XFS_SB_VERSION_5)
>  		sb->s_flags |= SB_I_VERSION;
>  
> -	if (xfs_sb_version_hasbigtime(&mp->m_sb))
> -		xfs_warn(mp,
> - "EXPERIMENTAL big timestamp feature in use. Use at your own risk!");
> -
>  	if (mp->m_flags & XFS_MOUNT_DAX_ALWAYS) {
>  		bool rtdev_is_dax = false, datadev_is_dax;
>  
> @@ -1598,10 +1594,6 @@ xfs_fs_fill_super(
>  		goto out_filestream_unmount;
>  	}
>  
> -	if (xfs_sb_version_hasinobtcounts(&mp->m_sb))
> -		xfs_warn(mp,
> - "EXPERIMENTAL inode btree counters feature in use. Use at your own risk!");
> -
>  	error = xfs_mountfs(mp);
>  	if (error)
>  		goto out_filestream_unmount;
> 

-- 
Carlos


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] xfs: drop experimental warnings for bigtime and inobtcount
  2021-07-07  0:23 [PATCH] xfs: drop experimental warnings for bigtime and inobtcount Darrick J. Wong
  2021-07-26 10:21 ` Carlos Maiolino
@ 2021-07-26 20:40 ` Bill O'Donnell
  1 sibling, 0 replies; 3+ messages in thread
From: Bill O'Donnell @ 2021-07-26 20:40 UTC (permalink / raw)
  To: Darrick J. Wong; +Cc: xfs, Dave Chinner, Eric Sandeen

On Tue, Jul 06, 2021 at 05:23:13PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@kernel.org>
> 
> These two features were merged a year ago, userspace tooling have been
> merged, and no serious errors have been reported by the developers.
> Drop the experimental tag to encourage wider testing.
> 
> Signed-off-by: Darrick J. Wong <djwong@kernel.org>

Reviewed-by: Bill O'Donnell <billodo@redhat.com>

> ---
>  fs/xfs/xfs_super.c |    8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
> index cbc1f0157bcd..321e3590c6fe 100644
> --- a/fs/xfs/xfs_super.c
> +++ b/fs/xfs/xfs_super.c
> @@ -1539,10 +1539,6 @@ xfs_fs_fill_super(
>  	if (XFS_SB_VERSION_NUM(&mp->m_sb) == XFS_SB_VERSION_5)
>  		sb->s_flags |= SB_I_VERSION;
>  
> -	if (xfs_sb_version_hasbigtime(&mp->m_sb))
> -		xfs_warn(mp,
> - "EXPERIMENTAL big timestamp feature in use. Use at your own risk!");
> -
>  	if (mp->m_flags & XFS_MOUNT_DAX_ALWAYS) {
>  		bool rtdev_is_dax = false, datadev_is_dax;
>  
> @@ -1598,10 +1594,6 @@ xfs_fs_fill_super(
>  		goto out_filestream_unmount;
>  	}
>  
> -	if (xfs_sb_version_hasinobtcounts(&mp->m_sb))
> -		xfs_warn(mp,
> - "EXPERIMENTAL inode btree counters feature in use. Use at your own risk!");
> -
>  	error = xfs_mountfs(mp);
>  	if (error)
>  		goto out_filestream_unmount;
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-07-26 20:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-07  0:23 [PATCH] xfs: drop experimental warnings for bigtime and inobtcount Darrick J. Wong
2021-07-26 10:21 ` Carlos Maiolino
2021-07-26 20:40 ` Bill O'Donnell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.