> On Nov 15, 2017, at 12:32 AM, Hanoch Haim (hhaim) <hh...@cisco.com> wrote: > > I would add this too > > - } else if (rte_atomic16_add_return(&m->refcnt_atomic, -1) == 0) > > Should be : > > + } else if (likely(rte_mbuf_refcnt_update(m, -1) == 0)) { > > > Hanoh >
Why not to send the separate patch for the change?