DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] autotests: fix pmd ring
@ 2016-03-15 12:25 Olivier Matz
  2016-03-24 20:15 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Olivier Matz @ 2016-03-15 12:25 UTC (permalink / raw)
  To: dev

This test expects that a vdev is instanciated on the command
line. If it's not the case, just skip this part.

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
 app/test/test_pmd_ring.c | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/app/test/test_pmd_ring.c b/app/test/test_pmd_ring.c
index 5568759..34fea09 100644
--- a/app/test/test_pmd_ring.c
+++ b/app/test/test_pmd_ring.c
@@ -425,7 +425,7 @@ static int
 test_pmd_ring(void)
 {
 	struct rte_ring *rxtx[NUM_RINGS];
-	int cmdl_port0 = 0;
+	int port, cmdl_port0 = -1;
 	uint8_t nb_ports;
 
 	nb_ports = rte_eth_dev_count();
@@ -501,8 +501,18 @@ test_pmd_ring(void)
 	if (test_pmd_ring_pair_create_attach(rxtx_portd, rxtx_porte) < 0)
 		return -1;
 
-	if (nb_ports > 0) {
-		/* test port 0 created with the --vdev=eth_ring0 command line option */
+	/* find a port created with the --vdev=eth_ring0 command line option */
+	for (port = 0; port < nb_ports; port++) {
+		struct rte_eth_dev_info dev_info;
+
+		rte_eth_dev_info_get(port, &dev_info);
+		if (!strcmp(dev_info.driver_name, "Rings PMD")) {
+			printf("found a command line ring port=%d\n", port);
+			cmdl_port0 = port;
+			break;
+		}
+	}
+	if (cmdl_port0 != -1) {
 		if (test_ethdev_configure_port(cmdl_port0) < 0)
 			return -1;
 		if (test_send_basic_packets_port(cmdl_port0) < 0)
-- 
2.1.4

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] autotests: fix pmd ring
  2016-03-15 12:25 [dpdk-dev] [PATCH] autotests: fix pmd ring Olivier Matz
@ 2016-03-24 20:15 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2016-03-24 20:15 UTC (permalink / raw)
  To: Olivier Matz; +Cc: dev

2016-03-15 13:25, Olivier Matz:
> This test expects that a vdev is instanciated on the command
> line. If it's not the case, just skip this part.
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>

Applied, thanks

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-24 20:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-15 12:25 [dpdk-dev] [PATCH] autotests: fix pmd ring Olivier Matz
2016-03-24 20:15 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).