diff mbox series

[04/51] kni: check status of getting ethdev info

Message ID	1566915962-5472-5-git-send-email-arybchenko@solarflare.com (mailing list archive)
State	Superseded, archived
Delegated to:	Ferruh Yigit
Headers	From: Andrew Rybchenko <arybchenko@solarflare.com> To: Ferruh Yigit <ferruh.yigit@intel.com> CC: <dev@dpdk.org>, Ivan Ilchenko <Ivan.Ilchenko@oktetlabs.ru> Date: Tue, 27 Aug 2019 15:25:15 +0100 Message-ID: <1566915962-5472-5-git-send-email-arybchenko@solarflare.com> In-Reply-To: <1566915962-5472-1-git-send-email-arybchenko@solarflare.com> References: <1566915962-5472-1-git-send-email-arybchenko@solarflare.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [dpdk-dev] [PATCH 04/51] kni: check status of getting ethdev info Precedence: list Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org>
Series	ethdev: change rte_eth_dev_info_get() return value to int \| [00/51] ethdev: change rte_eth_dev_info_get() return value to int [01/51] ethdev: change rte_eth_dev_info_get() return value to int [02/51] app/testpmd: check status of getting ethdev info [03/51] app/eventdev: check status of getting ethdev info [04/51] kni: check status of getting ethdev info [05/51] latency: check status of getting ethdev info [06/51] pdump: check status of getting ethdev info [07/51] ring: check status of getting ethdev info [08/51] app/procinfo: check status of getting ethdev info [09/51] app/test: check status of getting ethdev info in bonding [10/51] app/test: check ethdev info get result in event Rx adapter [11/51] net/bnxt: check status of getting ethdev info [12/51] net/bonding: check status of getting ethdev info [13/51] net/netvsc: check status of getting ethdev info [14/51] net/softnic: check status of getting ethdev info [15/51] examples/rxtx_callbacks: check status of getting ethdev info [16/51] examples/l3fwd: check status of getting ethdev info [17/51] examples/qos_meter: check status of getting ethdev info [18/51] examples/ip_frag: check status of getting ethdev info [19/51] examples/performance-thread: check dev info get result [20/51] examples/vmdq: check status of getting ethdev info [21/51] examples/distributor: check status of getting ethdev info [22/51] examples/l3fwd-acl: check status of getting ethdev info [23/51] examples/vm_power: check status of getting ethdev info [24/51] examples/qos_sched: check status of getting ethdev info [25/51] examples/flow_filtering: check status of getting ethdev info [26/51] examples/l3fwd-power: check status of getting ethdev info [27/51] examples/l2fwd: check status of getting ethdev info [28/51] examples/skeleton: check status of getting ethdev info [29/51] examples/vmdq_dcb: check status of getting ethdev info [30/51] examples/ipv4_multicast: check status of getting ethdev info [31/51] examples/l2fwd-jobstats: check status of getting ethdev info [32/51] examples/bond: check status of getting ethdev info [33/51] examples/eventdev: check status of getting ethdev info [34/51] examples/ip_reassembly: check status of getting ethdev info [35/51] examples/vhost: check status of getting ethdev info [36/51] examples/ptpclient: check status of getting ethdev info [37/51] examples/link_status_interrupt: check dev info get result [38/51] examples/tep_termination: check dev info get result [39/51] examples/server_node_efd: check dev info get result [40/51] examples/flow_classify: check status of getting ethdev info [41/51] examples/packet_ordering: check dev info get result [42/51] examples/l2fwd-crypto: check status of getting ethdev info [43/51] examples/multi_process: check status of getting ethdev info [44/51] examples/ipsec-secgw: check status of getting ethdev info [45/51] examples/netmap_compat: check status of getting ethdev info [46/51] examples/l2fwd-keepalive: check dev info get result [47/51] examples/ip_pipeline: check status of getting ethdev info [48/51] examples/load_balancer: check status of getting ethdev info [49/51] examples/kni: check status of getting ethdev info [50/51] examples/ethtool: check status of getting ethdev info [51/51] examples/exception_path: check status of getting ethdev info

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK
ci/Intel-compilation	success	Compilation OK

Commit Message

Andrew Rybchenko Aug. 27, 2019, 2:25 p.m. UTC

  From: Ivan Ilchenko <Ivan.Ilchenko@oktetlabs.ru>

rte_eth_dev_info_get() return value was changed from void to int,
so this patch modify rte_eth_dev_info_get() usage across
app/test/test_kni.c according to its new return type.

Signed-off-by: Ivan Ilchenko <Ivan.Ilchenko@oktetlabs.ru>
Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
---
 app/test/test_kni.c | 27 ++++++++++++++++++++++++---
 1 file changed, 24 insertions(+), 3 deletions(-)

diff mbox series

Patch

diff --git a/app/test/test_kni.c b/app/test/test_kni.c
index 7a65de1..2c33374 100644
--- a/app/test/test_kni.c
+++ b/app/test/test_kni.c
@@ -436,7 +436,13 @@  struct test_kni_stats {
 	memset(&info, 0, sizeof(info));
 	memset(&ops, 0, sizeof(ops));
 
-	rte_eth_dev_info_get(port_id, &info);
+	ret = rte_eth_dev_info_get(port_id, &info);
+	if (ret != 0) {
+		printf("Error during getting device (port %u) info: %s\n",
+				port_id, strerror(-ret));
+		return -1;
+	}
+
 	if (info.device)
 		bus = rte_bus_find_by_device(info.device);
 	if (bus && !strcmp(bus->name, "pci")) {
@@ -622,7 +628,14 @@  struct test_kni_stats {
 	memset(&info, 0, sizeof(info));
 	memset(&conf, 0, sizeof(conf));
 	memset(&ops, 0, sizeof(ops));
-	rte_eth_dev_info_get(port_id, &info);
+
+	ret = rte_eth_dev_info_get(port_id, &info);
+	if (ret != 0) {
+		printf("Error during getting device (port %u) info: %s\n",
+				port_id, strerror(-ret));
+		return -1;
+	}
+
 	if (info.device)
 		bus = rte_bus_find_by_device(info.device);
 	else
@@ -658,7 +671,15 @@  struct test_kni_stats {
 	memset(&conf, 0, sizeof(conf));
 	memset(&info, 0, sizeof(info));
 	memset(&ops, 0, sizeof(ops));
-	rte_eth_dev_info_get(port_id, &info);
+
+	ret = rte_eth_dev_info_get(port_id, &info);
+	if (ret != 0) {
+		printf("Error during getting device (port %u) info: %s\n",
+				port_id, strerror(-ret));
+		ret = -1;
+		goto fail;
+	}
+
 	if (info.device)
 		bus = rte_bus_find_by_device(info.device);
 	else