]> git.proxmox.com Git - mirror_ubuntu-eoan-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)
commita5a16e43529b5040760ebf9bd9b056dd34861f93
treec58f1a5a9029fd5adcd6e6fffaaa694f195dbaa3
parent23316a366e1654e4ad05817c6075bc1019efb30a
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: 423f38329d267969 ("xsk: add umem fill queue support and mmap")
Fixes: fe2308328cd2f26e ("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