]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
xsk: proper AF_XDP socket teardown ordering
authorBjörn Töpel <bjorn.topel@intel.com>
Fri, 5 Oct 2018 11:25:15 +0000 (13:25 +0200)
committerDaniel Borkmann <daniel@iogearbox.net>
Mon, 8 Oct 2018 08:09:22 +0000 (10:09 +0200)
commit541d7fdd7694560404c502f64298a90ffe017e6b
tree899c2c76d4f59f42a3b1ea0b1339e32f936426ca
parentdf1ea77bc51ddb571530fd1b765b4c132095c419
xsk: proper AF_XDP socket teardown ordering

The AF_XDP socket struct can exist in three different, implicit
states: setup, bound and released. Setup is prior the socket has been
bound to a device. Bound is when the socket is active for receive and
send. Released is when the process/userspace side of the socket is
released, but the sock object is still lingering, e.g. when there is a
reference to the socket in an XSKMAP after process termination.

The Rx fast-path code uses the "dev" member of struct xdp_sock to
check whether a socket is bound or relased, and the Tx code uses the
struct xdp_umem "xsk_list" member in conjunction with "dev" to
determine the state of a socket.

However, the transition from bound to released did not tear the socket
down in correct order.

On the Rx side "dev" was cleared after synchronize_net() making the
synchronization useless. On the Tx side, the internal queues were
destroyed prior removing them from the "xsk_list".

This commit corrects the cleanup order, and by doing so
xdp_del_sk_umem() can be simplified and one synchronize_net() can be
removed.

Fixes: 965a99098443 ("xsk: add support for bind for Rx")
Fixes: ac98d8aab61b ("xsk: wire upp Tx zero-copy functions")
Reported-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: Björn Töpel <bjorn.topel@intel.com>
Acked-by: Song Liu <songliubraving@fb.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
net/xdp/xdp_umem.c
net/xdp/xsk.c