[Cluster-devel] [PATCH] mkfs.gfs2: print message about BKLDISCARD ioctl taking a long time
Andrew Price
anprice at redhat.com
Fri Sep 25 08:47:18 UTC 2015
Hi Paul,
On 24/09/15 19:27, Paul Evans wrote:
> When running mkfs.gfs2 the BLKDISCARD ioctl() can sit waiting for a
> long time without feedback.
>
> Added a message in mkfs.gfs2 to the user just before the ioctl begins
> to warn the user that the operation could take a while to complete.
>
> Signed-off-by: Paul Evans <pevans at redhat.com>
> ---
> gfs2/mkfs/main_mkfs.c | 10 +++++++++-
> 1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/gfs2/mkfs/main_mkfs.c b/gfs2/mkfs/main_mkfs.c
> index d3d8edf..005b6f8 100644
> --- a/gfs2/mkfs/main_mkfs.c
> +++ b/gfs2/mkfs/main_mkfs.c
> @@ -904,9 +904,17 @@ int main(int argc, char *argv[])
> if (opts.confirm && !opts.override)
> are_you_sure();
>
> - if (!S_ISREG(opts.dev.stat.st_mode) && opts.discard)
> + if (!S_ISREG(opts.dev.stat.st_mode) && opts.discard) {
> + if (!opts.quiet) {
> + printf("%s", _("\nDiscarding old blocks (may take a while on large devices): "));
"Discarding device contents" is probably more accurate. The string
shouldn't need to start with a '\n' though. If a previous message didn't
end with a '\n' then that message should be fixed instead, but if it's
just to add a blank I would prefer to remove it.
Otherwise, it looks good.
Thanks,
Andy
> + fflush(stdout);
> + }
> discard_blocks(opts.dev.fd, opts.dev.size, opts.debug);
>
> + if (!opts.quiet)
> + printf("%s", _("Done\n"));
> + }
> +
> error = place_rgrps(&sbd, rgs, &opts);
> if (error) {
> fprintf(stderr, _("Failed to build resource groups\n"));
>
More information about the Cluster-devel
mailing list