diff options
| author | Yongqiang Liu <liuyongqiang13@huawei.com> | 2024-08-21 19:27:37 +0800 |
|---|---|---|
| committer | Sean Christopherson <seanjc@google.com> | 2024-08-22 11:35:09 -0700 |
| commit | c501062bb22ba325b7b77c91433d79574b4a3dcc (patch) | |
| tree | 668cf78c578a3ca75befd2ee93f1ec021471f2f9 /arch/x86/kvm/svm/nested.c | |
| parent | 47ac09b91befbb6a235ab620c32af719f8208399 (diff) | |
| download | linux-c501062bb22ba325b7b77c91433d79574b4a3dcc.tar.gz linux-c501062bb22ba325b7b77c91433d79574b4a3dcc.tar.bz2 linux-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 <liuyongqiang13@huawei.com>
Link: https://lore.kernel.org/r/20240821112737.3649937-1-liuyongqiang13@huawei.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
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; |