ping
> -----Original Message----- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of EXT Ivan > Khoronzhuk > Sent: Friday, January 15, 2016 6:24 PM > To: Stuart Haslam <stuart.has...@linaro.org>; lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCH] validation: pktio: reduce stdout noise > > > > 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 _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp
