[PATCH v2] KVM: arm64: Fix kvm_get_vtcr() kernel-doc

Fuad Tabba fuad.tabba at linux.dev
Thu Sep 10 23:56:04 PDT 2026


On Thu, 10 Sept 2026 at 20:05, Karl Mehltretter <kmehltretter at gmail.com> wrote:
>
> kvm_get_vtcr() takes a parameter named phys_shift, but its kernel-doc
> describes phys_shfit. This leaves the real parameter undescribed and
> adds an excess parameter description to the generated documentation.
>
> Correct the spelling and describe phys_shift as the IPA size in bits.
> The value written to VTCR_EL2.T0SZ is 64 - phys_shift, rather than
> phys_shift itself.
>
> Fixes: bcb25a2b86b4 ("KVM: arm64: Refactor kvm_arm_setup_stage2()")
> Suggested-by: Fuad Tabba <fuad.tabba at linux.dev>
> Assisted-by: LLM
> Signed-off-by: Karl Mehltretter <kmehltretter at gmail.com>

Reviewed-by: Fuad Tabba <fuad.tabba at linux.dev>

Cheers,
/fuad
> ---
> Changes in v2:
> - Describe phys_shift as the IPA size in bits (Fuad).
>
>  arch/arm64/include/asm/kvm_pgtable.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/include/asm/kvm_pgtable.h b/arch/arm64/include/asm/kvm_pgtable.h
> index 41a8687938eb..20d12da3d28e 100644
> --- a/arch/arm64/include/asm/kvm_pgtable.h
> +++ b/arch/arm64/include/asm/kvm_pgtable.h
> @@ -532,7 +532,7 @@ u64 kvm_pgtable_hyp_unmap(struct kvm_pgtable *pgt, u64 addr, u64 size);
>   * kvm_get_vtcr() - Helper to construct VTCR_EL2
>   * @mmfr0:     Sanitized value of SYS_ID_AA64MMFR0_EL1 register.
>   * @mmfr1:     Sanitized value of SYS_ID_AA64MMFR1_EL1 register.
> - * @phys_shfit:        Value to set in VTCR_EL2.T0SZ.
> + * @phys_shift:        IPA size, in bits.
>   *
>   * The VTCR value is common across all the physical CPUs on the system.
>   * We use system wide sanitised values to fill in different fields,
>
> base-commit: 893e11787f78e43b534e252249ac3fff4d1333f8
> --
> 2.53.0



More information about the linux-arm-kernel mailing list