It should be as of per device so we should do it for each port alone since
several ports can have different queues.
Moreover, if the port that has the registration was closed or unplugged we'll
not be able to sync qeues for other ports.
Kindest regards,
Raslan Darawsheh
-----Original Message-----
From: Thomas Monjalon <tho...@monjalon.net>
Sent: Tuesday, October 2, 2018 1:41 PM
To: Raslan Darawsheh <rasl...@mellanox.com>
Cc: dev@dpdk.org; keith.wi...@intel.com; Shahaf Shuler <shah...@mellanox.com>;
Ori Kam <or...@mellanox.com>
Subject: Re: [dpdk-dev] [PATCH v4 2/2] net/tap: add queues when attaching from
secondary process
02/10/2018 12:34, Raslan Darawsheh:
> @@ -2056,6 +2179,13 @@ rte_pmd_tap_probe(struct rte_vdev_device *dev)
> TAP_LOG(NOTICE, "Initializing pmd_tap for %s as %s",
> name, tap_name);
>
> + /* Register IPC feed callback */
> + ret = rte_mp_action_register(TAP_MP_KEY, tap_mp_sync_queues);
> + if (ret < 0 && rte_errno != EEXIST) {
> + TAP_LOG(ERR, "%s: Failed to register IPC callback: %s",
> + tuntap_name, strerror(rte_errno));
> + goto leave;
> + }
> ret = eth_dev_tap_create(dev, tap_name, remote_iface, &user_mac,
> ETH_TUNTAP_TYPE_TAP);
Is it an issue registering tap_mp_sync_queues at each tap probing?
Should we do it only once?
