diff options
author | Krzysztof Kazimierczak <krzysztof.kazimierczak@intel.com> | 2019-01-10 20:29:02 +0100 |
---|---|---|
committer | Daniel Borkmann <daniel@iogearbox.net> | 2019-01-15 20:51:57 +0100 |
commit | cc5b5d3565048ae57d14e5674a5fb085b2ab0193 (patch) | |
tree | 802382d715a03e5f6e1f20558a700df5800533a3 /tools/testing | |
parent | 7939f8beecf1cfb10d255be65bfbd3793fae82a1 (diff) | |
download | linux-stable-cc5b5d3565048ae57d14e5674a5fb085b2ab0193.tar.gz linux-stable-cc5b5d3565048ae57d14e5674a5fb085b2ab0193.tar.bz2 linux-stable-cc5b5d3565048ae57d14e5674a5fb085b2ab0193.zip |
xsk: Check if a queue exists during umem setup
In the xdp_umem_assign_dev() path, the xsk code does not
check if a queue for which umem is to be created exists.
It leads to a situation where umem is not assigned to any
Tx/Rx queue of a netdevice, without notifying the stack
about an error. This affects both XDP_SKB and XDP_DRV
modes - in case of XDP_DRV_ZC, queue index is checked by
the driver.
This patch fixes xsk code, so that in both XDP_SKB and
XDP_DRV mode of AF_XDP, an error is returned when requested
queue index exceedes an existing maximum.
Fixes: c9b47cc1fabca ("xsk: fix bug when trying to use both copy and zero-copy on one queue id")
Reported-by: Jakub Spizewski <jakub.spizewski@intel.com>
Signed-off-by: Krzysztof Kazimierczak <krzysztof.kazimierczak@intel.com>
Acked-by: Björn Töpel <bjorn.topel@intel.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Diffstat (limited to 'tools/testing')
0 files changed, 0 insertions, 0 deletions