diff mbox series

[v1,3/3] linux-gen: pktio: do not set pktio ops on not allocated state

Message ID 1508338806-1394-4-git-send-email-odpbot@yandex.ru
State New
Headers show
Series [v1,1/3] linux-gen: pktio: add missing unlock | expand

Commit Message

Github ODP bot Oct. 18, 2017, 3 p.m. UTC 
From: Maxim Uvarov <maxim.uvarov@linaro.org>


Simplify ret code.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

---
/** Email created from pull request 239 (muvarov:master_bug3242)
 ** https://github.com/Linaro/odp/pull/239
 ** Patch: https://github.com/Linaro/odp/pull/239.patch
 ** Base sha: ec0c3145fcafa09ae3a79875e7e07dd4794583cc
 ** Merge commit sha: 13cc3007dc36c8942ffe7dd834f3812b32ae2ae3
 **/
 platform/linux-generic/odp_packet_io.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)
diff mbox series

Patch

diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c
index 4363711fb..f972bb78f 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -215,27 +215,25 @@  static odp_pktio_t setup_pktio_entry(const char *name, odp_pool_t pool,
 	for (pktio_if = 0; pktio_if_ops[pktio_if]; ++pktio_if) {
 		ret = pktio_if_ops[pktio_if]->open(hdl, pktio_entry, name,
 						   pool);
-
-		if (!ret) {
-			pktio_entry->s.ops = pktio_if_ops[pktio_if];
-			ODP_DBG("%s uses %s\n",
-				name, pktio_if_ops[pktio_if]->name);
+		if (!ret)
 			break;
-		}
 	}
 
 	if (ret != 0) {
 		pktio_entry->s.state = PKTIO_STATE_FREE;
-		hdl = ODP_PKTIO_INVALID;
+		unlock_entry(pktio_entry);
 		ODP_ERR("Unable to init any I/O type.\n");
-	} else {
-		snprintf(pktio_entry->s.name,
-			 sizeof(pktio_entry->s.name), "%s", name);
-		pktio_entry->s.state = PKTIO_STATE_OPENED;
+		return ODP_PKTIO_INVALID;
 	}
 
+	snprintf(pktio_entry->s.name,
+		 sizeof(pktio_entry->s.name), "%s", name);
+	pktio_entry->s.state = PKTIO_STATE_OPENED;
+	pktio_entry->s.ops = pktio_if_ops[pktio_if];
 	unlock_entry(pktio_entry);
 
+	ODP_DBG("%s uses %s\n", name, pktio_if_ops[pktio_if]->name);
+
 	return hdl;
 }