Hi all (replying below) On Mon, May 06, 2013 at 09:28:33AM -0400, Han, Dongsu wrote: > Sounds good. With the two bug fixes I submitted txonly now runs perfectly. > Thanks! > -Dongsu > On May 6, 2013 8:47 AM, "Thomas Monjalon" <thomas.monjalon at 6wind.com> > wrote: > > > Hi Dongsu Han, > > > > I think your fix is right. > > I've just removed tx_mbuf_alloc() and directly called rte_pktmbuf_alloc() > > instead. > > Is it OK for you ? > > Could you also review this (modified) description ? [...]
While using rte_pktmbuf_alloc() is the correct fix, it's much slower than __rte_mbuf_raw_alloc() due to the unnecessary call to rte_pktmbuf_reset(). Since testpmd is often used for performance testing, we should consider a wrapper function calling __rte_mbuf_raw_alloc() directly instead, as in rte_rxmbuf_alloc() implemented in igb and ixgbe PMDs. -- Adrien Mazarguil 6WIND
