mbox series

[v2,0/3] refactoring ring library

Message ID 20210126081746.1580627-1-feifei.wang2@arm.com (mailing list archive)
Headers
Series refactoring ring library |

Message

Feifei Wang Jan. 26, 2021, 8:17 a.m. UTC
  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)

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} |  88 +++-
 lib/librte_ring/rte_ring_peek.h               |  71 +---
 ...eek_c11_mem.h => rte_ring_peek_elem_pvt.h} |  75 +++-
 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} |  88 +++-
 13 files changed, 649 insertions(+), 635 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%)
  

Comments

David Marchand Jan. 28, 2021, 11:05 a.m. UTC | #1
On Tue, Jan 26, 2021 at 9:18 AM Feifei Wang <feifei.wang2@arm.com> wrote:
>
> 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)

This series seems simple enough and safe for taking in rc2.
Honnappa and Konstantin seem ok with it, so I'll proceed unless I hear
any objection by tomorrow.


Thanks.