> + pkt_pool = mbuf_pool_create(pkt_dpdk->pool_name, > + pool_entry->num, cache, > + pool_entry->max_seg_len + > + CONFIG_PACKET_HEADROOM, > + pkt_dpdk); > instead of passing the whole pkt_dpdk struct, can you just pass odp_pool_t & > calculate just data_room in the dequeue_bulk ?. This way the pool > configuration will be more clearer & less dependent on pkt_dpdk. >
The value of 'data_room' is constant, so it doesn't make sense to recalculate it in the fast path every time pool_dequeue_bulk() is called. mbuf_pool_create() is only used by the dpdk pktio, so the dependency is not a problem. -Matias
