9ae3a8
From 94787daf53d6eb2936704f7d9089b26cc5803699 Mon Sep 17 00:00:00 2001
9ae3a8
From: John Snow <jsnow@redhat.com>
9ae3a8
Date: Mon, 23 Nov 2015 17:38:22 +0100
9ae3a8
Subject: [PATCH 03/27] qemu-io: Make cvtnum() a wrapper around
9ae3a8
 strtosz_suffix()
9ae3a8
9ae3a8
RH-Author: John Snow <jsnow@redhat.com>
9ae3a8
Message-id: <1448300320-7772-4-git-send-email-jsnow@redhat.com>
9ae3a8
Patchwork-id: 68432
9ae3a8
O-Subject: [RHEL-7.3 qemu-kvm PATCH v2 03/21] qemu-io: Make cvtnum() a wrapper around strtosz_suffix()
9ae3a8
Bugzilla: 1272523
9ae3a8
RH-Acked-by: Thomas Huth <thuth@redhat.com>
9ae3a8
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
9ae3a8
RH-Acked-by: Max Reitz <mreitz@redhat.com>
9ae3a8
9ae3a8
From: Kevin Wolf <kwolf@redhat.com>
9ae3a8
9ae3a8
No reason to implement the same thing multiple times. A nice side effect
9ae3a8
is that fractional numbers like 0.5M can be used in qemu-io now.
9ae3a8
9ae3a8
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
9ae3a8
Reviewed-by: Eric Blake <eblake@redhat.com>
9ae3a8
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9ae3a8
(cherry picked from commit b6e356aa25c81d928e1c463292048d29cf25f04e)
9ae3a8
Signed-off-by: John Snow <jsnow@redhat.com>
9ae3a8
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
9ae3a8
---
9ae3a8
 cmd.c     | 37 -------------------------------------
9ae3a8
 cmd.h     |  1 -
9ae3a8
 qemu-io.c |  6 ++++++
9ae3a8
 3 files changed, 6 insertions(+), 38 deletions(-)
9ae3a8
9ae3a8
diff --git a/cmd.c b/cmd.c
9ae3a8
index 4e7579b..214c6f7 100644
9ae3a8
--- a/cmd.c
9ae3a8
+++ b/cmd.c
9ae3a8
@@ -344,43 +344,6 @@ doneline(
9ae3a8
 #define MEGABYTES(x)	((long long)(x) << 20)
9ae3a8
 #define KILOBYTES(x)	((long long)(x) << 10)
9ae3a8
 
9ae3a8
-long long
9ae3a8
-cvtnum(
9ae3a8
-	char		*s)
9ae3a8
-{
9ae3a8
-	long long	i;
9ae3a8
-	char		*sp;
9ae3a8
-	int		c;
9ae3a8
-
9ae3a8
-	i = strtoll(s, &sp, 0);
9ae3a8
-	if (i == 0 && sp == s)
9ae3a8
-		return -1LL;
9ae3a8
-	if (*sp == '\0')
9ae3a8
-		return i;
9ae3a8
-
9ae3a8
-	if (sp[1] != '\0')
9ae3a8
-		return -1LL;
9ae3a8
-
9ae3a8
-	c = qemu_tolower(*sp);
9ae3a8
-	switch (c) {
9ae3a8
-	default:
9ae3a8
-		return i;
9ae3a8
-	case 'k':
9ae3a8
-		return KILOBYTES(i);
9ae3a8
-	case 'm':
9ae3a8
-		return MEGABYTES(i);
9ae3a8
-	case 'g':
9ae3a8
-		return GIGABYTES(i);
9ae3a8
-	case 't':
9ae3a8
-		return TERABYTES(i);
9ae3a8
-	case 'p':
9ae3a8
-		return PETABYTES(i);
9ae3a8
-	case 'e':
9ae3a8
-		return  EXABYTES(i);
9ae3a8
-	}
9ae3a8
-	return -1LL;
9ae3a8
-}
9ae3a8
-
9ae3a8
 #define TO_EXABYTES(x)	((x) / EXABYTES(1))
9ae3a8
 #define TO_PETABYTES(x)	((x) / PETABYTES(1))
9ae3a8
 #define TO_TERABYTES(x)	((x) / TERABYTES(1))
9ae3a8
diff --git a/cmd.h b/cmd.h
9ae3a8
index 8e6f753..4dcfe88 100644
9ae3a8
--- a/cmd.h
9ae3a8
+++ b/cmd.h
9ae3a8
@@ -58,7 +58,6 @@ char **breakline(char *input, int *count);
9ae3a8
 void doneline(char *input, char **vec);
9ae3a8
 char *fetchline(void);
9ae3a8
 
9ae3a8
-long long cvtnum(char *s);
9ae3a8
 void cvtstr(double value, char *str, size_t sz);
9ae3a8
 
9ae3a8
 struct timeval tsub(struct timeval t1, struct timeval t2);
9ae3a8
diff --git a/qemu-io.c b/qemu-io.c
9ae3a8
index 9f66a78..3b5890e 100644
9ae3a8
--- a/qemu-io.c
9ae3a8
+++ b/qemu-io.c
9ae3a8
@@ -33,6 +33,12 @@ static BlockDriverState *bs;
9ae3a8
 
9ae3a8
 static int misalign;
9ae3a8
 
9ae3a8
+static int64_t cvtnum(const char *s)
9ae3a8
+{
9ae3a8
+    char *end;
9ae3a8
+    return strtosz_suffix(s, &end, STRTOSZ_DEFSUFFIX_B);
9ae3a8
+}
9ae3a8
+
9ae3a8
 /*
9ae3a8
  * Parse the pattern argument to various sub-commands.
9ae3a8
  *
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8