On Tue, 2 Feb 2016 14:57:00 +0800 Wenzhuo Lu <wenzhuo.lu at intel.com> wrote:
> +/** > + * l2 tunnel configuration. > + */ > +struct rte_eth_l2_tunnel { > + enum rte_eth_l2_tunnel_type l2_tunnel_type; > + uint16_t ether_type; > +}; Building an API with an indirect (structure) parameter with only two elements seems like overkill. If you are building it to support future features, that won't work because it will break ABI. Why not just? int rte_eth_dev_etag_enable(uint8_t port_id, uint16_t ether_type)