[dpdk-dev] [PATCH v2 00/15] Unit tests fixes for CI

David Marchand david.marchand at redhat.com
Sat Jun 15 08:42:15 CEST 2019


This is a joint effort to make the unit tests ready for CI.
The first patches are fixes that I had accumulated.
Then the second part of the series focuses on skipping tests when some
requirements are not fulfilled so that we can start them in a restrained
environment like Travis virtual machines that gives us two cores and does
not have specific hw devices.

We are still not ready for enabling those tests in Travis.
At least, the following issues remain:
- some fixes on librte_acl have not been merged yet [1],
- the tests on --file-prefix are still ko, and have been isolated in a
  test that we could disable while waiting for the fixes,
- rwlock_autotest and hash_readwrite_lf_autotest are taking a little more
  than 10s,
- librte_table unit test crashes on ipv6 [2],
- the "perf" tests are taking way too long for my taste,
- the shared build unit tests all fail when depending on mempool since
  the mempool drivers are not loaded,

1: http://patchwork.dpdk.org/project/dpdk/list/?series=4242
2: https://bugs.dpdk.org/show_bug.cgi?id=285

Changelog since v1:
- removed limit on 128 cores in rcu tests,
- reworked Michael patch on eal test and started splitting big tests into
  subtests: when a subtest fails, it does not impact the other subtests;
  plus, subtests are shorter to run, so easier to make them fit in 10s,

Comments/reviews welcome!

-- 
David Marchand

David Marchand (13):
  test/bonding: add missing sources for link bonding RSS
  test/crypto: move tests to the driver specific list
  test/eventdev: move tests to the driver specific list
  test/hash: fix off-by-one check on core count
  test/hash: clean remaining trace of scaling autotest
  test/latencystats: fix stack smashing
  test/rcu: remove arbitrary limit on max core count
  test/stack: fix lock-free test name
  test/eal: set memory channel config only in dedicated test
  test/eal: set core mask/list config only in dedicated test
  test: split into shorter subtests for CI
  test: do not start tests in parallel
  test: skip tests when missing requirements

Dharmik Thakkar (1):
  test/hash: rectify slaveid to point to valid cores

Michael Santana (1):
  test/eal: check number of cores before running subtests

 app/test/autotest.py                |   2 +-
 app/test/autotest_data.py           |   4 +-
 app/test/meson.build                |  84 ++++++------
 app/test/test.c                     |  24 ++--
 app/test/test_compressdev.c         |   4 +-
 app/test/test_cryptodev.c           |   4 +-
 app/test/test_distributor.c         |   4 +-
 app/test/test_distributor_perf.c    |   4 +-
 app/test/test_eal_flags.c           | 265 ++++++++++++++++++++----------------
 app/test/test_event_timer_adapter.c |   5 +-
 app/test/test_eventdev.c            |   2 +
 app/test/test_func_reentrancy.c     |   6 +-
 app/test/test_hash_multiwriter.c    |   7 +-
 app/test/test_hash_readwrite.c      |   7 +-
 app/test/test_hash_readwrite_lf.c   |  34 +++--
 app/test/test_ipsec.c               |   4 +-
 app/test/test_latencystats.c        |  18 ---
 app/test/test_mbuf.c                |  13 +-
 app/test/test_rcu_qsbr.c            | 139 +++++++++----------
 app/test/test_rcu_qsbr_perf.c       |  73 ++++------
 app/test/test_rwlock.c              |   6 +
 app/test/test_service_cores.c       |  14 ++
 app/test/test_stack.c               |   8 +-
 app/test/test_timer.c               |  10 +-
 app/test/test_timer_secondary.c     |  10 +-
 25 files changed, 381 insertions(+), 370 deletions(-)

-- 
1.8.3.1



More information about the dev mailing list