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