KVM: x86: fix size of x86_fpu_cache objects
authorPaolo Bonzini <pbonzini@redhat.com>
Fri, 21 Dec 2018 10:25:59 +0000 (11:25 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 21 Dec 2018 10:28:19 +0000 (11:28 +0100)
The memory allocation in b666a4b69739 ("kvm: x86: Dynamically allocate
guest_fpu", 2018-11-06) is wrong, there are other members in struct fpu
before the fpregs_state union and the patch should be doing something
similar to the code in fpu__init_task_struct_size.  It's enough to run
a guest and then rmmod kvm to see slub errors which are actually caused
by memory corruption.

For now let's revert it to sizeof(struct fpu), which is conservative.
I have plans to move fsave/fxsave/xsave directly in KVM, without using
the kernel FPU helpers, and once it's done, the size of the object in
the cache will be something like kvm_xstate_size.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/x86.c

index 4f786fcc620e2f23358f8d42d79587e32f26544f..70faa3cdc4dc70941324bdcf2844ee34cbc4377a 100644 (file)
@@ -6867,7 +6867,7 @@ int kvm_arch_init(void *opaque)
        }
 
        r = -ENOMEM;
-       x86_fpu_cache = kmem_cache_create("x86_fpu", fpu_kernel_xstate_size,
+       x86_fpu_cache = kmem_cache_create("x86_fpu", sizeof(struct fpu),
                                          __alignof__(struct fpu), SLAB_ACCOUNT,
                                          NULL);
        if (!x86_fpu_cache) {