On Thu, Apr 24, 2014 at 05:36:38PM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> We have had this code in the kernel for over a year now and have
> shaken all the known issues out of the code over the past few
> releases. It's now time to remove the experimental warnings during
> mount and fully support the new filesystem format in production
> systems.
>
> Remove the experimental warning, and add a version number to the
> initial "mounting filesystem" message to tell use what type of
> filesystem is being mounted. Also, remove the temporary inode
> cluster size output at mount time now we know that this code works
> fine.
>
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> ---
Seems reasonable to me at this point.
Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx>
> fs/xfs/xfs_log.c | 10 ++++++----
> fs/xfs/xfs_mount.c | 2 --
> fs/xfs/xfs_sb.c | 4 ----
> 3 files changed, 6 insertions(+), 10 deletions(-)
>
> diff --git a/fs/xfs/xfs_log.c b/fs/xfs/xfs_log.c
> index 08624dc..a5f8bd9 100644
> --- a/fs/xfs/xfs_log.c
> +++ b/fs/xfs/xfs_log.c
> @@ -616,11 +616,13 @@ xfs_log_mount(
> int error = 0;
> int min_logfsbs;
>
> - if (!(mp->m_flags & XFS_MOUNT_NORECOVERY))
> - xfs_notice(mp, "Mounting Filesystem");
> - else {
> + if (!(mp->m_flags & XFS_MOUNT_NORECOVERY)) {
> + xfs_notice(mp, "Mounting V%d Filesystem",
> + XFS_SB_VERSION_NUM(&mp->m_sb));
> + } else {
> xfs_notice(mp,
> -"Mounting filesystem in no-recovery mode. Filesystem will be
> inconsistent.");
> +"Mounting V%d filesystem in no-recovery mode. Filesystem will be
> inconsistent.",
> + XFS_SB_VERSION_NUM(&mp->m_sb));
> ASSERT(mp->m_flags & XFS_MOUNT_RDONLY);
> }
>
> diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c
> index 993cb19..944f3d9 100644
> --- a/fs/xfs/xfs_mount.c
> +++ b/fs/xfs/xfs_mount.c
> @@ -743,8 +743,6 @@ xfs_mountfs(
> new_size *= mp->m_sb.sb_inodesize / XFS_DINODE_MIN_SIZE;
> if (mp->m_sb.sb_inoalignmt >= XFS_B_TO_FSBT(mp, new_size))
> mp->m_inode_cluster_size = new_size;
> - xfs_info(mp, "Using inode cluster size of %d bytes",
> - mp->m_inode_cluster_size);
> }
>
> /*
> diff --git a/fs/xfs/xfs_sb.c b/fs/xfs/xfs_sb.c
> index 0c0e41b..8baf61a 100644
> --- a/fs/xfs/xfs_sb.c
> +++ b/fs/xfs/xfs_sb.c
> @@ -201,10 +201,6 @@ xfs_mount_validate_sb(
> * write validation, we don't need to check feature masks.
> */
> if (check_version && XFS_SB_VERSION_NUM(sbp) == XFS_SB_VERSION_5) {
> - xfs_alert(mp,
> -"Version 5 superblock detected. This kernel has EXPERIMENTAL support
> enabled!\n"
> -"Use of these features in this kernel is at your own risk!");
> -
> if (xfs_sb_has_compat_feature(sbp,
> XFS_SB_FEAT_COMPAT_UNKNOWN)) {
> xfs_warn(mp,
> --
> 1.9.0
>
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs
|