]> git.baikalelectronics.ru Git - kernel.git/commit
xsk: Fix umem fill/completion queue mmap on 32-bit
authorGeert Uytterhoeven <geert@linux-m68k.org>
Thu, 7 Jun 2018 13:37:34 +0000 (15:37 +0200)
committerDaniel Borkmann <daniel@iogearbox.net>
Thu, 7 Jun 2018 22:18:02 +0000 (00:18 +0200)
commit296f17eab23245450af7806f86dd0a142561d613
treec58f1a5a9029fd5adcd6e6fffaaa694f195dbaa3
parent9e4510105c8e73168a248c7ce2c8d04e7cf009fd
xsk: Fix umem fill/completion queue mmap on 32-bit

With gcc-4.1.2 on 32-bit:

    net/xdp/xsk.c:663: warning: integer constant is too large for ‘long’ type
    net/xdp/xsk.c:665: warning: integer constant is too large for ‘long’ type

Add the missing "ULL" suffixes to the large XDP_UMEM_PGOFF_*_RING values
to fix this.

    net/xdp/xsk.c:663: warning: comparison is always false due to limited range of data type
    net/xdp/xsk.c:665: warning: comparison is always false due to limited range of data type

"unsigned long" is 32-bit on 32-bit systems, hence the offset is
truncated, and can never be equal to any of the XDP_UMEM_PGOFF_*_RING
values.  Use loff_t (and the required cast) to fix this.

Fixes: 34666db2a3bd3a72 ("xsk: add umem fill queue support and mmap")
Fixes: 165ede7e9b511fc8 ("xsk: add umem completion queue support and mmap")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Björn Töpel <bjorn.topel@intel.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
include/uapi/linux/if_xdp.h
net/xdp/xsk.c