[lvc-project] [PATCH] RDMA/erdma: handle ib_umem_find_best_pgsz() return value

[Fedor Pchelkin]

On Fri, 21. Feb 07:03, Ваторопин Андрей wrote:
> From: Andrey Vatoropin <a.vatoropin at crpt.ru>
> 
> The ib_umem_find_best_pgsz function is necessary for obtaining the optimal
> hardware page size. In the comment above, function has statement: 
> "Drivers always supporting PAGE_SIZE or smaller will never see a 0 result."
> 
> But it's hard to prove this holds true for the erdma driver.
> 
> Similar to other drivers that use ib_umem_find_best_pgsz, it is essential 
> to add an error handler to manage potential error situations in the future.
> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> 
> Fixes: 155055771704 ("RDMA/erdma: Add verbs implementation")
> Signed-off-by: Andrey Vatoropin <a.vatoropin at crpt.ru>

Просьба адаптировать данный патч для linux-5.10.y и linux-6.1.y веток,
т.к. текущий его вариант не применяется к ним из-за гит конфликта в
связи с произведёнными в master ветке изменениями.

Отправить стоит только в lvc-patches at linuxtesting.org

Описание патча остаётся без изменений.

В теме письма стоит пометить целевую стабильную ветку
[PATCH 5.10] и/или [PATCH 6.1]

Если для 5.10 и 6.1 патч будет одинаковый, можно объединить, указав в
теме письма [PATCH 5.10/6.1].

Если для 5.10 патч не требуется, то для этой ветки проблема и,
следовательно, патч для неё, не являются актуальными.

> ---
>  drivers/infiniband/hw/erdma/erdma_verbs.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/infiniband/hw/erdma/erdma_verbs.c b/drivers/infiniband/hw/erdma/erdma_verbs.c
> index 51d619edb6c5..7ad38fb84661 100644
> --- a/drivers/infiniband/hw/erdma/erdma_verbs.c
> +++ b/drivers/infiniband/hw/erdma/erdma_verbs.c
> @@ -781,6 +781,10 @@ static int get_mtt_entries(struct erdma_dev *dev, struct erdma_mem *mem,
>  	mem->page_size = ib_umem_find_best_pgsz(mem->umem, req_page_size, virt);
> +	if (!mem->page_size) {
> +		ret = -EINVAL;
> +		goto error_ret;
> +	}
>  	mem->page_offset = start & (mem->page_size - 1);
>  	mem->mtt_nents = ib_umem_num_dma_blocks(mem->umem, mem->page_size);
>  	mem->page_cnt = mem->mtt_nents;
>  	mem->mtt = erdma_create_mtt(dev, MTT_SIZE(mem->page_cnt),
>  				    force_continuous);
> -- 
> 2.43.0