This is just cosmetic, but I'd prefer to keep the call toI'm not sure exactly what you mean and I could certainly do a better
kvm_tdp_mmu_invalidate_roots at the original place, so that it's clear
in the next patch that it's two separate parts because of the different
locking requirements.
job explaining in the commit description, but it's actually quite
important that kvm_tdp_mmu_invalidate_roots at least precede
kvm_zap_obsolete_pages as kvm_zap_obsolete_pages drops the lock and
yields. If kvm_tdp_mmu_invalidate_roots doesn't go first then vCPUs
could wind up dropping their ref on an old root and then picking it up
again before the last root had a chance to drop its ref.
Explaining in the description that kvm_tdp_mmu_zap_all is being
dropped because it is no longer necessary (as opposed to being moved)
might help make that cleaner.