[PATCH v2 0/3] KVM: arm64: vgic-its: Some fixes about vgic-its

[Jing Zhang]

Hi Kunkun,

Do you have a plan to respin this patch series?

We encountered an issue during the VM suspend/resume test that can be
resolved by this patch series.
During VM suspending, the ITS table saving is called multiple times.
Concurrently, device table and ITT table entries are mapped/unmapped
due to devices' suspend/resume. This leaves some free entries dirty
(valid bit equals 1).
The table restore starts from the first entry to find a valid one.
There is a chance that the entry found is a free one, but with the
valid bit as 1.

Jing

On Thu, Jul 4, 2024 at 7:23 AM Kunkun Jiang <jiangkunkun at huawei.com> wrote:
>
> There are three fixes about vgic-its:
> Patch 1: A data length check is added in vgic_its_save_*
> Patch 2-3: Fix two bugs in the scenario of continuous save and restore
>
> History:
> v1 -> v2:
> - Replaced BUG_ON() with KVM_BUG_ON()
>
> Kunkun Jiang (3):
>   KVM: arm64: vgic-its: Add a data length check in vgic_its_save_*
>   KVM: arm64: vgic-its: Clear dte when mapd unmaps a device
>   KVM: arm64: vgic-its: Clear ite when discard frees an ite
>
>  arch/arm64/kvm/vgic/vgic-its.c | 44 ++++++++++++++++++++++++++++++----
>  1 file changed, 40 insertions(+), 4 deletions(-)
>
> --
> 2.27.0
>
>