Re: [PATCH v2 0/4] KVM: arm64: vgic-its: Make the ITS table save reliable
From: Oliver Upton
Date: Tue Sep 15 2026 - 18:27:17 EST
On Fri, 21 Aug 2026 07:44:41 +0100, Fuad Tabba wrote:
> Changes since v1 [1]:
> - Free the LPI translation cache in vgic_mmio_write_its_baser() too,
> inside the its_lock section. (Eric)
> - Reworded patch 1 to name the register the write is to. (Eric)
> - Rebased onto kvmarm/next.
>
> Marc pointed out on the v2 ITS series [2] that patch 3 fixed the wrong
> end of the problem: KVM should drop the collections when the guest
> changes the table that described them, rather than validating them at
> save time. He also called that patch's -EINVAL a mistake, since it
> aborts a save userspace has to be able to issue. This series does both,
> following his answers [3] to the two open questions: purge the whole
> list, and skip the offending device rather than fail.
>
> [...]
Applied to fixes, thanks!
[1/4] KVM: arm64: vgic-its: Free the caches when GITS_BASER changes
https://git.kernel.org/kvmarm/kvmarm/c/8cd92f77ae4f
[2/4] Revert "KVM: arm64: vgic-its: Don't save collections the table cannot hold"
https://git.kernel.org/kvmarm/kvmarm/c/30908e727247
[3/4] KVM: arm64: vgic-its: Skip unreachable devices instead of failing the save
https://git.kernel.org/kvmarm/kvmarm/c/cc5d96036e01
[4/4] KVM: arm64: selftests: Add ITS table save tests
https://git.kernel.org/kvmarm/kvmarm/c/6f182db39fb0
--
Best,
Oliver