Re: [PATCH] sample: vfio-mdev: avoid deadlock in mdev_access()

2018-07-11 Thread Alex Williamson
On Fri, 6 Jul 2018 23:50:06 +0300 Alexey Khoroshilov wrote: > mdev_access() calls mbochs_get_page() with mdev_state->ops_lock held, > while mbochs_get_page() locks the mutex by itself. > It leads to unavoidable deadlock. > > Found by Linux Driver Verification project (linuxtesting.org). > >

Re: [PATCH] sample: vfio-mdev: avoid deadlock in mdev_access()

2018-07-11 Thread Alex Williamson
On Fri, 6 Jul 2018 23:50:06 +0300 Alexey Khoroshilov wrote: > mdev_access() calls mbochs_get_page() with mdev_state->ops_lock held, > while mbochs_get_page() locks the mutex by itself. > It leads to unavoidable deadlock. > > Found by Linux Driver Verification project (linuxtesting.org). > >

[PATCH] sample: vfio-mdev: avoid deadlock in mdev_access()

2018-07-06 Thread Alexey Khoroshilov
mdev_access() calls mbochs_get_page() with mdev_state->ops_lock held, while mbochs_get_page() locks the mutex by itself. It leads to unavoidable deadlock. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- samples/vfio-mdev/mbochs.c | 4 +++- 1

[PATCH] sample: vfio-mdev: avoid deadlock in mdev_access()

2018-07-06 Thread Alexey Khoroshilov
mdev_access() calls mbochs_get_page() with mdev_state->ops_lock held, while mbochs_get_page() locks the mutex by itself. It leads to unavoidable deadlock. Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshilov --- samples/vfio-mdev/mbochs.c | 4 +++- 1