Skip to content

Commit 81e7907

Browse files
Quentin Perretgregkh
authored andcommitted
KVM: arm64: Fix initialisation order in __pkvm_init_finalise()
commit 5bb0aed upstream. fix_host_ownership() walks the hypervisor's stage-1 page-table to adjust the host's stage-2 accordingly. Any such adjustment that requires cache maintenance operations depends on the per-CPU hyp fixmap being present. However, fix_host_ownership() is currently called before fix_hyp_pgtable_refcnt() and hyp_create_fixmap(), so the fixmap does not yet exist when it runs. This is benign today because the host stage-2 starts empty and no CMOs are needed, but it becomes a latent crash as soon as fix_host_ownership() is extended to operate on a non-empty page-table. Reorder the calls so that fix_hyp_pgtable_refcnt() and hyp_create_fixmap() complete before fix_host_ownership() is invoked. Fixes: 0d16d12 ("KVM: arm64: Fix-up hyp stage-1 refcounts for all pages mapped at EL2") Signed-off-by: Quentin Perret <qperret@google.com> Signed-off-by: Fuad Tabba <tabba@google.com> Link: https://patch.msgid.link/20260424084908.370776-7-tabba@google.com Signed-off-by: Marc Zyngier <maz@kernel.org> Cc: stable@vger.kernel.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 7b99369 commit 81e7907

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

arch/arm64/kvm/hyp/nvhe/setup.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -312,15 +312,15 @@ void __noreturn __pkvm_init_finalise(void)
312312
};
313313
pkvm_pgtable.mm_ops = &pkvm_pgtable_mm_ops;
314314

315-
ret = fix_host_ownership();
315+
ret = fix_hyp_pgtable_refcnt();
316316
if (ret)
317317
goto out;
318318

319-
ret = fix_hyp_pgtable_refcnt();
319+
ret = hyp_create_fixmap();
320320
if (ret)
321321
goto out;
322322

323-
ret = hyp_create_fixmap();
323+
ret = fix_host_ownership();
324324
if (ret)
325325
goto out;
326326

0 commit comments

Comments
 (0)