diff options
| author |   Stefano Stabellini <sstabellini@kernel.org> | 2018-02-14 10:28:24 -0800 |
|---|---|---|
| committer |   Juergen Gross <jgross@suse.com> | 2018-02-17 09:41:06 +0100 |
| commit | d1a75e0896f5e9f5cb6a979caaea39f1f4b9feb1 (patch) | |
| tree | 64a06c2c248dfcfff3da8a9b249005cc573b722a /drivers | |
| parent | pvcalls-front: introduce a per sock_mapping refcount (diff) | |
| download | linux-dev-d1a75e0896f5e9f5cb6a979caaea39f1f4b9feb1.tar.xz linux-dev-d1a75e0896f5e9f5cb6a979caaea39f1f4b9feb1.zip | |
pvcalls-front: wait for other operations to return when release passive sockets
Passive sockets can have ongoing operations on them, specifically, we
have two wait_event_interruptable calls in pvcalls_front_accept.
Add two wake_up calls in pvcalls_front_release, then wait for the
potential waiters to return and release the sock_mapping refcount.
Signed-off-by: Stefano Stabellini <stefano@aporeto.com>
Acked-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Juergen Gross <jgross@suse.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/xen/pvcalls-front.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/xen/pvcalls-front.c b/drivers/xen/pvcalls-front.c index 11ce470b41a5..aedbee3b2838 100644 --- a/drivers/xen/pvcalls-front.c +++ b/drivers/xen/pvcalls-front.c @@ -1018,6 +1018,12 @@ int pvcalls_front_release(struct socket *sock) pvcalls_front_free_map(bedata, map); } else { + wake_up(&bedata->inflight_req); + wake_up(&map->passive.inflight_accept_req); + + while (atomic_read(&map->refcount) > 1) + cpu_relax(); + spin_lock(&bedata->socket_lock); list_del(&map->list); spin_unlock(&bedata->socket_lock); |