]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
KVM: s390: selftests: fix ucontrol memory region test
authorClaudio Imbrenda <imbrenda@linux.ibm.com>
Thu, 23 Jan 2025 14:46:16 +0000 (15:46 +0100)
committerClaudio Imbrenda <imbrenda@linux.ibm.com>
Fri, 31 Jan 2025 11:03:52 +0000 (12:03 +0100)
With the latest patch, attempting to create a memslot from userspace
will result in an EEXIST error for UCONTROL VMs, instead of EINVAL,
since the new memslot will collide with the internal memslot. There is
no simple way to bring back the previous behaviour.

This is not a problem, but the test needs to be fixed accordingly.

Reviewed-by: Christoph Schlameuss <schlameuss@linux.ibm.com>
Link: https://lore.kernel.org/r/20250123144627.312456-5-imbrenda@linux.ibm.com
Signed-off-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Message-ID: <20250123144627.312456-5-imbrenda@linux.ibm.com>

tools/testing/selftests/kvm/s390/ucontrol_test.c

index 135ee22856cf1a4c57b2967cdf69c4cfe6daf987..22ce9219620ce8a6e596990b3129fffe0572a862 100644 (file)
@@ -459,10 +459,14 @@ TEST_F(uc_kvm, uc_no_user_region)
        };
 
        ASSERT_EQ(-1, ioctl(self->vm_fd, KVM_SET_USER_MEMORY_REGION, &region));
-       ASSERT_EQ(EINVAL, errno);
+       ASSERT_TRUE(errno == EEXIST || errno == EINVAL)
+               TH_LOG("errno %s (%i) not expected for ioctl KVM_SET_USER_MEMORY_REGION",
+                      strerror(errno), errno);
 
        ASSERT_EQ(-1, ioctl(self->vm_fd, KVM_SET_USER_MEMORY_REGION2, &region2));
-       ASSERT_EQ(EINVAL, errno);
+       ASSERT_TRUE(errno == EEXIST || errno == EINVAL)
+               TH_LOG("errno %s (%i) not expected for ioctl KVM_SET_USER_MEMORY_REGION2",
+                      strerror(errno), errno);
 }
 
 TEST_F(uc_kvm, uc_map_unmap)