| Message ID | 20200311131056.140016-2-borntraeger@de.ibm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | format string fixups for selftests | expand |
On 3/11/20 2:10 PM, Christian Borntraeger wrote: > value is u64 and not string. > > Reported-by: Andrew Jones <drjones@redhat.com> > Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com> Reviewed-by: Janosch Frank <frankja@linux.ibm.com> > --- > tools/testing/selftests/kvm/s390x/resets.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/kvm/s390x/resets.c b/tools/testing/selftests/kvm/s390x/resets.c > index b567705f0d41..221e9c9a8bd2 100644 > --- a/tools/testing/selftests/kvm/s390x/resets.c > +++ b/tools/testing/selftests/kvm/s390x/resets.c > @@ -66,7 +66,7 @@ static void test_one_reg(uint64_t id, uint64_t value) > reg.addr = (uintptr_t)&eval_reg; > reg.id = id; > vcpu_get_reg(vm, VCPU_ID, ®); > - TEST_ASSERT(eval_reg == value, "value == %s", value); > + TEST_ASSERT(eval_reg == value, "value == 0x%lx", value); > } > > static void assert_noirq(void) >
diff --git a/tools/testing/selftests/kvm/s390x/resets.c b/tools/testing/selftests/kvm/s390x/resets.c index b567705f0d41..221e9c9a8bd2 100644 --- a/tools/testing/selftests/kvm/s390x/resets.c +++ b/tools/testing/selftests/kvm/s390x/resets.c @@ -66,7 +66,7 @@ static void test_one_reg(uint64_t id, uint64_t value) reg.addr = (uintptr_t)&eval_reg; reg.id = id; vcpu_get_reg(vm, VCPU_ID, ®); - TEST_ASSERT(eval_reg == value, "value == %s", value); + TEST_ASSERT(eval_reg == value, "value == 0x%lx", value); } static void assert_noirq(void)
value is u64 and not string. Reported-by: Andrew Jones <drjones@redhat.com> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com> --- tools/testing/selftests/kvm/s390x/resets.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)