On 25/08/2020 21:17, Jason Gunthorpe wrote:
> The original function returns unsigned long and 0 on failure.
>
> Fixes: 4a35339958f1 ("RDMA/umem: Add API to find best driver supported page size in an MR")
> Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxx>
> ---
> include/rdma/ib_umem.h | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/include/rdma/ib_umem.h b/include/rdma/ib_umem.h
> index 71f573a418bf06..07a764eb692eed 100644
> --- a/include/rdma/ib_umem.h
> +++ b/include/rdma/ib_umem.h
> @@ -68,10 +68,11 @@ static inline int ib_umem_copy_from(void *dst, struct ib_umem *umem, size_t offs
> size_t length) {
> return -EINVAL;
> }
> -static inline int ib_umem_find_best_pgsz(struct ib_umem *umem,
> - unsigned long pgsz_bitmap,
> - unsigned long virt) {
> - return -EINVAL;
> +static inline unsigned long ib_umem_find_best_pgsz(struct ib_umem *umem,
> + unsigned long pgsz_bitmap,
> + unsigned long virt)
> +{
> + return 0;
> }
>
> #endif /* CONFIG_INFINIBAND_USER_MEM */
>
Reviewed-by: Gal Pressman <galpress@xxxxxxxxxx>
