Re: [PATCH 1/2] KVM: arm64: nv: Allocate the shadow S2 MMUs individually
From: Karl Mehltretter
Date: Mon Aug 10 2026 - 18:32:55 EST
On Mon, Aug 10, 2026 at 08:58:12AM +0100, Marc Zyngier wrote:
> > The increment expression of the for loop still runs after an error.
>
> Yup. Should be fixed with
> + for (i = 0; !ret && i < S2_MMU_PER_VCPU; i++) {
> ret = init_nested_s2_mmu(kvm, &tmp[i]);
> + if (ret)
> + break;
> + }
Yes, that fixes the cleanup issue. I have no other concerns with the patch.
Thanks,
Karl