Re: [PATCH 3/3] KVM: arm64: Sanitise MTE tags when a permission fault installs a mapping

From: Lorenzo Stoakes (ARM)

Date: Tue Oct 20 2026 - 09:40:55 EST


On Tue, Oct 20, 2026 at 02:17:00PM +0100, Fuad Tabba wrote:

> > In the else branch? Seems neater and clearer.
>
> Agree, I'll move it there in v2.

Great thanks!

> > if (is_mte_fault(kvm, s2vi))
> > sanitise_mte_tags(kvm, pfn, mapping_size);
>
> I'd rather leave that out: the fix doesn't change that part of the
> condition, and it only appears twice.

Ack, just a nice-to-have that. Not a big deal.

--
Cheers, Lorenzo