diff options
| author | Marc Orr | 2018-11-06 23:53:56 +0100 |
|---|---|---|
| committer | Paolo Bonzini | 2018-12-14 18:00:08 +0100 |
| commit | b666a4b697397f8492dc11a2a1877557d3e0af56 (patch) | |
| tree | d79fd9e812f8b3fc6adf74fd587de062a062ca6b /arch/x86/include/asm | |
| parent | kvm: x86: Use task structs fpu field for user (diff) | |
| download | kernel-qcow2-linux-b666a4b697397f8492dc11a2a1877557d3e0af56.tar.gz kernel-qcow2-linux-b666a4b697397f8492dc11a2a1877557d3e0af56.tar.xz kernel-qcow2-linux-b666a4b697397f8492dc11a2a1877557d3e0af56.zip | |
kvm: x86: Dynamically allocate guest_fpu
Previously, the guest_fpu field was embedded in the kvm_vcpu_arch
struct. Unfortunately, the field is quite large, (e.g., 4352 bytes on my
current setup). This bloats the kvm_vcpu_arch struct for x86 into an
order 3 memory allocation, which can become a problem on overcommitted
machines. Thus, this patch moves the fpu state outside of the
kvm_vcpu_arch struct.
With this patch applied, the kvm_vcpu_arch struct is reduced to 15168
bytes for vmx on my setup when building the kernel with kvmconfig.
Suggested-by: Dave Hansen <dave.hansen@intel.com>
Signed-off-by: Marc Orr <marcorr@google.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch/x86/include/asm')
| -rw-r--r-- | arch/x86/include/asm/kvm_host.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h index c143dfe75869..bca77c25a19a 100644 --- a/arch/x86/include/asm/kvm_host.h +++ b/arch/x86/include/asm/kvm_host.h @@ -610,7 +610,7 @@ struct kvm_vcpu_arch { * "guest_fpu" state here contains the guest FPU context, with the * host PRKU bits. */ - struct fpu guest_fpu; + struct fpu *guest_fpu; u64 xcr0; u64 guest_supported_xcr0; @@ -1196,6 +1196,7 @@ struct kvm_arch_async_pf { }; extern struct kvm_x86_ops *kvm_x86_ops; +extern struct kmem_cache *x86_fpu_cache; #define __KVM_HAVE_ARCH_VM_ALLOC static inline struct kvm *kvm_arch_alloc_vm(void) |