On Fri, Mar 11, 2022 at 12:25:13AM +0000, David Matlack wrote:
> static struct kvm_mmu_page *tdp_mmu_alloc_sp(struct kvm_vcpu *vcpu)
> {
> - struct kvm_mmu_page *sp;
> -
> - sp = kvm_mmu_memory_cache_alloc(&vcpu->arch.mmu_page_header_cache);
> - sp->spt = kvm_mmu_memory_cache_alloc(&vcpu->arch.mmu_shadow_page_cache);
> - set_page_private(virt_to_page(sp->spt), (unsigned long)sp);
> -
> - return sp;
> + return kvm_mmu_alloc_shadow_page(vcpu, true);
> }
Similarly I had a feeling we could drop tdp_mmu_alloc_sp() too.. anyway:
Reviewed-by: Peter Xu <peterx@xxxxxxxxxx>
--
Peter Xu
