On 11/17/2014 05:07 PM, Ciprian Barbu wrote:
         nbr = queue_deq_multi(qentry, buf_hdr, num);
>+       if (nbr >= num)
The geq comparison here is scary, queue_deq_multi should return at
most num buffer. Perhaps an assert should be added?

Ciprian, thanks for review. I just kept original code logic and looks like the was bug. I.e. nbr == num is valid case. I.e. queue return as many packets as requested. That
can be reproducible with ping -f.

Will send new version with all comments updated.

Maxim.

_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to