Skip to content

refactoring ring library

Feifei Wang dpdk-dev 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest3 patches 1 replyIn review
  2. v23 patches 2 repliesSuperseded
  3. v13 patches 8 repliesSuperseded

Changes in v3 from the cover letter

1. fix compile issue of file name bug

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3test/ring: reduce iteration numbers to make test duration shorterapp/test/test_ring_perf.c+1 -10 replies
2/3ring: add rte prefix before update tail APIlib/librte_ring/rte_ring_c11_mem.h lib/librte_ring/rte_ring_elem.h lib/librte_ring/rte_ring_generic.h+6 -60 replies
3/3ring: refactor ring librarylib/librte_ring/meson.build lib/librte_ring/rte_ring_c11_pvt.h lib/librte_ring/rte_ring_elem.h +9 more+647 -6330 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchHonnappa NagarahalliRuifeng WangKonstantin Ananyev
1/3R-b R-b A-b
2/3R-b R-b A-b
3/3R-b R-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v3 thread

Cover letter [dpdk-dev] [PATCH v3 0/3] refactoring ring library

Do some work for ring refactoring, which includes:
1. add rte prefix before update tail API
2. Move all private APIs into new header files

v2:
1. add rte_ prefixes for private files (Konstantin)

v3:
1. fix compile issue of file name bug

Feifei Wang (3):
test/ring: reduce iteration numbers to make test duration shorter
ring: add rte prefix before update tail API
ring: refactor ring library

 app/test/test_ring_perf.c                     |   2 +-
 lib/librte_ring/meson.build                   |  11 +-
 ...{rte_ring_c11_mem.h => rte_ring_c11_pvt.h} |  10 +-
 lib/librte_ring/rte_ring_elem.h               | 374 +----------------
 lib/librte_ring/rte_ring_elem_pvt.h           | 385 ++++++++++++++++++
 ..._ring_generic.h => rte_ring_generic_pvt.h} |  10 +-
 lib/librte_ring/rte_ring_hts.h                |  84 +---
 ..._hts_c11_mem.h => rte_ring_hts_elem_pvt.h} |  90 +++-
 lib/librte_ring/rte_ring_peek.h               |  71 +---
 ...eek_c11_mem.h => rte_ring_peek_elem_pvt.h} |  77 +++-
 lib/librte_ring/rte_ring_peek_zc.h            |   2 +-
 lib/librte_ring/rte_ring_rts.h                |  84 +---
 ..._rts_c11_mem.h => rte_ring_rts_elem_pvt.h} |  90 +++-
 13 files changed, 652 insertions(+), 638 deletions(-)
 rename lib/librte_ring/{rte_ring_c11_mem.h => rte_ring_c11_pvt.h} (96%)
 create mode 100644 lib/librte_ring/rte_ring_elem_pvt.h
 rename lib/librte_ring/{rte_ring_generic.h => rte_ring_generic_pvt.h} (95%)
 rename lib/librte_ring/{rte_ring_hts_c11_mem.h => rte_ring_hts_elem_pvt.h} (60%)
 rename lib/librte_ring/{rte_ring_peek_c11_mem.h => rte_ring_peek_elem_pvt.h} (61%)
 rename lib/librte_ring/{rte_ring_rts_c11_mem.h => rte_ring_rts_elem_pvt.h} (62%)
Signature · 1 line
2.25.1