mbox series

[0/2] Adding a missing queue count API in rawdev

Message ID 20180731103303.14988-1-shreyansh.jain@nxp.com (mailing list archive)
Headers
Series Adding a missing queue count API in rawdev |

Message

Shreyansh Jain July 31, 2018, 10:33 a.m. UTC 
  Recently, off the list, Keith pointed out that the rte_rawdev_queue_count
API was incomplete - only the declaration existed but no definition. This
is an old miss (and embarassingly, almost two release cycle long miss!)

This patch adds that and the relevant unit test case.

Ferruh: Though I am sending it as a fix hoping it would be possible to
merge as 'fix' for 18.08, if not, I will resend with map file changes
for early merge in 1811 window.
The changes are completely independent of any other API or driver impl.

Shreyansh Jain (2):
  rawdev: fix missing queue count API
  raw/skeleton: implement queue count API

 drivers/raw/skeleton_rawdev/skeleton_rawdev.c | 13 ++++++++++++
 .../skeleton_rawdev/skeleton_rawdev_test.c    | 13 ++++++++++++
 lib/librte_rawdev/rte_rawdev.c                | 12 +++++++++++
 lib/librte_rawdev/rte_rawdev.h                |  1 +
 lib/librte_rawdev/rte_rawdev_pmd.h            | 20 +++++++++++++++++++
 lib/librte_rawdev/rte_rawdev_version.map      |  1 +
 6 files changed, 60 insertions(+)

Comments

Thomas Monjalon Aug. 1, 2018, 4:26 p.m. UTC | #1 
31/07/2018 12:33, Shreyansh Jain:
> Recently, off the list, Keith pointed out that the rte_rawdev_queue_count
> API was incomplete - only the declaration existed but no definition. This
> is an old miss (and embarassingly, almost two release cycle long miss!)
> 
> This patch adds that and the relevant unit test case.
> 
> Ferruh: Though I am sending it as a fix hoping it would be possible to
> merge as 'fix' for 18.08, if not, I will resend with map file changes
> for early merge in 1811 window.
> The changes are completely independent of any other API or driver impl.
> 
> Shreyansh Jain (2):
>   rawdev: fix missing queue count API
>   raw/skeleton: implement queue count API

Applied, thanks