Re: [PATCH v2 6/6] selftests: kvm: s390: Add has device attr check to uc_attr_mem_limit selftest

[Hariharan Mari]

On 2024-12-16 10:21, Christoph Schlameuss wrote:

Fixup the uc_attr_mem_limit test case to also cover the
KVM_HAS_DEVICE_ATTR ioctl.

Signed-off-by: Christoph Schlameuss <schlame...@linux.ibm.com>

Tested-by: Hariharan Mari <har...@linux.ibm.com>

---
 tools/testing/selftests/kvm/s390x/ucontrol_test.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tools/testing/selftests/kvm/s390x/ucontrol_test.c
b/tools/testing/selftests/kvm/s390x/ucontrol_test.c
index ef3e391811b3..087496fe264d 100644
--- a/tools/testing/selftests/kvm/s390x/ucontrol_test.c
+++ b/tools/testing/selftests/kvm/s390x/ucontrol_test.c
@@ -206,10 +206,13 @@ TEST_F(uc_kvm, uc_attr_mem_limit)
        struct kvm_device_attr attr = {
                .group = KVM_S390_VM_MEM_CTRL,
                .attr = KVM_S390_VM_MEM_LIMIT_SIZE,
-               .addr = (unsigned long)&limit,
+               .addr = (u64)&limit,
        };
        int rc;

+       rc = ioctl(self->vm_fd, KVM_HAS_DEVICE_ATTR, &attr);
+       EXPECT_EQ(0, rc);
+
        rc = ioctl(self->vm_fd, KVM_GET_DEVICE_ATTR, &attr);
        EXPECT_EQ(0, rc);
        EXPECT_EQ(~0UL, limit);