[dpdk-dev] [PATCHv4 1/5] ethdev: add new API to retrieve RX/TX queue information
Thomas Monjalon
thomas.monjalon at 6wind.com
Thu Jul 23 18:26:24 CEST 2015
2015-07-22 19:28, Konstantin Ananyev:
> + if (!rte_eth_dev_is_valid_port(port_id)) {
> + PMD_DEBUG_TRACE("Invalid port_id=%d\n", port_id);
> + return -EINVAL;
> + }
Please use VALID_PORTID_OR_ERR_RET.
> + * Ethernet device RX queue information strcuture.
Typo here (and same for TX).
> +struct rte_eth_rxq_info {
> + struct rte_mempool *mp; /**< mempool used by that queue. */
> + struct rte_eth_rxconf conf; /**< queue config parameters. */
> + uint8_t scattered_rx; /**< scattered packets RX supported. */
> + uint16_t nb_desc; /**< configured number of RXDs. */
Shouldn't we move nb_desc in rte_eth_rxconf?
So rte_eth_rx_queue_setup() would have less parameters.
> -#ifdef __cplusplus
> -}
> -#endif
> -
> /**
> * Set the list of multicast addresses to filter on an Ethernet device.
> *
> @@ -3882,4 +3952,9 @@ extern int rte_eth_timesync_read_rx_timestamp(uint8_t port_id,
> */
> extern int rte_eth_timesync_read_tx_timestamp(uint8_t port_id,
> struct timespec *timestamp);
> +
> +#ifdef __cplusplus
> +}
> +#endif
> +
> #endif /* _RTE_ETHDEV_H_ */
Please send this change in a separate patch alone.
More information about the dev
mailing list