On 1/4/16, 9:14 PM, "Stephen Hemminger" <stephen at networkplumber.org> wrote:
>On Mon, 4 Jan 2016 18:28:18 -0800
>Yong Wang <yongwang at vmware.com> wrote:
>
>> + mbuf = txq->cmd_ring.buf_info[eop_idx].m;
>> + if (unlikely(mbuf == NULL))
>> + rte_panic("EOP desc does not point to a valid mbuf");
>> + else
>
>The unlikely is really not needed with rte_panic since it is declared
>with cold attribute which has same effect.
>
>Else is unnecessary because rte_panic never returns.
Done.
