[dpdk-dev] [PATCH v5 0/8] Infrastructure to support octeontx HW mempool manager

Santosh Shukla santosh.shukla at caviumnetworks.com
Wed Sep 6 13:28:26 CEST 2017


V5:
Includes v4 review change, suggested by Olivier.

v4:
Include 
- mempool deprecation changes, refer [1], 
- patches are rebased against v17.11-rc0.

v3:
In order to support octeontx HW mempool manager, the common mempool layer must
meet below condition.
- Object start address should be block size (total elem size) aligned.
- Object must have the physically contiguous address within the pool.

And right now mempool doesn't support both.

Patchset adds infrastrucure to support both condition in a _generic_ way.
Proposed solution won't effect existing mempool drivers or its functionality.

Summary:
Introducing capability flag. Now mempool drivers can advertise their
capabilities to common mempool layer(at the pool creation time).
Handlers are introduced in order to support capability flag.

Flags:
* MEMPOOL_F_CAPA_PHYS_CONTIG - If flag is set then Detect whether the object
has the physically contiguous address with in a hugepage.

* MEMPOOL_F_BLK_ALIGNED_OBJECTS - If flag is set then make sure that object
addresses are block size aligned.

API:
Two handles are introduced:
* rte_mempool_ops_get_capability - advertise mempool manager capability.
* rte_mempool_ops_update_range - Update start and end address range to
HW mempool manager.

Change History:
v4 --> v5:
- Replaced mp param with flags param in xmem_size/_usage() api. (Suggested by
  Olivier)
- Renamed flags from MEMPOOL_F_POOL_BLK_SZ_ALIGNED to
  MEMPOOL_F_BLK_ALIGNED_OBJECTS (suggested by Olivier)
- added flag param in get_capabilities() handle (suggested by Olivier)

Refer individual patch for detailed change history.

v3 --> v4:
* [01 - 02 - 03/07] mempool deprecation notice changes.
* [04 - 05 - 06 - 07/07] are v3 patches.

v2 --> v3:
(Note: v3 work is based on deprecation notice [1], It's for 17.11)
* Changed _version.map from 17.08 to 17.11.
* build fixes reported by stv_sys.
* Patchset rebased on upstream commit: da94a999.


v1 --> v2 :
* [01/06] Per deprecation notice [1], Changed rte_mempool 'flag'
  data type from int to unsigned int and removed flag param
  from _xmem_size/usage api.
* [02/06] Incorporated review feedback from v1 [2] (Suggested by Olivier)
* [03/06] Renamed flag to MEMPOOL_F_CAPA_PHYS_CONTIG
  and comment reworded. (Suggested by Olivier per v1 [3])
* [04/06] added new mempool arg in xmem_size/usage. (Suggested by Olivier)
* [05/06] patch description changed.
        - Removed elseif brakcet mix
        - removed sanity check for alignment
        - removed extra var delta
        - Removed __rte_unused from xmem_usage/size and added _BLK_SZ_ALIGN check.
        (Suggeted by Olivier per v1[4])
* [06/06] Added RTE_FUNC_PTR_OR_RET in rte_mempool_ops_update_ops.

Checkpatch status:
CLEAN.

Thanks.

[1] deprecation notice v2: http://dpdk.org/dev/patchwork/patch/27079/
[2] v1: http://dpdk.org/dev/patchwork/patch/25603/
[3] v1: http://dpdk.org/dev/patchwork/patch/25604/
[4] v1: http://dpdk.org/dev/patchwork/patch/25605/

Santosh Shukla (8):
  mempool: remove unused flags argument
  mempool: change flags from int to unsigned int
  mempool: add flags arg in xmem size and usage
  doc: remove mempool notice
  mempool: get the mempool capability
  mempool: detect physical contiguous object in pool
  mempool: introduce block size align flag
  mempool: update range info to pool

 doc/guides/rel_notes/deprecation.rst       |  9 ---
 doc/guides/rel_notes/release_17_11.rst     |  7 +++
 drivers/net/xenvirt/rte_mempool_gntalloc.c |  7 ++-
 lib/librte_mempool/rte_mempool.c           | 47 +++++++++++---
 lib/librte_mempool/rte_mempool.h           | 99 ++++++++++++++++++++++--------
 lib/librte_mempool/rte_mempool_ops.c       | 28 +++++++++
 lib/librte_mempool/rte_mempool_version.map |  8 +++
 test/test/test_mempool.c                   | 25 ++++----
 test/test/test_mempool_perf.c              |  4 +-
 9 files changed, 176 insertions(+), 58 deletions(-)

-- 
2.11.0



More information about the dev mailing list