From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46403 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1P3ryv-0008Uy-QJ for qemu-devel@nongnu.org; Thu, 07 Oct 2010 11:02:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1P3rys-0005FJ-Q7 for qemu-devel@nongnu.org; Thu, 07 Oct 2010 11:02:05 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54005) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1P3rys-0005F8-Jc for qemu-devel@nongnu.org; Thu, 07 Oct 2010 11:02:02 -0400 From: Jes.Sorensen@redhat.com Date: Thu, 7 Oct 2010 17:01:44 +0200 Message-Id: <1286463710-28262-1-git-send-email-Jes.Sorensen@redhat.com> Subject: [Qemu-devel] [PATCH v3 0/6] Introduce strtosz and make use of it List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: pbonzini@redhat.com, armbru@redhat.com From: Jes Sorensen This patch introduces cutils.c: strtosz() and gets rid of the multiple custom hacks for parsing byte sizes. In addition it adds supports for specifying human style sizes such as 1.5G. Last it eliminates the horrible abuse of a float to store the byte size for migrate_set_speed in the monitor. New in v3 I tried to apply the changes suggested by Markus, notably I changed the name from strtobytes() to strtosz(), as well as added better error handling. Jes Jes Sorensen (6): Introduce strtosz() library function to convert a string to a byte count. Support human unit formats in strtosz, eg. 1.0G Add more error handling to strtosz() Add support for 'o' octet (bytes) format as monitor parameter. Switch migrate_set_speed() to take an 'o' argument rather than a float. Remove obsolete 'f' double parameter type cutils.c | 61 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ hmp-commands.hx | 2 +- migration.c | 4 +- monitor.c | 46 ++++++++++++++++++++++++++--------------- qemu-common.h | 1 + vl.c | 31 +++++++++------------------- 6 files changed, 104 insertions(+), 41 deletions(-) -- 1.7.2.3