Re: NAT performance regression caused by vlan GRO support

2019-04-04 Thread Rafał Miłecki
On 05.04.2019 06:26, Toshiaki Makita wrote: On 2019/04/05 5:22, Rafał Miłecki wrote: On 04.04.2019 17:17, Toshiaki Makita wrote: On 19/04/04 (木) 21:57:15, Rafał Miłecki wrote: I'd like to report a regression that goes back to the 2015. I know it's damn late, but the good thing is, the regressi

Re: NAT performance regression caused by vlan GRO support

2019-04-04 Thread Toshiaki Makita
On 2019/04/05 5:22, Rafał Miłecki wrote: > On 04.04.2019 17:17, Toshiaki Makita wrote: >> On 19/04/04 (木) 21:57:15, Rafał Miłecki wrote: >>> I'd like to report a regression that goes back to the 2015. I know >>> it's damn >>> late, but the good thing is, the regression is still easy to >>> reproduc

Re: [PATCH net] ipv6: un-do: defrag: drop non-last frags smaller than min mtu

2019-04-04 Thread Greg Kroah-Hartman
On Thu, Apr 04, 2019 at 06:18:30PM -0600, Captain Wiggum wrote: > Hi Greg, > > A previous bad patch breaks 18 test cases for IPv6 fragment headers. > This has already been un-done in upstream, but not in any of the LTS. > However two upstream patches are first needed to cover a DoS vulnerability.

Re: [PATCH net-next 0/2] sctp: fully support memory accounting

2019-04-04 Thread Xin Long
On Fri, Apr 5, 2019 at 1:38 AM David Miller wrote: > > From: Xin Long > Date: Thu, 4 Apr 2019 17:45:07 +0800 > > > We're thinking about adding some sctp tests into selftests, > > but these ones are not good to be the first one, I think. > > sysctl_sockets_memctl.sh is a multi-host test, even netn

Re: cellular modem driver APIs

2019-04-04 Thread Subash Abhinov Kasiviswanathan
On 2019-04-04 14:38, Johannes Berg wrote: Hi, The normal mode of operation of rmnet is using the rmnet netdevices in an embedded device. Sure. Can you say what driver this would typically live on top of? I'm actually a bit surprised to find out this isn't really a driver :-) This needs a p

Re: [PATCH] igb: Debugging for setting rate negotiation settings.

2019-04-04 Thread kbuild test robot
Hi, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on jkirsher-next-queue/dev-queue] [also build test WARNING on v5.1-rc3 next-20190404] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

Re: [PATCH bpf-next] selftests: bpf: add VRF test cases to lwt_ip_encap test.

2019-04-04 Thread David Ahern
On 4/3/19 9:43 AM, Peter Oskolkov wrote: > This patch adds tests validating that VRF and BPF-LWT > encap work together well, as requested by David Ahern. > > Signed-off-by: Peter Oskolkov > --- > .../selftests/bpf/test_lwt_ip_encap.sh| 134 +++--- > 1 file changed, 86 inserti

Re: [PATCH] igb: Debugging for setting rate negotiation settings.

2019-04-04 Thread kbuild test robot
Hi, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on jkirsher-next-queue/dev-queue] [also build test WARNING on v5.1-rc3 next-20190404] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

Re: [PATCH net] ibmvnic: Fix completion structure initialization

2019-04-04 Thread David Miller
From: Thomas Falcon Date: Thu, 4 Apr 2019 18:58:26 -0500 > Fix device initialization completion handling for vNIC adapters. > Initialize the completion structure on probe and reinitialize when needed. > This also fixes a race condition during kdump where the driver can attempt > to access the co

Re: [PATCH net] ipv6: sit: reset ip header pointer in ipip6_rcv

2019-04-04 Thread David Miller
From: Lorenzo Bianconi Date: Thu, 4 Apr 2019 16:37:53 +0200 > ipip6 tunnels run iptunnel_pull_header on received skbs. This can > determine the following use-after-free accessing iph pointer since > the packet will be 'uncloned' running pskb_expand_head if it is a > cloned gso skb (e.g if the pa

Re: [next-queue PATCH v1] igb: Fix limiting the number of queues to number of cpus

2019-04-04 Thread Vinicius Costa Gomes
Hi Jeff, Jeff Kirsher writes: > A module parameter maybe fine for our out-of-tree driver, but not for the > kernel driver. > > NACK on the basis that a new module parameter is being introduced for the > driver. This is not acceptable by Dave Miller or myself. As of now, I do > not have a alter

Re: [PATCH net-next v2 0/2] net: phy: use generic PHY ability readers if callback get_features isn't set

2019-04-04 Thread David Miller
From: Heiner Kallweit Date: Wed, 3 Apr 2019 23:13:18 +0200 > Meanwhile we have generic functions for reading the abilities of > Clause 22 / 45 PHY's. This allows to use them as fallback in case > callback get_features isn't set. Benefit is the reduction of > boilerplate code in PHY drivers. > >

Re: [PATCH net] net: thunderx: don't allow jumbo frames with XDP

2019-04-04 Thread Matteo Croce
On Fri, Apr 5, 2019 at 2:20 AM David Miller wrote: > > From: Matteo Croce > Date: Wed, 3 Apr 2019 01:11:36 +0200 > > > The thunderx driver forbids to load an eBPF program if the MTU is > > higher than 1500 bytes, but this can be circumvented by first > > loading the eBPF, and then raising the MT

Re: [PATCH net-next] Revert "r8169: use netif_receive_skb_list batching"

2019-04-04 Thread David Miller
From: Heiner Kallweit Date: Wed, 3 Apr 2019 19:54:12 +0200 > This reverts commit 6578229d4efb7ea6287861bfc2bd306140458e07. > netif_receive_skb_list() doesn't support GRO, therefore we may have > scenarios with decreased performance. See discussion here [0]. > > [0] https://marc.info/?t=155403847

Re: [patch net-next v4 00/13] net: extend devlink port attrs with switch ID

2019-04-04 Thread David Miller
From: Jiri Pirko Date: Wed, 3 Apr 2019 14:24:14 +0200 > From: Jiri Pirko > > Extend devlink port attrs to contain switch ID and change drivers that > register devlink ports to use that. Series applied, thanks Jiri.

Re: [PATCH net] libcxgb: fix incorrect ppmax calculation

2019-04-04 Thread David Miller
From: Varun Prakash Date: Wed, 3 Apr 2019 17:30:14 +0530 > BITS_TO_LONGS() uses DIV_ROUND_UP() because of > this ppmax value can be greater than available > per cpu page pods. > > This patch removes BITS_TO_LONGS() to fix this > issue. > > Signed-off-by: Varun Prakash Applied.

Re: [PATCH] net: Remove inclusion of pci.h

2019-04-04 Thread David Miller
From: Yuval Shaia Date: Wed, 3 Apr 2019 12:15:07 +0300 > This header is not in use - remove it. > > Signed-off-by: Yuval Shaia Applied to net-next.

Re: [PATCH net-next] net: use correct this_cpu primitive in dev_recursion_level

2019-04-04 Thread David Miller
From: Florian Westphal Date: Wed, 3 Apr 2019 08:28:35 +0200 > syzbot reports: > BUG: using __this_cpu_read() in preemptible code: > caller is dev_recursion_level include/linux/netdevice.h:3052 [inline] > __this_cpu_preempt_check+0x246/0x270 lib/smp_processor_id.c:47 > dev_recursion_level inclu

Re: [PATCH net-next 0/3] exthdrs: Make ext. headers & options useful - Part I

2019-04-04 Thread David Miller
From: Tom Herbert Date: Tue, 2 Apr 2019 19:36:25 -0700 > This patch series is Part I. It reorganizes the extension header related > files to make them easier to maintain and update. They are: If we don't see the patches that add ipv4 support alongside this I'm not going to apply patches that ju

Re: [PATCH net] net: thunderx: don't allow jumbo frames with XDP

2019-04-04 Thread David Miller
From: Matteo Croce Date: Wed, 3 Apr 2019 01:11:36 +0200 > The thunderx driver forbids to load an eBPF program if the MTU is higher > than 1500 bytes, but this can be circumvented by first loading the eBPF, > and then raising the MTU. > > XDP assumes that SKBs are linear and fit in a single page

Re: [PATCH net-next 1/1] net: sched: ensure tc flower reoffload takes filter ref

2019-04-04 Thread David Miller
From: John Hurley Date: Tue, 2 Apr 2019 23:53:20 +0100 > Recent changes to TC flower remove the requirement for rtnl lock when > accessing and modifying filters. Refcounts now ensure access and deletion > do not happen concurrently. However, the reoffload function which cycles > through all filt

Re: [PATCH] vlan: conditional inclusion of FCoE hooks to match netdevice.h and bnx2x

2019-04-04 Thread David Miller
From: Chris Leech Date: Tue, 2 Apr 2019 15:06:12 -0700 > Way back in 3c9c36bcedd426f2be2826da43e5163de61735f7 the > ndo_fcoe_get_wwn pointer was switched from depending on CONFIG_FCOE to > CONFIG_LIBFCOE in order to allow building FCoE support into the bnx2x > driver and used by bnx2fc without i

[PATCH net] ipv6: un-do: defrag: drop non-last frags smaller than min mtu

2019-04-04 Thread Captain Wiggum
Hi Greg, A previous bad patch breaks 18 test cases for IPv6 fragment headers. This has already been un-done in upstream, but not in any of the LTS. However two upstream patches are first needed to cover a DoS vulnerability. For background, there are two mail threads in [netdev] on this subject: 1

Re: pull-request: bpf 2019-04-04

2019-04-04 Thread David Miller
From: Daniel Borkmann Date: Thu, 4 Apr 2019 17:21:41 +0200 > The following pull-request contains BPF updates for your *net* tree. > > The main changes are: > > 1) Batch of fixes to the existing BPF flow dissector API to support >calling BPF programs from the eth_get_headlen context (suppor

Re: [next-queue PATCH v1] igb: Fix limiting the number of queues to number of cpus

2019-04-04 Thread Jeff Kirsher
On Thu, 2019-04-04 at 14:56 -0700, Vinicius Costa Gomes wrote: > We have seen some reports[1] of users complaining that they aren't > able to use some queues when their machines have less than 4 cpus. > This affects some TSN workloads, as different traffic classes are > assigned different queues. T

[PATCH net] ibmvnic: Fix completion structure initialization

2019-04-04 Thread Thomas Falcon
Fix device initialization completion handling for vNIC adapters. Initialize the completion structure on probe and reinitialize when needed. This also fixes a race condition during kdump where the driver can attempt to access the completion struct before it is initialized: Unable to handle kernel p

Re: r8169 driver from kernel 5.0 crashing - napi_consume_skb

2019-04-04 Thread VDR User
> > Unfortunately I have to report that the crashing has resurfaced. I'm > > currently using kernel 5.0 with Heiner's fix applied. In the last few > > days I've had the crash occur 4 times now. I'm not sure how to further > > investigate this but I'm guessing a patch that adds more debugging > > ou

[next-queue PATCH v1] igb: Fix limiting the number of queues to number of cpus

2019-04-04 Thread Vinicius Costa Gomes
We have seen some reports[1] of users complaining that they aren't able to use some queues when their machines have less than 4 cpus. This affects some TSN workloads, as different traffic classes are assigned different queues. The current behavior limits the number of traffic classes that can be re

[PATCH bpf-next] samples/bpf: fix build with new clang

2019-04-04 Thread Alexei Starovoitov
clang started to error on invalid asm clobber usage in x86 headers and many bpf program samples failed to build with the message: CLANG-bpf /data/users/ast/bpf-next/samples/bpf/xdp_redirect_kern.o In file included from /data/users/ast/bpf-next/samples/bpf/xdp_redirect_kern.c:14: In file includ

Re: cellular modem driver APIs

2019-04-04 Thread Johannes Berg
On Thu, 2019-04-04 at 22:38 +0200, Johannes Berg wrote: > > The bridge mode is used only for testing by sending frames > > without de-muxing to some other driver such as a USB netdev so packets > > can be parsed on a tethered PC. > > Yeah, I get it, it's just done in a strange way. You'd think ad

Re: cellular modem driver APIs

2019-04-04 Thread Johannes Berg
Hi, > The normal mode of operation of rmnet is using the rmnet netdevices > in an embedded device. Sure. Can you say what driver this would typically live on top of? I'm actually a bit surprised to find out this isn't really a driver :-) In my view right now, I'd recommend splitting rmnet into t

Re: [PATCH v4 net-next 0/5] net: More movement to fib_nh_common

2019-04-04 Thread David Miller
From: David Ahern Date: Thu, 4 Apr 2019 09:07:04 -0600 > On 4/3/19 10:50 PM, David Miller wrote: >> From: David Ahern >> Date: Tue, 2 Apr 2019 14:11:53 -0700 >> >>> From: David Ahern >>> >>> Second set of three with the end goal of enabling IPv6 gateways with IPv4 >>> routes. >>> >>> This set

Re: NAT performance regression caused by vlan GRO support

2019-04-04 Thread Rafał Miłecki
On 04.04.2019 17:17, Toshiaki Makita wrote: On 19/04/04 (木) 21:57:15, Rafał Miłecki wrote: I'd like to report a regression that goes back to the 2015. I know it's damn late, but the good thing is, the regression is still easy to reproduce, verify & revert. Long story short, starting with the co

Re: r8169 driver from kernel 5.0 crashing - napi_consume_skb

2019-04-04 Thread Heiner Kallweit
On 04.04.2019 16:28, VDR User wrote: > Hi, > > Unfortunately I have to report that the crashing has resurfaced. I'm > currently using kernel 5.0 with Heiner's fix applied. In the last few > days I've had the crash occur 4 times now. I'm not sure how to further > investigate this but I'm guessing a

Re: cellular modem driver APIs

2019-04-04 Thread Subash Abhinov Kasiviswanathan
On 2019-04-04 09:52, Dan Williams wrote: On Thu, 2019-04-04 at 11:00 +0200, Johannes Berg wrote: Hi, > Thanks a lot for doing this! Being responsible for most of the > issues > you point out, I can only say that you have my full support if you > want > to change any of it. :-) > My pathetic

[PATCH bpf-next 3/3] selftests: bpf: add selftest for __sk_buff context in BPF_PROG_TEST_RUN

2019-04-04 Thread Stanislav Fomichev
Simple test that sets cb to {1,2,3,4,5} and priority to 6, runs bpf program that fails if cb is not what we expect and increments cb[i] and priority. When the test finishes, we check that cb is now {2,3,4,5,6} and priority is 7. We also test the sanity checks: * ctx_in is provided, but ctx_size_in

[PATCH bpf-next 2/3] libbpf: add support for ctx_{size,}_{in,out} in BPF_PROG_TEST_RUN

2019-04-04 Thread Stanislav Fomichev
Support recently introduced input/output context for test runs. We extend only bpf_prog_test_run_xattr. bpf_prog_test_run is unextendable and left as is. Signed-off-by: Stanislav Fomichev --- tools/include/uapi/linux/bpf.h | 7 +++ tools/lib/bpf/bpf.c| 5 + tools/lib/bpf/bpf.

[PATCH bpf-next 1/3] bpf: support input __sk_buff context in BPF_PROG_TEST_RUN

2019-04-04 Thread Stanislav Fomichev
Add new set of arguments to bpf_attr for BPF_PROG_TEST_RUN: * ctx_in/ctx_size_in - input context * ctx_out/ctx_size_out - output context The intended use case is to pass some meta data to the test runs that operate on skb (this has being brought up on recent LPC). For programs that use bpf_prog_t

Re: [PATCH net] team: set slave to promisc if team is already in promisc mode

2019-04-04 Thread Marcelo Ricardo Leitner
On Thu, Apr 04, 2019 at 08:47:02PM +0800, Hangbin Liu wrote: > After adding a team interface to bridge, the team interface will enter > promisc mode. Then if we add a new slave to team0, the slave will keep > promisc off. Fix it by setting slave to promisc on if team master is > already in promisc

[PATCH net-next 07/18] ipv4: Add support to fib_config for IPv6 gateway

2019-04-04 Thread David Ahern
From: David Ahern Add support for an IPv6 gateway to fib_config. Since a gateway is either IPv4 or IPv6, make it a union with fc_gw4 where fc_gw_family decides which address is in use. Update current checks on family and gw4 to handle ipv6 as well. Signed-off-by: David Ahern --- include/net/ip

Re: [PATCH net] net: vrf: Fix ping failed when vrf mtu is set to 0

2019-04-04 Thread David Miller
From: linmiaohe Date: Thu, 4 Apr 2019 20:01:13 +0800 > When the mtu of a vrf device is set to 0, it would cause ping > failed.So I think we should limit dev->min_mtu to ETH_MIN_MTU > to solve this problem. And if dev->max_mtu still be 0 can be > confusing, so I limit dev->max_mtu to ETH_MAX_MTU.

Re: [PATCH net-next] net: bridge: update multicast stats from maybe_deliver()

2019-04-04 Thread David Miller
From: Pablo Neira Ayuso Date: Thu, 4 Apr 2019 13:56:38 +0200 > Simplify this code by updating bridge multicast stats from > maybe_deliver(). > > Note that commit 6db6f0eae605 ("bridge: multicast to unicast"), in case > the port flag BR_MULTICAST_TO_UNICAST is set, never updates the previous > p

[PATCH net-next 03/18] net: Replace nhc_has_gw with nhc_gw_family

2019-04-04 Thread David Ahern
From: David Ahern Allow the gateway in a fib_nh_common to be from a different address family than the outer fib{6}_nh. To that end, replace nhc_has_gw with nhc_gw_family and update users of nhc_has_gw to check nhc_gw_family. Now nhc_family is used to know if the nh_common is part of a fib_nh or f

[PATCH net-next 00/18] ipv4: Enable support for IPv6 gateway with IPv4 routes

2019-04-04 Thread David Ahern
From: David Ahern Last set of three with the end goal of enabling IPv6 gateways with IPv4 routes. This set adds fib6_nh_init and release to the IPv6 stubs, and adds neighbor helpers that IPv4 code invokes to resolve an IPv6 address. When using an IPv6 neighbor entry the hh_cache is bypassed as i

[PATCH net-next 08/18] ipv4: Refactor fib_check_nh

2019-04-04 Thread David Ahern
From: David Ahern fib_check_nh is currently huge covering multiple uses cases - device only, device + gateway, and device + gateway with ONLINK. The next patch adds validation checks for IPv6 which only further complicates it. So, break fib_check_nh into 2 helpers - one for gateway validation and

[PATCH net-next 05/18] ipv4: Prepare fib_config for IPv6 gateway

2019-04-04 Thread David Ahern
From: David Ahern Similar to rtable, fib_config needs to allow the gateway to be either an IPv4 or an IPv6 address. To that end, rename fc_gw to fc_gw4 to mean an IPv4 address and add fc_gw_family. Check on 'is a gateway set' are changed to see if fc_gw_family is set. In the process prepare the c

Re: [PATCH net-next] net: use kfree_skb_list() from ip_do_fragment()

2019-04-04 Thread David Miller
From: Pablo Neira Ayuso Date: Thu, 4 Apr 2019 13:54:20 +0200 > Just like 46cfd725c377 ("net: use kfree_skb_list() helper in more places"). > > Signed-off-by: Pablo Neira Ayuso > Acked-by: Florian Westphal Applied.

[PATCH net-next 02/18] ipv6: Add neighbor helpers that use the ipv6 stub

2019-04-04 Thread David Ahern
From: David Ahern Add ipv6 helpers to handle ndisc references via the stub. Update bpf_ipv6_fib_lookup to use __ipv6_neigh_lookup_noref_stub instead of the open code ___neigh_lookup_noref with the stub. Signed-off-by: David Ahern --- include/net/ndisc.h | 40 +++

[PATCH net-next 01/18] ipv6: Add fib6_nh_init and release to stubs

2019-04-04 Thread David Ahern
From: David Ahern Signed-off-by: David Ahern --- include/net/ipv6_stubs.h | 6 ++ net/ipv6/addrconf_core.c | 9 + net/ipv6/af_inet6.c | 2 ++ 3 files changed, 17 insertions(+) diff --git a/include/net/ipv6_stubs.h b/include/net/ipv6_stubs.h index d8d9c0b0e8c0..453b55bf6723 100

[PATCH net-next 06/18] ipv4: Add support to rtable for ipv6 gateway

2019-04-04 Thread David Ahern
From: David Ahern Add support for an IPv6 gateway to rtable. Since a gateway is either IPv4 or IPv6, make it a union with rt_gw4 where rt_gw_family decides which address is in use. When dumping the route data, encode an ipv6 nexthop using RTA_VIA. Signed-off-by: David Ahern --- .../net/ethern

[PATCH net-next 12/18] bpf: Handle ipv6 gateway in bpf_ipv4_fib_lookup

2019-04-04 Thread David Ahern
From: David Ahern Update bpf_ipv4_fib_lookup to handle an ipv6 gateway. Signed-off-by: David Ahern --- net/core/filter.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/net/core/filter.c b/net/core/filter.c index eb07a02596c5..b706e9799dfe 100644 --- a/ne

[PATCH net-next 10/18] neighbor: Add skip_cache argument to neigh_output

2019-04-04 Thread David Ahern
From: David Ahern A later patch allows an IPv6 gateway with an IPv4 route. The neighbor entry will exist in the v6 ndisc table and the cached header will contain the ipv6 protocol which is wrong for an IPv4 packet. For an IPv4 packet to use the v6 neighbor entry, neigh_output needs to skip the ca

[PATCH net-next 09/18] ipv4: Add fib_check_nh_v6_gw

2019-04-04 Thread David Ahern
From: David Ahern Add helper to validate nexthop spec with an IPv6 gateway. Signed-off-by: David Ahern --- net/ipv4/fib_semantics.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/net/ipv4/fib_semantics.c b/net/ipv4/fib_semantics.c index b9f0388a2c70..acbdf0ed6

[PATCH net-next 11/18] ipv4: Add helpers for neigh lookup for nexthop

2019-04-04 Thread David Ahern
From: David Ahern A common them in the output path is looking up a neigh entry for a nexthop, either the gateway in an rtable or a fallback to the daddr in the skb: nexthop = (__force u32)rt_nexthop(rt, ip_hdr(skb)->daddr); neigh = __ipv4_neigh_lookup_noref(dev, nexthop);

[PATCH net-next 13/18] ipv4: Handle ipv6 gateway in ipv4_confirm_neigh

2019-04-04 Thread David Ahern
From: David Ahern Update ipv4_confirm_neigh to handle an ipv6 gateway. Signed-off-by: David Ahern --- net/ipv4/route.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/net/ipv4/route.c b/net/ipv4/route.c index 32ecb4c1c7e3..efa6a36cbfff 100644 --- a/net/ipv4/route

[PATCH net-next 18/18] selftests: fib_tests: Add tests for ipv6 gateway with ipv4 route

2019-04-04 Thread David Ahern
From: David Ahern Add tests for ipv6 gateway with ipv4 route. Tests include basic single path with ping to verify connectivity and multipath. Signed-off-by: David Ahern --- tools/testing/selftests/net/fib_tests.sh | 70 +++- 1 file changed, 69 insertions(+), 1 delet

[PATCH net-next 04/18] ipv4: Prepare rtable for IPv6 gateway

2019-04-04 Thread David Ahern
From: David Ahern To allow the gateway to be either an IPv4 or IPv6 address, remove rt_uses_gateway from rtable and replace with rt_gw_family. If rt_gw_family is set it implies rt_uses_gateway. Rename rt_gateway to rt_gw4 to represent the IPv4 version. Signed-off-by: David Ahern --- drivers/in

[PATCH net-next 15/18] ipv4: Handle ipv6 gateway in fib_good_nh

2019-04-04 Thread David Ahern
From: David Ahern Update fib_good_nh to handle an ipv6 gateway. Signed-off-by: David Ahern --- net/ipv4/fib_semantics.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/net/ipv4/fib_semantics.c b/net/ipv4/fib_semantics.c index f3f11f55e905..6b48a0b92eef 100644 ---

[PATCH net-next 16/18] ipv4: Flag fib_info with a fib_nh using IPv6 gateway

2019-04-04 Thread David Ahern
From: David Ahern Until support is added to the offload drivers, they need to be able to reject routes with an IPv6 gateway. To that end add a flag to fib_info that indicates if any fib_nh has a v6 gateway. The flag allows the drivers to efficiently know the use of a v6 gateway without walking al

[PATCH net-next 14/18] ipv4: Handle ipv6 gateway in fib_detect_death

2019-04-04 Thread David Ahern
From: David Ahern Update fib_detect_death to handle an ipv6 gateway. Signed-off-by: David Ahern --- net/ipv4/fib_semantics.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/net/ipv4/fib_semantics.c b/net/ipv4/fib_semantics.c index acbdf0ed6ff8..f3f11f55e905 100644

[PATCH net-next 17/18] ipv4: Allow ipv6 gateway with ipv4 routes

2019-04-04 Thread David Ahern
From: David Ahern Add support for RTA_VIA and allow an IPv6 nexthop for v4 routes: $ ip ro add 172.16.1.0/24 via inet6 2001:db8::1 dev eth0 $ ip ro ls ... 172.16.1.0/24 via inet6 2001:db8::1 dev eth0 For convenience and simplicity, userspace can use RTA_VIA to specify AF_INET or AF_I

Re: [PATCH net] net/sched: act_sample: fix divide by zero in the traffic path

2019-04-04 Thread David Miller
From: Davide Caratti Date: Thu, 4 Apr 2019 12:31:35 +0200 > the control path of 'sample' action does not validate the value of 'rate' > provided by the user, but then it uses it as divisor in the traffic path. > Validate it in tcf_sample_init(), and return -EINVAL with a proper extack > message

Re: [PATCH bpf] bpf: report verifier bugs as warnings

2019-04-04 Thread Alexei Starovoitov
On Thu, Apr 04, 2019 at 12:41:32AM +0200, Daniel Borkmann wrote: > On 04/03/2019 07:30 PM, Alexei Starovoitov wrote: > > On Wed, Apr 03, 2019 at 04:52:40PM +0100, Edward Cree wrote: > >> On 02/04/2019 15:37, Daniel Borkmann wrote: > >>> If we really want to have a kernel warn, then lets add a > >>>

Re: [PATCH net] net: thunderx: fix NULL pointer dereference in nicvf_open/nicvf_stop

2019-04-04 Thread David Miller
From: Lorenzo Bianconi Date: Thu, 4 Apr 2019 12:16:27 +0200 > When a bpf program is uploaded, the driver computes the number of > xdp tx queues resulting in the allocation of additional qsets. > Starting from commit '2ecbe4f4a027 ("net: thunderx: replace global > nicvf_rx_mode_wq work queue for

Re: [PATCH rdma-next 00/12] Move IB representors to single IB device multiple ports

2019-04-04 Thread Leon Romanovsky
On Thu, Apr 04, 2019 at 10:02:21AM -0300, Jason Gunthorpe wrote: > On Thu, Mar 28, 2019 at 03:27:30PM +0200, Leon Romanovsky wrote: > > From: Leon Romanovsky > > > > >From Mark, > > > > Hi, > > > > This series starts from two refactoring patches performed in mlx5_core > > and they are part of a la

Re: [PATCH net-next 0/2] sctp: fully support memory accounting

2019-04-04 Thread David Miller
From: Xin Long Date: Thu, 4 Apr 2019 17:45:07 +0800 > We're thinking about adding some sctp tests into selftests, > but these ones are not good to be the first one, I think. > sysctl_sockets_memctl.sh is a multi-host test, even netns can't work for it. You can't set per-netns memory controls app

Re: [PATCH iproute2-next] ip xfrm: support setting/printing XFRMA_IF_ID attribute in states/policies

2019-04-04 Thread Stephen Hemminger
On Thu, 4 Apr 2019 19:07:38 +0300 Eyal Birger wrote: > The XFRMA_IF_ID attribute is set in policies/states for them to be > associated with an XFRM interface (4.19+). > > Add support for setting / displaying this attribute. > > Note that 0 is a valid value therefore set XFRMA_IF_ID if any valu

Re: [iproute2] ip: support for xfrm interfaces

2019-04-04 Thread Antony Antony
Tested-by: Antony Antony Question: is it easy to add "if_id" to "ip link show" output? currently: ip link show ipsec0 4: ipsec0@eth1: mtu 1500 qdisc noqueue state UNKNOWN mode DEFAULT group default qlen 1000 link/none da:25:61:2e:0c:98 brd ff:ff:ff:ff:ff:ff proposed: 4: ipsec0@eth1: mtu 1

[PATCH iproute2-next] ip xfrm: support setting/printing XFRMA_IF_ID attribute in states/policies

2019-04-04 Thread Eyal Birger
The XFRMA_IF_ID attribute is set in policies/states for them to be associated with an XFRM interface (4.19+). Add support for setting / displaying this attribute. Note that 0 is a valid value therefore set XFRMA_IF_ID if any value was provided in command line. Tested-by: Antony Antony Signed-of

Re: cellular modem driver APIs

2019-04-04 Thread Dan Williams
On Thu, 2019-04-04 at 11:00 +0200, Johannes Berg wrote: > Hi, > > > Thanks a lot for doing this! Being responsible for most of the > > issues > > you point out, I can only say that you have my full support if you > > want > > to change any of it. > > :-) > > > My pathetic excuses below are just

Re: [PATCH net-next] selftests: Add debugging options to pmtu.sh

2019-04-04 Thread David Ahern
On 4/4/19 6:16 AM, Stefano Brivio wrote: >> diff --git a/tools/testing/selftests/net/pmtu.sh >> b/tools/testing/selftests/net/pmtu.sh >> index 912b2dc50be3..28e8c97b5c9e 100755 >> --- a/tools/testing/selftests/net/pmtu.sh >> +++ b/tools/testing/selftests/net/pmtu.sh >> @@ -116,6 +116,9 @@ >> # Ks

Re: [PATCH v3 1/2] selftests/bpf: ksym_search won't check symbols exists

2019-04-04 Thread Daniel Borkmann
On 04/04/2019 12:17 AM, Daniel T. Lee wrote: > Currently, ksym_search located at trace_helpers won't check symbols are > existing or not. > > In ksym_search, when symbol is not found, it will return &syms[0](_stext). > But when the kernel symbols are not loaded, it will return NULL, which is > not

pull-request: bpf 2019-04-04

2019-04-04 Thread Daniel Borkmann
Hi David, The following pull-request contains BPF updates for your *net* tree. The main changes are: 1) Batch of fixes to the existing BPF flow dissector API to support calling BPF programs from the eth_get_headlen context (support for latter is planned to be added in bpf-next), from Stani

Re: NAT performance regression caused by vlan GRO support

2019-04-04 Thread Toshiaki Makita
Hi Rafał, On 19/04/04 (木) 21:57:15, Rafał Miłecki wrote: Hello, I'd like to report a regression that goes back to the 2015. I know it's damn late, but the good thing is, the regression is still easy to reproduce, verify & revert. Long story short, starting with the commit 66e5133f19e9 ("vla

Re: [PATCH v4 net-next 0/5] net: More movement to fib_nh_common

2019-04-04 Thread David Ahern
On 4/3/19 10:50 PM, David Miller wrote: > From: David Ahern > Date: Tue, 2 Apr 2019 14:11:53 -0700 > >> From: David Ahern >> >> Second set of three with the end goal of enabling IPv6 gateways with IPv4 >> routes. >> >> This set moves: >> - the ipv4 tracepoint to take a fib_nh_common and updates

Re: [PATCH bpf-next] selftests: bpf: add VRF test cases to lwt_ip_encap test.

2019-04-04 Thread David Ahern
On 4/3/19 11:09 PM, Martin Lau wrote: > On Wed, Apr 03, 2019 at 08:43:38AM -0700, Peter Oskolkov wrote: >> This patch adds tests validating that VRF and BPF-LWT >> encap work together well, as requested by David Ahern. > LGTM. David Ahern, please review also. > looked at it last night and hit so

Re: [PATCH bpf-next] selftests: bpf: add VRF test cases to lwt_ip_encap test.

2019-04-04 Thread David Ahern
On 4/3/19 11:09 PM, Martin Lau wrote: > On Wed, Apr 03, 2019 at 08:43:38AM -0700, Peter Oskolkov wrote: >> This patch adds tests validating that VRF and BPF-LWT >> encap work together well, as requested by David Ahern. > LGTM. David Ahern, please review also. > looking at it last night and hit s

[PATCH net] ipv6: sit: reset ip header pointer in ipip6_rcv

2019-04-04 Thread Lorenzo Bianconi
ipip6 tunnels run iptunnel_pull_header on received skbs. This can determine the following use-after-free accessing iph pointer since the packet will be 'uncloned' running pskb_expand_head if it is a cloned gso skb (e.g if the packet has been sent though a veth device) [ 706.369655] BUG: KASAN: us

RE: bnx2x - odd behaviour

2019-04-04 Thread Sudarsana Reddy Kalluru
Hi, We are not aware of this issue. Please collect the register dump i.e., "ethtool -d " output when this issue happens (before performing link-flap) and share it for the analysis. Thanks, Sudarsana > -Original Message- > From: netdev-ow...@vger.kernel.org On > Behalf Of Ian Kumlien

Re: r8169 driver from kernel 5.0 crashing - napi_consume_skb

2019-04-04 Thread VDR User
Hi, Unfortunately I have to report that the crashing has resurfaced. I'm currently using kernel 5.0 with Heiner's fix applied. In the last few days I've had the crash occur 4 times now. I'm not sure how to further investigate this but I'm guessing a patch that adds more debugging output will be ne

[PATCH v3 iproute2] ip: support for xfrm interfaces

2019-04-04 Thread Matt Ellison
Interfaces take a 'if_id' which is an interface id which can be set on an xfrm policy as its interface lookup key (XFRMA_IF_ID). Signed-off-by: Matt Ellison --- My apologies for taking so long to get v3 out: v3 Changes: * Only set IF_ID if set on command line. ip/Makefile

Re: [PATCH] v2 Check for SKBTX_HW_TSTAMP in macb driver

2019-04-04 Thread Paul Thomas
On Thu, Apr 4, 2019 at 9:23 AM Paul Thomas wrote: > > > Could you please resend the patch addressing Dave's review comments > > about subject line and Fixes tag? > Yup, sorry about that. I'm not subscribed to netdev, but I thought > replies would have me in the to:line, but I see the comments on t

Re: [PATCH bpf] flow_dissector: rst'ify documentation

2019-04-04 Thread Daniel Borkmann
On 04/03/2019 10:53 PM, Stanislav Fomichev wrote: > Rename bpf_flow_dissector.txt to bpf_flow_dissector.rst and fix > formatting. Also, link it from the Documentation/networking/index.rst. > > Tested with 'make htmldocs' to make sure it looks reasonable. > > Fixes: ae82899bbe92 ("flow_dissector:

Re: [PATCH] v2 Check for SKBTX_HW_TSTAMP in macb driver

2019-04-04 Thread Paul Thomas
> Could you please resend the patch addressing Dave's review comments > about subject line and Fixes tag? Yup, sorry about that. I'm not subscribed to netdev, but I thought replies would have me in the to:line, but I see the comments on the archive. I'll send this out shortly. -Paul

Re: [PATCH] v2 Check for SKBTX_HW_TSTAMP in macb driver

2019-04-04 Thread Harini Katakam
Hi Paul, On Thu, Apr 4, 2019 at 6:35 PM Paul Thomas wrote: > > On Tue, Mar 19, 2019 at 1:07 PM Paul Thomas wrote: > > > Hi All, just a reminder on this. Could you please resend the patch addressing Dave's review comments about subject line and Fixes tag? Regards, Harini

Re: [PATCH] v2 Check for SKBTX_HW_TSTAMP in macb driver

2019-04-04 Thread Paul Thomas
On Tue, Mar 19, 2019 at 1:07 PM Paul Thomas wrote: > > Make sure SKBTX_HW_TSTAMP (i.e. SOF_TIMESTAMPING_TX_HARDWARE) has been > enabled for this skb It does fix the issue where normal socks that > aren't expecting a timestamp will not wake up on select. And when a > user does want a SOF_TIMESTAMPI

Re: [PATCH rdma-next 00/12] Move IB representors to single IB device multiple ports

2019-04-04 Thread Jason Gunthorpe
On Thu, Mar 28, 2019 at 03:27:30PM +0200, Leon Romanovsky wrote: > From: Leon Romanovsky > > >From Mark, > > Hi, > > This series starts from two refactoring patches performed in mlx5_core > and they are part of a larger series intended to expose the ability > to insert FDB steering rules by the

NAT performance regression caused by vlan GRO support

2019-04-04 Thread Rafał Miłecki
Hello, I'd like to report a regression that goes back to the 2015. I know it's damn late, but the good thing is, the regression is still easy to reproduce, verify & revert. Long story short, starting with the commit 66e5133f19e9 ("vlan: Add GRO support for non hardware accelerated vlan") - which

[PATCH net] team: set slave to promisc if team is already in promisc mode

2019-04-04 Thread Hangbin Liu
After adding a team interface to bridge, the team interface will enter promisc mode. Then if we add a new slave to team0, the slave will keep promisc off. Fix it by setting slave to promisc on if team master is already in promisc mode, also do the same for allmulti. Fixes: 3d249d4ca7d0 ("net: intr

[PATCH net] netdevice: Fix promiscuity and allmulti negative overflow

2019-04-04 Thread Hangbin Liu
Similarly to dad9b335c694 ("netdevice: Fix promiscuity and allmulti overflow"), we should not decrease promiscuity if it is already 0. An example is after adding a team interface to bridge, the team interface will enter promisc mode. Then if we add a slave to team0, the slave will keep promisc off

Re: [PATCH net] net: thunderx: fix NULL pointer dereference in nicvf_open/nicvf_stop

2019-04-04 Thread Matteo Croce
On Thu, Apr 4, 2019 at 12:16 PM Lorenzo Bianconi wrote: > > When a bpf program is uploaded, the driver computes the number of > xdp tx queues resulting in the allocation of additional qsets. > Starting from commit '2ecbe4f4a027 ("net: thunderx: replace global > nicvf_rx_mode_wq work queue for all

Re: [PATCH net-next] selftests: Add debugging options to pmtu.sh

2019-04-04 Thread Stefano Brivio
Hi David, On Wed, 3 Apr 2019 18:18:24 -0700 David Ahern wrote: > From: David Ahern > > pmtu.sh script runs a number of tests and dumps a summary of pass/fail. > If a test fails, it is near impossible to debug why. For example: > > TEST: ipv6: PMTU exceptions [FAIL]

[PATCH net-next] net: use kfree_skb_list() from ip_do_fragment()

2019-04-04 Thread Pablo Neira Ayuso
Just like 46cfd725c377 ("net: use kfree_skb_list() helper in more places"). Signed-off-by: Pablo Neira Ayuso Acked-by: Florian Westphal --- net/ipv4/ip_output.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c index c80188875

Re: [net-next 13/15] net/mlx5: A write memory barrier is sufficient in EQ ci update

2019-04-04 Thread Gal Pressman
On 03-Apr-19 02:03, Saeed Mahameed wrote: > From: Tariq Toukan > > Soften the memory barrier call of mb() by a sufficient wmb() in the > consumer index update of the event queues. > > Signed-off-by: Tariq Toukan > Signed-off-by: Saeed Mahameed > --- > drivers/net/ethernet/mellanox/mlx5/core/e

Re: [PATCH net-next] net: phy: improve handling link_change_notify callback

2019-04-04 Thread David Wu
Hi Heiner, Today i apply this patch, I test it on the RK3328 which has the phy, it looks good, and link state switching between 10M/100M is correct. 在 2019/3/21 上午1:50, David Miller 写道: From: Heiner Kallweit Date: Tue, 19 Mar 2019 19:56:51 +0100 Currently the Phy driver's link_change_notify

Re: [RFC net-next 1/1] net: sched: fix hw filter offload in tc flower

2019-04-04 Thread John Hurley
On Wed, Apr 3, 2019 at 5:42 PM Vlad Buslov wrote: > > > On Wed 03 Apr 2019 at 15:37, John Hurley wrote: > > Recent refactoring of fl_change aims to use the classifier spinlock to > > avoid the need for rtnl lock. In doing so, the fl_hw_replace_filer() > > function was moved to before the lock is

Re: [PATCH net] net/sched: act_sample: fix divide by zero in the traffic path

2019-04-04 Thread yotam gigi
On Thu, Apr 4, 2019 at 1:32 PM Davide Caratti wrote: > > the control path of 'sample' action does not validate the value of 'rate' > provided by the user, but then it uses it as divisor in the traffic path. > Validate it in tcf_sample_init(), and return -EINVAL with a proper extack > message in ca

[PATCH net] net/sched: act_sample: fix divide by zero in the traffic path

2019-04-04 Thread Davide Caratti
the control path of 'sample' action does not validate the value of 'rate' provided by the user, but then it uses it as divisor in the traffic path. Validate it in tcf_sample_init(), and return -EINVAL with a proper extack message in case that value is zero, to fix a splat with the script below: #

iproute2: Issue with link type link/generic for NETROM links

2019-04-04 Thread Ralf Baechle
For a NETROM "ip link show dev nr0" will show 4: nr0: mtu 236 qdisc noqueue state UNKNOWN mode DEFAULT group default qlen 1000 link/generic 88:98:6a:a4:84:40:0a brd 00:00:00:00:00:00:00 But rather link/netrom is expected to be displayed. I have below patch to change ip such that link/netro

[PATCH net] net: thunderx: fix NULL pointer dereference in nicvf_open/nicvf_stop

2019-04-04 Thread Lorenzo Bianconi
When a bpf program is uploaded, the driver computes the number of xdp tx queues resulting in the allocation of additional qsets. Starting from commit '2ecbe4f4a027 ("net: thunderx: replace global nicvf_rx_mode_wq work queue for all VFs to private for each of them")' the driver runs link state polli

  1   2   >