mempool: fix alignment of memzone length when populating
authorOlivier Matz <olivier.matz@6wind.com>
Mon, 7 May 2018 08:18:01 +0000 (10:18 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 8 May 2018 13:58:20 +0000 (15:58 +0200)
commit5751ff40fed0f7dc359e983d80c768126328561f
treecb854f81eae6a319ea5c05d653057ddb8b6fd547
parent13c5cbafa60a90fa4f5bb519ef213f1c7c08e745
mempool: fix alignment of memzone length when populating

When populating a mempool with the default function, if there is not
enough virtually contiguous memory for the whole mempool, it will be
populated with several chunks. A chunk of the maximum available length
is requested with:

  mz = rte_memzone_reserve_aligned(..., len=0, ..., align=x)

If align is smaller than the page size, the address and the length of
the memzone may not be a multiple of the page size. This makes
rte_mempool_populate_virt() to fail because it requires them to be
page-aligned. This patch fixes that.

The problem can be reproduced easily by allocating more than available
memory:
  ./build/app/testpmd -l 0,1 -- --total-num-mbufs=65536
  ...
  Cause: Creation of mbuf pool for socket 0 failed: Invalid argument

After the patch, the error code is correct:
  ./build/app/testpmd -l 0,1 -- --total-num-mbufs=65536
  ...
  Cause: Creation of mbuf pool for socket 0 failed: Cannot allocate memory

Fixes: ba0009560c30 ("mempool: support new allocation methods")

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
Reviewed-by: Andrew Rybchenko <arybchenko@solarflare.com>
lib/librte_mempool/rte_mempool.c