public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] xfs_repair: correct docs for "-t" units
@ 2013-11-14 20:38 Eric Sandeen
  2013-11-14 22:09 ` Dave Chinner
  2013-11-18 19:03 ` Rich Johnston
  0 siblings, 2 replies; 3+ messages in thread
From: Eric Sandeen @ 2013-11-14 20:38 UTC (permalink / raw)
  To: xfs-oss

usage() says that the xfs_repair -t reporting interval is
in minutes, and the manpage says nothing at all about units.

Actually, it's in seconds.

Clarify in both places that the interval is specified in
seconds.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---

diff --git a/man/man8/xfs_repair.8 b/man/man8/xfs_repair.8
index 47436ec..ed20fb7 100644
--- a/man/man8/xfs_repair.8
+++ b/man/man8/xfs_repair.8
@@ -152,7 +152,7 @@ in no modify mode first.
 .RE
 .TP
 .B \-t " interval"
-Modify reporting interval. During long runs
+Modify reporting interval, specified in seconds. During long runs
 .B xfs_repair
 outputs its progress every 15 minutes. Reporting is only activated when
 ag_stride is enabled.
diff --git a/repair/xfs_repair.c b/repair/xfs_repair.c
index a863337..7beffcb 100644
--- a/repair/xfs_repair.c
+++ b/repair/xfs_repair.c
@@ -90,7 +90,7 @@ usage(void)
 "  -v           Verbose output.\n"
 "  -c subopts   Change filesystem parameters - use xfs_admin.\n"
 "  -o subopts   Override default behaviour, refer to man page.\n"
-"  -t interval  Reporting interval in minutes.\n"
+"  -t interval  Reporting interval in seconds.\n"
 "  -d           Repair dangerously.\n"
 "  -V           Reports version and exits.\n"), progname);
 	exit(1);

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: [PATCH] xfs_repair: correct docs for "-t" units
  2013-11-14 20:38 [PATCH] xfs_repair: correct docs for "-t" units Eric Sandeen
@ 2013-11-14 22:09 ` Dave Chinner
  2013-11-18 19:03 ` Rich Johnston
  1 sibling, 0 replies; 3+ messages in thread
From: Dave Chinner @ 2013-11-14 22:09 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: xfs-oss

On Thu, Nov 14, 2013 at 02:38:08PM -0600, Eric Sandeen wrote:
> usage() says that the xfs_repair -t reporting interval is
> in minutes, and the manpage says nothing at all about units.
> 
> Actually, it's in seconds.
> 
> Clarify in both places that the interval is specified in
> seconds.
> 
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
> ---

Looks good.

Reviewed-by: Dave Chinner <dchinner@redhat.com>

-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: [PATCH] xfs_repair: correct docs for "-t" units
  2013-11-14 20:38 [PATCH] xfs_repair: correct docs for "-t" units Eric Sandeen
  2013-11-14 22:09 ` Dave Chinner
@ 2013-11-18 19:03 ` Rich Johnston
  1 sibling, 0 replies; 3+ messages in thread
From: Rich Johnston @ 2013-11-18 19:03 UTC (permalink / raw)
  To: Eric Sandeen, xfs-oss

This has been committed.

Thanks
--Rich

commit 79e106f05f0fbffb89ab5401fb97af5391c6291f
Author: Eric Sandeen <sandeen@redhat.com>
Date:   Thu Nov 14 20:38:08 2013 +0000

     xfs_repair: correct docs for "-t" units

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

end of thread, other threads:[~2013-11-18 19:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-14 20:38 [PATCH] xfs_repair: correct docs for "-t" units Eric Sandeen
2013-11-14 22:09 ` Dave Chinner
2013-11-18 19:03 ` Rich Johnston

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox