]> xenbits.xensource.com Git - qemu-xen.git/commitdiff
Merge remote-tracking branch 'remotes/thuth-gitlab/tags/pull-request-2021-03-12'...
authorPeter Maydell <peter.maydell@linaro.org>
Sun, 14 Mar 2021 15:13:53 +0000 (15:13 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Sun, 14 Mar 2021 15:13:53 +0000 (15:13 +0000)
* Move unit and bench tests into separate directories
* Clean-up and improve gitlab-ci jobs
* Drop the non-working "check-speed" makefile target
* Minor documentation updates

# gpg: Signature made Fri 12 Mar 2021 17:18:45 GMT
# gpg:                using RSA key 27B88847EEE0250118F3EAB92ED9D774FE702DB5
# gpg:                issuer "thuth@redhat.com"
# gpg: Good signature from "Thomas Huth <th.huth@gmx.de>" [full]
# gpg:                 aka "Thomas Huth <thuth@redhat.com>" [full]
# gpg:                 aka "Thomas Huth <huth@tuxfamily.org>" [full]
# gpg:                 aka "Thomas Huth <th.huth@posteo.de>" [unknown]
# Primary key fingerprint: 27B8 8847 EEE0 2501 18F3  EAB9 2ED9 D774 FE70 2DB5

* remotes/thuth-gitlab/tags/pull-request-2021-03-12:
  README: Add Documentation blurb
  MAINTAINERS: Merge the Gitlab-CI section into the generic CI section
  tests: remove "make check-speed" in favor of "make bench"
  gitlab-ci.yml: Merge check-crypto-old jobs into the build-crypto-old jobs
  gitlab-ci.yml: Merge one of the coroutine jobs with the tcg-disabled job
  gitlab-ci.yml: Add some missing dependencies to the jobs
  gitlab-ci.yml: Move build-tools-and-docs-debian to a better place
  tests: Move benchmarks into a separate folder
  tests: Move unit tests into a separate directory

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
1  2 
.gitlab-ci.yml
MAINTAINERS
tests/unit/test-cutils.c
tests/unit/test-keyval.c
tests/unit/test-qemu-opts.c

diff --cc .gitlab-ci.yml
index 293a81065691b33e4dabbd19c2f1c50cb897ff7a,0ada3dbb90a4180f33273f43787f2ff4c1e2bf1f..f65cb11c4d377641ed8b8322db1b5c86301a0469
@@@ -485,127 -475,10 +479,129 @@@ clang-user
        --extra-cflags=-fsanitize=undefined --extra-cflags=-fno-sanitize-recover=undefined
      MAKE_CHECK_ARGS: check-unit check-tcg
  
 +# Set LD_JOBS=1 because this requires LTO and ld consumes a large amount of memory.
 +# On gitlab runners, default value sometimes end up calling 2 lds concurrently and
 +# triggers an Out-Of-Memory error
 +#
 +# Since slirp callbacks are used in QEMU Timers, slirp needs to be compiled together
 +# with QEMU and linked as a static library to avoid false positives in CFI checks.
 +# This can be accomplished by using -enable-slirp=git, which avoids the use of
 +# a system-wide version of the library
 +#
 +# Split in three sets of build/check/acceptance to limit the execution time of each
 +# job
 +build-cfi-aarch64:
 +  <<: *native_build_job_definition
 +  needs:
 +  - job: amd64-fedora-container
 +  variables:
 +    LD_JOBS: 1
 +    AR: llvm-ar
 +    IMAGE: fedora
 +    CONFIGURE_ARGS: --cc=clang --cxx=clang++ --enable-cfi --enable-cfi-debug
 +      --enable-safe-stack --enable-slirp=git
 +    TARGETS: aarch64-softmmu
 +    MAKE_CHECK_ARGS: check-build
 +  artifacts:
 +    expire_in: 2 days
 +    paths:
 +      - build
 +
 +check-cfi-aarch64:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-aarch64
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check
 +
 +acceptance-cfi-aarch64:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-aarch64
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check-acceptance
 +  <<: *acceptance_definition
 +
 +build-cfi-ppc64-s390x:
 +  <<: *native_build_job_definition
 +  needs:
 +  - job: amd64-fedora-container
 +  variables:
 +    LD_JOBS: 1
 +    AR: llvm-ar
 +    IMAGE: fedora
 +    CONFIGURE_ARGS: --cc=clang --cxx=clang++ --enable-cfi --enable-cfi-debug
 +      --enable-safe-stack --enable-slirp=git
 +    TARGETS: ppc64-softmmu s390x-softmmu
 +    MAKE_CHECK_ARGS: check-build
 +  artifacts:
 +    expire_in: 2 days
 +    paths:
 +      - build
 +
 +check-cfi-ppc64-s390x:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-ppc64-s390x
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check
 +
 +acceptance-cfi-ppc64-s390x:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-ppc64-s390x
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check-acceptance
 +  <<: *acceptance_definition
 +
 +build-cfi-x86_64:
 +  <<: *native_build_job_definition
 +  needs:
 +  - job: amd64-fedora-container
 +  variables:
 +    LD_JOBS: 1
 +    AR: llvm-ar
 +    IMAGE: fedora
 +    CONFIGURE_ARGS: --cc=clang --cxx=clang++ --enable-cfi --enable-cfi-debug
 +      --enable-safe-stack --enable-slirp=git
 +    TARGETS: x86_64-softmmu
 +    MAKE_CHECK_ARGS: check-build
 +  artifacts:
 +    expire_in: 2 days
 +    paths:
 +      - build
 +
 +check-cfi-x86_64:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-x86_64
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check
 +
 +acceptance-cfi-x86_64:
 +  <<: *native_test_job_definition
 +  needs:
 +    - job: build-cfi-x86_64
 +      artifacts: true
 +  variables:
 +    IMAGE: fedora
 +    MAKE_CHECK_ARGS: check-acceptance
 +  <<: *acceptance_definition
 +
  tsan-build:
    <<: *native_build_job_definition
+   needs:
+     job: amd64-ubuntu2004-container
    variables:
      IMAGE: ubuntu2004
      CONFIGURE_ARGS: --enable-tsan --cc=clang-10 --cxx=clang++-10 --disable-docs
diff --cc MAINTAINERS
Simple merge
index 0000000000000000000000000000000000000000,1aa8351520ae33201a3f7fb0b67841498aefad06..bad3a609938975bc10c0f11cadc9c5d1715cefc0
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,2460 +1,2574 @@@
 -    str = "12345";
+ /*
+  * cutils.c unit-tests
+  *
+  * Copyright (C) 2013 Red Hat Inc.
+  *
+  * Authors:
+  *  Eduardo Habkost <ehabkost@redhat.com>
+  *
+  * Permission is hereby granted, free of charge, to any person obtaining a copy
+  * of this software and associated documentation files (the "Software"), to deal
+  * in the Software without restriction, including without limitation the rights
+  * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+  * copies of the Software, and to permit persons to whom the Software is
+  * furnished to do so, subject to the following conditions:
+  *
+  * The above copyright notice and this permission notice shall be included in
+  * all copies or substantial portions of the Software.
+  *
+  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+  * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+  * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+  * THE SOFTWARE.
+  */
+ #include "qemu/osdep.h"
+ #include "qemu/units.h"
+ #include "qemu/cutils.h"
+ #include "qemu/units.h"
+ static void test_parse_uint_null(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     int r;
+     r = parse_uint(NULL, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -EINVAL);
+     g_assert_cmpint(i, ==, 0);
+     g_assert(endptr == NULL);
+ }
+ static void test_parse_uint_empty(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -EINVAL);
+     g_assert_cmpint(i, ==, 0);
+     g_assert(endptr == str);
+ }
+ static void test_parse_uint_whitespace(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "   \t   ";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -EINVAL);
+     g_assert_cmpint(i, ==, 0);
+     g_assert(endptr == str);
+ }
+ static void test_parse_uint_invalid(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = " \t xxx";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -EINVAL);
+     g_assert_cmpint(i, ==, 0);
+     g_assert(endptr == str);
+ }
+ static void test_parse_uint_trailing(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "123xxx";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_parse_uint_correct(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "123";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_parse_uint_octal(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "0123";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_parse_uint_decimal(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "0123";
+     int r;
+     r = parse_uint(str, &i, &endptr, 10);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_parse_uint_llong_max(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     char *str = g_strdup_printf("%llu", (unsigned long long)LLONG_MAX + 1);
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, (unsigned long long)LLONG_MAX + 1);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_parse_uint_overflow(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = "99999999999999999999999999999999999999";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -ERANGE);
+     g_assert_cmpint(i, ==, ULLONG_MAX);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_parse_uint_negative(void)
+ {
+     unsigned long long i = 999;
+     char f = 'X';
+     char *endptr = &f;
+     const char *str = " \t -321";
+     int r;
+     r = parse_uint(str, &i, &endptr, 0);
+     g_assert_cmpint(r, ==, -ERANGE);
+     g_assert_cmpint(i, ==, 0);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_parse_uint_full_trailing(void)
+ {
+     unsigned long long i = 999;
+     const char *str = "123xxx";
+     int r;
+     r = parse_uint_full(str, &i, 0);
+     g_assert_cmpint(r, ==, -EINVAL);
+     g_assert_cmpint(i, ==, 0);
+ }
+ static void test_parse_uint_full_correct(void)
+ {
+     unsigned long long i = 999;
+     const char *str = "123";
+     int r;
+     r = parse_uint_full(str, &i, 0);
+     g_assert_cmpint(r, ==, 0);
+     g_assert_cmpint(i, ==, 123);
+ }
+ static void test_qemu_strtoi_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtoi_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtoi_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtoi_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi_max(void)
+ {
+     char *str = g_strdup_printf("%d", INT_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, INT_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoi_overflow(void)
+ {
+     char *str = g_strdup_printf("%lld", (long long)INT_MAX + 1ll);
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, INT_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoi_underflow(void)
+ {
+     char *str = g_strdup_printf("%lld", (long long)INT_MIN - 1ll);
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err  = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, INT_MIN);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoi_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi_full_correct(void)
+ {
+     const char *str = "123";
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+ }
+ static void test_qemu_strtoi_full_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     int res = 999;
+     int err;
+     err = qemu_strtoi(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtoi_full_empty(void)
+ {
+     const char *str = "";
+     int res = 999L;
+     int err;
+     err =  qemu_strtoi(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoi_full_negative(void)
+ {
+     const char *str = " \t -321";
+     int res = 999;
+     int err;
+     err = qemu_strtoi(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+ }
+ static void test_qemu_strtoi_full_trailing(void)
+ {
+     const char *str = "123xxx";
+     int res;
+     int err;
+     err = qemu_strtoi(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoi_full_max(void)
+ {
+     char *str = g_strdup_printf("%d", INT_MAX);
+     int res;
+     int err;
+     err = qemu_strtoi(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, INT_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtoui_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtoui_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtoui_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoui_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoui_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoui_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtoui_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoui_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoui_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoui_max(void)
+ {
+     char *str = g_strdup_printf("%u", UINT_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, UINT_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoui_overflow(void)
+ {
+     char *str = g_strdup_printf("%lld", (long long)UINT_MAX + 1ll);
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmphex(res, ==, UINT_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoui_underflow(void)
+ {
+     char *str = g_strdup_printf("%lld", (long long)INT_MIN - 1ll);
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err  = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpuint(res, ==, (unsigned int)-1);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoui_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, (unsigned int)-321);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoui_full_correct(void)
+ {
+     const char *str = "123";
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+ }
+ static void test_qemu_strtoui_full_null(void)
+ {
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(NULL, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoui_full_empty(void)
+ {
+     const char *str = "";
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoui_full_negative(void)
+ {
+     const char *str = " \t -321";
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, (unsigned int)-321);
+ }
+ static void test_qemu_strtoui_full_trailing(void)
+ {
+     const char *str = "123xxx";
+     unsigned int res;
+     int err;
+     err = qemu_strtoui(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoui_full_max(void)
+ {
+     char *str = g_strdup_printf("%u", UINT_MAX);
+     unsigned int res = 999;
+     int err;
+     err = qemu_strtoui(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, UINT_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtol_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtol_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtol_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtol_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtol_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtol_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtol_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     res = 999;
+     endptr = &f;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_max(void)
+ {
+     char *str = g_strdup_printf("%ld", LONG_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, LONG_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtol_overflow(void)
+ {
+     const char *str = "99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, LONG_MAX);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_underflow(void)
+ {
+     const char *str = "-99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err  = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, LONG_MIN);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtol_full_correct(void)
+ {
+     const char *str = "123";
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+ }
+ static void test_qemu_strtol_full_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     long res = 999;
+     int err;
+     err = qemu_strtol(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtol_full_empty(void)
+ {
+     const char *str = "";
+     long res = 999L;
+     int err;
+     err =  qemu_strtol(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtol_full_negative(void)
+ {
+     const char *str = " \t -321";
+     long res = 999;
+     int err;
+     err = qemu_strtol(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+ }
+ static void test_qemu_strtol_full_trailing(void)
+ {
+     const char *str = "123xxx";
+     long res;
+     int err;
+     err = qemu_strtol(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtol_full_max(void)
+ {
+     char *str = g_strdup_printf("%ld", LONG_MAX);
+     long res;
+     int err;
+     err = qemu_strtol(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, LONG_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtoul_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtoul_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtoul_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoul_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoul_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoul_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtoul_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     res = 999;
+     endptr = &f;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_max(void)
+ {
+     char *str = g_strdup_printf("%lu", ULONG_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, ULONG_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoul_overflow(void)
+ {
+     const char *str = "99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmphex(res, ==, ULONG_MAX);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_underflow(void)
+ {
+     const char *str = "-99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err  = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpuint(res, ==, -1ul);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, -321ul);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoul_full_correct(void)
+ {
+     const char *str = "123";
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+ }
+ static void test_qemu_strtoul_full_null(void)
+ {
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(NULL, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoul_full_empty(void)
+ {
+     const char *str = "";
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoul_full_negative(void)
+ {
+     const char *str = " \t -321";
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, -321ul);
+ }
+ static void test_qemu_strtoul_full_trailing(void)
+ {
+     const char *str = "123xxx";
+     unsigned long res;
+     int err;
+     err = qemu_strtoul(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoul_full_max(void)
+ {
+     char *str = g_strdup_printf("%lu", ULONG_MAX);
+     unsigned long res = 999;
+     int err;
+     err = qemu_strtoul(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, ULONG_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtoi64_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtoi64_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtoi64_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi64_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi64_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtoi64_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtoi64_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     endptr = &f;
+     res = 999;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     endptr = &f;
+     res = 999;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     endptr = &f;
+     res = 999;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_max(void)
+ {
+     char *str = g_strdup_printf("%lld", LLONG_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, LLONG_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtoi64_overflow(void)
+ {
+     const char *str = "99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, LLONG_MAX);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_underflow(void)
+ {
+     const char *str = "-99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err  = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmpint(res, ==, LLONG_MIN);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtoi64_full_correct(void)
+ {
+     const char *str = "123";
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 123);
+ }
+ static void test_qemu_strtoi64_full_null(void)
+ {
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(NULL, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoi64_full_empty(void)
+ {
+     const char *str = "";
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoi64_full_negative(void)
+ {
+     const char *str = " \t -321";
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, -321);
+ }
+ static void test_qemu_strtoi64_full_trailing(void)
+ {
+     const char *str = "123xxx";
+     int64_t res = 999;
+     int err;
+     err = qemu_strtoi64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtoi64_full_max(void)
+ {
+     char *str = g_strdup_printf("%lld", LLONG_MAX);
+     int64_t res;
+     int err;
+     err = qemu_strtoi64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, LLONG_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtou64_correct(void)
+ {
+     const char *str = "12345 foo";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 12345);
+     g_assert(endptr == str + 5);
+ }
+ static void test_qemu_strtou64_null(void)
+ {
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(NULL, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == NULL);
+ }
+ static void test_qemu_strtou64_empty(void)
+ {
+     const char *str = "";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtou64_whitespace(void)
+ {
+     const char *str = "  \t  ";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtou64_invalid(void)
+ {
+     const char *str = "   xxxx  \t abc";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+ }
+ static void test_qemu_strtou64_trailing(void)
+ {
+     const char *str = "123xxx";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtou64_octal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 8, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+     endptr = &f;
+     res = 999;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 0123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_decimal(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 10, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+     str = "123";
+     endptr = &f;
+     res = 999;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_hex(void)
+ {
+     const char *str = "0123";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 16, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+     str = "0x123";
+     endptr = &f;
+     res = 999;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, 0x123);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_max(void)
+ {
+     char *str = g_strdup_printf("%llu", ULLONG_MAX);
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, ULLONG_MAX);
+     g_assert(endptr == str + strlen(str));
+     g_free(str);
+ }
+ static void test_qemu_strtou64_overflow(void)
+ {
+     const char *str = "99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmphex(res, ==, ULLONG_MAX);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_underflow(void)
+ {
+     const char *str = "-99999999999999999999999999999999999999999999";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err  = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert_cmphex(res, ==, -1ull);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_negative(void)
+ {
+     const char *str = "  \t -321";
+     char f = 'X';
+     const char *endptr = &f;
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, &endptr, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, -321ull);
+     g_assert(endptr == str + strlen(str));
+ }
+ static void test_qemu_strtou64_full_correct(void)
+ {
+     const char *str = "18446744073709551614";
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, 18446744073709551614ull);
+ }
+ static void test_qemu_strtou64_full_null(void)
+ {
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(NULL, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtou64_full_empty(void)
+ {
+     const char *str = "";
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtou64_full_negative(void)
+ {
+     const char *str = " \t -321";
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpuint(res, ==, -321ull);
+ }
+ static void test_qemu_strtou64_full_trailing(void)
+ {
+     const char *str = "18446744073709551614xxxxxx";
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtou64_full_max(void)
+ {
+     char *str = g_strdup_printf("%lld", ULLONG_MAX);
+     uint64_t res = 999;
+     int err;
+     err = qemu_strtou64(str, NULL, 0, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmphex(res, ==, ULLONG_MAX);
+     g_free(str);
+ }
+ static void test_qemu_strtosz_simple(void)
+ {
+     const char *str;
+     const char *endptr;
+     int err;
+     uint64_t res = 0xbaadf00d;
+     str = "0";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0);
+     g_assert(endptr == str + 1);
 -    g_assert(endptr == str + 5);
++    /* Leading 0 gives decimal results, not octal */
++    str = "08";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 8);
++    g_assert(endptr == str + 2);
++
++    /* Leading space is ignored */
++    str = " 12345";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345);
 -    /* Note: precision is 53 bits since we're parsing with strtod() */
 -
++    g_assert(endptr == str + 6);
+     err = qemu_strtosz(str, NULL, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345);
 -    g_assert_cmpint(res, ==, 0x20000000000000); /* rounded to 53 bits */
+     str = "9007199254740991"; /* 2^53-1 */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x1fffffffffffff);
+     g_assert(endptr == str + 16);
+     str = "9007199254740992"; /* 2^53 */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0x20000000000000);
+     g_assert(endptr == str + 16);
+     str = "9007199254740993"; /* 2^53+1 */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
 -    g_assert_cmpint(res, ==, 0xfffffffffffff800); /* rounded to 53 bits */
++    g_assert_cmpint(res, ==, 0x20000000000001);
+     g_assert(endptr == str + 16);
+     str = "18446744073709549568"; /* 0xfffffffffffff800 (53 msbs set) */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 0xfffffffffffff800);
+     g_assert(endptr == str + 20);
+     str = "18446744073709550591"; /* 0xfffffffffffffbff */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
 -    /* 0x7ffffffffffffe00..0x7fffffffffffffff get rounded to
 -     * 0x8000000000000000, thus -ERANGE; see test_qemu_strtosz_erange() */
++    g_assert_cmpint(res, ==, 0xfffffffffffffbff);
+     g_assert(endptr == str + 20);
 -    const char *str = "12.345M";
++    str = "18446744073709551615"; /* 0xffffffffffffffff */
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 0xffffffffffffffff);
++    g_assert(endptr == str + 20);
++}
++
++static void test_qemu_strtosz_hex(void)
++{
++    const char *str;
++    const char *endptr;
++    int err;
++    uint64_t res = 0xbaadf00d;
++
++    str = "0x0";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 0);
++    g_assert(endptr == str + 3);
++
++    str = "0xab";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 171);
++    g_assert(endptr == str + 4);
++
++    str = "0xae";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 174);
++    g_assert(endptr == str + 4);
+ }
+ static void test_qemu_strtosz_units(void)
+ {
+     const char *none = "1";
+     const char *b = "1B";
+     const char *k = "1K";
+     const char *m = "1M";
+     const char *g = "1G";
+     const char *t = "1T";
+     const char *p = "1P";
+     const char *e = "1E";
+     int err;
+     const char *endptr;
+     uint64_t res = 0xbaadf00d;
+     /* default is M */
+     err = qemu_strtosz_MiB(none, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, MiB);
+     g_assert(endptr == none + 1);
+     err = qemu_strtosz(b, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 1);
+     g_assert(endptr == b + 2);
+     err = qemu_strtosz(k, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, KiB);
+     g_assert(endptr == k + 2);
+     err = qemu_strtosz(m, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, MiB);
+     g_assert(endptr == m + 2);
+     err = qemu_strtosz(g, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, GiB);
+     g_assert(endptr == g + 2);
+     err = qemu_strtosz(t, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, TiB);
+     g_assert(endptr == t + 2);
+     err = qemu_strtosz(p, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, PiB);
+     g_assert(endptr == p + 2);
+     err = qemu_strtosz(e, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, EiB);
+     g_assert(endptr == e + 2);
+ }
+ static void test_qemu_strtosz_float(void)
+ {
 -    g_assert_cmpint(res, ==, 12.345 * MiB);
++    const char *str;
+     int err;
+     const char *endptr;
+     uint64_t res = 0xbaadf00d;
++    str = "0.5E";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, EiB / 2);
++    g_assert(endptr == str + 4);
++
++    /* For convenience, a fraction of 0 is tolerated even on bytes */
++    str = "1.0B";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 1);
++    g_assert(endptr == str + 4);
++
++    /* An empty fraction is tolerated */
++    str = "1.k";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
 -    str = "-1";
 -    err = qemu_strtosz(str, &endptr, &res);
 -    g_assert_cmpint(err, ==, -ERANGE);
 -    g_assert(endptr == str + 2);
 -
 -    str = "18446744073709550592"; /* 0xfffffffffffffc00 */
 -    err = qemu_strtosz(str, &endptr, &res);
 -    g_assert_cmpint(err, ==, -ERANGE);
 -    g_assert(endptr == str + 20);
 -
 -    str = "18446744073709551615"; /* 2^64-1 */
 -    err = qemu_strtosz(str, &endptr, &res);
 -    g_assert_cmpint(err, ==, -ERANGE);
 -    g_assert(endptr == str + 20);
 -
 -    str = "18446744073709551616"; /* 2^64 */
++    g_assert_cmpint(res, ==, 1024);
++    g_assert(endptr == str + 3);
++
++    /* For convenience, we permit values that are not byte-exact */
++    str = "12.345M";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, (uint64_t) (12.345 * MiB));
+     g_assert(endptr == str + 7);
+ }
+ static void test_qemu_strtosz_invalid(void)
+ {
+     const char *str;
+     const char *endptr;
+     int err;
+     uint64_t res = 0xbaadf00d;
+     str = "";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+     str = " \t ";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+     str = "crap";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+     str = "inf";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
+     str = "NaN";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     g_assert(endptr == str);
++
++    /* Fractional values require scale larger than bytes */
++    str = "1.1B";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    str = "1.1";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    /* No floating point exponents */
++    str = "1.5e1k";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    str = "1.5E+0k";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    /* No hex fractions */
++    str = "0x1.8k";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    /* No negative values */
++    str = "-0";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
++
++    str = "-1";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++    g_assert(endptr == str);
+ }
+ static void test_qemu_strtosz_trailing(void)
+ {
+     const char *str;
+     const char *endptr;
+     int err;
+     uint64_t res = 0xbaadf00d;
+     str = "123xxx";
+     err = qemu_strtosz_MiB(str, &endptr, &res);
+     g_assert_cmpint(res, ==, 123 * MiB);
+     g_assert(endptr == str + 3);
+     err = qemu_strtosz(str, NULL, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
+     str = "1kiB";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 1024);
+     g_assert(endptr == str + 2);
+     err = qemu_strtosz(str, NULL, &res);
+     g_assert_cmpint(err, ==, -EINVAL);
++
++    str = "0x";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(res, ==, 0);
++    g_assert(endptr == str + 1);
++
++    err = qemu_strtosz(str, NULL, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++
++    str = "0.NaN";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert(endptr == str + 2);
++
++    err = qemu_strtosz(str, NULL, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
++
++    str = "123-45";
++    err = qemu_strtosz(str, &endptr, &res);
++    g_assert_cmpint(res, ==, 123);
++    g_assert(endptr == str + 3);
++
++    err = qemu_strtosz(str, NULL, &res);
++    g_assert_cmpint(err, ==, -EINVAL);
+ }
+ static void test_qemu_strtosz_erange(void)
+ {
+     const char *str;
+     const char *endptr;
+     int err;
+     uint64_t res = 0xbaadf00d;
 -    const char *str = "12345k";
++    str = "18446744073709551616"; /* 2^64; see strtosz_simple for 2^64-1 */
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert(endptr == str + 20);
+     str = "20E";
+     err = qemu_strtosz(str, &endptr, &res);
+     g_assert_cmpint(err, ==, -ERANGE);
+     g_assert(endptr == str + 3);
+ }
+ static void test_qemu_strtosz_metric(void)
+ {
++    const char *str;
+     int err;
+     const char *endptr;
+     uint64_t res = 0xbaadf00d;
++    str = "12345k";
+     err = qemu_strtosz_metric(str, &endptr, &res);
+     g_assert_cmpint(err, ==, 0);
+     g_assert_cmpint(res, ==, 12345000);
+     g_assert(endptr == str + 6);
++
++    str = "12.345M";
++    err = qemu_strtosz_metric(str, &endptr, &res);
++    g_assert_cmpint(err, ==, 0);
++    g_assert_cmpint(res, ==, 12345000);
++    g_assert(endptr == str + 7);
+ }
+ int main(int argc, char **argv)
+ {
+     g_test_init(&argc, &argv, NULL);
+     g_test_add_func("/cutils/parse_uint/null", test_parse_uint_null);
+     g_test_add_func("/cutils/parse_uint/empty", test_parse_uint_empty);
+     g_test_add_func("/cutils/parse_uint/whitespace",
+                     test_parse_uint_whitespace);
+     g_test_add_func("/cutils/parse_uint/invalid", test_parse_uint_invalid);
+     g_test_add_func("/cutils/parse_uint/trailing", test_parse_uint_trailing);
+     g_test_add_func("/cutils/parse_uint/correct", test_parse_uint_correct);
+     g_test_add_func("/cutils/parse_uint/octal", test_parse_uint_octal);
+     g_test_add_func("/cutils/parse_uint/decimal", test_parse_uint_decimal);
+     g_test_add_func("/cutils/parse_uint/llong_max", test_parse_uint_llong_max);
+     g_test_add_func("/cutils/parse_uint/overflow", test_parse_uint_overflow);
+     g_test_add_func("/cutils/parse_uint/negative", test_parse_uint_negative);
+     g_test_add_func("/cutils/parse_uint_full/trailing",
+                     test_parse_uint_full_trailing);
+     g_test_add_func("/cutils/parse_uint_full/correct",
+                     test_parse_uint_full_correct);
+     /* qemu_strtoi() tests */
+     g_test_add_func("/cutils/qemu_strtoi/correct",
+                     test_qemu_strtoi_correct);
+     g_test_add_func("/cutils/qemu_strtoi/null",
+                     test_qemu_strtoi_null);
+     g_test_add_func("/cutils/qemu_strtoi/empty",
+                     test_qemu_strtoi_empty);
+     g_test_add_func("/cutils/qemu_strtoi/whitespace",
+                     test_qemu_strtoi_whitespace);
+     g_test_add_func("/cutils/qemu_strtoi/invalid",
+                     test_qemu_strtoi_invalid);
+     g_test_add_func("/cutils/qemu_strtoi/trailing",
+                     test_qemu_strtoi_trailing);
+     g_test_add_func("/cutils/qemu_strtoi/octal",
+                     test_qemu_strtoi_octal);
+     g_test_add_func("/cutils/qemu_strtoi/decimal",
+                     test_qemu_strtoi_decimal);
+     g_test_add_func("/cutils/qemu_strtoi/hex",
+                     test_qemu_strtoi_hex);
+     g_test_add_func("/cutils/qemu_strtoi/max",
+                     test_qemu_strtoi_max);
+     g_test_add_func("/cutils/qemu_strtoi/overflow",
+                     test_qemu_strtoi_overflow);
+     g_test_add_func("/cutils/qemu_strtoi/underflow",
+                     test_qemu_strtoi_underflow);
+     g_test_add_func("/cutils/qemu_strtoi/negative",
+                     test_qemu_strtoi_negative);
+     g_test_add_func("/cutils/qemu_strtoi_full/correct",
+                     test_qemu_strtoi_full_correct);
+     g_test_add_func("/cutils/qemu_strtoi_full/null",
+                     test_qemu_strtoi_full_null);
+     g_test_add_func("/cutils/qemu_strtoi_full/empty",
+                     test_qemu_strtoi_full_empty);
+     g_test_add_func("/cutils/qemu_strtoi_full/negative",
+                     test_qemu_strtoi_full_negative);
+     g_test_add_func("/cutils/qemu_strtoi_full/trailing",
+                     test_qemu_strtoi_full_trailing);
+     g_test_add_func("/cutils/qemu_strtoi_full/max",
+                     test_qemu_strtoi_full_max);
+     /* qemu_strtoui() tests */
+     g_test_add_func("/cutils/qemu_strtoui/correct",
+                     test_qemu_strtoui_correct);
+     g_test_add_func("/cutils/qemu_strtoui/null",
+                     test_qemu_strtoui_null);
+     g_test_add_func("/cutils/qemu_strtoui/empty",
+                     test_qemu_strtoui_empty);
+     g_test_add_func("/cutils/qemu_strtoui/whitespace",
+                     test_qemu_strtoui_whitespace);
+     g_test_add_func("/cutils/qemu_strtoui/invalid",
+                     test_qemu_strtoui_invalid);
+     g_test_add_func("/cutils/qemu_strtoui/trailing",
+                     test_qemu_strtoui_trailing);
+     g_test_add_func("/cutils/qemu_strtoui/octal",
+                     test_qemu_strtoui_octal);
+     g_test_add_func("/cutils/qemu_strtoui/decimal",
+                     test_qemu_strtoui_decimal);
+     g_test_add_func("/cutils/qemu_strtoui/hex",
+                     test_qemu_strtoui_hex);
+     g_test_add_func("/cutils/qemu_strtoui/max",
+                     test_qemu_strtoui_max);
+     g_test_add_func("/cutils/qemu_strtoui/overflow",
+                     test_qemu_strtoui_overflow);
+     g_test_add_func("/cutils/qemu_strtoui/underflow",
+                     test_qemu_strtoui_underflow);
+     g_test_add_func("/cutils/qemu_strtoui/negative",
+                     test_qemu_strtoui_negative);
+     g_test_add_func("/cutils/qemu_strtoui_full/correct",
+                     test_qemu_strtoui_full_correct);
+     g_test_add_func("/cutils/qemu_strtoui_full/null",
+                     test_qemu_strtoui_full_null);
+     g_test_add_func("/cutils/qemu_strtoui_full/empty",
+                     test_qemu_strtoui_full_empty);
+     g_test_add_func("/cutils/qemu_strtoui_full/negative",
+                     test_qemu_strtoui_full_negative);
+     g_test_add_func("/cutils/qemu_strtoui_full/trailing",
+                     test_qemu_strtoui_full_trailing);
+     g_test_add_func("/cutils/qemu_strtoui_full/max",
+                     test_qemu_strtoui_full_max);
+     /* qemu_strtol() tests */
+     g_test_add_func("/cutils/qemu_strtol/correct",
+                     test_qemu_strtol_correct);
+     g_test_add_func("/cutils/qemu_strtol/null",
+                     test_qemu_strtol_null);
+     g_test_add_func("/cutils/qemu_strtol/empty",
+                     test_qemu_strtol_empty);
+     g_test_add_func("/cutils/qemu_strtol/whitespace",
+                     test_qemu_strtol_whitespace);
+     g_test_add_func("/cutils/qemu_strtol/invalid",
+                     test_qemu_strtol_invalid);
+     g_test_add_func("/cutils/qemu_strtol/trailing",
+                     test_qemu_strtol_trailing);
+     g_test_add_func("/cutils/qemu_strtol/octal",
+                     test_qemu_strtol_octal);
+     g_test_add_func("/cutils/qemu_strtol/decimal",
+                     test_qemu_strtol_decimal);
+     g_test_add_func("/cutils/qemu_strtol/hex",
+                     test_qemu_strtol_hex);
+     g_test_add_func("/cutils/qemu_strtol/max",
+                     test_qemu_strtol_max);
+     g_test_add_func("/cutils/qemu_strtol/overflow",
+                     test_qemu_strtol_overflow);
+     g_test_add_func("/cutils/qemu_strtol/underflow",
+                     test_qemu_strtol_underflow);
+     g_test_add_func("/cutils/qemu_strtol/negative",
+                     test_qemu_strtol_negative);
+     g_test_add_func("/cutils/qemu_strtol_full/correct",
+                     test_qemu_strtol_full_correct);
+     g_test_add_func("/cutils/qemu_strtol_full/null",
+                     test_qemu_strtol_full_null);
+     g_test_add_func("/cutils/qemu_strtol_full/empty",
+                     test_qemu_strtol_full_empty);
+     g_test_add_func("/cutils/qemu_strtol_full/negative",
+                     test_qemu_strtol_full_negative);
+     g_test_add_func("/cutils/qemu_strtol_full/trailing",
+                     test_qemu_strtol_full_trailing);
+     g_test_add_func("/cutils/qemu_strtol_full/max",
+                     test_qemu_strtol_full_max);
+     /* qemu_strtoul() tests */
+     g_test_add_func("/cutils/qemu_strtoul/correct",
+                     test_qemu_strtoul_correct);
+     g_test_add_func("/cutils/qemu_strtoul/null",
+                     test_qemu_strtoul_null);
+     g_test_add_func("/cutils/qemu_strtoul/empty",
+                     test_qemu_strtoul_empty);
+     g_test_add_func("/cutils/qemu_strtoul/whitespace",
+                     test_qemu_strtoul_whitespace);
+     g_test_add_func("/cutils/qemu_strtoul/invalid",
+                     test_qemu_strtoul_invalid);
+     g_test_add_func("/cutils/qemu_strtoul/trailing",
+                     test_qemu_strtoul_trailing);
+     g_test_add_func("/cutils/qemu_strtoul/octal",
+                     test_qemu_strtoul_octal);
+     g_test_add_func("/cutils/qemu_strtoul/decimal",
+                     test_qemu_strtoul_decimal);
+     g_test_add_func("/cutils/qemu_strtoul/hex",
+                     test_qemu_strtoul_hex);
+     g_test_add_func("/cutils/qemu_strtoul/max",
+                     test_qemu_strtoul_max);
+     g_test_add_func("/cutils/qemu_strtoul/overflow",
+                     test_qemu_strtoul_overflow);
+     g_test_add_func("/cutils/qemu_strtoul/underflow",
+                     test_qemu_strtoul_underflow);
+     g_test_add_func("/cutils/qemu_strtoul/negative",
+                     test_qemu_strtoul_negative);
+     g_test_add_func("/cutils/qemu_strtoul_full/correct",
+                     test_qemu_strtoul_full_correct);
+     g_test_add_func("/cutils/qemu_strtoul_full/null",
+                     test_qemu_strtoul_full_null);
+     g_test_add_func("/cutils/qemu_strtoul_full/empty",
+                     test_qemu_strtoul_full_empty);
+     g_test_add_func("/cutils/qemu_strtoul_full/negative",
+                     test_qemu_strtoul_full_negative);
+     g_test_add_func("/cutils/qemu_strtoul_full/trailing",
+                     test_qemu_strtoul_full_trailing);
+     g_test_add_func("/cutils/qemu_strtoul_full/max",
+                     test_qemu_strtoul_full_max);
+     /* qemu_strtoi64() tests */
+     g_test_add_func("/cutils/qemu_strtoi64/correct",
+                     test_qemu_strtoi64_correct);
+     g_test_add_func("/cutils/qemu_strtoi64/null",
+                     test_qemu_strtoi64_null);
+     g_test_add_func("/cutils/qemu_strtoi64/empty",
+                     test_qemu_strtoi64_empty);
+     g_test_add_func("/cutils/qemu_strtoi64/whitespace",
+                     test_qemu_strtoi64_whitespace);
+     g_test_add_func("/cutils/qemu_strtoi64/invalid"
+                     ,
+                     test_qemu_strtoi64_invalid);
+     g_test_add_func("/cutils/qemu_strtoi64/trailing",
+                     test_qemu_strtoi64_trailing);
+     g_test_add_func("/cutils/qemu_strtoi64/octal",
+                     test_qemu_strtoi64_octal);
+     g_test_add_func("/cutils/qemu_strtoi64/decimal",
+                     test_qemu_strtoi64_decimal);
+     g_test_add_func("/cutils/qemu_strtoi64/hex",
+                     test_qemu_strtoi64_hex);
+     g_test_add_func("/cutils/qemu_strtoi64/max",
+                     test_qemu_strtoi64_max);
+     g_test_add_func("/cutils/qemu_strtoi64/overflow",
+                     test_qemu_strtoi64_overflow);
+     g_test_add_func("/cutils/qemu_strtoi64/underflow",
+                     test_qemu_strtoi64_underflow);
+     g_test_add_func("/cutils/qemu_strtoi64/negative",
+                     test_qemu_strtoi64_negative);
+     g_test_add_func("/cutils/qemu_strtoi64_full/correct",
+                     test_qemu_strtoi64_full_correct);
+     g_test_add_func("/cutils/qemu_strtoi64_full/null",
+                     test_qemu_strtoi64_full_null);
+     g_test_add_func("/cutils/qemu_strtoi64_full/empty",
+                     test_qemu_strtoi64_full_empty);
+     g_test_add_func("/cutils/qemu_strtoi64_full/negative",
+                     test_qemu_strtoi64_full_negative);
+     g_test_add_func("/cutils/qemu_strtoi64_full/trailing",
+                     test_qemu_strtoi64_full_trailing);
+     g_test_add_func("/cutils/qemu_strtoi64_full/max",
+                     test_qemu_strtoi64_full_max);
+     /* qemu_strtou64() tests */
+     g_test_add_func("/cutils/qemu_strtou64/correct",
+                     test_qemu_strtou64_correct);
+     g_test_add_func("/cutils/qemu_strtou64/null",
+                     test_qemu_strtou64_null);
+     g_test_add_func("/cutils/qemu_strtou64/empty",
+                     test_qemu_strtou64_empty);
+     g_test_add_func("/cutils/qemu_strtou64/whitespace",
+                     test_qemu_strtou64_whitespace);
+     g_test_add_func("/cutils/qemu_strtou64/invalid",
+                     test_qemu_strtou64_invalid);
+     g_test_add_func("/cutils/qemu_strtou64/trailing",
+                     test_qemu_strtou64_trailing);
+     g_test_add_func("/cutils/qemu_strtou64/octal",
+                     test_qemu_strtou64_octal);
+     g_test_add_func("/cutils/qemu_strtou64/decimal",
+                     test_qemu_strtou64_decimal);
+     g_test_add_func("/cutils/qemu_strtou64/hex",
+                     test_qemu_strtou64_hex);
+     g_test_add_func("/cutils/qemu_strtou64/max",
+                     test_qemu_strtou64_max);
+     g_test_add_func("/cutils/qemu_strtou64/overflow",
+                     test_qemu_strtou64_overflow);
+     g_test_add_func("/cutils/qemu_strtou64/underflow",
+                     test_qemu_strtou64_underflow);
+     g_test_add_func("/cutils/qemu_strtou64/negative",
+                     test_qemu_strtou64_negative);
+     g_test_add_func("/cutils/qemu_strtou64_full/correct",
+                     test_qemu_strtou64_full_correct);
+     g_test_add_func("/cutils/qemu_strtou64_full/null",
+                     test_qemu_strtou64_full_null);
+     g_test_add_func("/cutils/qemu_strtou64_full/empty",
+                     test_qemu_strtou64_full_empty);
+     g_test_add_func("/cutils/qemu_strtou64_full/negative",
+                     test_qemu_strtou64_full_negative);
+     g_test_add_func("/cutils/qemu_strtou64_full/trailing",
+                     test_qemu_strtou64_full_trailing);
+     g_test_add_func("/cutils/qemu_strtou64_full/max",
+                     test_qemu_strtou64_full_max);
+     g_test_add_func("/cutils/strtosz/simple",
+                     test_qemu_strtosz_simple);
++    g_test_add_func("/cutils/strtosz/hex",
++                    test_qemu_strtosz_hex);
+     g_test_add_func("/cutils/strtosz/units",
+                     test_qemu_strtosz_units);
+     g_test_add_func("/cutils/strtosz/float",
+                     test_qemu_strtosz_float);
+     g_test_add_func("/cutils/strtosz/invalid",
+                     test_qemu_strtosz_invalid);
+     g_test_add_func("/cutils/strtosz/trailing",
+                     test_qemu_strtosz_trailing);
+     g_test_add_func("/cutils/strtosz/erange",
+                     test_qemu_strtosz_erange);
+     g_test_add_func("/cutils/strtosz/metric",
+                     test_qemu_strtosz_metric);
+     return g_test_run();
+ }
index 0000000000000000000000000000000000000000,ee927fe4e427154f1d14f9f47a16b33788e93861..e20c07cf3ea54855f21132b76cafacb4d3f16cbb
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,750 +1,765 @@@
 -    /* Note: precision is 53 bits since we're parsing with strtod() */
+ /*
+  * Unit tests for parsing of KEY=VALUE,... strings
+  *
+  * Copyright (C) 2017 Red Hat Inc.
+  *
+  * Authors:
+  *  Markus Armbruster <armbru@redhat.com>,
+  *
+  * This work is licensed under the terms of the GNU GPL, version 2 or later.
+  * See the COPYING file in the top-level directory.
+  */
+ #include "qemu/osdep.h"
+ #include "qemu/units.h"
+ #include "qapi/error.h"
+ #include "qapi/qmp/qdict.h"
+ #include "qapi/qmp/qlist.h"
+ #include "qapi/qmp/qstring.h"
+ #include "qapi/qobject-input-visitor.h"
+ #include "test-qapi-visit.h"
+ #include "qemu/cutils.h"
+ #include "qemu/option.h"
+ static void test_keyval_parse(void)
+ {
+     Error *err = NULL;
+     QDict *qdict, *sub_qdict;
+     char long_key[129];
+     char *params;
+     bool help;
+     /* Nothing */
+     qdict = keyval_parse("", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 0);
+     qobject_unref(qdict);
+     /* Empty key (qemu_opts_parse() accepts this) */
+     qdict = keyval_parse("=val", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Empty key fragment */
+     qdict = keyval_parse(".", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     qdict = keyval_parse("key.", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Invalid non-empty key (qemu_opts_parse() doesn't care) */
+     qdict = keyval_parse("7up=val", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Overlong key */
+     memset(long_key, 'a', 127);
+     long_key[127] = 'z';
+     long_key[128] = 0;
+     params = g_strdup_printf("k.%s=v", long_key);
+     qdict = keyval_parse(params + 2, NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Overlong key fragment */
+     qdict = keyval_parse(params, NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     g_free(params);
+     /* Long key (qemu_opts_parse() accepts and truncates silently) */
+     params = g_strdup_printf("k.%s=v", long_key + 1);
+     qdict = keyval_parse(params + 2, NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, long_key + 1), ==, "v");
+     qobject_unref(qdict);
+     /* Long key fragment */
+     qdict = keyval_parse(params, NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     sub_qdict = qdict_get_qdict(qdict, "k");
+     g_assert(sub_qdict);
+     g_assert_cmpuint(qdict_size(sub_qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(sub_qdict, long_key + 1), ==, "v");
+     qobject_unref(qdict);
+     g_free(params);
+     /* Crap after valid key */
+     qdict = keyval_parse("key[0]=val", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Multiple keys, last one wins */
+     qdict = keyval_parse("a=1,b=2,,x,a=3", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 2);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "a"), ==, "3");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "b"), ==, "2,x");
+     qobject_unref(qdict);
+     /* Even when it doesn't in qemu_opts_parse() */
+     qdict = keyval_parse("id=foo,id=bar", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "id"), ==, "bar");
+     qobject_unref(qdict);
+     /* Dotted keys */
+     qdict = keyval_parse("a.b.c=1,a.b.c=2,d=3", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 2);
+     sub_qdict = qdict_get_qdict(qdict, "a");
+     g_assert(sub_qdict);
+     g_assert_cmpuint(qdict_size(sub_qdict), ==, 1);
+     sub_qdict = qdict_get_qdict(sub_qdict, "b");
+     g_assert(sub_qdict);
+     g_assert_cmpuint(qdict_size(sub_qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(sub_qdict, "c"), ==, "2");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "d"), ==, "3");
+     qobject_unref(qdict);
+     /* Inconsistent dotted keys */
+     qdict = keyval_parse("a.b=1,a=2", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     qdict = keyval_parse("a.b=1,a.b.c=2", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Trailing comma is ignored */
+     qdict = keyval_parse("x=y,", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "x"), ==, "y");
+     qobject_unref(qdict);
+     /* Except when it isn't */
+     qdict = keyval_parse(",", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Value containing ,id= not misinterpreted as qemu_opts_parse() does */
+     qdict = keyval_parse("x=,,id=bar", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "x"), ==, ",id=bar");
+     qobject_unref(qdict);
+     /* Anti-social ID is left to caller (qemu_opts_parse() rejects it) */
+     qdict = keyval_parse("id=666", NULL, NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "id"), ==, "666");
+     qobject_unref(qdict);
+     /* Implied value not supported (unlike qemu_opts_parse()) */
+     qdict = keyval_parse("an,noaus,noaus=", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Implied value, key "no" (qemu_opts_parse(): negated empty key) */
+     qdict = keyval_parse("no", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Implied key */
+     qdict = keyval_parse("an,aus=off,noaus=", "implied", NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 3);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "implied"), ==, "an");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "aus"), ==, "off");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "noaus"), ==, "");
+     qobject_unref(qdict);
+     /* Implied dotted key */
+     qdict = keyval_parse("val", "eins.zwei", NULL, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     sub_qdict = qdict_get_qdict(qdict, "eins");
+     g_assert(sub_qdict);
+     g_assert_cmpuint(qdict_size(sub_qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(sub_qdict, "zwei"), ==, "val");
+     qobject_unref(qdict);
+     /* Implied key with empty value (qemu_opts_parse() accepts this) */
+     qdict = keyval_parse(",", "implied", NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Likewise (qemu_opts_parse(): implied key with comma value) */
+     qdict = keyval_parse(",,,a=1", "implied", NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Implied key's value can't have comma (qemu_opts_parse(): it can) */
+     qdict = keyval_parse("val,,ue", "implied", NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Empty key is not an implied key */
+     qdict = keyval_parse("=val", "implied", NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* "help" by itself, without implied key */
+     qdict = keyval_parse("help", NULL, &help, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 0);
+     g_assert(help);
+     qobject_unref(qdict);
+     /* "help" by itself, with implied key */
+     qdict = keyval_parse("help", "implied", &help, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 0);
+     g_assert(help);
+     qobject_unref(qdict);
+     /* "help" when no help is available, without implied key */
+     qdict = keyval_parse("help", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* "help" when no help is available, with implied key */
+     qdict = keyval_parse("help", "implied", NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Key "help" */
+     qdict = keyval_parse("help=on", NULL, &help, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "help"), ==, "on");
+     g_assert(!help);
+     qobject_unref(qdict);
+     /* "help" followed by crap, without implied key */
+     qdict = keyval_parse("help.abc", NULL, &help, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* "help" followed by crap, with implied key */
+     qdict = keyval_parse("help.abc", "implied", &help, &err);
+     g_assert_cmpuint(qdict_size(qdict), ==, 1);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "implied"), ==, "help.abc");
+     g_assert(!help);
+     qobject_unref(qdict);
+     /* "help" with other stuff, without implied key */
+     qdict = keyval_parse("number=42,help,foo=bar", NULL, &help, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 2);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "number"), ==, "42");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "foo"), ==, "bar");
+     g_assert(help);
+     qobject_unref(qdict);
+     /* "help" with other stuff, with implied key */
+     qdict = keyval_parse("val,help,foo=bar", "implied", &help, &error_abort);
+     g_assert_cmpuint(qdict_size(qdict), ==, 2);
+     g_assert_cmpstr(qdict_get_try_str(qdict, "implied"), ==, "val");
+     g_assert_cmpstr(qdict_get_try_str(qdict, "foo"), ==, "bar");
+     g_assert(help);
+     qobject_unref(qdict);
+ }
+ static void check_list012(QList *qlist)
+ {
+     static const char *expected[] = { "null", "eins", "zwei" };
+     int i;
+     QString *qstr;
+     g_assert(qlist);
+     for (i = 0; i < ARRAY_SIZE(expected); i++) {
+         qstr = qobject_to(QString, qlist_pop(qlist));
+         g_assert(qstr);
+         g_assert_cmpstr(qstring_get_str(qstr), ==, expected[i]);
+         qobject_unref(qstr);
+     }
+     g_assert(qlist_empty(qlist));
+ }
+ static void test_keyval_parse_list(void)
+ {
+     Error *err = NULL;
+     QDict *qdict, *sub_qdict;
+     /* Root can't be a list */
+     qdict = keyval_parse("0=1", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* List elements need not be in order */
+     qdict = keyval_parse("list.0=null,list.2=zwei,list.1=eins", NULL, NULL,
+                          &error_abort);
+     g_assert_cmpint(qdict_size(qdict), ==, 1);
+     check_list012(qdict_get_qlist(qdict, "list"));
+     qobject_unref(qdict);
+     /* Multiple indexes, last one wins */
+     qdict = keyval_parse("list.1=goner,list.0=null,list.01=eins,list.2=zwei",
+                          NULL, NULL, &error_abort);
+     g_assert_cmpint(qdict_size(qdict), ==, 1);
+     check_list012(qdict_get_qlist(qdict, "list"));
+     qobject_unref(qdict);
+     /* List at deeper nesting */
+     qdict = keyval_parse("a.list.1=eins,a.list.00=null,a.list.2=zwei", NULL,
+                          NULL, &error_abort);
+     g_assert_cmpint(qdict_size(qdict), ==, 1);
+     sub_qdict = qdict_get_qdict(qdict, "a");
+     g_assert_cmpint(qdict_size(sub_qdict), ==, 1);
+     check_list012(qdict_get_qlist(sub_qdict, "list"));
+     qobject_unref(qdict);
+     /* Inconsistent dotted keys: both list and dictionary */
+     qdict = keyval_parse("a.b.c=1,a.b.0=2", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     qdict = keyval_parse("a.0.c=1,a.b.c=2", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     /* Missing list indexes */
+     qdict = keyval_parse("list.1=lonely", NULL, NULL, &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+     qdict = keyval_parse("list.0=null,list.2=eins,list.02=zwei", NULL, NULL,
+                          &err);
+     error_free_or_abort(&err);
+     g_assert(!qdict);
+ }
+ static void test_keyval_visit_bool(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     bool b;
+     qdict = keyval_parse("bool1=on,bool2=off", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_bool(v, "bool1", &b, &error_abort);
+     g_assert(b);
+     visit_type_bool(v, "bool2", &b, &error_abort);
+     g_assert(!b);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     qdict = keyval_parse("bool1=offer", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_bool(v, "bool1", &b, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_number(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     uint64_t u;
+     /* Lower limit zero */
+     qdict = keyval_parse("number1=0", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &error_abort);
+     g_assert_cmpuint(u, ==, 0);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Upper limit 2^64-1 */
+     qdict = keyval_parse("number1=18446744073709551615,number2=-1", NULL,
+                          NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &error_abort);
+     g_assert_cmphex(u, ==, UINT64_MAX);
+     visit_type_uint64(v, "number2", &u, &error_abort);
+     g_assert_cmphex(u, ==, UINT64_MAX);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Above upper limit */
+     qdict = keyval_parse("number1=18446744073709551616", NULL, NULL,
+                          &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Below lower limit */
+     qdict = keyval_parse("number1=-18446744073709551616", NULL, NULL,
+                          &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Hex and octal */
+     qdict = keyval_parse("number1=0x2a,number2=052", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &error_abort);
+     g_assert_cmpuint(u, ==, 42);
+     visit_type_uint64(v, "number2", &u, &error_abort);
+     g_assert_cmpuint(u, ==, 42);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Trailing crap */
+     qdict = keyval_parse("number1=3.14,number2=08", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_uint64(v, "number1", &u, &err);
+     error_free_or_abort(&err);
+     visit_type_uint64(v, "number2", &u, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_size(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     uint64_t sz;
+     /* Lower limit zero */
+     qdict = keyval_parse("sz1=0", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &error_abort);
+     g_assert_cmpuint(sz, ==, 0);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
 -    /* Around limit of precision: 2^53-1, 2^53, 2^53+1 */
++    /* Note: full 64 bits of precision */
 -    g_assert_cmphex(sz, ==, 0x20000000000000);
++    /* Around double limit of precision: 2^53-1, 2^53, 2^53+1 */
+     qdict = keyval_parse("sz1=9007199254740991,"
+                          "sz2=9007199254740992,"
+                          "sz3=9007199254740993",
+                          NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, 0x1fffffffffffff);
+     visit_type_size(v, "sz2", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, 0x20000000000000);
+     visit_type_size(v, "sz3", &sz, &error_abort);
 -    /* Close to signed upper limit 0x7ffffffffffffc00 (53 msbs set) */
 -    qdict = keyval_parse("sz1=9223372036854774784," /* 7ffffffffffffc00 */
 -                         "sz2=9223372036854775295", /* 7ffffffffffffdff */
++    g_assert_cmphex(sz, ==, 0x20000000000001);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
 -    g_assert_cmphex(sz, ==, 0x7ffffffffffffc00);
++    /* Close to signed integer limit 2^63 */
++    qdict = keyval_parse("sz1=9223372036854775807," /* 7fffffffffffffff */
++                         "sz2=9223372036854775808," /* 8000000000000000 */
++                         "sz3=9223372036854775809", /* 8000000000000001 */
+                          NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &error_abort);
 -    g_assert_cmphex(sz, ==, 0x7ffffffffffffc00);
++    g_assert_cmphex(sz, ==, 0x7fffffffffffffff);
+     visit_type_size(v, "sz2", &sz, &error_abort);
 -    g_assert_cmphex(sz, ==, 0xfffffffffffff800);
++    g_assert_cmphex(sz, ==, 0x8000000000000000);
++    visit_type_size(v, "sz3", &sz, &error_abort);
++    g_assert_cmphex(sz, ==, 0x8000000000000001);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Close to actual upper limit 0xfffffffffffff800 (53 msbs set) */
+     qdict = keyval_parse("sz1=18446744073709549568," /* fffffffffffff800 */
+                          "sz2=18446744073709550591", /* fffffffffffffbff */
+                          NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, 0xfffffffffffff800);
+     visit_type_size(v, "sz2", &sz, &error_abort);
 -                         "sz2=18446744073709550592", /* fffffffffffffc00 */
++    g_assert_cmphex(sz, ==, 0xfffffffffffffbff);
++    visit_check_struct(v, &error_abort);
++    visit_end_struct(v, NULL);
++    visit_free(v);
++
++    /* Actual limit 2^64-1*/
++    qdict = keyval_parse("sz1=18446744073709551615", /* ffffffffffffffff */
++                         NULL, NULL, &error_abort);
++    v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
++    qobject_unref(qdict);
++    visit_start_struct(v, NULL, NULL, 0, &error_abort);
++    visit_type_size(v, "sz1", &sz, &error_abort);
++    g_assert_cmphex(sz, ==, 0xffffffffffffffff);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Beyond limits */
+     qdict = keyval_parse("sz1=-1,"
++                         "sz2=18446744073709551616", /* 2^64 */
+                          NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &err);
+     error_free_or_abort(&err);
+     visit_type_size(v, "sz2", &sz, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Suffixes */
+     qdict = keyval_parse("sz1=8b,sz2=1.5k,sz3=2M,sz4=0.1G,sz5=16777215T",
+                          NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &error_abort);
+     g_assert_cmpuint(sz, ==, 8);
+     visit_type_size(v, "sz2", &sz, &error_abort);
+     g_assert_cmpuint(sz, ==, 1536);
+     visit_type_size(v, "sz3", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, 2 * MiB);
+     visit_type_size(v, "sz4", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, GiB / 10);
+     visit_type_size(v, "sz5", &sz, &error_abort);
+     g_assert_cmphex(sz, ==, 16777215ULL * TiB);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Beyond limit with suffix */
+     qdict = keyval_parse("sz1=16777216T", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     /* Trailing crap */
+     qdict = keyval_parse("sz1=0Z,sz2=16Gi", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_size(v, "sz1", &sz, &err);
+     error_free_or_abort(&err);
+     visit_type_size(v, "sz2", &sz, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_dict(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     int64_t i;
+     qdict = keyval_parse("a.b.c=1,a.b.c=2,d=3", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_start_struct(v, "a", NULL, 0, &error_abort);
+     visit_start_struct(v, "b", NULL, 0, &error_abort);
+     visit_type_int(v, "c", &i, &error_abort);
+     g_assert_cmpint(i, ==, 2);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_type_int(v, "d", &i, &error_abort);
+     g_assert_cmpint(i, ==, 3);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     qdict = keyval_parse("a.b=", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_start_struct(v, "a", NULL, 0, &error_abort);
+     visit_type_int(v, "c", &i, &err);   /* a.c missing */
+     error_free_or_abort(&err);
+     visit_check_struct(v, &err);
+     error_free_or_abort(&err);          /* a.b unexpected */
+     visit_end_struct(v, NULL);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_list(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     char *s;
+     qdict = keyval_parse("a.0=,a.1=I,a.2.0=II", NULL, NULL, &error_abort);
+     /* TODO empty list */
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_start_list(v, "a", NULL, 0, &error_abort);
+     visit_type_str(v, NULL, &s, &error_abort);
+     g_assert_cmpstr(s, ==, "");
+     g_free(s);
+     visit_type_str(v, NULL, &s, &error_abort);
+     g_assert_cmpstr(s, ==, "I");
+     g_free(s);
+     visit_start_list(v, NULL, NULL, 0, &error_abort);
+     visit_type_str(v, NULL, &s, &error_abort);
+     g_assert_cmpstr(s, ==, "II");
+     g_free(s);
+     visit_check_list(v, &error_abort);
+     visit_end_list(v, NULL);
+     visit_check_list(v, &error_abort);
+     visit_end_list(v, NULL);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+     qdict = keyval_parse("a.0=,b.0.0=head", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_start_list(v, "a", NULL, 0, &error_abort);
+     visit_check_list(v, &err);  /* a[0] unexpected */
+     error_free_or_abort(&err);
+     visit_end_list(v, NULL);
+     visit_start_list(v, "b", NULL, 0, &error_abort);
+     visit_start_list(v, NULL, NULL, 0, &error_abort);
+     visit_type_str(v, NULL, &s, &error_abort);
+     g_assert_cmpstr(s, ==, "head");
+     g_free(s);
+     visit_type_str(v, NULL, &s, &err); /* b[0][1] missing */
+     error_free_or_abort(&err);
+     visit_end_list(v, NULL);
+     visit_end_list(v, NULL);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_optional(void)
+ {
+     Visitor *v;
+     QDict *qdict;
+     bool present;
+     int64_t i;
+     qdict = keyval_parse("a.b=1", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_optional(v, "b", &present);
+     g_assert(!present);         /* b missing */
+     visit_optional(v, "a", &present);
+     g_assert(present);          /* a present */
+     visit_start_struct(v, "a", NULL, 0, &error_abort);
+     visit_optional(v, "b", &present);
+     g_assert(present);          /* a.b present */
+     visit_type_int(v, "b", &i, &error_abort);
+     g_assert_cmpint(i, ==, 1);
+     visit_optional(v, "a", &present);
+     g_assert(!present);         /* a.a missing */
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_alternate(void)
+ {
+     Error *err = NULL;
+     Visitor *v;
+     QDict *qdict;
+     AltStrObj *aso;
+     AltNumEnum *ane;
+     AltEnumBool *aeb;
+     /*
+      * Can't do scalar alternate variants other than string.  You get
+      * the string variant if there is one, else an error.
+      * TODO make it work for unambiguous cases like AltEnumBool below
+      */
+     qdict = keyval_parse("a=1,b=2,c=on", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_AltStrObj(v, "a", &aso, &error_abort);
+     g_assert_cmpint(aso->type, ==, QTYPE_QSTRING);
+     g_assert_cmpstr(aso->u.s, ==, "1");
+     qapi_free_AltStrObj(aso);
+     visit_type_AltNumEnum(v, "b", &ane, &err);
+     error_free_or_abort(&err);
+     visit_type_AltEnumBool(v, "c", &aeb, &err);
+     error_free_or_abort(&err);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ static void test_keyval_visit_any(void)
+ {
+     Visitor *v;
+     QDict *qdict;
+     QObject *any;
+     QList *qlist;
+     QString *qstr;
+     qdict = keyval_parse("a.0=null,a.1=1", NULL, NULL, &error_abort);
+     v = qobject_input_visitor_new_keyval(QOBJECT(qdict));
+     qobject_unref(qdict);
+     visit_start_struct(v, NULL, NULL, 0, &error_abort);
+     visit_type_any(v, "a", &any, &error_abort);
+     qlist = qobject_to(QList, any);
+     g_assert(qlist);
+     qstr = qobject_to(QString, qlist_pop(qlist));
+     g_assert_cmpstr(qstring_get_str(qstr), ==, "null");
+     qobject_unref(qstr);
+     qstr = qobject_to(QString, qlist_pop(qlist));
+     g_assert_cmpstr(qstring_get_str(qstr), ==, "1");
+     g_assert(qlist_empty(qlist));
+     qobject_unref(qstr);
+     qobject_unref(any);
+     visit_check_struct(v, &error_abort);
+     visit_end_struct(v, NULL);
+     visit_free(v);
+ }
+ int main(int argc, char *argv[])
+ {
+     g_test_init(&argc, &argv, NULL);
+     g_test_add_func("/keyval/keyval_parse", test_keyval_parse);
+     g_test_add_func("/keyval/keyval_parse/list", test_keyval_parse_list);
+     g_test_add_func("/keyval/visit/bool", test_keyval_visit_bool);
+     g_test_add_func("/keyval/visit/number", test_keyval_visit_number);
+     g_test_add_func("/keyval/visit/size", test_keyval_visit_size);
+     g_test_add_func("/keyval/visit/dict", test_keyval_visit_dict);
+     g_test_add_func("/keyval/visit/list", test_keyval_visit_list);
+     g_test_add_func("/keyval/visit/optional", test_keyval_visit_optional);
+     g_test_add_func("/keyval/visit/alternate", test_keyval_visit_alternate);
+     g_test_add_func("/keyval/visit/any", test_keyval_visit_any);
+     g_test_run();
+     return 0;
+ }
index 0000000000000000000000000000000000000000,8bbb17b1c726c9783d923b41dce975461c948700..6568e31a72290f03416a6a4416d07d5e7e936fdd
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,1046 +1,1057 @@@
 -    /* Note: precision is 53 bits since we're parsing with strtod() */
+ /*
+  * QemuOpts unit-tests.
+  *
+  * Copyright (C) 2014 Leandro Dorileo <l@dorileo.org>
+  *
+  * This work is licensed under the terms of the GNU LGPL, version 2.1 or later.
+  * See the COPYING.LIB file in the top-level directory.
+  */
+ #include "qemu/osdep.h"
+ #include "qemu/units.h"
+ #include "qemu/option.h"
+ #include "qemu/option_int.h"
+ #include "qapi/error.h"
+ #include "qapi/qmp/qdict.h"
+ #include "qapi/qmp/qstring.h"
+ #include "qemu/config-file.h"
+ static QemuOptsList opts_list_01 = {
+     .name = "opts_list_01",
+     .head = QTAILQ_HEAD_INITIALIZER(opts_list_01.head),
+     .desc = {
+         {
+             .name = "str1",
+             .type = QEMU_OPT_STRING,
+             .help = "Help texts are preserved in qemu_opts_append",
+             .def_value_str = "default",
+         },{
+             .name = "str2",
+             .type = QEMU_OPT_STRING,
+         },{
+             .name = "str3",
+             .type = QEMU_OPT_STRING,
+         },{
+             .name = "number1",
+             .type = QEMU_OPT_NUMBER,
+             .help = "Having help texts only for some options is okay",
+         },{
+             .name = "number2",
+             .type = QEMU_OPT_NUMBER,
+         },
+         { /* end of list */ }
+     },
+ };
+ static QemuOptsList opts_list_02 = {
+     .name = "opts_list_02",
+     .head = QTAILQ_HEAD_INITIALIZER(opts_list_02.head),
+     .desc = {
+         {
+             .name = "str1",
+             .type = QEMU_OPT_STRING,
+         },{
+             .name = "str2",
+             .type = QEMU_OPT_STRING,
+         },{
+             .name = "bool1",
+             .type = QEMU_OPT_BOOL,
+         },{
+             .name = "bool2",
+             .type = QEMU_OPT_BOOL,
+         },{
+             .name = "size1",
+             .type = QEMU_OPT_SIZE,
+         },{
+             .name = "size2",
+             .type = QEMU_OPT_SIZE,
+         },{
+             .name = "size3",
+             .type = QEMU_OPT_SIZE,
+         },
+         { /* end of list */ }
+     },
+ };
+ static QemuOptsList opts_list_03 = {
+     .name = "opts_list_03",
+     .implied_opt_name = "implied",
+     .head = QTAILQ_HEAD_INITIALIZER(opts_list_03.head),
+     .desc = {
+         /* no elements => accept any params */
+         { /* end of list */ }
+     },
+ };
+ static QemuOptsList opts_list_04 = {
+     .name = "opts_list_04",
+     .head = QTAILQ_HEAD_INITIALIZER(opts_list_04.head),
+     .merge_lists = true,
+     .desc = {
+         {
+             .name = "str3",
+             .type = QEMU_OPT_STRING,
+         },
+         { /* end of list */ }
+     },
+ };
+ static void register_opts(void)
+ {
+     qemu_add_opts(&opts_list_01);
+     qemu_add_opts(&opts_list_02);
+     qemu_add_opts(&opts_list_03);
+     qemu_add_opts(&opts_list_04);
+ }
+ static void test_find_unknown_opts(void)
+ {
+     QemuOptsList *list;
+     Error *err = NULL;
+     /* should not return anything, we don't have an "unknown" option */
+     list = qemu_find_opts_err("unknown", &err);
+     g_assert(list == NULL);
+     error_free_or_abort(&err);
+ }
+ static void test_qemu_find_opts(void)
+ {
+     QemuOptsList *list;
+     /* we have an "opts_list_01" option, should return it */
+     list = qemu_find_opts("opts_list_01");
+     g_assert(list != NULL);
+     g_assert_cmpstr(list->name, ==, "opts_list_01");
+ }
+ static void test_qemu_opts_create(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     list = qemu_find_opts("opts_list_01");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_01");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* now we've create the opts, must find it */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts != NULL);
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opt_get(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     const char *opt = NULL;
+     list = qemu_find_opts("opts_list_01");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_01");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* haven't set anything to str2 yet */
+     opt = qemu_opt_get(opts, "str2");
+     g_assert(opt == NULL);
+     qemu_opt_set(opts, "str2", "value", &error_abort);
+     /* now we have set str2, should know about it */
+     opt = qemu_opt_get(opts, "str2");
+     g_assert_cmpstr(opt, ==, "value");
+     qemu_opt_set(opts, "str2", "value2", &error_abort);
+     /* having reset the value, the returned should be the reset one */
+     opt = qemu_opt_get(opts, "str2");
+     g_assert_cmpstr(opt, ==, "value2");
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opt_get_bool(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     bool opt;
+     list = qemu_find_opts("opts_list_02");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_02");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* haven't set anything to bool1 yet, so defval should be returned */
+     opt = qemu_opt_get_bool(opts, "bool1", false);
+     g_assert(opt == false);
+     qemu_opt_set_bool(opts, "bool1", true, &error_abort);
+     /* now we have set bool1, should know about it */
+     opt = qemu_opt_get_bool(opts, "bool1", false);
+     g_assert(opt == true);
+     /* having reset the value, opt should be the reset one not defval */
+     qemu_opt_set_bool(opts, "bool1", false, &error_abort);
+     opt = qemu_opt_get_bool(opts, "bool1", true);
+     g_assert(opt == false);
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opt_get_number(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     uint64_t opt;
+     list = qemu_find_opts("opts_list_01");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_01");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* haven't set anything to number1 yet, so defval should be returned */
+     opt = qemu_opt_get_number(opts, "number1", 5);
+     g_assert(opt == 5);
+     qemu_opt_set_number(opts, "number1", 10, &error_abort);
+     /* now we have set number1, should know about it */
+     opt = qemu_opt_get_number(opts, "number1", 5);
+     g_assert(opt == 10);
+     /* having reset it, the returned should be the reset one not defval */
+     qemu_opt_set_number(opts, "number1", 15, &error_abort);
+     opt = qemu_opt_get_number(opts, "number1", 5);
+     g_assert(opt == 15);
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opt_get_size(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     uint64_t opt;
+     QDict *dict;
+     list = qemu_find_opts("opts_list_02");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_02");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* haven't set anything to size1 yet, so defval should be returned */
+     opt = qemu_opt_get_size(opts, "size1", 5);
+     g_assert(opt == 5);
+     dict = qdict_new();
+     g_assert(dict != NULL);
+     qdict_put_str(dict, "size1", "10");
+     qemu_opts_absorb_qdict(opts, dict, &error_abort);
+     g_assert(error_abort == NULL);
+     /* now we have set size1, should know about it */
+     opt = qemu_opt_get_size(opts, "size1", 5);
+     g_assert(opt == 10);
+     /* reset value */
+     qdict_put_str(dict, "size1", "15");
+     qemu_opts_absorb_qdict(opts, dict, &error_abort);
+     g_assert(error_abort == NULL);
+     /* test the reset value */
+     opt = qemu_opt_get_size(opts, "size1", 5);
+     g_assert(opt == 15);
+     qdict_del(dict, "size1");
+     g_free(dict);
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opt_unset(void)
+ {
+     QemuOpts *opts;
+     const char *value;
+     int ret;
+     /* dynamically initialized (parsed) opts */
+     opts = qemu_opts_parse(&opts_list_03, "key=value", false, NULL);
+     g_assert(opts != NULL);
+     /* check default/parsed value */
+     value = qemu_opt_get(opts, "key");
+     g_assert_cmpstr(value, ==, "value");
+     /* reset it to value2 */
+     qemu_opt_set(opts, "key", "value2", &error_abort);
+     value = qemu_opt_get(opts, "key");
+     g_assert_cmpstr(value, ==, "value2");
+     /* unset, valid only for "accept any" */
+     ret = qemu_opt_unset(opts, "key");
+     g_assert(ret == 0);
+     /* after reset the value should be the parsed/default one */
+     value = qemu_opt_get(opts, "key");
+     g_assert_cmpstr(value, ==, "value");
+     qemu_opts_del(opts);
+ }
+ static void test_qemu_opts_reset(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     uint64_t opt;
+     list = qemu_find_opts("opts_list_01");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_01");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* create the opts */
+     opts = qemu_opts_create(list, NULL, 0, &error_abort);
+     g_assert(opts != NULL);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* haven't set anything to number1 yet, so defval should be returned */
+     opt = qemu_opt_get_number(opts, "number1", 5);
+     g_assert(opt == 5);
+     qemu_opt_set_number(opts, "number1", 10, &error_abort);
+     /* now we have set number1, should know about it */
+     opt = qemu_opt_get_number(opts, "number1", 5);
+     g_assert(opt == 10);
+     qemu_opts_reset(list);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static void test_qemu_opts_set(void)
+ {
+     QemuOptsList *list;
+     QemuOpts *opts;
+     const char *opt;
+     list = qemu_find_opts("opts_list_04");
+     g_assert(list != NULL);
+     g_assert(QTAILQ_EMPTY(&list->head));
+     g_assert_cmpstr(list->name, ==, "opts_list_04");
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+     /* implicitly create opts and set str3 value */
+     qemu_opts_set(list, "str3", "value", &error_abort);
+     g_assert(!QTAILQ_EMPTY(&list->head));
+     /* get the just created opts */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts != NULL);
+     /* check the str3 value */
+     opt = qemu_opt_get(opts, "str3");
+     g_assert_cmpstr(opt, ==, "value");
+     qemu_opts_del(opts);
+     /* should not find anything at this point */
+     opts = qemu_opts_find(list, NULL);
+     g_assert(opts == NULL);
+ }
+ static int opts_count_iter(void *opaque, const char *name, const char *value,
+                            Error **errp)
+ {
+     (*(size_t *)opaque)++;
+     return 0;
+ }
+ static size_t opts_count(QemuOpts *opts)
+ {
+     size_t n = 0;
+     qemu_opt_foreach(opts, opts_count_iter, &n, NULL);
+     return n;
+ }
+ static void test_opts_parse(void)
+ {
+     Error *err = NULL;
+     QemuOpts *opts;
+     /* Nothing */
+     opts = qemu_opts_parse(&opts_list_03, "", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 0);
+     /* Empty key */
+     opts = qemu_opts_parse(&opts_list_03, "=val", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, ""), ==, "val");
+     /* Multiple keys, last one wins */
+     opts = qemu_opts_parse(&opts_list_03, "a=1,b=2,,x,a=3",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmpstr(qemu_opt_get(opts, "a"), ==, "3");
+     g_assert_cmpstr(qemu_opt_get(opts, "b"), ==, "2,x");
+     /* Except when it doesn't */
+     opts = qemu_opts_parse(&opts_list_03, "id=foo,id=bar",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 0);
+     g_assert_cmpstr(qemu_opts_id(opts), ==, "foo");
+     /* TODO Cover low-level access to repeated keys */
+     /* Trailing comma is ignored */
+     opts = qemu_opts_parse(&opts_list_03, "x=y,", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, "x"), ==, "y");
+     /* Except when it isn't */
+     opts = qemu_opts_parse(&opts_list_03, ",", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, ""), ==, "on");
+     /* Duplicate ID */
+     opts = qemu_opts_parse(&opts_list_03, "x=y,id=foo", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* TODO Cover .merge_lists = true */
+     /* Buggy ID recognition (fixed) */
+     opts = qemu_opts_parse(&opts_list_03, "x=,,id=bar", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert(!qemu_opts_id(opts));
+     g_assert_cmpstr(qemu_opt_get(opts, "x"), ==, ",id=bar");
+     /* Anti-social ID */
+     opts = qemu_opts_parse(&opts_list_01, "id=666", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Implied value (qemu_opts_parse warns but accepts it) */
+     opts = qemu_opts_parse(&opts_list_03, "an,noaus,noaus=",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmpstr(qemu_opt_get(opts, "an"), ==, "on");
+     g_assert_cmpstr(qemu_opt_get(opts, "aus"), ==, "off");
+     g_assert_cmpstr(qemu_opt_get(opts, "noaus"), ==, "");
+     /* Implied value, negated empty key */
+     opts = qemu_opts_parse(&opts_list_03, "no", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, ""), ==, "off");
+     /* Implied key */
+     opts = qemu_opts_parse(&opts_list_03, "an,noaus,noaus=", true,
+                            &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmpstr(qemu_opt_get(opts, "implied"), ==, "an");
+     g_assert_cmpstr(qemu_opt_get(opts, "aus"), ==, "off");
+     g_assert_cmpstr(qemu_opt_get(opts, "noaus"), ==, "");
+     /* Implied key with empty value */
+     opts = qemu_opts_parse(&opts_list_03, ",", true, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, "implied"), ==, "");
+     /* Implied key with comma value */
+     opts = qemu_opts_parse(&opts_list_03, ",,,a=1", true, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert_cmpstr(qemu_opt_get(opts, "implied"), ==, ",");
+     g_assert_cmpstr(qemu_opt_get(opts, "a"), ==, "1");
+     /* Empty key is not an implied key */
+     opts = qemu_opts_parse(&opts_list_03, "=val", true, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpstr(qemu_opt_get(opts, ""), ==, "val");
+     /* Unknown key */
+     opts = qemu_opts_parse(&opts_list_01, "nonexistent=", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     qemu_opts_reset(&opts_list_01);
+     qemu_opts_reset(&opts_list_03);
+ }
+ static void test_opts_parse_bool(void)
+ {
+     Error *err = NULL;
+     QemuOpts *opts;
+     opts = qemu_opts_parse(&opts_list_02, "bool1=on,bool2=off",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert(qemu_opt_get_bool(opts, "bool1", false));
+     g_assert(!qemu_opt_get_bool(opts, "bool2", true));
+     opts = qemu_opts_parse(&opts_list_02, "bool1=offer", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     qemu_opts_reset(&opts_list_02);
+ }
+ static void test_opts_parse_number(void)
+ {
+     Error *err = NULL;
+     QemuOpts *opts;
+     /* Lower limit zero */
+     opts = qemu_opts_parse(&opts_list_01, "number1=0", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpuint(qemu_opt_get_number(opts, "number1", 1), ==, 0);
+     /* Upper limit 2^64-1 */
+     opts = qemu_opts_parse(&opts_list_01,
+                            "number1=18446744073709551615,number2=-1",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert_cmphex(qemu_opt_get_number(opts, "number1", 1), ==, UINT64_MAX);
+     g_assert_cmphex(qemu_opt_get_number(opts, "number2", 0), ==, UINT64_MAX);
+     /* Above upper limit */
+     opts = qemu_opts_parse(&opts_list_01, "number1=18446744073709551616",
+                            false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Below lower limit */
+     opts = qemu_opts_parse(&opts_list_01, "number1=-18446744073709551616",
+                            false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Hex and octal */
+     opts = qemu_opts_parse(&opts_list_01, "number1=0x2a,number2=052",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert_cmpuint(qemu_opt_get_number(opts, "number1", 1), ==, 42);
+     g_assert_cmpuint(qemu_opt_get_number(opts, "number2", 0), ==, 42);
+     /* Invalid */
+     opts = qemu_opts_parse(&opts_list_01, "number1=", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     opts = qemu_opts_parse(&opts_list_01, "number1=eins", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Leading whitespace */
+     opts = qemu_opts_parse(&opts_list_01, "number1= \t42",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpuint(qemu_opt_get_number(opts, "number1", 1), ==, 42);
+     /* Trailing crap */
+     opts = qemu_opts_parse(&opts_list_01, "number1=3.14", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     opts = qemu_opts_parse(&opts_list_01, "number1=08", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     opts = qemu_opts_parse(&opts_list_01, "number1=0 ", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     qemu_opts_reset(&opts_list_01);
+ }
+ static void test_opts_parse_size(void)
+ {
+     Error *err = NULL;
+     QemuOpts *opts;
+     /* Lower limit zero */
+     opts = qemu_opts_parse(&opts_list_02, "size1=0", false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 1);
+     g_assert_cmpuint(qemu_opt_get_size(opts, "size1", 1), ==, 0);
 -    /* Around limit of precision: 2^53-1, 2^53, 2^54 */
++    /* Note: full 64 bits of precision */
 -                     ==, 0x20000000000000);
++    /* Around double limit of precision: 2^53-1, 2^53, 2^53+1 */
+     opts = qemu_opts_parse(&opts_list_02,
+                            "size1=9007199254740991,"
+                            "size2=9007199254740992,"
+                            "size3=9007199254740993",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size1", 1),
+                      ==, 0x1fffffffffffff);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size2", 1),
+                      ==, 0x20000000000000);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size3", 1),
 -    /* Close to signed upper limit 0x7ffffffffffffc00 (53 msbs set) */
++                     ==, 0x20000000000001);
 -                           "size1=9223372036854774784," /* 7ffffffffffffc00 */
 -                           "size2=9223372036854775295", /* 7ffffffffffffdff */
++    /* Close to signed int limit: 2^63-1, 2^63, 2^63+1 */
+     opts = qemu_opts_parse(&opts_list_02,
 -    g_assert_cmpuint(opts_count(opts), ==, 2);
++                           "size1=9223372036854775807," /* 7fffffffffffffff */
++                           "size2=9223372036854775808," /* 8000000000000000 */
++                           "size3=9223372036854775809", /* 8000000000000001 */
+                            false, &error_abort);
 -                     ==, 0x7ffffffffffffc00);
++    g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size1", 1),
 -                     ==, 0x7ffffffffffffc00);
++                     ==, 0x7fffffffffffffff);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size2", 1),
 -                     ==, 0xfffffffffffff800);
++                     ==, 0x8000000000000000);
++    g_assert_cmphex(qemu_opt_get_size(opts, "size3", 1),
++                     ==, 0x8000000000000001);
+     /* Close to actual upper limit 0xfffffffffffff800 (53 msbs set) */
+     opts = qemu_opts_parse(&opts_list_02,
+                            "size1=18446744073709549568," /* fffffffffffff800 */
+                            "size2=18446744073709550591", /* fffffffffffffbff */
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size1", 1),
+                      ==, 0xfffffffffffff800);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size2", 1),
 -                           "size1=18446744073709550592", /* fffffffffffffc00 */
++                     ==, 0xfffffffffffffbff);
++
++    /* Actual limit, 2^64-1 */
++    opts = qemu_opts_parse(&opts_list_02,
++                           "size1=18446744073709551615", /* ffffffffffffffff */
++                           false, &error_abort);
++    g_assert_cmpuint(opts_count(opts), ==, 1);
++    g_assert_cmphex(qemu_opt_get_size(opts, "size1", 1),
++                     ==, 0xffffffffffffffff);
+     /* Beyond limits */
+     opts = qemu_opts_parse(&opts_list_02, "size1=-1", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     opts = qemu_opts_parse(&opts_list_02,
++                           "size1=18446744073709551616", /* 2^64 */
+                            false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Suffixes */
+     opts = qemu_opts_parse(&opts_list_02, "size1=8b,size2=1.5k,size3=2M",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 3);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size1", 0), ==, 8);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size2", 0), ==, 1536);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size3", 0), ==, 2 * MiB);
+     opts = qemu_opts_parse(&opts_list_02, "size1=0.1G,size2=16777215T",
+                            false, &error_abort);
+     g_assert_cmpuint(opts_count(opts), ==, 2);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size1", 0), ==, GiB / 10);
+     g_assert_cmphex(qemu_opt_get_size(opts, "size2", 0), ==, 16777215ULL * TiB);
+     /* Beyond limit with suffix */
+     opts = qemu_opts_parse(&opts_list_02, "size1=16777216T",
+                            false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     /* Trailing crap */
+     opts = qemu_opts_parse(&opts_list_02, "size1=16E", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     opts = qemu_opts_parse(&opts_list_02, "size1=16Gi", false, &err);
+     error_free_or_abort(&err);
+     g_assert(!opts);
+     qemu_opts_reset(&opts_list_02);
+ }
+ static void test_has_help_option(void)
+ {
+     static const struct {
+         const char *params;
+         /* expected value of qemu_opt_has_help_opt() with implied=false */
+         bool expect;
+         /* expected value of qemu_opt_has_help_opt() with implied=true */
+         bool expect_implied;
+     } test[] = {
+         { "help", true, false },
+         { "?", true, false },
+         { "helpme", false, false },
+         { "?me", false, false },
+         { "a,help", true, true },
+         { "a,?", true, true },
+         { "a=0,help,b", true, true },
+         { "a=0,?,b", true, true },
+         { "help,b=1", true, false },
+         { "?,b=1", true, false },
+         { "a,b,,help", true, true },
+         { "a,b,,?", true, true },
+     };
+     int i;
+     QemuOpts *opts;
+     for (i = 0; i < ARRAY_SIZE(test); i++) {
+         g_assert_cmpint(has_help_option(test[i].params),
+                         ==, test[i].expect);
+         opts = qemu_opts_parse(&opts_list_03, test[i].params, false,
+                                &error_abort);
+         g_assert_cmpint(qemu_opt_has_help_opt(opts),
+                         ==, test[i].expect);
+         qemu_opts_del(opts);
+         opts = qemu_opts_parse(&opts_list_03, test[i].params, true,
+                                &error_abort);
+         g_assert_cmpint(qemu_opt_has_help_opt(opts),
+                         ==, test[i].expect_implied);
+         qemu_opts_del(opts);
+     }
+ }
+ static void append_verify_list_01(QemuOptDesc *desc, bool with_overlapping)
+ {
+     int i = 0;
+     if (with_overlapping) {
+         g_assert_cmpstr(desc[i].name, ==, "str1");
+         g_assert_cmpint(desc[i].type, ==, QEMU_OPT_STRING);
+         g_assert_cmpstr(desc[i].help, ==,
+                         "Help texts are preserved in qemu_opts_append");
+         g_assert_cmpstr(desc[i].def_value_str, ==, "default");
+         i++;
+         g_assert_cmpstr(desc[i].name, ==, "str2");
+         g_assert_cmpint(desc[i].type, ==, QEMU_OPT_STRING);
+         g_assert_cmpstr(desc[i].help, ==, NULL);
+         g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+         i++;
+     }
+     g_assert_cmpstr(desc[i].name, ==, "str3");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_STRING);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "number1");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_NUMBER);
+     g_assert_cmpstr(desc[i].help, ==,
+                     "Having help texts only for some options is okay");
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "number2");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_NUMBER);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, NULL);
+ }
+ static void append_verify_list_02(QemuOptDesc *desc)
+ {
+     int i = 0;
+     g_assert_cmpstr(desc[i].name, ==, "str1");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_STRING);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "str2");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_STRING);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "bool1");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_BOOL);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "bool2");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_BOOL);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "size1");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_SIZE);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "size2");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_SIZE);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+     i++;
+     g_assert_cmpstr(desc[i].name, ==, "size3");
+     g_assert_cmpint(desc[i].type, ==, QEMU_OPT_SIZE);
+     g_assert_cmpstr(desc[i].help, ==, NULL);
+     g_assert_cmpstr(desc[i].def_value_str, ==, NULL);
+ }
+ static void test_opts_append_to_null(void)
+ {
+     QemuOptsList *merged;
+     merged = qemu_opts_append(NULL, &opts_list_01);
+     g_assert(merged != &opts_list_01);
+     g_assert_cmpstr(merged->name, ==, NULL);
+     g_assert_cmpstr(merged->implied_opt_name, ==, NULL);
+     g_assert_false(merged->merge_lists);
+     append_verify_list_01(merged->desc, true);
+     qemu_opts_free(merged);
+ }
+ static void test_opts_append(void)
+ {
+     QemuOptsList *first, *merged;
+     first = qemu_opts_append(NULL, &opts_list_02);
+     merged = qemu_opts_append(first, &opts_list_01);
+     g_assert(first != &opts_list_02);
+     g_assert(merged != &opts_list_01);
+     g_assert_cmpstr(merged->name, ==, NULL);
+     g_assert_cmpstr(merged->implied_opt_name, ==, NULL);
+     g_assert_false(merged->merge_lists);
+     append_verify_list_02(&merged->desc[0]);
+     append_verify_list_01(&merged->desc[7], false);
+     qemu_opts_free(merged);
+ }
+ static void test_opts_to_qdict_basic(void)
+ {
+     QemuOpts *opts;
+     QDict *dict;
+     opts = qemu_opts_parse(&opts_list_01, "str1=foo,str2=,str3=bar,number1=42",
+                            false, &error_abort);
+     g_assert(opts != NULL);
+     dict = qemu_opts_to_qdict(opts, NULL);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "str1"), ==, "foo");
+     g_assert_cmpstr(qdict_get_str(dict, "str2"), ==, "");
+     g_assert_cmpstr(qdict_get_str(dict, "str3"), ==, "bar");
+     g_assert_cmpstr(qdict_get_str(dict, "number1"), ==, "42");
+     g_assert_false(qdict_haskey(dict, "number2"));
+     qobject_unref(dict);
+     qemu_opts_del(opts);
+ }
+ static void test_opts_to_qdict_filtered(void)
+ {
+     QemuOptsList *first, *merged;
+     QemuOpts *opts;
+     QDict *dict;
+     first = qemu_opts_append(NULL, &opts_list_02);
+     merged = qemu_opts_append(first, &opts_list_01);
+     opts = qemu_opts_parse(merged,
+                            "str1=foo,str2=,str3=bar,bool1=off,number1=42",
+                            false, &error_abort);
+     g_assert(opts != NULL);
+     /* Convert to QDict without deleting from opts */
+     dict = qemu_opts_to_qdict_filtered(opts, NULL, &opts_list_01, false);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "str1"), ==, "foo");
+     g_assert_cmpstr(qdict_get_str(dict, "str2"), ==, "");
+     g_assert_cmpstr(qdict_get_str(dict, "str3"), ==, "bar");
+     g_assert_cmpstr(qdict_get_str(dict, "number1"), ==, "42");
+     g_assert_false(qdict_haskey(dict, "number2"));
+     g_assert_false(qdict_haskey(dict, "bool1"));
+     qobject_unref(dict);
+     dict = qemu_opts_to_qdict_filtered(opts, NULL, &opts_list_02, false);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "str1"), ==, "foo");
+     g_assert_cmpstr(qdict_get_str(dict, "str2"), ==, "");
+     g_assert_cmpstr(qdict_get_str(dict, "bool1"), ==, "off");
+     g_assert_false(qdict_haskey(dict, "str3"));
+     g_assert_false(qdict_haskey(dict, "number1"));
+     g_assert_false(qdict_haskey(dict, "number2"));
+     qobject_unref(dict);
+     /* Now delete converted options from opts */
+     dict = qemu_opts_to_qdict_filtered(opts, NULL, &opts_list_01, true);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "str1"), ==, "foo");
+     g_assert_cmpstr(qdict_get_str(dict, "str2"), ==, "");
+     g_assert_cmpstr(qdict_get_str(dict, "str3"), ==, "bar");
+     g_assert_cmpstr(qdict_get_str(dict, "number1"), ==, "42");
+     g_assert_false(qdict_haskey(dict, "number2"));
+     g_assert_false(qdict_haskey(dict, "bool1"));
+     qobject_unref(dict);
+     dict = qemu_opts_to_qdict_filtered(opts, NULL, &opts_list_02, true);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "bool1"), ==, "off");
+     g_assert_false(qdict_haskey(dict, "str1"));
+     g_assert_false(qdict_haskey(dict, "str2"));
+     g_assert_false(qdict_haskey(dict, "str3"));
+     g_assert_false(qdict_haskey(dict, "number1"));
+     g_assert_false(qdict_haskey(dict, "number2"));
+     qobject_unref(dict);
+     g_assert_true(QTAILQ_EMPTY(&opts->head));
+     qemu_opts_del(opts);
+     qemu_opts_free(merged);
+ }
+ static void test_opts_to_qdict_duplicates(void)
+ {
+     QemuOpts *opts;
+     QemuOpt *opt;
+     QDict *dict;
+     opts = qemu_opts_parse(&opts_list_03, "foo=a,foo=b", false, &error_abort);
+     g_assert(opts != NULL);
+     /* Verify that opts has two options with the same name */
+     opt = QTAILQ_FIRST(&opts->head);
+     g_assert_cmpstr(opt->name, ==, "foo");
+     g_assert_cmpstr(opt->str , ==, "a");
+     opt = QTAILQ_NEXT(opt, next);
+     g_assert_cmpstr(opt->name, ==, "foo");
+     g_assert_cmpstr(opt->str , ==, "b");
+     opt = QTAILQ_NEXT(opt, next);
+     g_assert(opt == NULL);
+     /* In the conversion to QDict, the last one wins */
+     dict = qemu_opts_to_qdict(opts, NULL);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "foo"), ==, "b");
+     qobject_unref(dict);
+     /* The last one still wins if entries are deleted, and both are deleted */
+     dict = qemu_opts_to_qdict_filtered(opts, NULL, NULL, true);
+     g_assert(dict != NULL);
+     g_assert_cmpstr(qdict_get_str(dict, "foo"), ==, "b");
+     qobject_unref(dict);
+     g_assert_true(QTAILQ_EMPTY(&opts->head));
+     qemu_opts_del(opts);
+ }
+ int main(int argc, char *argv[])
+ {
+     register_opts();
+     g_test_init(&argc, &argv, NULL);
+     g_test_add_func("/qemu-opts/find_unknown_opts", test_find_unknown_opts);
+     g_test_add_func("/qemu-opts/find_opts", test_qemu_find_opts);
+     g_test_add_func("/qemu-opts/opts_create", test_qemu_opts_create);
+     g_test_add_func("/qemu-opts/opt_get", test_qemu_opt_get);
+     g_test_add_func("/qemu-opts/opt_get_bool", test_qemu_opt_get_bool);
+     g_test_add_func("/qemu-opts/opt_get_number", test_qemu_opt_get_number);
+     g_test_add_func("/qemu-opts/opt_get_size", test_qemu_opt_get_size);
+     g_test_add_func("/qemu-opts/opt_unset", test_qemu_opt_unset);
+     g_test_add_func("/qemu-opts/opts_reset", test_qemu_opts_reset);
+     g_test_add_func("/qemu-opts/opts_set", test_qemu_opts_set);
+     g_test_add_func("/qemu-opts/opts_parse/general", test_opts_parse);
+     g_test_add_func("/qemu-opts/opts_parse/bool", test_opts_parse_bool);
+     g_test_add_func("/qemu-opts/opts_parse/number", test_opts_parse_number);
+     g_test_add_func("/qemu-opts/opts_parse/size", test_opts_parse_size);
+     g_test_add_func("/qemu-opts/has_help_option", test_has_help_option);
+     g_test_add_func("/qemu-opts/append_to_null", test_opts_append_to_null);
+     g_test_add_func("/qemu-opts/append", test_opts_append);
+     g_test_add_func("/qemu-opts/to_qdict/basic", test_opts_to_qdict_basic);
+     g_test_add_func("/qemu-opts/to_qdict/filtered", test_opts_to_qdict_filtered);
+     g_test_add_func("/qemu-opts/to_qdict/duplicates", test_opts_to_qdict_duplicates);
+     g_test_run();
+     return 0;
+ }