From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:45712) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TvXwE-0000zM-EB for qemu-devel@nongnu.org; Wed, 16 Jan 2013 13:42:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TvXwD-0004Fo-Fn for qemu-devel@nongnu.org; Wed, 16 Jan 2013 13:42:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:1414) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TvXiM-0008Sb-8S for qemu-devel@nongnu.org; Wed, 16 Jan 2013 13:27:54 -0500 From: Eduardo Habkost Date: Wed, 16 Jan 2013 16:28:45 -0200 Message-Id: <1358360933-5323-1-git-send-email-ehabkost@redhat.com> Subject: [Qemu-devel] [PATCH 0/8] -numa option parsing fixes (v3) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, Anthony Liguori Cc: Chegu Vinod , Laszlo Ersek Changes v2 -> v3: - Add 'base' parameter to parse_uint*() (patch 1/8) - Keep existing base=10 behavior when parsing "nodeid" and "cpus" (patches 6/8, 8/8) - Trivial whitespace change on patch 1/8 - Fix fprintf() format string on patch 5/8 This series contains only the most important fixes from the previous "-numa option parsing fixes & improvements" series I have submitted. I have introduced parse_uint*() helpers that can be reused by other code, later. I plan to submit parse_int*() (for signed integers) and parse_double*() functions too, later, and change string-input-visitor.c and opts-visitor.c to use those common functions instead of duplicating the number parsing code. Eduardo Habkost (8): cutils: unsigned int parsing functions vl.c: Fix off-by-one bug when handling "-numa node" argument vl.c: Abort on unknown -numa option type vl.c: Check for NUMA node limit inside numa_add() vl.c: numa_add(): Validate nodeid before using it vl.c: Use parse_uint_full() for NUMA nodeid vl.c: Extract -numa "cpus" parsing to separate function vl.c: validate -numa "cpus" parameter properly include/qemu-common.h | 4 + tests/Makefile | 3 + tests/test-cutils.c | 216 ++++++++++++++++++++++++++++++++++++++++++++++++++ util/cutils.c | 79 ++++++++++++++++++ vl.c | 93 ++++++++++++++++------ 5 files changed, 370 insertions(+), 25 deletions(-) create mode 100644 tests/test-cutils.c -- 1.7.11.7