diff options
author | Bhanuprakash Bodireddy <bhanuprakash.bodireddy@intel.com> | 2017-09-20 14:12:56 +0100 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2017-11-03 13:22:00 -0700 |
commit | e0a00cee33aca9f8abe3b521d85f93244589fafb (patch) | |
tree | c754b5593e24cbd2f0ceafbc4453cd424c9339b8 /lib | |
parent | 13708b2183825f06d07c929244c7708032001c18 (diff) | |
download | openvswitch-e0a00cee33aca9f8abe3b521d85f93244589fafb.tar.gz |
netdev-linux: Clean up netdev_linux_sock_batch_send().
Use DP_PACKET_BATCH_FOR_EACH macro and dp_packet_batch_size() API
in netdev_linux_sock_batch_send().
Signed-off-by: Bhanuprakash Bodireddy <bhanuprakash.bodireddy@intel.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/netdev-linux.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c index 8fcc36336..fbbbb7205 100644 --- a/lib/netdev-linux.c +++ b/lib/netdev-linux.c @@ -1186,16 +1186,17 @@ static int netdev_linux_sock_batch_send(int sock, int ifindex, struct dp_packet_batch *batch) { + const size_t size = dp_packet_batch_size(batch); /* We don't bother setting most fields in sockaddr_ll because the * kernel ignores them for SOCK_RAW. */ struct sockaddr_ll sll = { .sll_family = AF_PACKET, .sll_ifindex = ifindex }; - struct mmsghdr *mmsg = xmalloc(sizeof(*mmsg) * batch->count); - struct iovec *iov = xmalloc(sizeof(*iov) * batch->count); + struct mmsghdr *mmsg = xmalloc(sizeof(*mmsg) * size); + struct iovec *iov = xmalloc(sizeof(*iov) * size); - for (int i = 0; i < batch->count; i++) { - struct dp_packet *packet = batch->packets[i]; + struct dp_packet *packet; + DP_PACKET_BATCH_FOR_EACH (packet, batch) { iov[i].iov_base = dp_packet_data(packet); iov[i].iov_len = dp_packet_get_send_len(packet); mmsg[i].msg_hdr = (struct msghdr) { .msg_name = &sll, @@ -1205,10 +1206,10 @@ netdev_linux_sock_batch_send(int sock, int ifindex, } int error = 0; - for (uint32_t ofs = 0; ofs < batch->count; ) { + for (uint32_t ofs = 0; ofs < size; ) { ssize_t retval; do { - retval = sendmmsg(sock, mmsg + ofs, batch->count - ofs, 0); + retval = sendmmsg(sock, mmsg + ofs, size - ofs, 0); error = retval < 0 ? errno : 0; } while (error == EINTR); if (error) { |