]> git.baikalelectronics.ru Git - kernel.git/commit
VSOCK: Don't call vsock_stream_has_data in atomic context
authorJorgen Hansen <jhansen@vmware.com>
Fri, 24 Nov 2017 14:25:28 +0000 (06:25 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sat, 25 Nov 2017 19:21:50 +0000 (04:21 +0900)
commit1302e2b9787ed87b2c2749084aa055c77a07a8ed
tree5c56e35dc86f14af77db9983856e39f4485f5a4a
parent10f6aabe7ca2015818d0401d7744fab21bc2966a
VSOCK: Don't call vsock_stream_has_data in atomic context

When using the host personality, VMCI will grab a mutex for any
queue pair access. In the detach callback for the vmci vsock
transport, we call vsock_stream_has_data while holding a spinlock,
and vsock_stream_has_data will access a queue pair.

To avoid this, we can simply omit calling vsock_stream_has_data
for host side queue pairs, since the QPs are empty per default
when the guest has detached.

This bug affects users of VMware Workstation using kernel version
4.4 and later.

Testing: Ran vsock tests between guest and host, and verified that
with this change, the host isn't calling vsock_stream_has_data
during detach. Ran mixedTest between guest and host using both
guest and host as server.

v2: Rebased on top of recent change to sk_state values
Reviewed-by: Adit Ranadive <aditr@vmware.com>
Reviewed-by: Aditya Sarwade <asarwade@vmware.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Jorgen Hansen <jhansen@vmware.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/vmw_vsock/vmci_transport.c