Messages in this thread | | | Date | Thu, 24 Sep 2020 07:39:24 -0700 | From | Sean Christopherson <> | Subject | Re: [PATCH] KVM: x86/mmu: Stash 'kvm' in a local variable in kvm_mmu_free_roots() |
| |
On Thu, Sep 24, 2020 at 02:42:19PM +0200, Vitaly Kuznetsov wrote: > What about kvm_mmu_get_page(), make_mmu_pages_available(), > mmu_alloc_root(), kvm_mmu_sync_roots(), direct_page_fault(), > kvm_mmu_pte_write() which seem to be using the same ugly pattern? :-)
Heh, good question. I guess only kvm_mmu_free_roots() managed to cross over the threshold from "that's ugly" to "this is ridiculous".
| |