[PATCH 2/4] KVM: arm64: vgic-v5: Correctly reset h_lpi_ist to NULL
Fuad Tabba
fuad.tabba at linux.dev
Sun Sep 20 07:42:46 PDT 2026
On Sat, 19 Sept 2026 at 13:32, Marc Zyngier <maz at kernel.org> wrote:
>
> Resetting a pointer to 0 is not completely wrong, but sparse
> legitimately shouts about it integer/pointer mix.
>
> Use NULL as expected.
>
> Reported-by: kernel test robot <lkp at intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202609190952.glUgOBvp-lkp@intel.com/
> Signed-off-by: Marc Zyngier <maz at kernel.org>
Reviewed-by: Fuad Tabba <fuad.tabba at linux.dev>
Cheers,
/fuad
> ---
> arch/arm64/kvm/vgic/vgic-v5-tables.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/kvm/vgic/vgic-v5-tables.c b/arch/arm64/kvm/vgic/vgic-v5-tables.c
> index ba8a8d5fcc2a3..d7196ed5d1bfd 100644
> --- a/arch/arm64/kvm/vgic/vgic-v5-tables.c
> +++ b/arch/arm64/kvm/vgic/vgic-v5-tables.c
> @@ -1066,7 +1066,7 @@ static int vgic_v5_alloc_two_level_lpi_ist(struct kvm *kvm, unsigned int id_bits
> /* Free the L1 IST again */
> vmi = xa_load(&vm_info, vm_id);
> kfree(vmi->h_lpi_ist);
> - vmi->h_lpi_ist = 0;
> + vmi->h_lpi_ist = NULL;
>
> return ret;
> }
> --
> 2.47.3
>
>
More information about the linux-arm-kernel
mailing list