diff mbox

odp_packet_io.c: Fix unreachable code

Message ID 54403DC2.2040203@linaro.org
State New
Headers show

Commit Message

Maxim Uvarov Oct. 16, 2014, 9:50 p.m. UTC 
Mike this patch needed update to the latest odp code.

Also it will be good to split it on 2: one for indent and other to 
exactly problem. Just for better review.
When applying I think that your fix is not quite correct. And I suggest 
to do it in that way:



Maxim.

On 09/17/2014 04:11 AM, Mike Holmes wrote:
> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
> ---
> Also fixed indentation
>
>   platform/linux-generic/odp_packet_io.c | 22 +++++++++++-----------
>   1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c
> index 33ade10..78d5243 100644
> --- a/platform/linux-generic/odp_packet_io.c
> +++ b/platform/linux-generic/odp_packet_io.c
> @@ -122,7 +122,7 @@ static void init_pktio_entry(pktio_entry_t *entry, odp_pktio_params_t *params)
>   	case ODP_PKTIO_TYPE_SOCKET_MMAP:
>   		memset(&entry->s.pkt_sock, 0, sizeof(entry->s.pkt_sock));
>   		memset(&entry->s.pkt_sock_mmap, 0,
> -		      sizeof(entry->s.pkt_sock_mmap));
> +		       sizeof(entry->s.pkt_sock_mmap));
>   		break;
>   #ifdef ODP_HAVE_NETMAP
>   	case ODP_PKTIO_TYPE_NETMAP:
> @@ -220,7 +220,7 @@ odp_pktio_t odp_pktio_open(const char *dev, odp_buffer_pool_t pool,
>   		break;
>   	case ODP_PKTIO_TYPE_SOCKET_MMAP:
>   		res = setup_pkt_sock_mmap(&pktio_entry->s.pkt_sock_mmap, dev,
> -				pool, params->sock_params.fanout);
> +					  pool, params->sock_params.fanout);
>   		if (res == -1) {
>   			close_pkt_sock_mmap(&pktio_entry->s.pkt_sock_mmap);
>   			free_pktio_entry(id);
> @@ -231,7 +231,7 @@ odp_pktio_t odp_pktio_open(const char *dev, odp_buffer_pool_t pool,
>   	case ODP_PKTIO_TYPE_NETMAP:
>   
>   		res = setup_pkt_netmap(&pktio_entry->s.pkt_nm, dev,
> -				pool, &params->nm_params);
> +				       pool, &params->nm_params);
>   		if (res == -1) {
>   			close_pkt_netmap(&pktio_entry->s.pkt_nm);
>   			free_pktio_entry(id);
> @@ -275,8 +275,8 @@ int odp_pktio_close(odp_pktio_t id)
>   			break;
>   #endif
>   		default:
> +			res |= free_pktio_entry(id);
>   			break;
> -		res |= free_pktio_entry(id);
>   		}
>   	}
>   	unlock_entry(entry);
> @@ -310,15 +310,15 @@ int odp_pktio_recv(odp_pktio_t id, odp_packet_t pkt_table[], unsigned len)
>   	switch (pktio_entry->s.params.type) {
>   	case ODP_PKTIO_TYPE_SOCKET_BASIC:
>   		pkts = recv_pkt_sock_basic(&pktio_entry->s.pkt_sock,
> -				pkt_table, len);
> +					   pkt_table, len);
>   		break;
>   	case ODP_PKTIO_TYPE_SOCKET_MMSG:
>   		pkts = recv_pkt_sock_mmsg(&pktio_entry->s.pkt_sock,
> -				pkt_table, len);
> +					  pkt_table, len);
>   		break;
>   	case ODP_PKTIO_TYPE_SOCKET_MMAP:
>   		pkts = recv_pkt_sock_mmap(&pktio_entry->s.pkt_sock_mmap,
> -				pkt_table, len);
> +					  pkt_table, len);
>   		break;
>   #ifdef ODP_HAVE_NETMAP
>   	case ODP_PKTIO_TYPE_NETMAP:
> @@ -352,20 +352,20 @@ int odp_pktio_send(odp_pktio_t id, odp_packet_t pkt_table[], unsigned len)
>   	switch (pktio_entry->s.params.type) {
>   	case ODP_PKTIO_TYPE_SOCKET_BASIC:
>   		pkts = send_pkt_sock_basic(&pktio_entry->s.pkt_sock,
> -				pkt_table, len);
> +					   pkt_table, len);
>   		break;
>   	case ODP_PKTIO_TYPE_SOCKET_MMSG:
>   		pkts = send_pkt_sock_mmsg(&pktio_entry->s.pkt_sock,
> -				pkt_table, len);
> +					  pkt_table, len);
>   		break;
>   	case ODP_PKTIO_TYPE_SOCKET_MMAP:
>   		pkts = send_pkt_sock_mmap(&pktio_entry->s.pkt_sock_mmap,
> -				pkt_table, len);
> +					  pkt_table, len);
>   		break;
>   #ifdef ODP_HAVE_NETMAP
>   	case ODP_PKTIO_TYPE_NETMAP:
>   		pkts = send_pkt_netmap(&pktio_entry->s.pkt_nm,
> -				pkt_table, len);
> +				       pkt_table, len);
>   		break;
>   #endif
>   	default:
diff mbox

Patch

--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -227,9 +227,11 @@  int odp_pktio_close(odp_pktio_t id)
                         res  = 
close_pkt_sock_mmap(&entry->s.pkt_sock_mmap);
                         break;
                 default:
+                       ODP_ABORT("wrong packet i/o branch\n");
                         break;
-               res |= free_pktio_entry(id);
                 }
+               /* free packet i/o entry on close */
+               res |= free_pktio_entry(id);
         }
         unlock_entry(entry);