kvm: selftests: ensure vcpu file is released
authorPaolo Bonzini <pbonzini@redhat.com>
Thu, 26 Jul 2018 11:02:24 +0000 (13:02 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Mon, 6 Aug 2018 15:32:04 +0000 (17:32 +0200)
The selftests were not munmap-ing the kvm_run area from the vcpu file descriptor.
The result was that kvm_vcpu_release was not called and a reference was left in the
parent "struct kvm".  Ultimately this was visible in the upcoming state save/restore
test as an error when KVM attempted to create a duplicate debugfs entry.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
tools/testing/selftests/kvm/lib/kvm_util.c

index 610d1326f03d0c48e5be20553d45d9fb1846b4b0..1634828733639fac64abfadd07e53feb28234545 100644 (file)
@@ -238,8 +238,12 @@ struct vcpu *vcpu_find(struct kvm_vm *vm,
 static void vm_vcpu_rm(struct kvm_vm *vm, uint32_t vcpuid)
 {
        struct vcpu *vcpu = vcpu_find(vm, vcpuid);
+       int ret;
 
-       int ret = close(vcpu->fd);
+       ret = munmap(vcpu->state, sizeof(*vcpu->state));
+       TEST_ASSERT(ret == 0, "munmap of VCPU fd failed, rc: %i "
+               "errno: %i", ret, errno);
+       close(vcpu->fd);
        TEST_ASSERT(ret == 0, "Close of VCPU fd failed, rc: %i "
                "errno: %i", ret, errno);
 
@@ -295,6 +299,10 @@ void kvm_vm_free(struct kvm_vm *vmp)
        TEST_ASSERT(ret == 0, "Close of vm fd failed,\n"
                "  vmp->fd: %i rc: %i errno: %i", vmp->fd, ret, errno);
 
+       close(vmp->kvm_fd);
+       TEST_ASSERT(ret == 0, "Close of /dev/kvm fd failed,\n"
+               "  vmp->kvm_fd: %i rc: %i errno: %i", vmp->kvm_fd, ret, errno);
+
        /* Free the structure describing the VM. */
        free(vmp);
 }