KVM: selftests: Dedup MSR index list helpers, simplify dedicated test

Consolidate the helper for retrieving the list of save/restore MSRs and
the list of feature MSRs, and use the common helpers in the related
get_msr_index_features test.  Switching to the common helpers eliminates
the testcase that KVM returns the same -E2BIG result if the input number
of MSRs is '1' versus '0', but considered that testcase isn't very
interesting, e.g. '0' and '1' are equally arbitrary, and certainly not
worth the additional code.

Signed-off-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

diff --git a/tools/testing/selftests/kvm/include/x86_64/processor.h b/tools/testing/selftests/kvm/include/x86_64/processor.h
index b8023c30bdeda2d51fa082d2f850ad16c1584a38..9145da0bc61e577c083edbec9118d390483b874a 100644 (file)
--- a/tools/testing/selftests/kvm/include/x86_64/processor.h
+++ b/tools/testing/selftests/kvm/include/x86_64/processor.h
@@ -428,6 +428,7 @@ void vcpu_load_state(struct kvm_vm *vm, uint32_t vcpuid,
 void kvm_x86_state_cleanup(struct kvm_x86_state *state);
 
 const struct kvm_msr_list *kvm_get_msr_index_list(void);
+const struct kvm_msr_list *kvm_get_feature_msr_index_list(void);
 bool kvm_msr_is_in_save_restore_list(uint32_t msr_index);
 uint64_t kvm_get_feature_msr(uint64_t msr_index);
 

diff --git a/tools/testing/selftests/kvm/lib/x86_64/processor.c b/tools/testing/selftests/kvm/lib/x86_64/processor.c
index 3f83857009a393c2b736b6db60dab6732ab2de58..18fce5e8a5e52119752fdc89530647335ed73c6a 100644 (file)
--- a/tools/testing/selftests/kvm/lib/x86_64/processor.c
+++ b/tools/testing/selftests/kvm/lib/x86_64/processor.c
@@ -891,19 +891,20 @@ void vcpu_dump(FILE *stream, struct kvm_vm *vm, uint32_t vcpuid, uint8_t indent)
        sregs_dump(stream, &sregs, indent + 4);
 }
 
-const struct kvm_msr_list *kvm_get_msr_index_list(void)
+static struct kvm_msr_list *__kvm_get_msr_index_list(bool feature_msrs)
 {
-       static struct kvm_msr_list *list;
+       struct kvm_msr_list *list;
        struct kvm_msr_list nmsrs;
        int kvm_fd, r;
 
-       if (list)
-               return list;
-
        kvm_fd = open_kvm_dev_path_or_exit();
 
        nmsrs.nmsrs = 0;
-       r = __kvm_ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, &nmsrs);
+       if (!feature_msrs)
+               r = __kvm_ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, &nmsrs);
+       else
+               r = __kvm_ioctl(kvm_fd, KVM_GET_MSR_FEATURE_INDEX_LIST, &nmsrs);
+
        TEST_ASSERT(r == -1 && errno == E2BIG,
                    "Expected -E2BIG, got rc: %i errno: %i (%s)",
                    r, errno, strerror(errno));
@@ -912,15 +913,37 @@ const struct kvm_msr_list *kvm_get_msr_index_list(void)
        TEST_ASSERT(list, "-ENOMEM when allocating MSR index list");
        list->nmsrs = nmsrs.nmsrs;
 
-       kvm_ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, list);
+       if (!feature_msrs)
+               kvm_ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, list);
+       else
+               kvm_ioctl(kvm_fd, KVM_GET_MSR_FEATURE_INDEX_LIST, list);
        close(kvm_fd);
 
        TEST_ASSERT(list->nmsrs == nmsrs.nmsrs,
-                   "Number of save/restore MSRs changed, was %d, now %d",
+                   "Number of MSRs in list changed, was %d, now %d",
                    nmsrs.nmsrs, list->nmsrs);
        return list;
 }
 
+const struct kvm_msr_list *kvm_get_msr_index_list(void)
+{
+       static const struct kvm_msr_list *list;
+
+       if (!list)
+               list = __kvm_get_msr_index_list(false);
+       return list;
+}
+
+
+const struct kvm_msr_list *kvm_get_feature_msr_index_list(void)
+{
+       static const struct kvm_msr_list *list;
+
+       if (!list)
+               list = __kvm_get_msr_index_list(true);
+       return list;
+}
+
 bool kvm_msr_is_in_save_restore_list(uint32_t msr_index)
 {
        const struct kvm_msr_list *list = kvm_get_msr_index_list();

diff --git a/tools/testing/selftests/kvm/x86_64/get_msr_index_features.c b/tools/testing/selftests/kvm/x86_64/get_msr_index_features.c
index 4ef60adbe108f36f02683962d18f909e563c581e..1e366fdfe7bef7cbb1c2e25b9a21eacdd947848c 100644 (file)
--- a/tools/testing/selftests/kvm/x86_64/get_msr_index_features.c
+++ b/tools/testing/selftests/kvm/x86_64/get_msr_index_features.c
@@ -15,108 +15,24 @@
 #include "kvm_util.h"
 #include "processor.h"
 
-static int kvm_num_index_msrs(int kvm_fd, int nmsrs)
-{
-       struct kvm_msr_list *list;
-       int r;
-
-       list = malloc(sizeof(*list) + nmsrs * sizeof(list->indices[0]));
-       list->nmsrs = nmsrs;
-       r = ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, list);
-       TEST_ASSERT(r == -1 && errno == E2BIG,
-                               "Unexpected result from KVM_GET_MSR_INDEX_LIST probe, r: %i",
-                               r);
-
-       r = list->nmsrs;
-       free(list);
-       return r;
-}
-
-static void test_get_msr_index(void)
-{
-       int old_res, res, kvm_fd;
-       struct kvm_msr_list *list;
-
-       kvm_fd = open_kvm_dev_path_or_exit();
-
-       old_res = kvm_num_index_msrs(kvm_fd, 0);
-       TEST_ASSERT(old_res != 0, "Expecting nmsrs to be > 0");
-
-       if (old_res != 1) {
-               res = kvm_num_index_msrs(kvm_fd, 1);
-               TEST_ASSERT(res > 1, "Expecting nmsrs to be > 1");
-               TEST_ASSERT(res == old_res, "Expecting nmsrs to be identical");
-       }
-
-       list = malloc(sizeof(*list) + old_res * sizeof(list->indices[0]));
-       list->nmsrs = old_res;
-       kvm_ioctl(kvm_fd, KVM_GET_MSR_INDEX_LIST, list);
-
-       TEST_ASSERT(list->nmsrs == old_res, "Expecting nmsrs to be identical");
-       free(list);
-
-       close(kvm_fd);
-}
-
-static int kvm_num_feature_msrs(int kvm_fd, int nmsrs)
-{
-       struct kvm_msr_list *list;
-       int r;
-
-       list = malloc(sizeof(*list) + nmsrs * sizeof(list->indices[0]));
-       list->nmsrs = nmsrs;
-       r = __kvm_ioctl(kvm_fd, KVM_GET_MSR_FEATURE_INDEX_LIST, list);
-       TEST_ASSERT(r == -1 && errno == E2BIG,
-               "Unexpected result from KVM_GET_MSR_FEATURE_INDEX_LIST probe, r: %i",
-                               r);
-
-       r = list->nmsrs;
-       free(list);
-       return r;
-}
-
-struct kvm_msr_list *kvm_get_msr_feature_list(int kvm_fd, int nmsrs)
-{
-       struct kvm_msr_list *list;
-
-       list = malloc(sizeof(*list) + nmsrs * sizeof(list->indices[0]));
-       list->nmsrs = nmsrs;
-       kvm_ioctl(kvm_fd, KVM_GET_MSR_FEATURE_INDEX_LIST, list);
-
-       return list;
-}
-
-static void test_get_msr_feature(void)
+int main(int argc, char *argv[])
 {
-       int res, old_res, i, kvm_fd;
-       struct kvm_msr_list *feature_list;
-
-       kvm_fd = open_kvm_dev_path_or_exit();
-
-       old_res = kvm_num_feature_msrs(kvm_fd, 0);
-       TEST_ASSERT(old_res != 0, "Expecting nmsrs to be > 0");
-
-       if (old_res != 1) {
-               res = kvm_num_feature_msrs(kvm_fd, 1);
-               TEST_ASSERT(res > 1, "Expecting nmsrs to be > 1");
-               TEST_ASSERT(res == old_res, "Expecting nmsrs to be identical");
+       const struct kvm_msr_list *feature_list;
+       int i;
+
+       /*
+        * Skip the entire test if MSR_FEATURES isn't supported, other tests
+        * will cover the "regular" list of MSRs, the coverage here is purely
+        * opportunistic and not interesting on its own.
+        */
+       if (!kvm_check_cap(KVM_CAP_GET_MSR_FEATURES)) {
+               print_skip("KVM_CAP_GET_MSR_FEATURES not supported");
+               exit(KSFT_SKIP);
        }
 
-       feature_list = kvm_get_msr_feature_list(kvm_fd, old_res);
-       TEST_ASSERT(old_res == feature_list->nmsrs,
-                               "Unmatching number of msr indexes");
+       (void)kvm_get_msr_index_list();
 
+       feature_list = kvm_get_feature_msr_index_list();
        for (i = 0; i < feature_list->nmsrs; i++)
                kvm_get_feature_msr(feature_list->indices[i]);
-
-       free(feature_list);
-       close(kvm_fd);
-}
-
-int main(int argc, char *argv[])
-{
-       if (kvm_check_cap(KVM_CAP_GET_MSR_FEATURES))
-               test_get_msr_feature();
-
-       test_get_msr_index();
 }