[v3,0/3] fixes and improvements for rawdev
mbox series

Message ID 20190702095608.20722-1-bruce.richardson@intel.com
Headers show
Series
  • fixes and improvements for rawdev
Related show

Message

Bruce Richardson July 2, 2019, 9:56 a.m. UTC
This set includes some general fixes and improvements for rawdev, the
rawdev autotests and the rawdev skeleton example implementation.
Specifically:

* the unit test were failing for me due to bad memory management
* the unit tests were not present in the meson builds due to missing
  file in file list
* the self-test function for a rawdev driver did not take the actual
  device id as a parameter:
  - because of this the skeleton driver always assumed device 0 was to
    be tested, an unsafe assumption

V3:
	* dropped patch 1 which has been independently fixed
	* added autotest to the drivers set rather than fast tests one

V2:
	extended the set with additional fixes to allow selftests to know
	what instance the tests are to be run on.

*** SUBJECT HERE ***

*** BLURB HERE ***

Bruce Richardson (3):
  app/test: add missing rawdev autotest to meson build
  raw/skeleton: remove compile-time constant for device id
  rawdev: pass the device id as parameter to selftest

 app/test/meson.build                          |  3 +
 drivers/raw/skeleton_rawdev/skeleton_rawdev.c |  8 ++-
 drivers/raw/skeleton_rawdev/skeleton_rawdev.h |  2 +-
 .../skeleton_rawdev/skeleton_rawdev_test.c    | 58 ++++++++++---------
 lib/librte_rawdev/rte_rawdev.c                |  2 +-
 lib/librte_rawdev/rte_rawdev_pmd.h            |  2 +-
 6 files changed, 41 insertions(+), 34 deletions(-)

Comments

Thomas Monjalon July 2, 2019, 3:05 p.m. UTC | #1
> Bruce Richardson (3):
>   app/test: add missing rawdev autotest to meson build
>   raw/skeleton: remove compile-time constant for device id
>   rawdev: pass the device id as parameter to selftest

Applied, thanks