examples/vdpa: remove useless device count
[dpdk.git] / examples / vdpa / main.c
index bb30796..2345f98 100644 (file)
@@ -27,7 +27,7 @@
 
 struct vdpa_port {
        char ifname[MAX_PATH_LEN];
-       int did;
+       struct rte_vdpa_device *dev;
        int vid;
        uint64_t flags;
        int stats_n;
@@ -38,7 +38,6 @@ struct vdpa_port {
 static struct vdpa_port vports[MAX_VDPA_SAMPLE_PORTS];
 
 static char iface[MAX_PATH_LEN];
-static int dev_total;
 static int devcnt;
 static int interactive;
 static int client_mode;
@@ -105,16 +104,23 @@ static int
 new_device(int vid)
 {
        char ifname[MAX_PATH_LEN];
+       struct rte_device *dev;
        int i;
 
        rte_vhost_get_ifname(vid, ifname, sizeof(ifname));
        for (i = 0; i < MAX_VDPA_SAMPLE_PORTS; i++) {
-               if (strncmp(ifname, vports[i].ifname, MAX_PATH_LEN) == 0) {
-                       printf("\nnew port %s, did: %d\n",
-                                       ifname, vports[i].did);
-                       vports[i].vid = vid;
-                       break;
+               if (strncmp(ifname, vports[i].ifname, MAX_PATH_LEN))
+                       continue;
+
+               dev = rte_vdpa_get_rte_device(vports[i].dev);
+               if (!dev) {
+                       RTE_LOG(ERR, VDPA,
+                               "Failed to get generic device for port %d\n", i);
+                       continue;
                }
+               printf("\nnew port %s, device : %s\n", ifname, dev->name);
+               vports[i].vid = vid;
+               break;
        }
 
        if (i >= MAX_VDPA_SAMPLE_PORTS)
@@ -126,16 +132,24 @@ new_device(int vid)
 static void
 destroy_device(int vid)
 {
+       struct rte_device *dev;
        char ifname[MAX_PATH_LEN];
        int i;
 
        rte_vhost_get_ifname(vid, ifname, sizeof(ifname));
        for (i = 0; i < MAX_VDPA_SAMPLE_PORTS; i++) {
-               if (strcmp(ifname, vports[i].ifname) == 0) {
-                       printf("\ndestroy port %s, did: %d\n",
-                                       ifname, vports[i].did);
-                       break;
+               if (strncmp(ifname, vports[i].ifname, MAX_PATH_LEN))
+                       continue;
+
+               dev = rte_vdpa_get_rte_device(vports[i].dev);
+               if (!dev) {
+                       RTE_LOG(ERR, VDPA,
+                               "Failed to get generic device for port %d\n", i);
+                       continue;
                }
+
+               printf("\ndestroy port %s, device: %s\n", ifname, dev->name);
+               break;
        }
 }
 
@@ -149,7 +163,6 @@ start_vdpa(struct vdpa_port *vport)
 {
        int ret;
        char *socket_path = vport->ifname;
-       int did = vport->did;
 
        if (client_mode)
                vport->flags |= RTE_VHOST_USER_CLIENT;
@@ -173,7 +186,7 @@ start_vdpa(struct vdpa_port *vport)
                        "register driver ops failed: %s\n",
                        socket_path);
 
-       ret = rte_vhost_driver_attach_vdpa_device(socket_path, did);
+       ret = rte_vhost_driver_attach_vdpa_device(socket_path, vport->dev);
        if (ret != 0)
                rte_exit(EXIT_FAILURE,
                        "attach vdpa device failed: %s\n",
@@ -213,7 +226,7 @@ static void
 vdpa_sample_quit(void)
 {
        int i;
-       for (i = 0; i < RTE_MIN(MAX_VDPA_SAMPLE_PORTS, dev_total); i++) {
+       for (i = 0; i < RTE_MIN(MAX_VDPA_SAMPLE_PORTS, devcnt); i++) {
                if (vports[i].ifname[0] != '\0')
                        close_vdpa(&vports[i]);
        }
@@ -276,34 +289,31 @@ static void cmd_list_vdpa_devices_parsed(
                struct cmdline *cl,
                __rte_unused void *data)
 {
-       int did;
        uint32_t queue_num;
        uint64_t features;
        struct rte_vdpa_device *vdev;
-       struct rte_pci_addr addr;
+       struct rte_device *dev;
+       struct rte_dev_iterator dev_iter;
 
-       cmdline_printf(cl, "device id\tdevice address\tqueue num\tsupported features\n");
-       for (did = 0; did < dev_total; did++) {
-               vdev = rte_vdpa_get_device(did);
+       cmdline_printf(cl, "device name\tqueue num\tsupported features\n");
+       RTE_DEV_FOREACH(dev, "class=vdpa", &dev_iter) {
+               vdev = rte_vdpa_find_device_by_name(dev->name);
                if (!vdev)
                        continue;
-               if (vdev->ops->get_queue_num(did, &queue_num) < 0) {
+               if (rte_vdpa_get_queue_num(vdev, &queue_num) < 0) {
                        RTE_LOG(ERR, VDPA,
                                "failed to get vdpa queue number "
-                               "for device id %d.\n", did);
+                               "for device %s.\n", dev->name);
                        continue;
                }
-               if (vdev->ops->get_features(did, &features) < 0) {
+               if (rte_vdpa_get_features(vdev, &features) < 0) {
                        RTE_LOG(ERR, VDPA,
                                "failed to get vdpa features "
-                               "for device id %d.\n", did);
+                               "for device %s.\n", dev->name);
                        continue;
                }
-               addr = vdev->addr.pci_addr;
-               cmdline_printf(cl,
-                       "%d\t\t" PCI_PRI_FMT "\t%" PRIu32 "\t\t0x%" PRIx64 "\n",
-                       did, addr.domain, addr.bus, addr.devid,
-                       addr.function, queue_num, features);
+               cmdline_printf(cl, "%s\t\t%" PRIu32 "\t\t0x%" PRIx64 "\n",
+                       dev->name, queue_num, features);
        }
 }
 
@@ -331,23 +341,18 @@ static void cmd_create_vdpa_port_parsed(void *parsed_result,
                struct cmdline *cl,
                __rte_unused void *data)
 {
-       int did;
+       struct rte_vdpa_device *dev;
        struct cmd_create_result *res = parsed_result;
-       struct rte_vdpa_dev_addr addr;
 
        rte_strscpy(vports[devcnt].ifname, res->socket_path, MAX_PATH_LEN);
-       if (rte_pci_addr_parse(res->bdf, &addr.pci_addr) != 0) {
-               cmdline_printf(cl, "Unable to parse the given bdf.\n");
-               return;
-       }
-       addr.type = VDPA_ADDR_PCI;
-       did = rte_vdpa_find_device_id(&addr);
-       if (did < 0) {
-               cmdline_printf(cl, "Unable to find vdpa device id.\n");
+       dev = rte_vdpa_find_device_by_name(res->bdf);
+       if (dev == NULL) {
+               cmdline_printf(cl, "Unable to find vdpa device id for %s.\n",
+                               res->bdf);
                return;
        }
 
-       vports[devcnt].did = did;
+       vports[devcnt].dev = dev;
 
        if (start_vdpa(&vports[devcnt]) == 0)
                devcnt++;
@@ -375,7 +380,7 @@ cmdline_parse_inst_t cmd_create_vdpa_port = {
 /* *** STATS *** */
 struct cmd_stats_result {
        cmdline_fixed_string_t stats;
-       uint16_t did;
+       cmdline_fixed_string_t bdf;
        uint16_t qid;
 };
 
@@ -383,25 +388,24 @@ static void cmd_device_stats_parsed(void *parsed_result, struct cmdline *cl,
                                    __rte_unused void *data)
 {
        struct cmd_stats_result *res = parsed_result;
-       struct rte_vdpa_device *vdev = rte_vdpa_get_device(res->did);
+       struct rte_vdpa_device *vdev = rte_vdpa_find_device_by_name(res->bdf);
        struct vdpa_port *vport = NULL;
        uint32_t first, last;
        int i;
 
        if (!vdev) {
-               RTE_LOG(ERR, VDPA, "Invalid device id %" PRIu16 ".\n",
-                       res->did);
+               RTE_LOG(ERR, VDPA, "Invalid device: %s.\n",
+                       res->bdf);
                return;
        }
-       for (i = 0; i < RTE_MIN(MAX_VDPA_SAMPLE_PORTS, dev_total); i++) {
-               if (vports[i].did == res->did) {
+       for (i = 0; i < RTE_MIN(MAX_VDPA_SAMPLE_PORTS, devcnt); i++) {
+               if (vports[i].dev == vdev) {
                        vport = &vports[i];
                        break;
                }
        }
        if (!vport) {
-               RTE_LOG(ERR, VDPA, "Device id %" PRIu16 " was not created.\n",
-                       res->did);
+               RTE_LOG(ERR, VDPA, "Device %s was not created.\n", res->bdf);
                return;
        }
        if (res->qid == 0xFFFF) {
@@ -409,7 +413,7 @@ static void cmd_device_stats_parsed(void *parsed_result, struct cmdline *cl,
                last = rte_vhost_get_vring_num(vport->vid);
                if (last == 0) {
                        RTE_LOG(ERR, VDPA, "Failed to get num of actual virtqs"
-                               " for device id %d.\n", (int)res->did);
+                               " for device %s.\n", res->bdf);
                        return;
                }
                last--;
@@ -418,10 +422,10 @@ static void cmd_device_stats_parsed(void *parsed_result, struct cmdline *cl,
                last = res->qid;
        }
        if (!vport->stats_names) {
-               vport->stats_n = rte_vdpa_get_stats_names(vport->did, NULL, 0);
+               vport->stats_n = rte_vdpa_get_stats_names(vport->dev, NULL, 0);
                if (vport->stats_n <= 0) {
                        RTE_LOG(ERR, VDPA, "Failed to get names number of "
-                               "device %d stats.\n", (int)res->did);
+                               "device %s stats.\n", res->bdf);
                        return;
                }
                vport->stats_names = rte_zmalloc(NULL,
@@ -429,26 +433,26 @@ static void cmd_device_stats_parsed(void *parsed_result, struct cmdline *cl,
                                                        vport->stats_n, 0);
                if (!vport->stats_names) {
                        RTE_LOG(ERR, VDPA, "Failed to allocate memory for stat"
-                               " names of device %d.\n", (int)res->did);
+                               " names of device %s.\n", res->bdf);
                        return;
                }
-               i = rte_vdpa_get_stats_names(vport->did, vport->stats_names,
+               i = rte_vdpa_get_stats_names(vport->dev, vport->stats_names,
                                                vport->stats_n);
                if (vport->stats_n != i) {
-                       RTE_LOG(ERR, VDPA, "Failed to get names of device %d "
-                               "stats.\n", (int)res->did);
+                       RTE_LOG(ERR, VDPA, "Failed to get names of device %s "
+                               "stats.\n", res->bdf);
                        return;
                }
                vport->stats = (struct rte_vdpa_stat *)
                                        (vport->stats_names + vport->stats_n);
        }
-       cmdline_printf(cl, "\nDevice %d:\n", (int)res->did);
+       cmdline_printf(cl, "\nDevice %s:\n", res->bdf);
        for (; first <= last; first++) {
                memset(vport->stats, 0, sizeof(*vport->stats) * vport->stats_n);
-               if (rte_vdpa_get_stats(vport->did, (int)first, vport->stats,
+               if (rte_vdpa_get_stats(vport->dev, (int)first, vport->stats,
                                        vport->stats_n) <= 0) {
                        RTE_LOG(ERR, VDPA, "Failed to get vdpa queue statistics"
-                               " for device id %d qid %d.\n", (int)res->did,
+                               " for device %s qid %d.\n", res->bdf,
                                (int)first);
                        return;
                }
@@ -464,8 +468,8 @@ static void cmd_device_stats_parsed(void *parsed_result, struct cmdline *cl,
 
 cmdline_parse_token_string_t cmd_device_stats_ =
        TOKEN_STRING_INITIALIZER(struct cmd_stats_result, stats, "stats");
-cmdline_parse_token_num_t cmd_device_id =
-       TOKEN_NUM_INITIALIZER(struct cmd_stats_result, did, UINT32);
+cmdline_parse_token_string_t cmd_device_bdf =
+       TOKEN_STRING_INITIALIZER(struct cmd_stats_result, bdf, NULL);
 cmdline_parse_token_num_t cmd_queue_id =
        TOKEN_NUM_INITIALIZER(struct cmd_stats_result, qid, UINT32);
 
@@ -475,7 +479,7 @@ cmdline_parse_inst_t cmd_device_stats = {
        .help_str = "stats: show device statistics",
        .tokens = {
                (void *)&cmd_device_stats_,
-               (void *)&cmd_device_id,
+               (void *)&cmd_device_bdf,
                (void *)&cmd_queue_id,
                NULL,
        },
@@ -519,9 +523,11 @@ int
 main(int argc, char *argv[])
 {
        char ch;
-       int i;
        int ret;
        struct cmdline *cl;
+       struct rte_vdpa_device *vdev;
+       struct rte_device *dev;
+       struct rte_dev_iterator dev_iter;
 
        ret = rte_eal_init(argc, argv);
        if (ret < 0)
@@ -529,10 +535,6 @@ main(int argc, char *argv[])
        argc -= ret;
        argv += ret;
 
-       dev_total = rte_vdpa_get_device_num();
-       if (dev_total <= 0)
-               rte_exit(EXIT_FAILURE, "No available vdpa device found\n");
-
        signal(SIGINT, signal_handler);
        signal(SIGTERM, signal_handler);
 
@@ -547,13 +549,18 @@ main(int argc, char *argv[])
                cmdline_interact(cl);
                cmdline_stdin_exit(cl);
        } else {
-               for (i = 0; i < RTE_MIN(MAX_VDPA_SAMPLE_PORTS, dev_total);
-                               i++) {
-                       vports[i].did = i;
-                       snprintf(vports[i].ifname, MAX_PATH_LEN, "%s%d",
-                                       iface, i);
+               RTE_DEV_FOREACH(dev, "class=vdpa", &dev_iter) {
+                       vdev = rte_vdpa_find_device_by_name(dev->name);
+                       if (vdev == NULL) {
+                               rte_panic("Failed to find vDPA dev for %s\n",
+                                               dev->name);
+                       }
+                       vports[devcnt].dev = vdev;
+                       snprintf(vports[devcnt].ifname, MAX_PATH_LEN, "%s%d",
+                                       iface, devcnt);
 
-                       start_vdpa(&vports[i]);
+                       start_vdpa(&vports[devcnt]);
+                       devcnt++;
                }
 
                printf("enter \'q\' to quit\n");