mem: limit use of address hint
authorShahaf Shuler <shahafs@mellanox.com>
Sun, 31 Mar 2019 08:43:48 +0000 (11:43 +0300)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 3 Apr 2019 17:10:47 +0000 (19:10 +0200)
commit237060c4ad15b4ee9002be3c0e56ac3070eceb48
tree119b645bd85b81ee2d194474311244b9052a986e
parentc0c055f5f773adca467d256298d8e777024f085f
mem: limit use of address hint

The commit below added an address hint as starting address for 64-bit
systems in case an explicit base virtual address was not set by the user.

The justification for such hint was to help devices that work in VA
mode and has a address range limitation to work smoothly with the eal
memory subsystem.

While the base address value selected may work fine for the eal
initialization, it easily breaks when trying to register external memory
using rte_extmem_register API.

Trying to register anonymous memory on RH x86_64 machine took several
minutes, during them the function eal_get_virtual_area repeatedly
scanned for a good VA candidate.

The attempt to guess which VA address will be free for mapping will
always result in not portable, error prone code:
* different application may use different libraries along w/ DPDK. One
  can never guess which library was called first and how much virtual
  memory it consumed.
* external memory can be registered at any time in the application run
  time.

In order not to break the existing secondary process design, this patch
only limits the max number of tries that will be done with the
address hint.
When the number of tries exceeds the threshold the code
will use the suggested address from kernel.

Fixes: 1df21702873d ("mem: use address hint for mapping hugepages")
Cc: stable@dpdk.org
Signed-off-by: Shahaf Shuler <shahafs@mellanox.com>
Tested-by: Anatoly Burakov <anatoly.burakov@intel.com>
Acked-by: Alejandro Lucero <alejandro.lucero@netronome.com>
lib/librte_eal/common/eal_common_memory.c