> > + > > + /* 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