On Wed, 01 Jun 2022 14:51:59 +0100, Sasha Levin <sas...@kernel.org> wrote: > > From: Ricardo Koller <ricar...@google.com> > > [ Upstream commit a1ccfd6f6e06eceb632cc29c4f15a32860f05a7e ] > > Restoring a corrupted collection entry (like an out of range ID) is > being ignored and treated as success. More specifically, a > vgic_its_restore_cte failure is treated as success by > vgic_its_restore_collection_table. vgic_its_restore_cte uses positive > and negative numbers to return error, and +1 to return success. The > caller then uses "ret > 0" to check for success. > > Fix this by having vgic_its_restore_cte only return negative numbers on > error. Do this by changing alloc_collection return codes to only return > negative numbers on error. > > Signed-off-by: Ricardo Koller <ricar...@google.com> > Reviewed-by: Oliver Upton <oup...@google.com> > Signed-off-by: Marc Zyngier <m...@kernel.org> > Link: https://lore.kernel.org/r/20220510001633.552496-4-ricar...@google.com > Signed-off-by: Sasha Levin <sas...@kernel.org>
Same thing here. This wasn't tagged for stable. I don't think there is much value in taking this in isolation. Thanks, M. -- Without deviation from the norm, progress is not possible. _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm