On Wed, 9 Oct 2019 16:29:09 +0100 Flavia Musatescu <flavia.musate...@intel.com> wrote:
> + if (sendto(pkt_q->sockfd, NULL, 0, MSG_DONTWAIT, NULL, 0) == -1 && > + errno != ENOBUFS) { > + /* Error sending. > + * When sendto call fails and ENOBUFS error is being set > + * some of the packets are actually successfully transmitted. > + * There is no available count of those packets, so in order > + * to make the statistics more accurate, all of the previously > + * enqueued packets will be considered successful, even though > + * this is not entirely correct. > + */ Agree with the change, but please keep only add a small code comment. Long comments are a personal pet peeve, it also tends to standout as "some different developer who had an issue left this comment"