On Tue, 27 Feb 2018 17:33:00 +0800
Qi Zhang <qi.z.zh...@intel.com> wrote:

> struct pmd_internals {
> +     int sfd;
> +     int if_index;
> +     char if_name[0x100];

why not IFNAMSIZ?

> +     struct ether_addr eth_addr;
> +     struct xdp_queue rx;
> +     struct xdp_queue tx;
> +     struct xdp_umem *umem;
> +     struct rte_mempool *mb_pool;
> +
> +     unsigned long rx_pkts;
> +     unsigned long rx_bytes;
> +     unsigned long rx_dropped;
> +
> +     unsigned long tx_pkts;
> +     unsigned long err_pkts;
> +     unsigned long tx_bytes;

why not per-queue stats? per-port stats are expensive

> +     uint16_t port_id;
> +     uint16_t queue_idx;
> +     int ring_size;
> +     struct rte_ring *buf_ring;
> +};

Reply via email to