vhost_net: introduce tx_can_batch()
authorJason Wang <jasowang@redhat.com>
Fri, 20 Jul 2018 00:15:17 +0000 (08:15 +0800)
committerDavid S. Miller <davem@davemloft.net>
Sun, 22 Jul 2018 16:43:31 +0000 (09:43 -0700)
Introduce tx_can_batch() to determine whether TX could be
batched. This will help to reduce the code duplication in the future.

Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/vhost/net.c

index a014ca0..f59b615 100644 (file)
@@ -514,6 +514,12 @@ static int get_tx_bufs(struct vhost_net *net,
        return ret;
 }
 
+static bool tx_can_batch(struct vhost_virtqueue *vq, size_t total_len)
+{
+       return total_len < VHOST_NET_WEIGHT &&
+              !vhost_vq_avail_empty(vq->dev, vq);
+}
+
 /* Expects to be always run from workqueue - which acts as
  * read-size critical section for our kind of RCU. */
 static void handle_tx(struct vhost_net *net)
@@ -598,8 +604,7 @@ static void handle_tx(struct vhost_net *net)
                        ubufs = NULL;
                }
                total_len += len;
-               if (total_len < VHOST_NET_WEIGHT &&
-                   !vhost_vq_avail_empty(&net->dev, vq) &&
+               if (tx_can_batch(vq, total_len) &&
                    likely(!vhost_exceeds_maxpend(net))) {
                        msg.msg_flags |= MSG_MORE;
                } else {