linux-kernel - Re: [PATCH vhost] vdpa/mlx5: Fix oversized null mkey longer than 32bit

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJaqyWcXcW9U7a1bMAngG-eEjw6t5T3XPUdn_hai5OWWTQW85Q@mail.gmail.com>
Date: Tue, 18 Feb 2025 09:14:43 +0100
From: Eugenio Perez Martin <eperezma@...hat.com>
To: Dragos Tatulea <dtatulea@...dia.com>
Cc: "Michael S . Tsirkin" <mst@...hat.com>, virtualization@...ts.linux.dev, 
	Jason Wang <jasowang@...hat.com>, Si-Wei Liu <si-wei.liu@...cle.com>, kvm@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Gal Pressman <gal@...dia.com>, 
	Xuan Zhuo <xuanzhuo@...ux.alibaba.com>, stable@...r.kernel.org
Subject: Re: [PATCH vhost] vdpa/mlx5: Fix oversized null mkey longer than 32bit

On Mon, Feb 17, 2025 at 8:45 PM Dragos Tatulea <dtatulea@...dia.com> wrote:
>
> From: Si-Wei Liu <si-wei.liu@...cle.com>
>
> create_user_mr() has correct code to count the number of null keys
> used to fill in a hole for the memory map. However, fill_indir()
> does not follow the same to cap the range up to the 1GB limit
> correspondinly.

s/correspondinly/correspondingly/g

Sounds to me the logic can be merged in a helper?

Either way,

Acked-by: Eugenio Pérez <eperezma@...hat.com>

Thanks!

> Fill in more null keys for the gaps in between,
> so that null keys are correctly populated.
>
> Fixes: 94abbccdf291 ("vdpa/mlx5: Add shared memory registration code")
> Cc: stable@...r.kernel.org
> Signed-off-by: Si-Wei Liu <si-wei.liu@...cle.com>
> Signed-off-by: Dragos Tatulea <dtatulea@...dia.com>
> ---
>  drivers/vdpa/mlx5/core/mr.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/vdpa/mlx5/core/mr.c b/drivers/vdpa/mlx5/core/mr.c
> index 8455f08f5d40..61424342c096 100644
> --- a/drivers/vdpa/mlx5/core/mr.c
> +++ b/drivers/vdpa/mlx5/core/mr.c
> @@ -190,9 +190,12 @@ static void fill_indir(struct mlx5_vdpa_dev *mvdev, struct mlx5_vdpa_mr *mkey, v
>                         klm->bcount = cpu_to_be32(klm_bcount(dmr->end - dmr->start));
>                         preve = dmr->end;
>                 } else {
> +                       u64 bcount = min_t(u64, dmr->start - preve, MAX_KLM_SIZE);
> +
>                         klm->key = cpu_to_be32(mvdev->res.null_mkey);
> -                       klm->bcount = cpu_to_be32(klm_bcount(dmr->start - preve));
> -                       preve = dmr->start;
> +                       klm->bcount = cpu_to_be32(klm_bcount(bcount));
> +                       preve += bcount;
> +
>                         goto again;
>                 }
>         }
> --
> 2.43.0
>