diff mbox

odp_packet_io.c: Fix unreachable code

Message ID 1410912693-31847-1-git-send-email-mike.holmes@linaro.org
State Rejected
Headers show

Commit Message

Mike Holmes Sept. 17, 2014, 12:11 a.m. UTC 
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(-)

Comments

Maxim Uvarov Sept. 22, 2014, 9:19 a.m. UTC | #1 
On 09/17/2014 04:11 AM, Mike Holmes wrote:
> @@ -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);
>   		}
>   	}

Mike, I think your fix is not correct. You need something like that:

         lock_entry(entry);
         if (!is_free(entry)) {
                 switch (entry->s.params.type) {
                 case ODP_PKTIO_TYPE_SOCKET_BASIC:
                 case ODP_PKTIO_TYPE_SOCKET_MMSG:
                  .....
                         break;
                 default:
                         break;
  -               res |= free_pktio_entry(id);
                 }
  + /*  set entry->s.taken = 0 when freeing entry */
  +    res |= free_pktio_entry(id);
         }
         unlock_entry(entry);


Maxim.
diff mbox

Patch

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: