OSDN Git Service

vhost_net: switch to use mutex_trylock() in vhost_net_busy_poll()
authorJason Wang <jasowang@redhat.com>
Thu, 13 Dec 2018 02:53:38 +0000 (10:53 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 13 Dec 2018 05:56:19 +0000 (21:56 -0800)
commit476e8ba712fa6a479192dfc4301cd3765d9ba713
treeffde46b13664033e1a41fffbe4c80720ee86868c
parent841df922417eb82c835e93d4b93eb6a68c99d599
vhost_net: switch to use mutex_trylock() in vhost_net_busy_poll()

We used to hold the mutex of paired virtqueue in
vhost_net_busy_poll(). But this will results an inconsistent lock
order which may cause deadlock if we try to bring back the protection
of device IOTLB with vq mutex that requires to hold mutex of all
virtqueues at the same time.

Fix this simply by switching to use mutex_trylock(), when fail just
skip the busy polling. This can happen when device IOTLB is under
updating which should be rare.

Fixes: commit 78139c94dc8c ("net: vhost: lock the vqs one by one")
Cc: Tonghao Zhang <xiangxia.m.yue@gmail.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/vhost/net.c