On 15.01.16 18:18, Stuart Haslam wrote:
Printing the pktio info every time a pktio is opened generates a load of noise on stdout and makes it difficult to see the status of individual test cases. Signed-off-by: Stuart Haslam <stuart.has...@linaro.org>
Reviewed-by: Ivan Khoronzhuk <ivan.khoronz...@linaro.org>
--- Log goes from this: http://people.linaro.org/~stuart.haslam/pktio1.log To this: http://people.linaro.org/~stuart.haslam/pktio2.log test/validation/pktio/pktio.c | 23 +++++++++++++++++++---- 1 file changed, 19 insertions(+), 4 deletions(-) diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index b3732bb..9c8c871 100644 --- a/test/validation/pktio/pktio.c +++ b/test/validation/pktio/pktio.c @@ -293,10 +293,6 @@ static odp_pktio_t create_pktio(int iface_idx, odp_pktio_input_mode_t imode, CU_ASSERT(pktio != ODP_PKTIO_INVALID); CU_ASSERT(odp_pktio_to_u64(pktio) != odp_pktio_to_u64(ODP_PKTIO_INVALID)); - /* Print pktio debug info and test that the odp_pktio_print() function - * is implemented. */ - if (pktio != ODP_PKTIO_INVALID) - odp_pktio_print(pktio); if (wait_for_network) spin_wait(ODP_TIME_SEC_IN_NS / 4); @@ -734,6 +730,24 @@ void pktio_test_lookup(void) CU_ASSERT(odp_pktio_lookup(iface_name[0]) == ODP_PKTIO_INVALID); } +static void pktio_test_print(void) +{ + odp_pktio_t pktio; + int i; + + for (i = 0; i < num_ifaces; ++i) { + pktio = create_pktio(i, ODP_PKTIN_MODE_POLL, + ODP_PKTOUT_MODE_SEND); + CU_ASSERT_FATAL(pktio != ODP_PKTIO_INVALID); + + /* Print pktio debug info and test that the + * odp_pktio_print() function is implemented. */ + odp_pktio_print(pktio); + + CU_ASSERT(odp_pktio_close(pktio) == 0); + } +} + void pktio_test_inq(void) { odp_pktio_t pktio; @@ -1217,6 +1231,7 @@ int pktio_suite_term(void) odp_testinfo_t pktio_suite_unsegmented[] = { ODP_TEST_INFO(pktio_test_open), ODP_TEST_INFO(pktio_test_lookup), + ODP_TEST_INFO(pktio_test_print), ODP_TEST_INFO(pktio_test_inq), ODP_TEST_INFO(pktio_test_poll_queue), ODP_TEST_INFO(pktio_test_poll_multi),
-- Regards, Ivan Khoronzhuk _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp
