[lng-odp] [PATCH] validation: pktio: fix check of pktio_stop() called twice

2016-01-13 Thread Ivan Khoronzhuk 
The odp_pktio_stop() called is supposed to return error.

Signed-off-by: Ivan Khoronzhuk 
---
 test/validation/pktio/pktio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 29ad4ea..1ca4979 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -737,7 +737,7 @@ void pktio_test_start_stop(void)
/* Interfaces are stopped by default,
 * Check that stop when stopped generates an error */
ret = odp_pktio_stop(pktio[0]);
-   CU_ASSERT(ret <= 0);
+   CU_ASSERT(ret < 0);
 
/* start first */
ret = odp_pktio_start(pktio[0]);
-- 
1.9.1

___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [PATCH] validation: pktio: fix check of pktio_stop() called twice

2016-01-18 Thread Ivan Khoronzhuk 

ping

On 13.01.16 19:38, Ivan Khoronzhuk wrote:

The odp_pktio_stop() called is supposed to return error.

Signed-off-by: Ivan Khoronzhuk 
---
  test/validation/pktio/pktio.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 29ad4ea..1ca4979 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -737,7 +737,7 @@ void pktio_test_start_stop(void)
/* Interfaces are stopped by default,
 * Check that stop when stopped generates an error */
ret = odp_pktio_stop(pktio[0]);
-   CU_ASSERT(ret <= 0);
+   CU_ASSERT(ret < 0);

/* start first */
ret = odp_pktio_start(pktio[0]);



--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [PATCH] validation: pktio: fix check of pktio_stop() called twice

2016-01-20 Thread Ivan Khoronzhuk 

ping

On 13.01.16 19:38, Ivan Khoronzhuk wrote:

The odp_pktio_stop() called is supposed to return error.

Signed-off-by: Ivan Khoronzhuk 
---
  test/validation/pktio/pktio.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 29ad4ea..1ca4979 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -737,7 +737,7 @@ void pktio_test_start_stop(void)
/* Interfaces are stopped by default,
 * Check that stop when stopped generates an error */
ret = odp_pktio_stop(pktio[0]);
-   CU_ASSERT(ret <= 0);
+   CU_ASSERT(ret < 0);

/* start first */
ret = odp_pktio_start(pktio[0]);



--
Regards,
Ivan Khoronzhuk
___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Re: [lng-odp] [PATCH] validation: pktio: fix check of pktio_stop() called twice

2016-01-20 Thread Bill Fischofer 
On Wed, Jan 13, 2016 at 11:38 AM, Ivan Khoronzhuk <
ivan.khoronz...@linaro.org> wrote:

> The odp_pktio_stop() called is supposed to return error.
>
> Signed-off-by: Ivan Khoronzhuk 
>

Reviewed-by: Bill Fischofer 


> ---
>  test/validation/pktio/pktio.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
> index 29ad4ea..1ca4979 100644
> --- a/test/validation/pktio/pktio.c
> +++ b/test/validation/pktio/pktio.c
> @@ -737,7 +737,7 @@ void pktio_test_start_stop(void)
> /* Interfaces are stopped by default,
>  * Check that stop when stopped generates an error */
> ret = odp_pktio_stop(pktio[0]);
> -   CU_ASSERT(ret <= 0);
> +   CU_ASSERT(ret < 0);
>
> /* start first */
> ret = odp_pktio_start(pktio[0]);
> --
> 1.9.1
>
> ___
> 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

Re: [lng-odp] [PATCH] validation: pktio: fix check of pktio_stop() called twice

2016-01-22 Thread Maxim Uvarov 

Merged,
Maxim.

On 01/21/2016 00:32, Bill Fischofer wrote:



On Wed, Jan 13, 2016 at 11:38 AM, Ivan Khoronzhuk 
mailto:ivan.khoronz...@linaro.org>> wrote:


The odp_pktio_stop() called is supposed to return error.

Signed-off-by: Ivan Khoronzhuk mailto:ivan.khoronz...@linaro.org>>


Reviewed-by: Bill Fischofer >


---
 test/validation/pktio/pktio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/validation/pktio/pktio.c
b/test/validation/pktio/pktio.c
index 29ad4ea..1ca4979 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -737,7 +737,7 @@ void pktio_test_start_stop(void)
/* Interfaces are stopped by default,
 * Check that stop when stopped generates an error */
ret = odp_pktio_stop(pktio[0]);
-   CU_ASSERT(ret <= 0);
+   CU_ASSERT(ret < 0);

/* start first */
ret = odp_pktio_start(pktio[0]);
--
1.9.1

___
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


___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp