Re: [RFC PATCH v3] RDMA/siw: use kzalloc_flex

[Leon Romanovsky <leon@kernel.org>]

On Thu, May 14, 2026 at 03:14:24PM +0200, Bernard Metzler wrote:
> On 13.05.2026 19:45, Leon Romanovsky wrote:
> > 
> > On Mon, 11 May 2026 16:11:49 +0200, bernard.metzler@linux.dev wrote:
> > > Simplify umem allocation by using flexible array member.
> > > Add __counted_by to get extra runtime analysis.
> > 
> > Applied, thanks!
> > 
> > [1/1] RDMA/siw: use kzalloc_flex
> >        https://git.kernel.org/rdma/rdma/c/79678bea399052
> > 
> > Best regards,
> Hi Leon,
> 
> per Jason's suggestion, please amend the patch by
> adding below line. We should NOT assume kzalloc_flex
> to initialize anything but 0. So we shall set
> umem->num_chunks explicitly.

done, thanks

> 
> Many thanks,
> Bernard.
> ----
> 
> diff --git a/drivers/infiniband/sw/siw/siw_mem.c b/drivers/infiniband/sw/siw/siw_mem.c
> index 88ec3cacfa00..40455616d9ec 100644
> --- a/drivers/infiniband/sw/siw/siw_mem.c
> +++ b/drivers/infiniband/sw/siw/siw_mem.c
> @@ -359,6 +359,7 @@ struct siw_umem *siw_umem_get(struct ib_device *base_dev, u64 start,
> 	umem->fp_addr = first_page_va;
> 	umem->base_mem = base_mem;
> 	umem->num_pages = num_pages;
> +	umem->num_chunks = num_chunks;
> 
> 	sgt = &base_mem->sgt_append.sgt;
> 	__sg_page_iter_start(&sg_iter, sgt->sgl, sgt->orig_nents, 0);
> 
>