On 14/10/20 20:26, Ben Gardon wrote: > + if (is_tdp_mmu_root(vcpu->kvm, vcpu->arch.mmu->root_hpa)) > + r = kvm_tdp_mmu_map(vcpu, gpa, error_code, map_writable, > + max_level, pfn, prefault, is_tdp); > + else > + r = __direct_map(vcpu, gpa, error_code, map_writable, max_level, > + pfn, prefault, is_tdp); I like the rename, but I guess is_tdp is clearly enough superfluous. Paolo