Re: [PATCH 1/2] selftests: KVM: s390: fixup fprintf format error in reset.c

[Janosch Frank <frankja@xxxxxxxxxxxxx>]

On 3/11/20 2:10 PM, Christian Borntraeger wrote:
> value is u64 and not string.
> 
> Reported-by: Andrew Jones <drjones@xxxxxxxxxx>
> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>

Reviewed-by: Janosch Frank <frankja@xxxxxxxxxxxxx>

> ---
>  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, &reg);
> -	TEST_ASSERT(eval_reg == value, "value == %s", value);
> +	TEST_ASSERT(eval_reg == value, "value == 0x%lx", value);
>  }
>  
>  static void assert_noirq(void)
>

Attachment: signature.asc
Description: OpenPGP digital signature