diff mbox

btrfs-progs: Adjust timing of safety delay countdown

Message ID 1462391006-4057-1-git-send-email-noah.massey@gmail.com (mailing list archive)
State Accepted
Headers show

Commit Message

Noah Massey May 4, 2016, 7:43 p.m. UTC
When printing the countdown in the safety delay, the number should
correspond to the number of seconds remaining to wait at the time the
delay is printed.

In other words, there should be a one second sleep after printing '1'.

Signed-off-by: Noah Massey <noah.massey@gmail.com>
---
 cmds-balance.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Sterba May 5, 2016, 8:16 a.m. UTC | #1
On Wed, May 04, 2016 at 03:43:26PM -0400, Noah Massey wrote:
> When printing the countdown in the safety delay, the number should
> correspond to the number of seconds remaining to wait at the time the
> delay is printed.
> 
> In other words, there should be a one second sleep after printing '1'.
> 
> Signed-off-by: Noah Massey <noah.massey@gmail.com>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/cmds-balance.c b/cmds-balance.c
index 5f05f60..8f3bf5b 100644
--- a/cmds-balance.c
+++ b/cmds-balance.c
@@ -445,9 +445,9 @@  static int do_balance(const char *path, struct btrfs_ioctl_balance_args *args,
 		printf("\twarning. The operation will start in %d seconds.\n", delay);
 		printf("\tUse Ctrl-C to stop it.\n");
 		while (delay) {
-			sleep(1);
 			printf("%2d", delay--);
 			fflush(stdout);
+			sleep(1);
 		}
 		printf("\nStarting balance without any filters.\n");
 	}