OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

qtest: Fix bad printf format specifiers

Browse Source 
We should use printf format specifier "%u" instead of "%d" for
argument of type "unsigned int".

Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Alex Chen <alex.chen@huawei.com>
Message-Id: <5FA28117.3020802@huawei.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
This commit is contained in:
AlexChen 2020-11-04 18:23:19 +08:00 committed by Thomas Huth
parent 8a47836548
commit 3dc057923d
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tests/qtest/arm-cpu-features.c
View File

@ -536,7 +536,7 @@ static void test_query_cpu_model_expansion_kvm(const void *data)
if (kvm_supports_sve) { if (kvm_supports_sve) {
g_assert(vls != 0); g_assert(vls != 0);
max_vq = 64 - __builtin_clzll(vls); max_vq = 64 - __builtin_clzll(vls);
sprintf(max_name, "sve%d", max_vq * 128); sprintf(max_name, "sve%u", max_vq * 128);
/* Enabling a supported length is of course fine. */ /* Enabling a supported length is of course fine. */
assert_sve_vls(qts, "host", vls, "{ %s: true }", max_name); assert_sve_vls(qts, "host", vls, "{ %s: true }", max_name);
@ -556,7 +556,7 @@ static void test_query_cpu_model_expansion_kvm(const void *data)
* unless all larger, supported vector lengths are also * unless all larger, supported vector lengths are also
* disabled. * disabled.
*/ */
sprintf(name, "sve%d", vq * 128); sprintf(name, "sve%u", vq * 128);
error = g_strdup_printf("cannot disable %s", name); error = g_strdup_printf("cannot disable %s", name);
assert_error(qts, "host", error, assert_error(qts, "host", error,
"{ %s: true, %s: false }", "{ %s: true, %s: false }",
@ -569,7 +569,7 @@ static void test_query_cpu_model_expansion_kvm(const void *data)
* we need at least one vector length enabled. * we need at least one vector length enabled.
*/ */
vq = __builtin_ffsll(vls); vq = __builtin_ffsll(vls);
sprintf(name, "sve%d", vq * 128); sprintf(name, "sve%u", vq * 128);
error = g_strdup_printf("cannot disable %s", name); error = g_strdup_printf("cannot disable %s", name);
assert_error(qts, "host", error, "{ %s: false }", name); assert_error(qts, "host", error, "{ %s: false }", name);
g_free(error); g_free(error);
@ -581,7 +581,7 @@ static void test_query_cpu_model_expansion_kvm(const void *data)
} }
} }
if (vq <= SVE_MAX_VQ) { if (vq <= SVE_MAX_VQ) {
sprintf(name, "sve%d", vq * 128); sprintf(name, "sve%u", vq * 128);
error = g_strdup_printf("cannot enable %s", name); error = g_strdup_printf("cannot enable %s", name);
assert_error(qts, "host", error, "{ %s: true }", name); assert_error(qts, "host", error, "{ %s: true }", name);
g_free(error); g_free(error);