[PATCH] xfs_growfs: don't grow data if only -m is specified
Dave Chinner
david at fromorbit.com
Thu Feb 27 19:19:01 CST 2014
On Thu, Feb 27, 2014 at 05:28:44PM -0600, Eric Sandeen wrote:
> While writing an xfstest to check imaxpct behavior, I realized
> that xfs_growfs -m XXX /mnt/point will not only change
> imaxpct, but also grow the filesystem if it's not currently
> occupying the entire block device. This doesn't seem like
> the expected behavior, so split it out such that if only
> -m, and not -d, is specified, only the imaxpct will be
> changed.
>
> This is a change from previous behavior, but I think it
> satisfies the principle of least surprise...
Seems reasonable to me. Code looks good, too.
Reviewed-by: Dave Chinner <dchinner at redhat.com>
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list