Re: [PATCH v9 1/6] KVM: arm64: Rename the device variable to s2_force_noncacheable

From: Jason Gunthorpe
Date: Fri Jul 04 2025 - 09:41:48 EST


On Sat, Jun 21, 2025 at 04:21:06AM +0000, ankita@xxxxxxxxxx wrote:
> From: Ankit Agrawal <ankita@xxxxxxxxxx>
>
> For cache maintenance on a region, ARM KVM relies on that
> region to be mapped to the Kernal virtual address as CMOs
> operate on VA.
>
> Currently the device variable is effectively trying to setup
> the S2 mapping as non cacheable for memory regions that are
> not mapped in the Kernel VA. This could be either device or
> Normal_NC depending on the VM_ALLOW_ANY_UNCACHED flag in the
> VMA.
>
> Thus "device" could be better renamed to s2_force_noncacheable
> which implies that it is ensuring that region be mapped as
> non-cacheable.
>
> CC: Catalin Marinas <catalin.marinas@xxxxxxx>
> Suggested-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> Signed-off-by: Ankit Agrawal <ankita@xxxxxxxxxx>
> ---
> arch/arm64/kvm/mmu.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)

I think this is much clearer

Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>

Jason