diff mbox

validation: buffer: fix invalid packet segment macro

Message ID 1419872035-12731-2-git-send-email-taras.kondratiuk@linaro.org
State Accepted
Commit fd46d82eb9f12405d919a0a45e6745391c52e18b
Headers show

Commit Message

Taras Kondratiuk Dec. 29, 2014, 4:53 p.m. UTC
ODP_PACKET_SEG_INVALID is a correct macro for an invalid packet segment.

Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
---
 test/validation/buffer/odp_packet_test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Balasubramanian Manoharan Dec. 30, 2014, 6:02 a.m. UTC | #1
Reviewed-by: Bala Manoharan <bala.manoharan@linaro.org>
On Monday 29 December 2014 10:23 PM, Taras Kondratiuk wrote:
> ODP_PACKET_SEG_INVALID is a correct macro for an invalid packet segment.
>
> Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
> ---
>   test/validation/buffer/odp_packet_test.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/validation/buffer/odp_packet_test.c b/test/validation/buffer/odp_packet_test.c
> index 57a9aa9..d073b49 100644
> --- a/test/validation/buffer/odp_packet_test.c
> +++ b/test/validation/buffer/odp_packet_test.c
> @@ -267,7 +267,7 @@ static void _verify_tailroom_shift(odp_packet_t pkt,
>   	tail_orig = odp_packet_tail(pkt);
>   
>   	seg = odp_packet_last_seg(pkt);
> -	CU_ASSERT(seg != ODP_SEGMENT_INVALID);
> +	CU_ASSERT(seg != ODP_PACKET_SEG_INVALID);
>   	seg_data_len = odp_packet_seg_data_len(pkt, seg);
>   
>   	if (shift >= 0) {
> @@ -309,7 +309,7 @@ static void packet_tailroom(void)
>   	uint32_t push_val, pull_val;
>   
>   	segment = odp_packet_last_seg(pkt);
> -	CU_ASSERT(segment != ODP_SEGMENT_INVALID);
> +	CU_ASSERT(segment != ODP_PACKET_SEG_INVALID);
>   	room = odp_packet_tailroom(pkt);
>   #if ODP_CONFIG_PACKET_TAILROOM != 0 /* Avoid 'always true' warning */
>   	CU_ASSERT(room >= ODP_CONFIG_PACKET_TAILROOM);
Mike Holmes Jan. 6, 2015, 4:20 p.m. UTC | #2
Applied

On 30 December 2014 at 01:02, Bala <bala.manoharan@linaro.org> wrote:

> Reviewed-by: Bala Manoharan <bala.manoharan@linaro.org>
>
> On Monday 29 December 2014 10:23 PM, Taras Kondratiuk wrote:
>
>> ODP_PACKET_SEG_INVALID is a correct macro for an invalid packet segment.
>>
>> Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
>> ---
>>   test/validation/buffer/odp_packet_test.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/test/validation/buffer/odp_packet_test.c
>> b/test/validation/buffer/odp_packet_test.c
>> index 57a9aa9..d073b49 100644
>> --- a/test/validation/buffer/odp_packet_test.c
>> +++ b/test/validation/buffer/odp_packet_test.c
>> @@ -267,7 +267,7 @@ static void _verify_tailroom_shift(odp_packet_t pkt,
>>         tail_orig = odp_packet_tail(pkt);
>>         seg = odp_packet_last_seg(pkt);
>> -       CU_ASSERT(seg != ODP_SEGMENT_INVALID);
>> +       CU_ASSERT(seg != ODP_PACKET_SEG_INVALID);
>>         seg_data_len = odp_packet_seg_data_len(pkt, seg);
>>         if (shift >= 0) {
>> @@ -309,7 +309,7 @@ static void packet_tailroom(void)
>>         uint32_t push_val, pull_val;
>>         segment = odp_packet_last_seg(pkt);
>> -       CU_ASSERT(segment != ODP_SEGMENT_INVALID);
>> +       CU_ASSERT(segment != ODP_PACKET_SEG_INVALID);
>>         room = odp_packet_tailroom(pkt);
>>   #if ODP_CONFIG_PACKET_TAILROOM != 0 /* Avoid 'always true' warning */
>>         CU_ASSERT(room >= ODP_CONFIG_PACKET_TAILROOM);
>>
>
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/lng-odp
>
diff mbox

Patch

diff --git a/test/validation/buffer/odp_packet_test.c b/test/validation/buffer/odp_packet_test.c
index 57a9aa9..d073b49 100644
--- a/test/validation/buffer/odp_packet_test.c
+++ b/test/validation/buffer/odp_packet_test.c
@@ -267,7 +267,7 @@  static void _verify_tailroom_shift(odp_packet_t pkt,
 	tail_orig = odp_packet_tail(pkt);
 
 	seg = odp_packet_last_seg(pkt);
-	CU_ASSERT(seg != ODP_SEGMENT_INVALID);
+	CU_ASSERT(seg != ODP_PACKET_SEG_INVALID);
 	seg_data_len = odp_packet_seg_data_len(pkt, seg);
 
 	if (shift >= 0) {
@@ -309,7 +309,7 @@  static void packet_tailroom(void)
 	uint32_t push_val, pull_val;
 
 	segment = odp_packet_last_seg(pkt);
-	CU_ASSERT(segment != ODP_SEGMENT_INVALID);
+	CU_ASSERT(segment != ODP_PACKET_SEG_INVALID);
 	room = odp_packet_tailroom(pkt);
 #if ODP_CONFIG_PACKET_TAILROOM != 0 /* Avoid 'always true' warning */
 	CU_ASSERT(room >= ODP_CONFIG_PACKET_TAILROOM);