linux-stable.git/drivers/vhost/vhost.c, branch v3.17

vhost: move memory pointer to VQs

2014-06-09T13:21:07+00:00

commit 2ae76693b8bcabf370b981cd00c36cd41d33fabc
    vhost: replace rcu with mutex
replaced rcu sync for memory accesses with VQ mutex locl/unlock.
This is correct since all accesses are under VQ mutex, but incomplete:
we still do useless rcu lock/unlock operations, someone might copy this
code into some other context where this won't be right.
This use of RCU is also non standard and hard to understand.
Let's copy the pointer to each VQ structure, this way
the access rules become straight-forward, and there's
no need for RCU anymore.

Reported-by: Eric Dumazet 
Signed-off-by: Michael S. Tsirkin

vhost: move acked_features to VQs

2014-06-09T13:21:06+00:00

Refactor code to make sure features are only accessed
under VQ mutex. This makes everything simpler, no need
for RCU here anymore.

Signed-off-by: Michael S. Tsirkin

vhost: replace rcu with mutex

2014-06-09T13:21:06+00:00

All memory accesses are done under some VQ mutex.
So lock/unlock all VQs is a faster equivalent of synchronize_rcu()
for memory access changes.
Some guests cause a lot of these changes, so it's helpful
to make them faster.

Reported-by: "Gonglei (Arei)" 
Signed-off-by: Michael S. Tsirkin

vhost: remove the dead branch

2013-12-06T20:22:05+00:00

Since vhost_dev_init() forever return 0, some branches are never run,
therefore need to be removed.

Signed-off-by: Zhi Yong Wu 
Acked-by: Michael S. Tsirkin 
Signed-off-by: David S. Miller

vhost: wake up worker outside spin_lock

2013-09-17T06:21:32+00:00

the wake_up_process func is included by spin_lock/unlock in
vhost_work_queue,
but it could be done outside the spin_lock.
I have test it with kernel 3.0.27 and guest suse11-sp2 using iperf,
the num as below.
                  original                 modified
thread_num  tp(Gbps)   vhost(%)  |  tp(Gbps)     vhost(%)
1           9.59        28.82    |   9.59        27.49
8           9.61        32.92    |   9.62        26.77
64          9.58        46.48    |   9.55        38.99
256         9.6         63.7     |   9.6         52.59

Signed-off-by: Chuanyu Qin 
Signed-off-by: Michael S. Tsirkin

vhost: switch to use vhost_add_used_n()

2013-09-04T02:46:57+00:00

Let vhost_add_used() to use vhost_add_used_n() to reduce the code
duplication. To avoid the overhead brought by __copy_to_user(). We will use
put_user() when one used need to be added.

Signed-off-by: Jason Wang 
Signed-off-by: David S. Miller

vhost: Include linux/uio.h instead of linux/socket.h

2013-08-20T22:05:04+00:00

memcpy_fromiovec is moved from net/core/iovec.c to lib/iovec.c.
linux/uio.h provides the declaration for memcpy_fromiovec.

Include linux/uio.h instead of inux/socket.h for it.

Signed-off-by: Asias He 
Acked-by: Michael S. Tsirkin 
Signed-off-by: David S. Miller

vhost: Make vhost a separate module

2013-07-07T14:33:44+00:00

Currently, vhost-net and vhost-scsi are sharing the vhost core code.
However, vhost-scsi shares the code by including the vhost.c file
directly.

Making vhost a separate module makes it is easier to share code with
other vhost devices.

Signed-off-by: Asias He 
Signed-off-by: Michael S. Tsirkin

vhost: Simplify dev->vqs[i] access

2013-07-07T11:38:26+00:00

Signed-off-by: Asias He 
Signed-off-by: Michael S. Tsirkin

vhost: check owner before we overwrite ubuf_info

2013-06-11T09:46:21+00:00

If device has an owner, we shouldn't touch ubuf_info
since it might be in use.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: David S. Miller