search for: tx_timer

...t; #include <linux/scatterlist.h> +#include <linux/hrtimer.h> /* FIXME: MTU in config. */ #define MAX_PACKET_LEN (ETH_HLEN+ETH_DATA_LEN) @@ -34,9 +35,15 @@ struct virtnet_info struct net_device *dev; struct napi_struct napi; + /* TX coalescing. */ + struct hrtimer tx_timer; + /* Number of input buffers, and max we've ever had. */ unsigned int num, max; + /* Number of queued output buffers, and max we've ever had. */ + unsigned int out_num, out_max; + /* Receive & send queues. */ struct sk_buff_head recv; struct sk_buff_hea...

...t; #include <linux/scatterlist.h> +#include <linux/hrtimer.h> /* FIXME: MTU in config. */ #define MAX_PACKET_LEN (ETH_HLEN+ETH_DATA_LEN) @@ -34,9 +35,15 @@ struct virtnet_info struct net_device *dev; struct napi_struct napi; + /* TX coalescing. */ + struct hrtimer tx_timer; + /* Number of input buffers, and max we've ever had. */ unsigned int num, max; + /* Number of queued output buffers, and max we've ever had. */ + unsigned int out_num, out_max; + /* Receive & send queues. */ struct sk_buff_head recv; struct sk_buff_hea...

hrtimer's only have a cb_mode if CONFIG_HIGH_RES_TIMER is enabled. This patch memset()s the tx_timer structure to 0 which has the same effect as setting cb_mode to HRTIMER_CB_SOFTIRQ without requiring CONFIG_HIGH_RES_TIMER to be set. This seems to be what other drivers do. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c...

hrtimer's only have a cb_mode if CONFIG_HIGH_RES_TIMER is enabled. This patch memset()s the tx_timer structure to 0 which has the same effect as setting cb_mode to HRTIMER_CB_SOFTIRQ without requiring CONFIG_HIGH_RES_TIMER to be set. This seems to be what other drivers do. Signed-off-by: Anthony Liguori <aliguori@us.ibm.com> diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c...

...c @@ -406,10 +405,10 @@ again: virtio_debug(vdebug, "%s: before calling kick %d\n", __FUNCTION__, __LINE__); vi->svq->vq_ops->kick(vi->svq); vi->out_num = 0; - } else { - vi->stats.hrtimer_starts++; - hrtimer_start(&vi->tx_timer, ktime_set(0,500000), - HRTIMER_MODE_REL); + } else if (!hrtimer_is_queued(&vi->tx_timer)) { + vi->stats.hrtimer_starts++; + hrtimer_start(&vi->tx_timer, ktime_set(0,500000), + HRTIMER_MODE_REL); } return 0;...

...c @@ -406,10 +405,10 @@ again: virtio_debug(vdebug, "%s: before calling kick %d\n", __FUNCTION__, __LINE__); vi->svq->vq_ops->kick(vi->svq); vi->out_num = 0; - } else { - vi->stats.hrtimer_starts++; - hrtimer_start(&vi->tx_timer, ktime_set(0,500000), - HRTIMER_MODE_REL); + } else if (!hrtimer_is_queued(&vi->tx_timer)) { + vi->stats.hrtimer_starts++; + hrtimer_start(&vi->tx_timer, ktime_set(0,500000), + HRTIMER_MODE_REL); } return 0;...

Hello all: This seires is an update of last version of multiqueue support to add multiqueue capability to both tap and virtio-net. Some kinds of tap backends has (macvatp in linux) or would (tap) support multiqueue. In such kind of tap backend, each file descriptor of a tap is a qeueu and ioctls were prodived to attach an exist tap file descriptor to the tun/tap device. So the patch let qemu to

Hello all: This seires is an update of last version of multiqueue support to add multiqueue capability to both tap and virtio-net. Some kinds of tap backends has (macvatp in linux) or would (tap) support multiqueue. In such kind of tap backend, each file descriptor of a tap is a qeueu and ioctls were prodived to attach an exist tap file descriptor to the tun/tap device. So the patch let qemu to

...--- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -392,10 +392,10 @@ again: vi->stats.sendq_kicks++; vi->svq->vq_ops->kick(vi->svq); vi->out_num = 0; - } else { - vi->stats.hrtimer_starts++; - hrtimer_start(&vi->tx_timer, ktime_set(0,500000), - HRTIMER_MODE_REL); + } else if (!hrtimer_is_queued(&vi->tx_timer)) { + vi->stats.hrtimer_starts++; + hrtimer_start(&vi->tx_timer, ktime_set(0,500000), + HRTIMER_MODE_REL);...

...--- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -392,10 +392,10 @@ again: vi->stats.sendq_kicks++; vi->svq->vq_ops->kick(vi->svq); vi->out_num = 0; - } else { - vi->stats.hrtimer_starts++; - hrtimer_start(&vi->tx_timer, ktime_set(0,500000), - HRTIMER_MODE_REL); + } else if (!hrtimer_is_queued(&vi->tx_timer)) { + vi->stats.hrtimer_starts++; + hrtimer_start(&vi->tx_timer, ktime_set(0,500000), + HRTIMER_MODE_REL);...

Hello all: This seires is an update of last version of multiqueue support to add multiqueue capability to both tap and virtio-net. Some kinds of tap backends has (macvatp in linux) or would (tap) support multiqueue. In such kind of tap backend, each file descriptor of a tap is a qeueu and ioctls were prodived to attach an exist tap file descriptor to the tun/tap device. So the patch let qemu to

Hello all: This seires is an update of last version of multiqueue support to add multiqueue capability to both tap and virtio-net. Some kinds of tap backends has (macvatp in linux) or would (tap) support multiqueue. In such kind of tap backend, each file descriptor of a tap is a qeueu and ioctls were prodived to attach an exist tap file descriptor to the tun/tap device. So the patch let qemu to

skb is only NULL the first time around: it's more correct to test for being under-budget. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> diff -r 2a94425ac7d5 drivers/net/virtio_net.c --- a/drivers/net/virtio_net.c Thu Nov 15 13:47:28 2007 +1100 +++ b/drivers/net/virtio_net.c Thu Nov 15 23:10:44 2007 +1100 @@ -198,8 +198,8 @@ again: if (vi->num < vi->max / 2)

skb is only NULL the first time around: it's more correct to test for being under-budget. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> diff -r 2a94425ac7d5 drivers/net/virtio_net.c --- a/drivers/net/virtio_net.c Thu Nov 15 13:47:28 2007 +1100 +++ b/drivers/net/virtio_net.c Thu Nov 15 23:10:44 2007 +1100 @@ -198,8 +198,8 @@ again: if (vi->num < vi->max / 2)