[PATCHv2,5/7] validation: pktio: pass interface index rather than name

Message ID 1441360547-18999-5-git-send-email-stuart.haslam@linaro.org
State New
Headers show

Commit Message

Stuart Haslam Sept. 4, 2015, 9:55 a.m.
Avoid the need to pass the interface name to the create_pktio() function
as it can be derived from the index which must also be passed. This
means the caller can't make the mistake of passing a mismatched name and
index - there was one instance of this which is now fixed.

Signed-off-by: Stuart Haslam <stuart.haslam@linaro.org>
---
 test/validation/pktio/pktio.c | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

Patch

diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c
index 50cd901..c74fab3 100644
--- a/test/validation/pktio/pktio.c
+++ b/test/validation/pktio/pktio.c
@@ -250,11 +250,11 @@  static int default_pool_create(void)
 	return 0;
 }
 
-static odp_pktio_t create_pktio(const char *iface, odp_queue_type_t q_type,
-				int num)
+static odp_pktio_t create_pktio(int iface_idx, odp_queue_type_t q_type)
 {
 	odp_pktio_t pktio;
 	odp_pktio_param_t pktio_param;
+	const char *iface = iface_name[iface_idx];
 
 	memset(&pktio_param, 0, sizeof(pktio_param));
 
@@ -263,7 +263,7 @@  static odp_pktio_t create_pktio(const char *iface, odp_queue_type_t q_type,
 	else
 		pktio_param.in_mode = ODP_PKTIN_MODE_SCHED;
 
-	pktio = odp_pktio_open(iface, pool[num], &pktio_param);
+	pktio = odp_pktio_open(iface, pool[iface_idx], &pktio_param);
 	if (pktio == ODP_PKTIO_INVALID)
 		pktio = odp_pktio_lookup(iface);
 	CU_ASSERT(pktio != ODP_PKTIO_INVALID);
@@ -461,7 +461,7 @@  static void test_txrx(odp_queue_type_t q_type, int num_pkts)
 		io = &pktios[i];
 
 		io->name = iface_name[i];
-		io->id   = create_pktio(iface_name[i], q_type, i);
+		io->id   = create_pktio(i, q_type);
 		if (io->id == ODP_PKTIO_INVALID) {
 			CU_FAIL("failed to open iface");
 			return;
@@ -520,7 +520,7 @@  void pktio_test_mtu(void)
 {
 	int ret;
 	int mtu;
-	odp_pktio_t pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+	odp_pktio_t pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 
 	mtu = odp_pktio_mtu(pktio);
 	CU_ASSERT(mtu > 0);
@@ -534,7 +534,7 @@  void pktio_test_mtu(void)
 void pktio_test_promisc(void)
 {
 	int ret;
-	odp_pktio_t pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+	odp_pktio_t pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 
 	ret = odp_pktio_promisc_mode_set(pktio, 1);
 	CU_ASSERT(0 == ret);
@@ -561,7 +561,7 @@  void pktio_test_mac(void)
 	int ret;
 	odp_pktio_t pktio;
 
-	pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+	pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 
 	mac_len = odp_pktio_mac_addr(pktio, mac_addr, sizeof(mac_addr));
 	CU_ASSERT(ODPH_ETHADDR_LEN == mac_len);
@@ -582,7 +582,7 @@  void pktio_test_inq_remdef(void)
 	uint64_t wait;
 	int i;
 
-	pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+	pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 	CU_ASSERT(pktio != ODP_PKTIO_INVALID);
 	CU_ASSERT(create_inq(pktio, ODP_QUEUE_TYPE_POLL) == 0);
 	inq = odp_pktio_inq_getdef(pktio);
@@ -610,7 +610,7 @@  void pktio_test_open(void)
 
 	/* test the sequence open->close->open->close() */
 	for (i = 0; i < 2; ++i) {
-		pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+		pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 		CU_ASSERT(pktio != ODP_PKTIO_INVALID);
 		CU_ASSERT(odp_pktio_close(pktio) == 0);
 	}
@@ -649,7 +649,7 @@  void pktio_test_inq(void)
 {
 	odp_pktio_t pktio;
 
-	pktio = create_pktio(iface_name[0], ODP_QUEUE_TYPE_SCHED, 0);
+	pktio = create_pktio(0, ODP_QUEUE_TYPE_SCHED);
 	CU_ASSERT(pktio != ODP_PKTIO_INVALID);
 
 	CU_ASSERT(create_inq(pktio, ODP_QUEUE_TYPE_POLL) == 0);
@@ -668,7 +668,7 @@  static void pktio_test_start_stop(void)
 	uint64_t wait = odp_schedule_wait_time(ODP_TIME_MSEC);
 
 	for (i = 0; i < num_ifaces; i++) {
-		pktio[i] = create_pktio(iface_name[i], ODP_QUEUE_TYPE_SCHED, 0);
+		pktio[i] = create_pktio(i, ODP_QUEUE_TYPE_SCHED);
 		CU_ASSERT(pktio[i] != ODP_PKTIO_INVALID);
 		create_inq(pktio[i],  ODP_QUEUE_TYPE_SCHED);
 	}