On 1/4/16, 9:14 PM, "Stephen Hemminger" [off-list ref] wrote:

On Mon,  4 Jan 2016 18:28:18 -0800
Yong Wang [off-list ref] wrote:

quoted

+	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.