validation: pktio: reduce stdout noise

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.haslam@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(-)

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.haslam@linaro.org>

Reviewed-by: Ivan Khoronzhuk <ivan.khoronzhuk@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),
>

Merged,
Maxim.

On 01/28/2016 12:42, Elo, Matias (Nokia - FI/Espoo) wrote:
> ping
>
>> -----Original Message-----
>> From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of EXT Ivan
>> Khoronzhuk
>> Sent: Friday, January 15, 2016 6:24 PM
>> To: Stuart Haslam <stuart.haslam@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.haslam@linaro.org>
>> Reviewed-by: Ivan Khoronzhuk <ivan.khoronzhuk@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