> > +
> > +            /* preserve the order of packet for flow batching */
> > +            index_map[packets_->count - 1] = map_cnt;
> 
> Using the "packets_->count" without accessor + inside a refill loop is highly
> inaccurate/potentially dangerous. Isn't it equal to "n_missed"?
> 
Hi Ilya,

Thanks for the review. However I could not understand your point about
inaccurate/potentially dangerous code. Can you elaborate on a scenario 
where this is likely to fail/crash ?
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to