diff options
| author |   Jason Wang <jasowang@redhat.com> | 2016-04-25 22:14:33 -0400 |
|---|---|---|
| committer |   Michael S. Tsirkin <mst@redhat.com> | 2016-08-01 21:44:51 +0300 |
| commit | 04b96e5528ca97199b429810fe963185a67dd40e (patch) | |
| tree | 35cef78e529e6fb49c34159f58d19fb07f54de3e /drivers/vhost/vhost.h | |
| parent | vhost: simplify work flushing (diff) | |
| download | linux-dev-04b96e5528ca97199b429810fe963185a67dd40e.tar.xz linux-dev-04b96e5528ca97199b429810fe963185a67dd40e.zip | |
vhost: lockless enqueuing
We use spinlock to synchronize the work list now which may cause
unnecessary contentions. So this patch switch to use llist to remove
this contention. Pktgen tests shows about 5% improvement:
Before:
~1300000 pps
After:
~1370000 pps
Signed-off-by: Jason Wang <jasowang@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'drivers/vhost/vhost.h')
| -rw-r--r-- | drivers/vhost/vhost.h | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h index d36d8beb3351..6690e645d2f8 100644 --- a/drivers/vhost/vhost.h +++ b/drivers/vhost/vhost.h @@ -15,13 +15,15 @@ struct vhost_work; typedef void (*vhost_work_fn_t)(struct vhost_work *work); +#define VHOST_WORK_QUEUED 1 struct vhost_work { - struct list_head node; + struct llist_node node; vhost_work_fn_t fn; wait_queue_head_t done; int flushing; unsigned queue_seq; unsigned done_seq; + unsigned long flags; }; /* Poll a file (eventfd or socket) */ @@ -126,8 +128,7 @@ struct vhost_dev { int nvqs; struct file *log_file; struct eventfd_ctx *log_ctx; - spinlock_t work_lock; - struct list_head work_list; + struct llist_head work_list; struct task_struct *worker; }; |