Re: [PATCH 01/14] KVM: selftests: Define literal to asm constraint in aarch64 as unsigned long

Philippe Mathieu-Daudé
2022-12-13T09:52:01Z

On 13/12/22 01:16, Sean Christopherson wrote:
> Define a literal '0' asm input constraint to aarch64/page_fault_test's
> guest_cas() as an unsigned long to make clang happy.
>
> tools/testing/selftests/kvm/aarch64/page_fault_test.c:120:16: error:
> value size does not match register size specified by the constraint
> and modifier [-Werror,-Wasm-operand-widths]
> :: "r" (0), "r" (TEST_DATA), "r" (guest_test_memory));
> ^
> tools/testing/selftests/kvm/aarch64/page_fault_test.c:119:15: note:
> use constraint modifier "w"
> "casal %0, %1, [%2]\n"
> ^~
> %w0
>
> Fixes: 35c581015712 ("KVM: selftests: aarch64: Add aarch64/page_fault_test")
> Cc: Ricardo Koller <ricarkol@google.com>
> Signed-off-by: Sean Christopherson <seanjc@google.com>
> ---
> tools/testing/selftests/kvm/aarch64/page_fault_test.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>