Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-20 Thread kbuild test robot
Hi David, I love your patch! Yet something to improve: [auto build test ERROR on bpf-next/master] url: https://github.com/0day-ci/linux/commits/David-Ahern/bpf-Add-MTU-check-to-fib-lookup-helper/20180520-103417 base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master c

Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-19 Thread kbuild test robot
Hi David, I love your patch! Yet something to improve: [auto build test ERROR on bpf-next/master] url: https://github.com/0day-ci/linux/commits/David-Ahern/bpf-Add-MTU-check-to-fib-lookup-helper/20180520-103417 base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master c

Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-18 Thread Daniel Borkmann
On 05/18/2018 02:34 AM, David Ahern wrote: > On 5/17/18 4:22 PM, Daniel Borkmann wrote: >> On 05/17/2018 06:09 PM, David Ahern wrote: >>> Add check that egress MTU can handle packet to be forwarded. If >>> the MTU is less than the packet lenght, return 0 meaning the >>> packet is expected to contin

Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-17 Thread David Ahern
On 5/17/18 4:22 PM, Daniel Borkmann wrote: > On 05/17/2018 06:09 PM, David Ahern wrote: >> Add check that egress MTU can handle packet to be forwarded. If >> the MTU is less than the packet lenght, return 0 meaning the >> packet is expected to continue up the stack for help - eg., >> fragmenting th

Re: [PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-17 Thread Daniel Borkmann
On 05/17/2018 06:09 PM, David Ahern wrote: > Add check that egress MTU can handle packet to be forwarded. If > the MTU is less than the packet lenght, return 0 meaning the > packet is expected to continue up the stack for help - eg., > fragmenting the packet or sending an ICMP. > > Signed-off-by:

[PATCH bpf-next 3/3] bpf: Add mtu checking to FIB forwarding helper

2018-05-17 Thread David Ahern
Add check that egress MTU can handle packet to be forwarded. If the MTU is less than the packet lenght, return 0 meaning the packet is expected to continue up the stack for help - eg., fragmenting the packet or sending an ICMP. Signed-off-by: David Ahern --- net/core/filter.c | 10 ++ 1