Re: [dpdk-dev] [PATCH v1 1/1] mempool/octeontx2: remove minimum chunck size limitation

[Jerin Jacob <jerinjacobk@gmail.com>]

On Tue, Sep 10, 2019 at 4:33 PM <vattunuru@marvell.com> wrote:
>
> From: Vamsi Attunuru <vattunuru@marvell.com>
>
> Memory required for the mempool objects need not be physically
> contiguous on octeontx2's HW, mempool pmd supports the minimum chunk
> size set by the default handler. Hence discarding the limitation set
> by the pmd on the min_chunk_size value.
>
> Signed-off-by: Vamsi Attunuru <vattunuru@marvell.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>

This patch needs to go through to master repo based on MAINTAINERS
file. Changing the "Delegate to" to Thomas in patchwork.

> ---
>  drivers/mempool/octeontx2/otx2_mempool_ops.c | 16 ++--------------
>  1 file changed, 2 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/mempool/octeontx2/otx2_mempool_ops.c b/drivers/mempool/octeontx2/otx2_mempool_ops.c
> index f5a4fe3..d769575 100644
> --- a/drivers/mempool/octeontx2/otx2_mempool_ops.c
> +++ b/drivers/mempool/octeontx2/otx2_mempool_ops.c
> @@ -713,24 +713,12 @@ static ssize_t
>  otx2_npa_calc_mem_size(const struct rte_mempool *mp, uint32_t obj_num,
>                        uint32_t pg_shift, size_t *min_chunk_size, size_t *align)
>  {
> -       ssize_t mem_size;
> -
>         /*
>          * Simply need space for one more object to be able to
>          * fulfill alignment requirements.
>          */
> -       mem_size = rte_mempool_op_calc_mem_size_default(mp, obj_num + 1,
> -                                                       pg_shift,
> -                                                       min_chunk_size, align);
> -       if (mem_size >= 0) {
> -               /*
> -                * Memory area which contains objects must be physically
> -                * contiguous.
> -                */
> -               *min_chunk_size = mem_size;
> -       }
> -
> -       return mem_size;
> +       return rte_mempool_op_calc_mem_size_default(mp, obj_num + 1, pg_shift,
> +                                                   min_chunk_size, align);
>  }
>
>  static int
> --
> 2.8.4
>