[dpdk-dev] [PATCH v4 2/2] net/tap: add queues when attaching from secondary process
Raslan Darawsheh
rasland at mellanox.com
Tue Oct 2 12:50:55 CEST 2018
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 <thomas at monjalon.net>
Sent: Tuesday, October 2, 2018 1:41 PM
To: Raslan Darawsheh <rasland at mellanox.com>
Cc: dev at dpdk.org; keith.wiles at intel.com; Shahaf Shuler <shahafs at mellanox.com>; Ori Kam <orika at 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?
More information about the dev
mailing list