[dpdk-dev] [PATCH v5 1/3] net/tap: add queue and port ids in Rx/Tx queues structures

Wiles, Keith keith.wiles at intel.com
Wed Oct 10 14:50:48 CEST 2018



> On Oct 10, 2018, at 2:03 AM, Raslan Darawsheh <rasland at mellanox.com> wrote:
> 
> Signed-off-by: Raslan Darawsheh <rasland at mellanox.com>

This title for the patch is the what we did not why we did it, should that be changed? To me it does not convey the reason or we would need to add a more complete comment body text to explain why we wanted the change. It is a bit of nit picking.

> ---
> drivers/net/tap/rte_eth_tap.c | 3 +++
> drivers/net/tap/rte_eth_tap.h | 3 +++
> 2 files changed, 6 insertions(+)
> 
> diff --git a/drivers/net/tap/rte_eth_tap.c b/drivers/net/tap/rte_eth_tap.c
> index ad5ae98..edfb7da 100644
> --- a/drivers/net/tap/rte_eth_tap.c
> +++ b/drivers/net/tap/rte_eth_tap.c
> @@ -1293,6 +1293,7 @@ tap_rx_queue_setup(struct rte_eth_dev *dev,
> 	rxq->mp = mp;
> 	rxq->trigger_seen = 1; /* force initial burst */
> 	rxq->in_port = dev->data->port_id;
> +	rxq->queue_id = rx_queue_id;
> 	rxq->nb_rx_desc = nb_desc;
> 	iovecs = rte_zmalloc_socket(dev->device->name, sizeof(*iovecs), 0,
> 				    socket_id);
> @@ -1359,6 +1360,8 @@ tap_tx_queue_setup(struct rte_eth_dev *dev,
> 		return -1;
> 	dev->data->tx_queues[tx_queue_id] = &internals->txq[tx_queue_id];
> 	txq = dev->data->tx_queues[tx_queue_id];
> +	txq->out_port = dev->data->port_id;
> +	txq->queue_id = tx_queue_id;
> 
> 	offloads = tx_conf->offloads | dev->data->dev_conf.txmode.offloads;
> 	txq->csum = !!(offloads &
> diff --git a/drivers/net/tap/rte_eth_tap.h b/drivers/net/tap/rte_eth_tap.h
> index 44e2773..4502e24 100644
> --- a/drivers/net/tap/rte_eth_tap.h
> +++ b/drivers/net/tap/rte_eth_tap.h
> @@ -46,6 +46,7 @@ struct rx_queue {
> 	struct rte_mempool *mp;         /* Mempool for RX packets */
> 	uint32_t trigger_seen;          /* Last seen Rx trigger value */
> 	uint16_t in_port;               /* Port ID */
> +	uint16_t queue_id;		/* queue ID*/
> 	int fd;
> 	struct pkt_stats stats;         /* Stats for this RX queue */
> 	uint16_t nb_rx_desc;            /* max number of mbufs available */
> @@ -62,6 +63,8 @@ struct tx_queue {
> 	uint16_t csum:1;                /* Enable checksum offloading */
> 	struct pkt_stats stats;         /* Stats for this TX queue */
> 	struct rte_gso_ctx gso_ctx;     /* GSO context */
> +	uint16_t out_port;              /* Port ID */
> +	uint16_t queue_id;		/* queue ID*/
> };
> 
> struct pmd_internals {
> -- 
> 2.7.4
> 

Regards,
Keith



More information about the dev mailing list