Message ID | 20221213001653.3852042-2-seanjc@google.com (mailing list archive) |
---|---|
State | RFC |
Headers | show |
Series | KVM: selftests: Clang fixes, Makefile cleanup | expand |
Context | Check | Description |
---|---|---|
conchuod/tree_selection | fail | Guessing tree name failed |
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>
diff --git a/tools/testing/selftests/kvm/aarch64/page_fault_test.c b/tools/testing/selftests/kvm/aarch64/page_fault_test.c index 95d22cfb7b41..beb944fa6fd4 100644 --- a/tools/testing/selftests/kvm/aarch64/page_fault_test.c +++ b/tools/testing/selftests/kvm/aarch64/page_fault_test.c @@ -117,7 +117,7 @@ static void guest_cas(void) GUEST_ASSERT(guest_check_lse()); asm volatile(".arch_extension lse\n" "casal %0, %1, [%2]\n" - :: "r" (0), "r" (TEST_DATA), "r" (guest_test_memory)); + :: "r" (0ul), "r" (TEST_DATA), "r" (guest_test_memory)); val = READ_ONCE(*guest_test_memory); GUEST_ASSERT_EQ(val, TEST_DATA); }
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(-)