diff options
| author | Yongqiang Liu <[email protected]> | 2024-08-21 11:27:37 +0000 |
|---|---|---|
| committer | Sean Christopherson <[email protected]> | 2024-08-22 18:35:09 +0000 |
| commit | c501062bb22ba325b7b77c91433d79574b4a3dcc (patch) | |
| tree | 668cf78c578a3ca75befd2ee93f1ec021471f2f9 /arch/x86/kvm/svm/nested.c | |
| parent | Linux 6.11-rc4 (diff) | |
| download | kernel-c501062bb22ba325b7b77c91433d79574b4a3dcc.tar.gz kernel-c501062bb22ba325b7b77c91433d79574b4a3dcc.zip | |
KVM: SVM: Remove unnecessary GFP_KERNEL_ACCOUNT in svm_set_nested_state()
The fixed size temporary variables vmcb_control_area and vmcb_save_area
allocated in svm_set_nested_state() are released when the function exits.
Meanwhile, svm_set_nested_state() also have vcpu mutex held to avoid
massive concurrency allocation, so we don't need to set GFP_KERNEL_ACCOUNT.
Signed-off-by: Yongqiang Liu <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Sean Christopherson <[email protected]>
Diffstat (limited to 'arch/x86/kvm/svm/nested.c')
| -rw-r--r-- | arch/x86/kvm/svm/nested.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/kvm/svm/nested.c b/arch/x86/kvm/svm/nested.c index 6f704c1037e5..d5314cb7dff4 100644 --- a/arch/x86/kvm/svm/nested.c +++ b/arch/x86/kvm/svm/nested.c @@ -1693,8 +1693,8 @@ static int svm_set_nested_state(struct kvm_vcpu *vcpu, return -EINVAL; ret = -ENOMEM; - ctl = kzalloc(sizeof(*ctl), GFP_KERNEL_ACCOUNT); - save = kzalloc(sizeof(*save), GFP_KERNEL_ACCOUNT); + ctl = kzalloc(sizeof(*ctl), GFP_KERNEL); + save = kzalloc(sizeof(*save), GFP_KERNEL); if (!ctl || !save) goto out_free; |