Re: [Patch net-next 2/2] net: dump whole skb data in netdev_rx_csum_fault()

2018-11-21 Thread Saeed Mahameed
On Wed, 2018-11-21 at 10:26 -0800, Eric Dumazet wrote: > On Wed, Nov 21, 2018 at 10:17 AM Cong Wang > wrote: > > On Wed, Nov 21, 2018 at 5:05 AM Eric Dumazet < > > eric.duma...@gmail.com> wrote: > > > > > > > > > On 11/20/2018 06:13 PM, Cong Wang wrote: > > > > Currently, we only dump a few

consistency for statistics with XDP mode

2018-11-21 Thread David Ahern
Paweł ran some more XDP tests yesterday and from it found a couple of issues. One is a panic in the mlx5 driver unloading the bpf program (mlx5e_xdp_xmit); he will send a send a separate email for that problem. The problem I wanted to discuss here is statistics for XDP context. The short of it is

Re: [PATCH net-next 2/4] netns: add support of NETNSA_TARGET_NSID

2018-11-21 Thread David Ahern
On 11/21/18 1:58 PM, Nicolas Dichtel wrote: > The target-nsid is not stored in net_cb (not needed). ref_net is set only if > tgt_net comes from TARGET_NSID. I can add a comment. ref_net is added by this patch and it is only used (unless I missed something) to know if the put_net is needed. ie/,

Re: [PATCH net-next 2/3] tcp: implement coalescing on backlog queue

2018-11-21 Thread Yuchung Cheng
On Wed, Nov 21, 2018 at 9:52 AM, Eric Dumazet wrote: > > In case GRO is not as efficient as it should be or disabled, > we might have a user thread trapped in __release_sock() while > softirq handler flood packets up to the point we have to drop. > > This patch balances work done from user thread

Re: [RFC v3 0/3] Add VRF support for VXLAN underlay

2018-11-21 Thread David Ahern
On 11/21/18 6:30 AM, Alexis Bauvin wrote: > Le 20 nov. 2018 à 22:45, David Ahern a écrit : >> >> On 11/20/18 7:23 AM, Alexis Bauvin wrote: >>> We are trying to isolate the VXLAN traffic from different VMs with VRF as >>> shown >>> in the schemas below: >>> >>> +-+

Re: [PATCH net-next 4/4] netns: enable to dump full nsid translation table

2018-11-21 Thread Nicolas Dichtel
Le 21/11/2018 à 19:09, David Ahern a écrit : > On 11/21/18 4:01 AM, Nicolas Dichtel wrote: >> diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c >> index 92730905886c..fc568cd0b560 100644 >> --- a/net/core/net_namespace.c >> +++ b/net/core/net_namespace.c >> @@ -740,7 +740,7 @@

Re: consistency for statistics with XDP mode

2018-11-21 Thread Paweł Staszewski
W dniu 21.11.2018 o 22:14, Toke Høiland-Jørgensen pisze: David Ahern writes: Paweł ran some more XDP tests yesterday and from it found a couple of issues. One is a panic in the mlx5 driver unloading the bpf program (mlx5e_xdp_xmit); he will send a send a separate email for that problem.

Re: [PATCH bpf-next] bpf: add read/write access to skb->tstamp from tc clsact progs

2018-11-21 Thread Vlad Dumitrescu
On Wed, Nov 21, 2018 at 5:08 AM Eric Dumazet wrote: > > > > On 11/20/2018 06:40 PM, Alexei Starovoitov wrote: > > > > > looks good to me. > > > > Any particular reason you decided to disable it for cg_skb ? > > It seems to me the same EDT approach will work from > > cgroup-bpf skb hooks just as

[net-next 3/6] docs-networking: fix typo in define

2018-11-21 Thread Jeff Kirsher
From: Jesse Brandeburg The #define for NETIF_F_GSO_UDP_L4 was incorrect in the documentation, fix it by making it match the actual code. Signed-off-by: Jesse Brandeburg Tested-by: Andrew Bowers Tested-by: Aaron Brown Signed-off-by: Jeff Kirsher ---

[net-next 4/6] ethernet/intel: consolidate NAPI and NAPI exit

2018-11-21 Thread Jeff Kirsher
From: Jesse Brandeburg While reviewing code, I noticed that Eric Dumazet recommends that drivers check the return code of napi_complete_done, and use that to decide to enable interrupts or not when exiting poll. One of the Intel drivers was already fixed (ixgbe). Upon looking at the Intel

[net-next 0/6][pull request] Intel Wired LAN Driver Updates 2018-11-21

2018-11-21 Thread Jeff Kirsher
This series contains updates to all of the Intel LAN drivers and documentation. Shannon Nelson updates the ixgbe kernel documentation to include IPsec hardware offload. Joe Perches cleans up whitespace issues in the igb driver. Jesse update the netdev kernel documentation for NETIF_F_GSO_UDP_L4

[net-next 1/6] ixgbe: add ipsec hw offload note to ixgbe Documentation

2018-11-21 Thread Jeff Kirsher
From: Shannon Nelson Add a short note about using IPsec Hardware Offload with the ixgbe driver. Signed-off-by: Shannon Nelson Tested-by: Andrew Bowers Signed-off-by: Jeff Kirsher --- Documentation/networking/ixgbe.rst | 13 + 1 file changed, 13 insertions(+) diff --git

[net-next 6/6] igc: Remove obsolete IGC_ERR define

2018-11-21 Thread Jeff Kirsher
From: Sasha Neftin Address community comment. Remove obsolete IGC_ERR define and use dev_err method. Suggested by Joe Perches. Signed-off-by: Sasha Neftin Tested-by: Aaron Brown Signed-off-by: Jeff Kirsher --- drivers/net/ethernet/intel/igc/igc.h | 2 --

[net-next 2/6] igb: Fix format with line continuation whitespace

2018-11-21 Thread Jeff Kirsher
From: Joe Perches The line continuation unintentionally adds whitespace so instead use a coalesced format to remove the whitespace. Miscellanea: o Use a more typical style for ternaries and arguments for this logging message Signed-off-by: Joe Perches Tested-by: Aaron Brown Acked-by:

[net-next 5/6] ixgbe: Replace synchronize_sched() with synchronize_rcu()

2018-11-21 Thread Jeff Kirsher
From: "Paul E. McKenney" Now that synchronize_rcu() waits for preempt-disable regions of code as well as RCU read-side critical sections, synchronize_sched() can be replaced by synchronize_rcu(). This commit therefore makes this change. Signed-off-by: "Paul E. McKenney" Tested-by: Andrew

Re: [PATCH bpf-next 1/2] libbpf: Add version script for DSO

2018-11-21 Thread Yonghong Song
On 11/21/18 9:40 AM, Andrey Ignatov wrote: > More and more projects use libbpf and one day it'll likely be packaged > and distributed as DSO and that requires ABI versioning so that both > compatible and incompatible changes to ABI can be introduced in a safe > way in the future without breaking

Re: [PATCH net-next 4/4] netns: enable to dump full nsid translation table

2018-11-21 Thread David Ahern
On 11/21/18 2:01 PM, Nicolas Dichtel wrote: > Le 21/11/2018 à 19:09, David Ahern a écrit : >> On 11/21/18 4:01 AM, Nicolas Dichtel wrote: >>> diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c >>> index 92730905886c..fc568cd0b560 100644 >>> --- a/net/core/net_namespace.c >>> +++

Re: [PATCH bpf-next] bpf: fix a libbpf loader issue

2018-11-21 Thread Daniel Borkmann
On 11/21/2018 08:22 PM, Yonghong Song wrote: > Commit 2993e0515bb4 ("tools/bpf: add support to read .BTF.ext sections") > added support to read .BTF.ext sections from an object file, create > and pass prog_btf_fd and func_info to the kernel. > > The program btf_fd (prog->btf_fd) is initialized to

Re: [PATCH net] sctp: hold transport before accessing its asoc in sctp_hash_transport

2018-11-21 Thread Marcelo Ricardo Leitner
On Wed, Nov 21, 2018 at 08:27:21AM -0500, Neil Horman wrote: > On Tue, Nov 20, 2018 at 10:46:26PM -0200, Marcelo Ricardo Leitner wrote: > > On Tue, Nov 20, 2018 at 07:52:48AM -0500, Neil Horman wrote: > > > On Tue, Nov 20, 2018 at 07:09:16PM +0800, Xin Long wrote: > > > > In sctp_hash_transport,

Re: [PATCH net-next,v3 04/12] cls_api: add translator to flow_action representation

2018-11-21 Thread Marcelo Ricardo Leitner
On Wed, Nov 21, 2018 at 03:51:24AM +0100, Pablo Neira Ayuso wrote: > This patch implements a new function to translate from native TC action > to the new flow_action representation. Moreover, this patch also updates > cls_flower to use this new function. > > Signed-off-by: Pablo Neira Ayuso >

Re: consistency for statistics with XDP mode

2018-11-21 Thread Toke Høiland-Jørgensen
David Ahern writes: > Paweł ran some more XDP tests yesterday and from it found a couple of > issues. One is a panic in the mlx5 driver unloading the bpf program > (mlx5e_xdp_xmit); he will send a send a separate email for that > problem. Same as this one, I guess?

Re: [iproute2-next PATCH v4] tc: flower: Classify packets based port ranges

2018-11-21 Thread David Ahern
On 11/20/18 11:17 PM, Amritha Nambiar wrote: > diff --git a/tc/f_flower.c b/tc/f_flower.c > index 65fca04..722647d 100644 > --- a/tc/f_flower.c > +++ b/tc/f_flower.c > @@ -494,6 +494,68 @@ static int flower_parse_port(char *str, __u8 ip_proto, > return 0; > } > > +static int

Re: [RFC v3 3/3] vxlan: handle underlay VRF changes

2018-11-21 Thread David Ahern
On 11/21/18 7:05 AM, Alexis Bauvin wrote: > Le 20 nov. 2018 à 18:09, David Ahern a écrit : >> On 11/20/18 9:58 AM, Alexis Bauvin wrote: >>> A socket bound to vrf-blue listens on *:4789, thus owning the port. If >>> moving an >>> underlay to the default vrf (ip link set dummy-b nomaster), a new

[PATCH v1 net] lan743x: Enable driver to work with LAN7431

2018-11-21 Thread Bryan Whitehead
This driver was designed to work with both LAN7430 and LAN7431. The only difference between the two is the LAN7431 has support for external phy. This change adds LAN7431 to the list of recognized devices supported by this driver. fixes: driver won't load for LAN7431 Signed-off-by: Bryan

Re: [PATCH V2 mlx5-next 00/12] mlx5 core generic EQ API for RDMA ODP

2018-11-21 Thread Jason Gunthorpe
On Tue, Nov 20, 2018 at 11:11:16AM -0700, Leon Romanovsky wrote: > On Mon, Nov 19, 2018 at 10:52:30AM -0800, Saeed Mahameed wrote: > > Hi, > > > > This patchset is for mlx5-next shared branch, and will be applied there > > once the review is done. > > > > This patchset introduces mostly

Re: [PATCH net-next 1/3] tcp: remove hdrlen argument from tcp_queue_rcv()

2018-11-21 Thread Yuchung Cheng
On Wed, Nov 21, 2018 at 9:52 AM, Eric Dumazet wrote: > Only one caller needs to pull TCP headers, so lets > move __skb_pull() to the caller side. > > Signed-off-by: Eric Dumazet > --- Acked-by: Yuchung Cheng > net/ipv4/tcp_input.c | 13 ++--- > 1 file changed, 6 insertions(+), 7

Re: [PATCH bpf-next] bpf: fix a libbpf loader issue

2018-11-21 Thread Martin Lau
On Wed, Nov 21, 2018 at 11:22:42AM -0800, Yonghong Song wrote: > Commit 2993e0515bb4 ("tools/bpf: add support to read .BTF.ext sections") > added support to read .BTF.ext sections from an object file, create > and pass prog_btf_fd and func_info to the kernel. > > The program btf_fd (prog->btf_fd)

Re: [PATCH net-next 2/4] netns: add support of NETNSA_TARGET_NSID

2018-11-21 Thread Nicolas Dichtel
Le 21/11/2018 à 19:05, David Ahern a écrit : > On 11/21/18 4:01 AM, Nicolas Dichtel wrote: >> static int rtnl_net_dumpid(struct sk_buff *skb, struct netlink_callback *cb) >> { >> -struct net *net = sock_net(skb->sk); >> struct rtnl_net_dump_cb net_cb = { >> -.net = net, >> +

Re: [PATCH v5 bpf-next 0/2] bpf: adding support for mapinmap in libbpf

2018-11-21 Thread Daniel Borkmann
On 11/21/2018 05:55 AM, Nikita V. Shirokov wrote: > in this patch series i'm adding a helper for libbpf which would allow > it to load map-in-map(BPF_MAP_TYPE_ARRAY_OF_MAPS and > BPF_MAP_TYPE_HASH_OF_MAPS). > first patch contains new helper + explains proposed workflow > second patch contains

Re: [PATCH bpf-next] bpf: add read/write access to skb->tstamp from tc clsact progs

2018-11-21 Thread Daniel Borkmann
On 11/21/2018 07:48 PM, Vlad Dumitrescu wrote: > On Wed, Nov 21, 2018 at 5:08 AM Eric Dumazet wrote: >> On 11/20/2018 06:40 PM, Alexei Starovoitov wrote: >>> >>> looks good to me. >>> >>> Any particular reason you decided to disable it for cg_skb ? >>> It seems to me the same EDT approach will

[PATCH bpf] tools: bpftool: fix potential NULL pointer dereference in do_load

2018-11-21 Thread Jakub Kicinski
This patch fixes a possible null pointer dereference in do_load, detected by the semantic patch deref_null.cocci, with the following warning: ./tools/bpf/bpftool/prog.c:1021:23-25: ERROR: map_replace is NULL but dereferenced. The following code has potential null pointer references: 881

Re: [PATCH bpf-next 1/2] libbpf: Add version script for DSO

2018-11-21 Thread Alexei Starovoitov
On 11/21/18 12:18 PM, Yonghong Song wrote: > > > On 11/21/18 9:40 AM, Andrey Ignatov wrote: >> More and more projects use libbpf and one day it'll likely be packaged >> and distributed as DSO and that requires ABI versioning so that both >> compatible and incompatible changes to ABI can be

Re: [PATCH net-next 3/3] tcp: implement head drops in backlog queue

2018-11-21 Thread Yuchung Cheng
On Wed, Nov 21, 2018 at 9:52 AM, Eric Dumazet wrote: > Under high stress, and if GRO or coalescing does not help, > we better make room in backlog queue to be able to keep latest > packet coming. > > This generally helps fast recovery, given that we often receive > packets in order. I like the

Re: [PATCH net-next 2/3] tcp: implement coalescing on backlog queue

2018-11-21 Thread Eric Dumazet
On 11/21/2018 02:31 PM, Yuchung Cheng wrote: > On Wed, Nov 21, 2018 at 9:52 AM, Eric Dumazet wrote: >> + > Really nice! would it make sense to re-use (some of) the similar > tcp_try_coalesce()? > Maybe, but it is a bit complex, since skbs in receive queues (regular or out of order) are

Re: [PATCH bpf-next] bpf: add read/write access to skb->tstamp from tc clsact progs

2018-11-21 Thread Alexei Starovoitov
On Wed, Nov 21, 2018 at 10:48:21AM -0800, Vlad Dumitrescu wrote: > On Wed, Nov 21, 2018 at 5:08 AM Eric Dumazet wrote: > > > > > > > > On 11/20/2018 06:40 PM, Alexei Starovoitov wrote: > > > > > > > > looks good to me. > > > > > > Any particular reason you decided to disable it for cg_skb ? > > >

[PATCH v2 net] lan743x: fix return value for lan743x_tx_napi_poll

2018-11-21 Thread Bryan Whitehead
The lan743x driver, when under heavy traffic load, has been noticed to sometimes hang, or cause a kernel panic. Debugging reveals that the TX napi poll routine was returning the wrong value, 'weight'. Most other drivers return 0. And call napi_complete, instead of napi_complete_done.

[PATCH net-next] net-gro: use ffs() to speedup napi_gro_flush()

2018-11-21 Thread Eric Dumazet
We very often have few flows/chains to look at, and we might increase GRO_HASH_BUCKETS to 32 or 64 in the future. Signed-off-by: Eric Dumazet --- net/core/dev.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index

Re: [PATCH net] net/sched: act_police: add missing spinlock initialization

2018-11-21 Thread Jiri Pirko
Wed, Nov 21, 2018 at 06:23:53PM CET, dcara...@redhat.com wrote: >commit f2cbd4852820 ("net/sched: act_police: fix race condition on state >variables") introduces a new spinlock, but forgets its initialization. >Ensure that tcf_police_init() initializes 'tcfp_lock' every time a 'police' >action is

Re: [PATCH bpf-next v3] libbpf: make sure bpf headers are c++ include-able

2018-11-21 Thread Daniel Borkmann
On 11/21/2018 06:29 PM, Stanislav Fomichev wrote: > Wrap headers in extern "C", to turn off C++ mangling. > This simplifies including libbpf in c++ and linking against it. > > v2 changes: > * do the same for btf.h > > v3 changes: > * test_libbpf.cpp to test for possible future c++ breakages > >

Re: [PATCH bpf-next v3 1/3] bpf, libbpf: introduce bpf_object__probe_caps to test BPF capabilities

2018-11-21 Thread Daniel Borkmann
On 11/21/2018 02:11 AM, Stanislav Fomichev wrote: > It currently only checks whether kernel supports map/prog names. > This capability check will be used in the next two commits to skip setting > prog/map names. > > Suggested-by: Daniel Borkmann > Signed-off-by: Stanislav Fomichev Looks great,

Re: [PATCH net] net: skb_scrub_packet(): Scrub offload_fwd_mark

2018-11-21 Thread David Miller
From: Petr Machata Date: Tue, 20 Nov 2018 11:39:56 + > When a packet is trapped and the corresponding SKB marked as > already-forwarded, it retains this marking even after it is forwarded > across veth links into another bridge. There, since it ingresses the > bridge over veth, which doesn't

[PATCH bpf-next] bpf: fix a libbpf loader issue

2018-11-21 Thread Yonghong Song
Commit 2993e0515bb4 ("tools/bpf: add support to read .BTF.ext sections") added support to read .BTF.ext sections from an object file, create and pass prog_btf_fd and func_info to the kernel. The program btf_fd (prog->btf_fd) is initialized to be -1 to please zclose so we do not need special

Re: [PATCH net-next 3/3] tcp: implement head drops in backlog queue

2018-11-21 Thread Eric Dumazet
On 11/21/2018 02:40 PM, Yuchung Cheng wrote: > On Wed, Nov 21, 2018 at 9:52 AM, Eric Dumazet wrote: >> Under high stress, and if GRO or coalescing does not help, >> we better make room in backlog queue to be able to keep latest >> packet coming. >> >> This generally helps fast recovery, given

Re: [PATCH net-next 0/4] VLAN tag handling cleanup

2018-11-21 Thread David Miller
From: Michał Mirosław Date: Tue, 20 Nov 2018 13:20:30 +0100 > This is a cleanup set after VLAN_TAG_PRESENT removal. The CFI bit > handling is made similar to how other tag fields are used. Series applied, thanks.

[PATCH net-next 4/4] netns: enable to dump full nsid translation table

2018-11-21 Thread Nicolas Dichtel
Like the previous patch, the goal is to ease to convert nsids from one netns to another netns. A new attribute (NETNSA_CURRENT_NSID) is added to the kernel answer when NETNSA_TARGET_NSID is provided, thus the user can easily convert nsids. Signed-off-by: Nicolas Dichtel ---

[PATCH net-next 1/4] netns: remove net arg from rtnl_net_fill()

2018-11-21 Thread Nicolas Dichtel
This argument is not used anymore. Fixes: cab3c8ec8d57 ("netns: always provide the id to rtnl_net_fill()") Signed-off-by: Nicolas Dichtel --- net/core/net_namespace.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c

[PATCH net-next 0/4] Ease to interpret net-nsid

2018-11-21 Thread Nicolas Dichtel
The goal of this series is to ease the interpretation of nsid received in netlink messages from other netns (when the user uses NETLINK_F_LISTEN_ALL_NSID). After this series, with a patched iproute2: $ ip netns add foo $ ip netns add bar $ touch /var/run/netns/init_net $ mount --bind

[PATCH net-next 2/4] netns: add support of NETNSA_TARGET_NSID

2018-11-21 Thread Nicolas Dichtel
Like it was done for link and address, add the ability to perform get/dump in another netns by specifying a target nsid attribute. Signed-off-by: Nicolas Dichtel --- include/uapi/linux/net_namespace.h | 1 + net/core/net_namespace.c | 97 -- 2 files

[PATCH net-next 3/4] netns: enable to specify a nsid for a get request

2018-11-21 Thread Nicolas Dichtel
Combined with NETNSA_TARGET_NSID, it enables to "translate" a nsid from one netns to a nsid of another netns. This is useful when using NETLINK_F_LISTEN_ALL_NSID because it helps the user to interpret a nsid received from an other netns. Signed-off-by: Nicolas Dichtel ---

Re: [RFC PATCH] net: don't keep lonely packets forever in the gro hash

2018-11-21 Thread Daniel Borkmann
Hi Eric, On 11/20/2018 02:49 PM, Eric Dumazet wrote: > On 11/20/2018 02:17 AM, Paolo Abeni wrote: >> Eric noted that with UDP GRO and napi timeout, we could keep a single >> UDP packet inside the GRO hash forever, if the related NAPI instance >> calls napi_gro_complete() at an higher frequency

selftests/bpf :get_cgroup_id_user: File not found: /sys/kernel/debug/tracing/events/syscalls/sys_enter_nanosleep/id

2018-11-21 Thread Naresh Kamboju
Kselftest bpf get_cgroup_id_user is failed on all devices. selftests: bpf: get_cgroup_id_user main:PASS:setup_cgroup_environment main:PASS:create_and_get_cgroup main:PASS:join_cgroup main:PASS:bpf_prog_load main:PASS:bpf_find_map main:PASS:bpf_find_map main:FAIL:open err -1 errno 2 not ok 1..15

Re: [RFC PATCH 0/6] Armada 38x comphy driver to support 2.5Gbps networking

2018-11-21 Thread Kishon Vijay Abraham I
Hi Russell, On 14/11/18 4:26 PM, Russell King - ARM Linux wrote: > On Wed, Nov 14, 2018 at 01:39:29PM +0530, Kishon Vijay Abraham I wrote: >> Hi, >> >> On 12/11/18 5:59 PM, Russell King - ARM Linux wrote: >>> Hi, >>> >>> This series adds support for dynamically switching between 1Gbps >>> and

Re: [RFC PATCH 5/6] net: marvell: neta: add support for 2500base-X

2018-11-21 Thread Kishon Vijay Abraham I
Hi, On 14/11/18 4:41 PM, Russell King - ARM Linux wrote: > On Wed, Nov 14, 2018 at 02:18:14PM +0530, Kishon Vijay Abraham I wrote: >> Hi, >> >> On 12/11/18 6:01 PM, Russell King wrote: >>> Signed-off-by: Russell King >>> --- >>> drivers/net/ethernet/marvell/mvneta.c | 58 >>>

selftests/bpf: libbpf: object file doesn't contain bpf program

2018-11-21 Thread Naresh Kamboju
The listed bpf test case failed due to, libbpf: object file doesn't contain bpf program We are cross compiling, installing selftests on device under test and running tests by using run_kselftest.sh script file. selftests: bpf: test_maps libbpf: object file doesn't contain bpf program Failed to

Re: [PATCH v1 2/2] netfilter: Add PSID mode to MASQUERADE

2018-11-21 Thread kbuild test robot
Hi Blair, Thank you for the patch! Yet something to improve: [auto build test ERROR on nf-next/master] [also build test ERROR on v4.20-rc3 next-20181121] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux

Re: [PATCHv2 1/2] can: xilinx: add can 2.0 support

2018-11-21 Thread Shubhrajyoti Datta
On Fri, Oct 12, 2018 at 10:06 AM wrote: > > From: Shubhrajyoti Datta > > Add support for can 2.0. > > Signed-off-by: Shubhrajyoti Datta > --- ping

Re: [PATCH net v2] net/sched: act_police: fix race condition on state variables

2018-11-21 Thread Davide Caratti
On Tue, 2018-11-20 at 20:13 -0800, Cong Wang wrote: > On Tue, Nov 20, 2018 at 3:30 PM Eric Dumazet wrote: > > On Tue, Nov 20, 2018 at 3:28 PM David Miller wrote: > > > Applied. > > > > We need a fix to make lockdep happy, as reported by Cong. > > > > Cong, do you want to handle this ? > > >

Re: [iproute2-next PATCH v3 2/2] man: tc-flower: Add explanation for range option

2018-11-21 Thread Jiri Pirko
Wed, Nov 21, 2018 at 05:59:45AM CET, dsah...@gmail.com wrote: >On 11/20/18 9:59 PM, Nambiar, Amritha wrote: >> Oops, submitted the v2 patch for man changes too soon, without seeing >> this. So, in this case, should I re-submit the iproute2-flower patch >> that was accepted removing the 'range'

[PATCH iproute2] ipnetns: parse nsid as a signed integer

2018-11-21 Thread Nicolas Dichtel
Don't confuse the user, nsid is a signed integer, this kind of command should return an error: 'ip netns set foo 0x'. Also, a valid value is a positive value. To let the kernel chooses a value, the keyword 'auto' must be used. Signed-off-by: Nicolas Dichtel --- ip/ipnetns.c | 8

Re: [PATCH bpf-next] bpf: add read/write access to skb->tstamp from tc clsact progs

2018-11-21 Thread Eric Dumazet
On 11/20/2018 06:40 PM, Alexei Starovoitov wrote: > > looks good to me. > > Any particular reason you decided to disable it for cg_skb ? > It seems to me the same EDT approach will work from > cgroup-bpf skb hooks just as well and then we can have neat > way of controlling traffic

[PATCH net] net/ipv6: re-do dad when interface has IFF_NOARP flag change

2018-11-21 Thread Hangbin Liu
When we add a new IPv6 address, we should also join corresponding solicited-node multicast address, unless the interface has IFF_NOARP flag, as function addrconf_join_solict() did. But if we remove IFF_NOARP flag later, we do not do dad and add the mcast address. So we will drop corresponding

[PATCH net-next] {net,IB}/mlx4: Initialize CQ buffers in the driver when possible

2018-11-21 Thread Tariq Toukan
From: Daniel Jurgens Perform CQ initialization in the driver when the capability is supported by the FW. When passing the CQ to HW indicate that the CQ buffer has been pre-initialized. Doing so decreases CQ creation time. Testing on P8 showed a single 2048 entry CQ creation time was reduced

[PATCH] samples: bpf: fix: error handling regarding kprobe_events

2018-11-21 Thread Daniel T. Lee
Currently, kprobe_events failure won't be handled properly. Due to calling system() indirectly to write to kprobe_events, it can't be identified whether an error is derived from kprobe or system. // buf = "echo '%c:%s %s' >> /s/k/d/t/kprobe_events" err = system(buf); if (err < 0) {

[PATCH net] net: thunderx: set xdp_prog to NULL if bpf_prog_add fails

2018-11-21 Thread Lorenzo Bianconi
Set xdp_prog pointer to NULL if bpf_prog_add fails since that routine reports the error code instead of NULL in case of failure and xdp_prog pointer value is used in the driver to verify if XDP is currently enabled. Moreover report the error code to userspace if nicvf_xdp_setup fails Fixes:

Hello My Beloved One

2018-11-21 Thread Aisha Gaddafi
Assalamu alaikum, I came across your e-mail contact prior a private search while in need of a trusted person. My name is Mrs. Aisha Gaddafi, a single Mother and a Widow with three Children. I am the only biological Daughter of late Libyan President (Late Colonel Muammar Gaddafi)I have a

Re: [RFC v3 0/3] Add VRF support for VXLAN underlay

2018-11-21 Thread Alexis Bauvin
Le 20 nov. 2018 à 22:45, David Ahern a écrit : > > On 11/20/18 7:23 AM, Alexis Bauvin wrote: >> We are trying to isolate the VXLAN traffic from different VMs with VRF as >> shown >> in the schemas below: >> >> +-+ ++ >> | +--+

[PATCH net-next] selftests: explicitly require kernel features needed by udpgro tests

2018-11-21 Thread Paolo Abeni
commit 3327a9c46352f1 ("selftests: add functionals test for UDP GRO") make use of ipv6 NAT, but such a feature is not currently implied by selftests. Since the 'ip[6]tables' commands may actually create nft rules, depending on the specific user-space version, let's pull both NF and NFT nat modules

Re: [PATCH bpf-next v2] filter: add BPF_ADJ_ROOM_DATA mode to bpf_skb_adjust_room()

2018-11-21 Thread Nicolas Dichtel
Le 20/11/2018 à 02:06, Daniel Borkmann a écrit : > On 11/13/2018 05:35 PM, Nicolas Dichtel wrote: >> This new mode enables to add or remove an l2 header in a programmatic way >> with cls_bpf. >> For example, it enables to play with mpls headers. >> >> Signed-off-by: Nicolas Dichtel >> Acked-by:

Re: hw csum failure + conntrack with more debugging information

2018-11-21 Thread Andre Tomt
On 18.11.2018 02:12, Eric Dumazet wrote: Please try this patch, we suspect mlx4 support for CHECKSUM_COMPLETE is wrong. (Only IPv4 handled, but I suspect a similar fix is needed for IPv6) Not conclusive, but.. Have not seen any splats or other weirdness since applying this patch 3 days ago.

Re: [Patch net-next 2/2] net: dump whole skb data in netdev_rx_csum_fault()

2018-11-21 Thread Eric Dumazet
On 11/20/2018 06:13 PM, Cong Wang wrote: > Currently, we only dump a few selected skb fields in > netdev_rx_csum_fault(). It is not suffient for debugging checksum > fault. This patch introduces skb_dump() which dumps skb mac header, > network header and its whole skb->data too. > > Cc:

[PATCH net] net/dim: Update DIM start sample after each DIM iteration

2018-11-21 Thread Tal Gilboa
On every iteration of net_dim, the algorithm may choose to check for the system state by comparing current data sample with previous data sample. After each of these comparison, regardless of the action taken, the sample used as baseline is needed to be updated. This patch fixes a bug that causes

Re: [RFC v3 3/3] vxlan: handle underlay VRF changes

2018-11-21 Thread Alexis Bauvin
Le 20 nov. 2018 à 18:09, David Ahern a écrit : > On 11/20/18 9:58 AM, Alexis Bauvin wrote: >> A socket bound to vrf-blue listens on *:4789, thus owning the port. If >> moving an >> underlay to the default vrf (ip link set dummy-b nomaster), a new socket >> will be >> created, unbound to any

Re: [PATCH net] sctp: hold transport before accessing its asoc in sctp_hash_transport

2018-11-21 Thread Neil Horman
On Tue, Nov 20, 2018 at 10:46:26PM -0200, Marcelo Ricardo Leitner wrote: > On Tue, Nov 20, 2018 at 07:52:48AM -0500, Neil Horman wrote: > > On Tue, Nov 20, 2018 at 07:09:16PM +0800, Xin Long wrote: > > > In sctp_hash_transport, it dereferences a transport's asoc only under > > > rcu_read_lock.

Re: [PATCH iproute2-next v3] rdma: Document IB device renaming option

2018-11-21 Thread Dennis Dalessandro
On 11/18/2018 6:05 AM, Leon Romanovsky wrote: On Fri, Nov 16, 2018 at 08:10:35PM +, Ruhl, Michael J wrote: -Original Message- From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- ow...@vger.kernel.org] On Behalf Of Leon Romanovsky Sent: Sunday, November 4, 2018 2:11 PM To:

[PATCH net v2] net: don't keep lonely packets forever in the gro hash

2018-11-21 Thread Paolo Abeni
Eric noted that with UDP GRO and NAPI timeout, we could keep a single UDP packet inside the GRO hash forever, if the related NAPI instance calls napi_gro_complete() at an higher frequency than the NAPI timeout. Willem noted that even TCP packets could be trapped there, till the next

[PATCH net] net/sched: act_police: add missing spinlock initialization

2018-11-21 Thread Davide Caratti
commit f2cbd4852820 ("net/sched: act_police: fix race condition on state variables") introduces a new spinlock, but forgets its initialization. Ensure that tcf_police_init() initializes 'tcfp_lock' every time a 'police' action is newly created, to avoid the following lockdep splat: INFO: trying

Re: [PATCH bpf-next] bpf: libbpf: retry program creation without the name

2018-11-21 Thread Stanislav Fomichev
On 11/21, Quentin Monnet wrote: > 2018-11-20 15:26 UTC-0800 ~ Stanislav Fomichev > > On 11/20, Alexei Starovoitov wrote: > >> On Wed, Nov 21, 2018 at 12:18:57AM +0100, Daniel Borkmann wrote: > >>> On 11/21/2018 12:04 AM, Alexei Starovoitov wrote: > On Tue, Nov 20, 2018 at 01:19:05PM -0800,

[PATCH bpf-next v3] libbpf: make sure bpf headers are c++ include-able

2018-11-21 Thread Stanislav Fomichev
Wrap headers in extern "C", to turn off C++ mangling. This simplifies including libbpf in c++ and linking against it. v2 changes: * do the same for btf.h v3 changes: * test_libbpf.cpp to test for possible future c++ breakages Signed-off-by: Stanislav Fomichev --- tools/lib/bpf/Makefile

Re: [PATCH net v2] net: don't keep lonely packets forever in the gro hash

2018-11-21 Thread Willem de Bruijn
On Wed, Nov 21, 2018 at 12:21 PM Paolo Abeni wrote: > > Eric noted that with UDP GRO and NAPI timeout, we could keep a single > UDP packet inside the GRO hash forever, if the related NAPI instance > calls napi_gro_complete() at an higher frequency than the NAPI timeout. > Willem noted that even

Re: [PATCH iproute2] ipnetns: parse nsid as a signed integer

2018-11-21 Thread Stephen Hemminger
On Wed, 21 Nov 2018 10:44:27 +0100 Nicolas Dichtel wrote: > Don't confuse the user, nsid is a signed integer, this kind of command > should return an error: 'ip netns set foo 0x'. > > Also, a valid value is a positive value. To let the kernel chooses a value, > the keyword 'auto' must

Re: [PATCH iproute2] bpf: initialise map symbol before retrieving and comparing its type

2018-11-21 Thread Stephen Hemminger
On Tue, 20 Nov 2018 01:26:27 + Quentin Monnet wrote: > In order to compare BPF map symbol type correctly in regard to the > latest LLVM, commit 7a04dd84a7f9 ("bpf: check map symbol type properly > with newer llvm compiler") compares map symbol type to both NOTYPE and > OBJECT. To do so, it

[PATCH bpf-next 2/2] libbpf: Verify versioned symbols

2018-11-21 Thread Andrey Ignatov
Since ABI versioning info is kept separately from the code it's easy to forget to update it while adding a new API. Add simple verification that all global symbols exported with LIBBPF_API are versioned in libbpf.map version script. The idea is to check that number of global symbols in

[PATCH bpf-next 0/2] libbpf: ABI versioning

2018-11-21 Thread Andrey Ignatov
This patch set adds ABI versioning to libbpf. Patch 1 adds version script and has more details on why it's needed. Patch 2 adds simple check that all global symbols are versioned. Andrey Ignatov (2): libbpf: Add version script for DSO libbpf: Verify versioned symbols

[PATCH bpf-next 1/2] libbpf: Add version script for DSO

2018-11-21 Thread Andrey Ignatov
More and more projects use libbpf and one day it'll likely be packaged and distributed as DSO and that requires ABI versioning so that both compatible and incompatible changes to ABI can be introduced in a safe way in the future without breaking executables dynamically linked with a previous

Re: [PATCH net v2] net: don't keep lonely packets forever in the gro hash

2018-11-21 Thread Eric Dumazet
On 11/21/2018 09:21 AM, Paolo Abeni wrote: > Eric noted that with UDP GRO and NAPI timeout, we could keep a single > UDP packet inside the GRO hash forever, if the related NAPI instance > calls napi_gro_complete() at an higher frequency than the NAPI timeout. Acked-by: Eric Dumazet

[PATCH net-next 2/3] tcp: implement coalescing on backlog queue

2018-11-21 Thread Eric Dumazet
In case GRO is not as efficient as it should be or disabled, we might have a user thread trapped in __release_sock() while softirq handler flood packets up to the point we have to drop. This patch balances work done from user thread and softirq, to give more chances to __release_sock() to

Re: [PATCH net] sctp: hold transport before accessing its asoc in sctp_hash_transport

2018-11-21 Thread Marcelo Ricardo Leitner
On Wed, Nov 21, 2018 at 03:47:33PM +0900, Xin Long wrote: > On Wed, Nov 21, 2018 at 9:46 AM Marcelo Ricardo Leitner > wrote: > > > > On Tue, Nov 20, 2018 at 07:52:48AM -0500, Neil Horman wrote: > > > On Tue, Nov 20, 2018 at 07:09:16PM +0800, Xin Long wrote: > > > > In sctp_hash_transport, it

[PATCH net-next 0/3] tcp: take a bit more care of backlog stress

2018-11-21 Thread Eric Dumazet
While working on the SACK compression issue Jean-Louis Dupond reported, we found that his linux box was suffering very hard from tail drops on the socket backlog queue, because the opposite TCP stack was ont implementing latest RFC recommendations. First patch is a cleanup Second patch is

[PATCH net-next 1/3] tcp: remove hdrlen argument from tcp_queue_rcv()

2018-11-21 Thread Eric Dumazet
Only one caller needs to pull TCP headers, so lets move __skb_pull() to the caller side. Signed-off-by: Eric Dumazet --- net/ipv4/tcp_input.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index

[PATCH net-next 3/3] tcp: implement head drops in backlog queue

2018-11-21 Thread Eric Dumazet
Under high stress, and if GRO or coalescing does not help, we better make room in backlog queue to be able to keep latest packet coming. This generally helps fast recovery, given that we often receive packets in order. Signed-off-by: Eric Dumazet Tested-by: Jean-Louis Dupond Cc: Neal Cardwell

Re: selftests/bpf :get_cgroup_id_user: File not found: /sys/kernel/debug/tracing/events/syscalls/sys_enter_nanosleep/id

2018-11-21 Thread Y Song
On Wed, Nov 21, 2018 at 3:44 AM Naresh Kamboju wrote: > > Kselftest bpf get_cgroup_id_user is failed on all devices. > > selftests: bpf: get_cgroup_id_user > main:PASS:setup_cgroup_environment > main:PASS:create_and_get_cgroup > main:PASS:join_cgroup > main:PASS:bpf_prog_load >

Re: [PATCH net-next,v3 04/12] cls_api: add translator to flow_action representation

2018-11-21 Thread Pablo Neira Ayuso
On Wed, Nov 21, 2018 at 07:15:41PM -0200, Marcelo Ricardo Leitner wrote: > On Wed, Nov 21, 2018 at 03:51:24AM +0100, Pablo Neira Ayuso wrote: [...] > > diff --git a/net/sched/cls_flower.c b/net/sched/cls_flower.c > > index d2971fbfc3d9..8898943b8ee6 100644 > > --- a/net/sched/cls_flower.c > > +++

Re: [PATCH net] tcp: defer SACK compression after DupThresh

2018-11-21 Thread David Miller
From: Eric Dumazet Date: Tue, 20 Nov 2018 05:53:59 -0800 > Jean-Louis reported a TCP regression and bisected to recent SACK > compression. > > After a loss episode (receiver not able to keep up and dropping > packets because its backlog is full), linux TCP stack is sending > a single SACK

[PATCH v3 5/5] Bluetooth: btusb: Use the hw_reset method to allow resetting the BT chip

2018-11-21 Thread Rajat Jain
If the platform provides it, use the reset gpio to reset the BT chip (requested by the HCI core if needed). This has been found helpful on some of Intel bluetooth controllers where the firmware gets stuck and the only way out is a hard reset pin provided by the platform. Signed-off-by: Rajat Jain

Re: [PATCH net-next 3/3] tcp: implement head drops in backlog queue

2018-11-21 Thread Eric Dumazet
On Wed, Nov 21, 2018 at 3:52 PM Eric Dumazet wrote: > This is basically what the patch does, the while loop breaks when we have > freed > just enough skbs. Also this is the patch we tested with Jean-Louis on his host, bring very nice results, even from an old stack sender (the one that had

Re: net: thunderx: nicvf_xdp_setup error code path

2018-11-21 Thread David Miller
From: Lorenzo Bianconi Date: Tue, 20 Nov 2018 18:56:36 +0100 > Hi all, > > looking at thunderx XDP support I noticed that nic->xdp_prog pointer in > nicvf_xdp_setup is not actually set to NULL if bpf_prog_add fails but it > is initialized with bpf_prog_add error code. xdp_prog pointer value is

Re: [Patch net] net: invert the check of detecting hardware RX checksum fault

2018-11-21 Thread Paweł Staszewski
W dniu 16.11.2018 o 21:06, Cong Wang pisze: On Thu, Nov 15, 2018 at 8:50 PM Herbert Xu wrote: On Thu, Nov 15, 2018 at 06:23:38PM -0800, Cong Wang wrote: Normally if the hardware's partial checksum is valid then we just trust it and send the packet along. However, if the partial checksum is

[PATCH bpf-next v2 1/1] bpf, lpm: make longest_prefix_match() faster

2018-11-21 Thread Eric Dumazet
At LPC 2018 in Vancouver, Vlad Dumitrescu mentioned that longest_prefix_match() has a high cost [1]. One reason for that cost is a loop handling one byte at a time. We can handle more bytes at a time, if enough attention is paid to endianness. I was able to remove ~55 % of

Re: [PATCH net-next,v3 09/12] flow_dissector: add basic ethtool_rx_flow_spec to flow_rule structure translator

2018-11-21 Thread Jiri Pirko
Thu, Nov 22, 2018 at 05:57:31AM CET, f.faine...@gmail.com wrote: > > >On 11/20/2018 6:51 PM, Pablo Neira Ayuso wrote: >> This patch adds a function to translate the ethtool_rx_flow_spec >> structure to the flow_rule representation. >> >> This allows us to reuse code from the driver side given

Re: [BUG] xfrm: unable to handle kernel NULL pointer dereference

2018-11-21 Thread Steffen Klassert
On Fri, Nov 16, 2018 at 08:12:46PM +0100, Steffen Klassert wrote: > On Fri, Nov 16, 2018 at 08:48:00PM +0200, Lennert Buytenhek wrote: > > On Sat, Nov 10, 2018 at 08:34:34PM +0100, Jean-Philippe Menil wrote: > > > > > we're seeing unexpected crashes from kernel 4.15 to 4.18.17, using > > > IPsec

  1   2   >