Re: [PATCH net-next 0/7] tcp: second round for EDT conversion

2018-10-15 Thread David Miller
From: Eric Dumazet Date: Mon, 15 Oct 2018 09:37:51 -0700 > First round of EDT patches left TCP stack in a non optimal state. > > - High speed flows suffered from loss of performance, addressed > by the first patch of this series. > > - Second patch brings pacing to the current state of

Re: [PATCH net] sctp: use the pmtu from the icmp packet to update transport pathmtu

2018-10-15 Thread David Miller
From: Xin Long Date: Mon, 15 Oct 2018 19:58:29 +0800 > Other than asoc pmtu sync from all transports, sctp_assoc_sync_pmtu > is also processing transport pmtu_pending by icmp packets. But it's > meaningless to use sctp_dst_mtu(t->dst) as new pmtu for a transport. > > The right pmtu value should

Re: [PATCH net,stable 1/1] net: fec: don't dump RX FIFO register when not available

2018-10-15 Thread David Miller
From: Andy Duan Date: Mon, 15 Oct 2018 05:19:00 + > From: Fugang Duan > > Commit db65f35f50e0 ("net: fec: add support of ethtool get_regs") introduce > ethool "--register-dump" interface to dump all FEC registers. > > But not all silicon implementations of the Freescale FEC hardware

[PATCH bpf-next] libbpf: Per-symbol visibility for DSO

2018-10-15 Thread Andrey Ignatov
Make global symbols in libbpf DSO hidden by default with -fvisibility=hidden and export symbols that are part of ABI explicitly with __attribute__((visibility("default"))). This is common practice that should prevent from accidentally exporting a symbol, that is not supposed to be a part of ABI

Re: [PATCH -next] fore200e: fix missing unlock on error in bsq_audit()

2018-10-15 Thread David Miller
From: Wei Yongjun Date: Mon, 15 Oct 2018 03:07:16 + > Add the missing unlock before return from function bsq_audit() > in the error handling case. > > Fixes: 1d9d8be91788 ("fore200e: check for dma mapping failures") > Signed-off-by: Wei Yongjun Applied.

Re: [PATCH net-next 00/23] bnxt_en: Add support for new 57500 chips.

2018-10-15 Thread David Miller
From: Michael Chan Date: Sun, 14 Oct 2018 07:02:36 -0400 > This patch-set is larger than normal because I wanted a complete series > to add basic support for the new 57500 chips. The new chips have the > following main differences compared to legacy chips: > > 1. Requires the PF driver to

Re: [PATCH net] ipv6: mcast: fix a use-after-free in inet6_mc_check

2018-10-15 Thread David Miller
From: Eric Dumazet Date: Fri, 12 Oct 2018 18:58:53 -0700 > syzbot found a use-after-free in inet6_mc_check [1] > > The problem here is that inet6_mc_check() uses rcu > and read_lock(>sflock) > > So the fact that ip6_mc_leave_src() is called under RTNL > and the socket lock does not help us, we

Re: [PATCH net-next 0/2] selftests: pmtu: Add test choice and captures

2018-10-15 Thread David Miller
From: Stefano Brivio Date: Fri, 12 Oct 2018 23:54:12 +0200 > This series adds a couple of features useful for debugging: 1/2 > allows selecting single tests and 2/2 adds optional traffic > captures. > > Semantics for current invocation of test script are preserved. M0AR SELF TESTS! I love it.

Re: [PATCH net-next] r8169: simplify rtl8169_set_magic_reg

2018-10-15 Thread David Miller
From: Heiner Kallweit Date: Fri, 12 Oct 2018 23:23:57 +0200 > Simplify this function, no functional change intended. > > Signed-off-by: Heiner Kallweit Applied.

Re: [PATCH net-next] r8169: remove unneeded call to netif_stop_queue in rtl8169_net_suspend

2018-10-15 Thread David Miller
From: Heiner Kallweit Date: Fri, 12 Oct 2018 23:30:52 +0200 > netif_device_detach() stops all tx queues already, so we don't need > this call. > > Signed-off-by: Heiner Kallweit Applied.

Re: [PATCH net-next] nfp: devlink port split support for 1x100G CXP NIC

2018-10-15 Thread David Miller
From: Jakub Kicinski Date: Fri, 12 Oct 2018 11:09:01 -0700 > From: Ryan C Goodfellow > > This commit makes it possible to use devlink to split the 100G CXP > Netronome into two 40G interfaces. Currently when you ask for 2 > interfaces, the math in src/nfp_devlink.c:nfp_devlink_port_split >

Re: [PATCH net-next 0/6] dpaa2-eth: code cleanup

2018-10-15 Thread David Miller
From: Ioana Ciornei Date: Fri, 12 Oct 2018 16:27:16 + > There are no functional changes in this patch set, only some cleanup > changes such as: unused parameters, uninitialized variables and > unnecessary Kconfig dependencies. Series applied.

Re: [PATCH net] ipv6: rate-limit probes for neighbourless routes

2018-10-15 Thread David Miller
From: Sabrina Dubroca Date: Fri, 12 Oct 2018 16:22:47 +0200 > When commit 270972554c91 ("[IPV6]: ROUTE: Add Router Reachability > Probing (RFC4191).") introduced router probing, the rt6_probe() function > required that a neighbour entry existed. This neighbour entry is used to > record the

Re: [PATCH net-next 0/2] net: phy: improve and simplify state machine

2018-10-15 Thread David Miller
From: Heiner Kallweit Date: Thu, 11 Oct 2018 22:35:35 +0200 > Improve / simplify handling of states PHY_RUNNING and PHY_RESUMING in > phylib state machine. Series applied.

Re: [PATCH net-next v2] vxlan: support NTF_USE refresh of fdb entries

2018-10-15 Thread David Miller
From: Roopa Prabhu Date: Thu, 11 Oct 2018 12:35:13 -0700 > From: Roopa Prabhu > > This makes use of NTF_USE in vxlan driver consistent > with bridge driver. > > Signed-off-by: Roopa Prabhu Applied.

Re: [Patch net] llc: set SOCK_RCU_FREE in llc_sap_add_socket()

2018-10-15 Thread David Miller
From: Cong Wang Date: Thu, 11 Oct 2018 11:15:13 -0700 > WHen an llc sock is added into the sk_laddr_hash of an llc_sap, > it is not marked with SOCK_RCU_FREE. > > This causes that the sock could be freed while it is still being > read by __llc_lookup_established() with RCU read lock. sock is >

Re: [PATCH net-next v7] net/ncsi: Extend NC-SI Netlink interface to allow user space to send NC-SI command

2018-10-15 Thread David Miller
From: Date: Thu, 11 Oct 2018 18:07:37 + > The new command (NCSI_CMD_SEND_CMD) is added to allow user space application > to send NC-SI command to the network card. > Also, add a new attribute (NCSI_ATTR_DATA) for transferring request and > response. > > The work flow is as below. > >

Re: [PATCH net-next] net: phy: trigger state machine immediately in phy_start_machine

2018-10-15 Thread David Miller
From: Heiner Kallweit Date: Thu, 11 Oct 2018 19:31:47 +0200 > When starting the state machine there may be work to be done > immediately, e.g. if the initial state is PHY_UP then the state > machine may trigger an autonegotiation. Having said that I see no need > to wait a second until the state

Re: [PATCH net-next 0/3] veth: XDP stats improvement

2018-10-15 Thread David Miller
From: Toshiaki Makita Date: Thu, 11 Oct 2018 18:36:47 +0900 > ndo_xdp_xmit in veth did not update packet counters as described in [1]. > Also, current implementation only updates counters on tx side so rx side > events like XDP_DROP were not collected. > This series implements the missing

Re: [pull request][net 0/3] Mellanox, mlx5 fixes 2018-10-10

2018-10-15 Thread David Miller
From: Saeed Mahameed Date: Wed, 10 Oct 2018 18:32:41 -0700 > This pull request includes some fixes to mlx5 driver, > Please pull and let me know if there's any problem. Pulled. > For -stable v4.11: > ('net/mlx5: Take only bit 24-26 of wqe.pftype_wq for page fault type') > For -stable v4.17: >

Re: [pull request][net-next 0/7] Mellanox, mlx5e and IPoIB netlink support fixes

2018-10-15 Thread David Miller
From: Saeed Mahameed Date: Wed, 10 Oct 2018 18:24:37 -0700 > This series was meant to go to -rc but due to this late submission and the > size/complexity of this patchset, I am submitting to net-next. > > This series came to fix a very serious regression in RDMA > IPoIB netlink child creation

Re: [PATCH net v3] net/sched: cls_api: add missing validation of netlink attributes

2018-10-15 Thread David Miller
From: Davide Caratti Date: Wed, 10 Oct 2018 22:00:58 +0200 > Similarly to what has been done in 8b4c3cdd9dd8 ("net: sched: Add policy > validation for tc attributes"), fix classifier code to add validation of > TCA_CHAIN and TCA_KIND netlink attributes. > > tested with: > # ./tdc.py -c filter

Re: [PATCH net-next v2 0/2] FDDI: DEC FDDIcontroller 700 TURBOchannel adapter support

2018-10-15 Thread David Miller
From: "Maciej W. Rozycki" Date: Tue, 9 Oct 2018 23:57:36 +0100 (BST) > Questions, comments? Otherwise, please apply. Series applied, thank you.

Re: [PATCH net-next] tun: Consistently configure generic netdev params via rtnetlink

2018-10-15 Thread David Miller
From: Serhey Popovych Date: Tue, 9 Oct 2018 21:21:01 +0300 > Configuring generic network device parameters on tun will fail in > presence of IFLA_INFO_KIND attribute in IFLA_LINKINFO nested attribute > since tun_validate() always return failure. > > This can be visualized with following

crash in xt_policy due to skb_dst_drop() in nf_ct_frag6_gather()

2018-10-15 Thread Maciej Żenczykowski
I believe that: commit ad8b1ffc3efae2f65080bdb11145c87d299b8f9a Author: Florian Westphal netfilter: ipv6: nf_defrag: drop skb dst before queueing +++ b/net/ipv6/netfilter/nf_conntrack_reasm.c @@ -618,6 +618,8 @@ int nf_ct_frag6_gather(struct net *net, struct sk_buff *skb, u32 user)

[PATCH v2 net-next 06/11] ipmr: Refactor mr_rtm_dumproute

2018-10-15 Thread David Ahern
From: David Ahern Move per-table loops from mr_rtm_dumproute to mr_table_dump and export mr_table_dump for dumps by specific table id. Signed-off-by: David Ahern --- include/linux/mroute_base.h | 6 net/ipv4/ipmr_base.c| 88 - 2 files

[PATCH v2 net-next 01/11] netlink: Add answer_flags to netlink_callback

2018-10-15 Thread David Ahern
From: David Ahern With dump filtering we need a way to ensure the NLM_F_DUMP_FILTERED flag is set on a message back to the user if the data returned is influenced by some input attributes. Normally this can be done as messages are added to the skb, but if the filter results in no data being

[PATCH v2 net-next 05/11] net/mpls: Plumb support for filtering route dumps

2018-10-15 Thread David Ahern
From: David Ahern Implement kernel side filtering of routes by egress device index and protocol. MPLS uses only a single table and route type. Signed-off-by: David Ahern --- net/mpls/af_mpls.c | 42 +- 1 file changed, 41 insertions(+), 1 deletion(-)

[PATCH v2 net-next 11/11] net/ipv4: Bail early if user only wants prefix entries

2018-10-15 Thread David Ahern
From: David Ahern Unlike IPv6, IPv4 does not have routes marked with RTF_PREFIX_RT. If the flag is set in the dump request, just return. In the process of this change, move the CLONE check to use the new filter flags. Signed-off-by: David Ahern --- net/ipv4/fib_frontend.c | 8 ++-- 1

[PATCH v2 net-next 07/11] net: Plumb support for filtering ipv4 and ipv6 multicast route dumps

2018-10-15 Thread David Ahern
From: David Ahern Implement kernel side filtering of routes by egress device index and table id. If the table id is given in the filter, lookup table and call mr_table_dump directly for it. Signed-off-by: David Ahern --- include/linux/mroute_base.h | 7 --- net/ipv4/ipmr.c |

[PATCH v2 net-next 10/11] net/ipv6: Bail early if user only wants cloned entries

2018-10-15 Thread David Ahern
From: David Ahern Similar to IPv4, IPv6 fib no longer contains cloned routes. If a user requests a route dump for only cloned entries, no sense walking the FIB and returning everything. Signed-off-by: David Ahern --- net/ipv6/ip6_fib.c | 7 +-- 1 file changed, 5 insertions(+), 2

[PATCH v2 net-next 09/11] net/mpls: Handle kernel side filtering of route dumps

2018-10-15 Thread David Ahern
From: David Ahern Update the dump request parsing in MPLS for the non-INET case to enable kernel side filtering. If INET is disabled the only filters that make sense for MPLS are protocol and nexthop device. Signed-off-by: David Ahern --- net/mpls/af_mpls.c | 33

[PATCH v2 net-next 04/11] net/ipv6: Plumb support for filtering route dumps

2018-10-15 Thread David Ahern
From: David Ahern Implement kernel side filtering of routes by table id, egress device index, protocol, and route type. If the table id is given in the filter, lookup the table and call fib6_dump_table directly for it. Move the existing route flags check for prefix only routes to the new

[PATCH v2 net-next 02/11] net: Add struct for fib dump filter

2018-10-15 Thread David Ahern
From: David Ahern Add struct fib_dump_filter for options on limiting which routes are returned in a dump request. The current list is table id, protocol, route type, rtm_flags and nexthop device index. struct net is needed to lookup the net_device from the index. Declare the filter for each

[PATCH v2 net-next 08/11] net: Enable kernel side filtering of route dumps

2018-10-15 Thread David Ahern
From: David Ahern Update parsing of route dump request to enable kernel side filtering. Allow filtering results by protocol (e.g., which routing daemon installed the route), route type (e.g., unicast), table id and nexthop device. These amount to the low hanging fruit, yet a huge improvement,

[PATCH v2 net-next 00/11] net: Kernel side filtering for route dumps

2018-10-15 Thread David Ahern
From: David Ahern Implement kernel side filtering of route dumps by protocol (e.g., which routing daemon installed the route), route type (e.g., unicast), table id and nexthop device. iproute2 has been doing this filtering in userspace for years; pushing the filters to the kernel side reduces

[PATCH v2 net-next 03/11] net/ipv4: Plumb support for filtering route dumps

2018-10-15 Thread David Ahern
From: David Ahern Implement kernel side filtering of routes by table id, egress device index, protocol and route type. If the table id is given in the filter, lookup the table and call fib_table_dump directly for it. Signed-off-by: David Ahern --- include/net/ip_fib.h| 2 +-

Re: [PATCH bpf-next v2 7/8] bpf: add tls support for testing in test_sockmap

2018-10-15 Thread Daniel Borkmann
On 10/16/2018 02:42 AM, Andrey Ignatov wrote: > Hi Daniel and John! > > Daniel Borkmann [Fri, 2018-10-12 17:46 -0700]: >> From: John Fastabend >> >> This adds a --ktls option to test_sockmap in order to enable the >> combination of ktls and sockmap to run, which makes for another >> batch of

Re: [PATCH bpf-next v2 7/8] bpf: add tls support for testing in test_sockmap

2018-10-15 Thread Andrey Ignatov
Hi Daniel and John! Daniel Borkmann [Fri, 2018-10-12 17:46 -0700]: > From: John Fastabend > > This adds a --ktls option to test_sockmap in order to enable the > combination of ktls and sockmap to run, which makes for another > batch of 648 test cases for both in combination. > >

pull-request: bpf-next 2018-10-16

2018-10-15 Thread Daniel Borkmann
Hi David, The following pull-request contains BPF updates for your *net-next* tree. The main changes are: 1) Convert BPF sockmap and kTLS to both use a new sk_msg API and enable sk_msg BPF integration for the latter, from Daniel and John. 2) Enable BPF syscall side to indicate for maps that

[PATCH net 3/3] nfp: flower: use offsets provided by pedit instead of index for ipv6

2018-10-15 Thread Jakub Kicinski
From: Pieter Jansen van Vuuren Previously when populating the set ipv6 address action, we incorrectly made use of pedit's key index to determine which 32bit word should be set. We now calculate which word has been selected based on the offset provided by the pedit action. Fixes: 354b82bb320e

[PATCH net 0/3] nfp: fix pedit set action offloads

2018-10-15 Thread Jakub Kicinski
Hi, Pieter says: This set fixes set actions when using multiple pedit actions with partial masks and with multiple keys per pedit action. Additionally it fixes set ipv6 pedit action offloads when using it in combination with other header keys. The problem would only trigger if one combines

[PATCH net 1/3] nfp: flower: fix pedit set actions for multiple partial masks

2018-10-15 Thread Jakub Kicinski
From: Pieter Jansen van Vuuren Previously we did not correctly change headers when using multiple pedit actions with partial masks. We now take this into account and no longer just commit the last pedit action. Fixes: c0b1bd9a8b8a ("nfp: add set ipv4 header action flower offload")

[PATCH net 2/3] nfp: flower: fix multiple keys per pedit action

2018-10-15 Thread Jakub Kicinski
From: Pieter Jansen van Vuuren Previously we only allowed a single header key per pedit action to change the header. This used to result in the last header key in the pedit action to overwrite previous headers. We now keep track of them and allow multiple header keys per pedit action. Fixes:

Re: [PATCH bpf-next v2] tools: bpftool: add map create command

2018-10-15 Thread Alexei Starovoitov
On Mon, Oct 15, 2018 at 04:30:36PM -0700, Jakub Kicinski wrote: > Add a way of creating maps from user space. The command takes > as parameters most of the attributes of the map creation system > call command. After map is created its pinned to bpffs. This makes > it possible to easily and

Re: [PATCH bpf-next 2/3] bpf: emit RECORD_MMAP events for bpf prog load/unload

2018-10-15 Thread Song Liu
On Fri, Sep 21, 2018 at 3:15 PM Alexei Starovoitov wrote: > > On Fri, Sep 21, 2018 at 09:25:00AM -0300, Arnaldo Carvalho de Melo wrote: > > > > > I have considered adding MUNMAP to match existing MMAP, but went > > > without it because I didn't want to introduce new bit in perf_event_attr > > >

[PATCH bpf-next v2] tools: bpftool: add map create command

2018-10-15 Thread Jakub Kicinski
Add a way of creating maps from user space. The command takes as parameters most of the attributes of the map creation system call command. After map is created its pinned to bpffs. This makes it possible to easily and dynamically (without rebuilding programs) test various corner cases related

Re: [bpf-next PATCH v3 0/2] bpftool support for sockmap use cases

2018-10-15 Thread Alexei Starovoitov
On Mon, Oct 15, 2018 at 11:19:44AM -0700, John Fastabend wrote: > The first patch adds support for attaching programs to maps. This is > needed to support sock{map|hash} use from bpftool. Currently, I carry > around custom code to do this so doing it using standard bpftool will > be great. > >

Re: [PATCH bpf-next 05/13] bpf: get better bpf_prog ksyms based on btf func type_id

2018-10-15 Thread Martin Lau
On Fri, Oct 12, 2018 at 11:54:42AM -0700, Yonghong Song wrote: > This patch added interface to load a program with the following > additional information: >. prog_btf_fd >. func_info and func_info_len > where func_info will provides function range and type_id > corresponding to each

Re: [PATCH bpf-next 0/2] IPv6 sk-lookup fixes

2018-10-15 Thread Alexei Starovoitov
On Mon, Oct 15, 2018 at 10:27:44AM -0700, Joe Stringer wrote: > This series includes a couple of fixups for the IPv6 socket lookup > helper, to make the API more consistent (always supply all arguments in > network byte-order) and to allow its use when IPv6 is compiled as a > module. Applied,

Re: [PATCH bpf-next] tools: bpftool: add map create command

2018-10-15 Thread Jakub Kicinski
On Mon, 15 Oct 2018 12:58:07 -0700, Alexei Starovoitov wrote: > > > > fprintf(stderr, > > > > "Usage: %s %s { show | list } [MAP]\n" > > > > + " %s %s create FILE type TYPE key KEY_SIZE > > > > value VALUE_SIZE \\\n" > > > > + "

Re: [PATCH bpf-next 02/13] bpf: btf: Add BTF_KIND_FUNC and BTF_KIND_FUNC_PROTO

2018-10-15 Thread Daniel Borkmann
On 10/12/2018 08:54 PM, Yonghong Song wrote: [...] > +static bool btf_name_valid_identifier(const struct btf *btf, u32 offset) > +{ > + /* offset must be valid */ > + const char *src = >strings[offset]; > + > + if (!isalpha(*src) && *src != '_') > + return false; > + > +

Re: Fw: [Bug 201423] New: eth0: hw csum failure

2018-10-15 Thread Fabio Rossi
On 15 October 2018 17:41:47 CEST, Eric Dumazet wrote: >On Mon, Oct 15, 2018 at 8:15 AM Stephen Hemminger > wrote: >> >> >> >> Begin forwarded message: >> >> Date: Sun, 14 Oct 2018 10:42:48 + >> From: bugzilla-dae...@bugzilla.kernel.org >> To: step...@networkplumber.org >> Subject: [Bug

Re: [PATCH bpf-next 02/13] bpf: btf: Add BTF_KIND_FUNC and BTF_KIND_FUNC_PROTO

2018-10-15 Thread Daniel Borkmann
On 10/12/2018 08:54 PM, Yonghong Song wrote: > This patch adds BTF_KIND_FUNC and BTF_KIND_FUNC_PROTO > support to the type section. BTF_KIND_FUNC_PROTO is used > to specify the type of a function pointer. With this, > BTF has a complete set of C types (except float). > > BTF_KIND_FUNC is used to

Re: [PATCH net] sctp: use the pmtu from the icmp packet to update transport pathmtu

2018-10-15 Thread Marcelo Ricardo Leitner
On Mon, Oct 15, 2018 at 07:58:29PM +0800, Xin Long wrote: > Other than asoc pmtu sync from all transports, sctp_assoc_sync_pmtu > is also processing transport pmtu_pending by icmp packets. But it's > meaningless to use sctp_dst_mtu(t->dst) as new pmtu for a transport. > > The right pmtu value

Re: [PATCH iproute2] macsec: fix off-by-one when parsing attributes

2018-10-15 Thread Sabrina Dubroca
2018-10-15, 09:36:58 -0700, Stephen Hemminger wrote: > On Fri, 12 Oct 2018 17:34:12 +0200 > Sabrina Dubroca wrote: > > > I seem to have had a massive brainfart with uses of > > parse_rtattr_nested(). The rtattr* array must have MAX+1 elements, and > > the call to parse_rtattr_nested must have

Re: [PATCH bpf-next 01/13] bpf: btf: Break up btf_type_is_void()

2018-10-15 Thread Daniel Borkmann
On 10/12/2018 08:54 PM, Yonghong Song wrote: > This patch breaks up btf_type_is_void() into > btf_type_is_void() and btf_type_is_fwd(). > > It also adds btf_type_nosize() to better describe it is > testing a type has nosize info. > > Signed-off-by: Martin KaFai Lau > --- Yonghong, your SoB is

Re: [PATCH bpf-next 0/2] IPv6 sk-lookup fixes

2018-10-15 Thread Daniel Borkmann
On 10/15/2018 07:27 PM, Joe Stringer wrote: > This series includes a couple of fixups for the IPv6 socket lookup > helper, to make the API more consistent (always supply all arguments in > network byte-order) and to allow its use when IPv6 is compiled as a > module. > > Joe Stringer (2): > bpf:

Re: [PATCH net-next 11/18] vxlan: Add netif_is_vxlan()

2018-10-15 Thread Stephen Hemminger
On Mon, 15 Oct 2018 13:30:41 -0700 Jakub Kicinski wrote: > On Mon, 15 Oct 2018 23:27:41 +0300, Ido Schimmel wrote: > > On Mon, Oct 15, 2018 at 01:16:42PM -0700, Stephen Hemminger wrote: > > > On Mon, 15 Oct 2018 22:57:48 +0300 > > > Ido Schimmel wrote: > > > > > > > On Mon, Oct 15, 2018

Re: [PATCH net-next 11/18] vxlan: Add netif_is_vxlan()

2018-10-15 Thread Jakub Kicinski
On Mon, 15 Oct 2018 23:27:41 +0300, Ido Schimmel wrote: > On Mon, Oct 15, 2018 at 01:16:42PM -0700, Stephen Hemminger wrote: > > On Mon, 15 Oct 2018 22:57:48 +0300 > > Ido Schimmel wrote: > > > > > On Mon, Oct 15, 2018 at 11:57:56AM -0700, Jakub Kicinski wrote: > > > > On Sat, 13 Oct 2018

Re: [PATCH net-next 11/18] vxlan: Add netif_is_vxlan()

2018-10-15 Thread Ido Schimmel
On Mon, Oct 15, 2018 at 01:16:42PM -0700, Stephen Hemminger wrote: > On Mon, 15 Oct 2018 22:57:48 +0300 > Ido Schimmel wrote: > > > On Mon, Oct 15, 2018 at 11:57:56AM -0700, Jakub Kicinski wrote: > > > On Sat, 13 Oct 2018 17:18:38 +, Ido Schimmel wrote: > > > > Add the ability to determine

[iproute PATCH] ip-addrlabel: Fix printing of label value

2018-10-15 Thread Phil Sutter
Passing the return value of RTA_DATA() to rta_getattr_u32() is wrong since that function will call RTA_DATA() by itself already. Fixes: a7ad1c8a6845d ("ipaddrlabel: add json support") Signed-off-by: Phil Sutter --- ip/ipaddrlabel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH net-next 11/18] vxlan: Add netif_is_vxlan()

2018-10-15 Thread Stephen Hemminger
On Mon, 15 Oct 2018 22:57:48 +0300 Ido Schimmel wrote: > On Mon, Oct 15, 2018 at 11:57:56AM -0700, Jakub Kicinski wrote: > > On Sat, 13 Oct 2018 17:18:38 +, Ido Schimmel wrote: > > > Add the ability to determine whether a netdev is a VxLAN netdev by > > > calling the above mentioned

Re: [PATCH bpf-next] tools: bpftool: add map create command

2018-10-15 Thread Alexei Starovoitov
On Mon, Oct 15, 2018 at 09:49:08AM -0700, Jakub Kicinski wrote: > On Fri, 12 Oct 2018 23:16:59 -0700, Alexei Starovoitov wrote: > > On Fri, Oct 12, 2018 at 11:06:14AM -0700, Jakub Kicinski wrote: > > > Add a way of creating maps from user space. The command takes > > > as parameters most of the

Re: [PATCH net-next 11/18] vxlan: Add netif_is_vxlan()

2018-10-15 Thread Ido Schimmel
On Mon, Oct 15, 2018 at 11:57:56AM -0700, Jakub Kicinski wrote: > On Sat, 13 Oct 2018 17:18:38 +, Ido Schimmel wrote: > > Add the ability to determine whether a netdev is a VxLAN netdev by > > calling the above mentioned function that checks the netdev's private > > flags. > > > > This will

Re: [PATCH bpf-next v2 0/8] sockmap integration for ktls

2018-10-15 Thread Alexei Starovoitov
On Sat, Oct 13, 2018 at 02:45:55AM +0200, Daniel Borkmann wrote: > This work adds a generic sk_msg layer and converts both sockmap > and later ktls over to make use of it as a common data structure > for application data (similarly as sk_buff for network packets). > With that in place the sk_msg

[PATCH net-next] net: phy: merge phy_start_aneg and phy_start_aneg_priv

2018-10-15 Thread Heiner Kallweit
After commit 9f2959b6b52d ("net: phy: improve handling delayed work") the sync parameter isn't needed any longer in phy_start_aneg_priv(). This allows to merge phy_start_aneg() and phy_start_aneg_priv(). Signed-off-by: Heiner Kallweit --- drivers/net/phy/phy.c | 21 +++-- 1 file

Re: [PATCH net] net/sched: properly init chain in case of multiple control actions

2018-10-15 Thread Cong Wang
On Sat, Oct 13, 2018 at 8:23 AM Davide Caratti wrote: > > On Fri, 2018-10-12 at 13:57 -0700, Cong Wang wrote: > > Why not just validate the fallback action in each action init()? > > For example, checking tcfg_paction in tcf_gact_init(). > > > > I don't see the need of making it generic. > >

Re: [bpf-next PATCH v3 2/2] bpf: bpftool, add flag to allow non-compat map definitions

2018-10-15 Thread Jakub Kicinski
On Mon, 15 Oct 2018 11:19:55 -0700, John Fastabend wrote: > Multiple map definition structures exist and user may have non-zero > fields in their definition that are not recognized by bpftool and > libbpf. The normal behavior is to then fail loading the map. Although > this is a good default

[bpf-next PATCH v3 2/2] bpf: bpftool, add flag to allow non-compat map definitions

2018-10-15 Thread John Fastabend
Multiple map definition structures exist and user may have non-zero fields in their definition that are not recognized by bpftool and libbpf. The normal behavior is to then fail loading the map. Although this is a good default behavior users may still want to load the map for debugging or other

[bpf-next PATCH v3 1/2] bpf: bpftool, add support for attaching programs to maps

2018-10-15 Thread John Fastabend
Sock map/hash introduce support for attaching programs to maps. To date I have been doing this with custom tooling but this is less than ideal as we shift to using bpftool as the single CLI for our BPF uses. This patch adds new sub commands 'attach' and 'detach' to the 'prog' command to attach

[bpf-next PATCH v3 0/2] bpftool support for sockmap use cases

2018-10-15 Thread John Fastabend
The first patch adds support for attaching programs to maps. This is needed to support sock{map|hash} use from bpftool. Currently, I carry around custom code to do this so doing it using standard bpftool will be great. The second patch adds a compat mode to ignore non-zero entries in the map def.

Re: [PATCH stable 4.9 v2 00/29] backport of IP fragmentation fixes

2018-10-15 Thread Eric Dumazet
On Mon, Oct 15, 2018 at 10:47 AM Florian Fainelli wrote: > > > > On 10/10/2018 12:29 PM, Florian Fainelli wrote: > > This is based on Stephen's v4.14 patches, with the necessary merge > > conflicts, and the lack of timer_setup() on the 4.9 baseline. > > > > Perf results on a gigabit capable

Re: [PATCH stable 4.9 v2 00/29] backport of IP fragmentation fixes

2018-10-15 Thread Florian Fainelli
On 10/10/2018 12:29 PM, Florian Fainelli wrote: > This is based on Stephen's v4.14 patches, with the necessary merge > conflicts, and the lack of timer_setup() on the 4.9 baseline. > > Perf results on a gigabit capable system, before and after are below. > > Series can also be found here: >

[PATCH bpf-next 1/2] bpf: Allow sk_lookup with IPv6 module

2018-10-15 Thread Joe Stringer
This is a more complete fix than d71019b54bff ("net: core: Fix build with CONFIG_IPV6=m"), so that IPv6 sockets may be looked up if the IPv6 module is loaded (not just if it's compiled in). Signed-off-by: Joe Stringer --- include/net/addrconf.h | 5 + net/core/filter.c | 12

[PATCH bpf-next 0/2] IPv6 sk-lookup fixes

2018-10-15 Thread Joe Stringer
This series includes a couple of fixups for the IPv6 socket lookup helper, to make the API more consistent (always supply all arguments in network byte-order) and to allow its use when IPv6 is compiled as a module. Joe Stringer (2): bpf: Allow sk_lookup with IPv6 module bpf: Fix IPv6 dport

[PATCH bpf-next 2/2] bpf: Fix IPv6 dport byte-order in bpf_sk_lookup

2018-10-15 Thread Joe Stringer
Commit 6acc9b432e67 ("bpf: Add helper to retrieve socket in BPF") mistakenly passed the destination port in network byte-order to the IPv6 TCP/UDP socket lookup functions, which meant that BPF writers would need to either manually swap the byte-order of this field or otherwise IPv6 sockets could

Read Business Letter

2018-10-15 Thread info
Steven Peter Walker(Esq) Stone Chambers, 4 Field Court, Gray's Inn, London, WC1R 5EF.. Email: stevenwalkerchamb...@workmail.co.za Greetings To You, This is a personal email directed to you and I request that it be treated as such. I am Steven Walker, a personal attorney/sole executor to the

Re: [PATCH bpf-next] tools: bpftool: add map create command

2018-10-15 Thread Jakub Kicinski
On Fri, 12 Oct 2018 23:16:59 -0700, Alexei Starovoitov wrote: > On Fri, Oct 12, 2018 at 11:06:14AM -0700, Jakub Kicinski wrote: > > Add a way of creating maps from user space. The command takes > > as parameters most of the attributes of the map creation system > > call command. After map is

Re: [PATCH iproute 2/2] utils: fix get_rtnl_link_stats_rta stats parsing

2018-10-15 Thread Stephen Hemminger
On Thu, 11 Oct 2018 14:24:03 +0200 Lorenzo Bianconi wrote: > > > iproute2 walks through the list of available tunnels using netlink > > > protocol in order to get device info instead of reading > > > them from proc filesystem. However the kernel reports device statistics > > > using

[PATCH net-next 7/7] tcp: cdg: use tcp high resolution clock cache

2018-10-15 Thread Eric Dumazet
We store in tcp socket a cache of most recent high resolution clock, there is no need to call local_clock() again, since this cache is good enough. Signed-off-by: Eric Dumazet --- net/ipv4/tcp_cdg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv4/tcp_cdg.c

[PATCH net-next 3/7] tcp: mitigate scheduling jitter in EDT pacing model

2018-10-15 Thread Eric Dumazet
In commit fefa569a9d4b ("net_sched: sch_fq: account for schedule/timers drifts") we added a mitigation for scheduling jitter in fq packet scheduler. This patch does the same in TCP stack, now it is using EDT model. Note that this mitigation is valid for both external (fq packet scheduler) or

[PATCH net-next 2/7] net: extend sk_pacing_rate to unsigned long

2018-10-15 Thread Eric Dumazet
sk_pacing_rate has beed introduced as a u32 field in 2013, effectively limiting per flow pacing to 34Gbit. We believe it is time to allow TCP to pace high speed flows on 64bit hosts, as we now can reach 100Gbit on one TCP flow. This patch adds no cost for 32bit kernels. The tcpi_pacing_rate and

[PATCH net-next 6/7] tcp_bbr: fix typo in bbr_pacing_margin_percent

2018-10-15 Thread Eric Dumazet
From: Neal Cardwell There was a typo in this parameter name. Signed-off-by: Neal Cardwell Signed-off-by: Eric Dumazet --- net/ipv4/tcp_bbr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/ipv4/tcp_bbr.c b/net/ipv4/tcp_bbr.c index

[PATCH net-next 4/7] net_sched: sch_fq: no longer use skb_is_tcp_pure_ack()

2018-10-15 Thread Eric Dumazet
With the new EDT model, sch_fq no longer has to special case TCP pure acks, since their skb->tstamp will allow them being sent without pacing delay. Signed-off-by: Eric Dumazet --- net/sched/sch_fq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/sched/sch_fq.c

[PATCH net-next 5/7] tcp: optimize tcp internal pacing

2018-10-15 Thread Eric Dumazet
When TCP implements its own pacing (when no fq packet scheduler is used), it is arming high resolution timer after a packet is sent. But in many cases (like TCP_RR kind of workloads), this high resolution timer expires before the application attempts to write the following packet. This overhead

[PATCH net-next 1/7] tcp: do not change tcp_wstamp_ns in tcp_mstamp_refresh

2018-10-15 Thread Eric Dumazet
In EDT design, I made the mistake of using tcp_wstamp_ns to store the last tcp_clock_ns() sample and to store the pacing virtual timer. This causes major regressions at high speed flows. Introduce tcp_clock_cache to store last tcp_clock_ns(). This is needed because some arches have slow

[PATCH net-next 0/7] tcp: second round for EDT conversion

2018-10-15 Thread Eric Dumazet
First round of EDT patches left TCP stack in a non optimal state. - High speed flows suffered from loss of performance, addressed by the first patch of this series. - Second patch brings pacing to the current state of networking, since we now reach ~100 Gbit on a single TCP flow. - Third

Re: [PATCH iproute2] macsec: fix off-by-one when parsing attributes

2018-10-15 Thread Stephen Hemminger
On Fri, 12 Oct 2018 17:34:12 +0200 Sabrina Dubroca wrote: > I seem to have had a massive brainfart with uses of > parse_rtattr_nested(). The rtattr* array must have MAX+1 elements, and > the call to parse_rtattr_nested must have MAX as its bound. Let's fix > those. > > Fixes: b26fc590ce62 ("ip:

Re: [PATCH iproute2] json: make 0xhex handle u64

2018-10-15 Thread Stephen Hemminger
On Fri, 12 Oct 2018 17:34:32 +0200 Sabrina Dubroca wrote: > Stephen converted macsec's sci to use 0xhex, but 0xhex handles > unsigned int's, not 64 bits ints. Thus, the output of the "ip macsec > show" command is mangled, with half of the SCI replaced with 0s: > > # ip macsec show > 11:

Re: [iproute PATCH] bridge: fdb: Fix for missing keywords in non-JSON output

2018-10-15 Thread Stephen Hemminger
On Tue, 9 Oct 2018 14:44:08 +0200 Phil Sutter wrote: > While migrating to JSON print library, some keywords were dropped from > standard output by accident. Add them back to unbreak output parsers. > > Fixes: c7c1a1ef51aea ("bridge: colorize output and use JSON print library") > Signed-off-by:

Re: BBR and TCP internal pacing causing interrupt storm with pfifo_fast

2018-10-15 Thread Eric Dumazet
On 10/15/2018 07:50 AM, Eric Dumazet wrote: > On Mon, Oct 15, 2018 at 3:26 AM Gasper Zejn wrote: >> >> >> I've tried to isolate the issue as best I could. There seems to be an >> issue if the TCP socket has keepalive set and send queue is not empty >> and the route goes away. >> >>

Re: [Bug 201423] New: eth0: hw csum failure

2018-10-15 Thread Stephen Hemminger
On Mon, 15 Oct 2018 08:41:47 -0700 Eric Dumazet wrote: > On Mon, Oct 15, 2018 at 8:15 AM Stephen Hemminger > wrote: > > > > > > > > Begin forwarded message: > > > > Date: Sun, 14 Oct 2018 10:42:48 + > > From: bugzilla-dae...@bugzilla.kernel.org > > To: step...@networkplumber.org > >

Re: Fw: [Bug 201423] New: eth0: hw csum failure

2018-10-15 Thread Dave Stevenson
Hi Eric. On Mon, 15 Oct 2018 at 16:42, Eric Dumazet wrote: > > On Mon, Oct 15, 2018 at 8:15 AM Stephen Hemminger > wrote: > > > > > > > > Begin forwarded message: > > > > Date: Sun, 14 Oct 2018 10:42:48 + > > From: bugzilla-dae...@bugzilla.kernel.org > > To: step...@networkplumber.org > >

Re: [bpf-next PATCH v2 2/2] bpf: bpftool, add flag to allow non-compat map definitions

2018-10-15 Thread Jakub Kicinski
On Mon, 15 Oct 2018 08:17:53 -0700, John Fastabend wrote: > Multiple map definition structures exist and user may have non-zero > fields in their definition that are not recognized by bpftool and > libbpf. The normal behavior is to then fail loading the map. Although > this is a good default

Re: [bpf-next PATCH v2 1/2] bpf: bpftool, add support for attaching programs to maps

2018-10-15 Thread Jakub Kicinski
On Mon, 15 Oct 2018 08:17:48 -0700, John Fastabend wrote: > Sock map/hash introduce support for attaching programs to maps. To > date I have been doing this with custom tooling but this is less than > ideal as we shift to using bpftool as the single CLI for our BPF uses. > This patch adds new sub

Re: Bug in MACSec - stops passing traffic after approx 5TB

2018-10-15 Thread Josh Coombs
And confirmed, starting with a high packet number results in a very short testbed run, 296 packets and then nothing, just as you surmised. Sorry for raising the alarm falsely. Looks like I need to roll my own build of wpa_supplicant as the ubuntu builds don't include the macsec driver, haven't

Re: Fw: [Bug 201423] New: eth0: hw csum failure

2018-10-15 Thread Eric Dumazet
On Mon, Oct 15, 2018 at 8:15 AM Stephen Hemminger wrote: > > > > Begin forwarded message: > > Date: Sun, 14 Oct 2018 10:42:48 + > From: bugzilla-dae...@bugzilla.kernel.org > To: step...@networkplumber.org > Subject: [Bug 201423] New: eth0: hw csum failure > > >

[bpf-next PATCH v2 2/2] bpf: bpftool, add flag to allow non-compat map definitions

2018-10-15 Thread John Fastabend
Multiple map definition structures exist and user may have non-zero fields in their definition that are not recognized by bpftool and libbpf. The normal behavior is to then fail loading the map. Although this is a good default behavior users may still want to load the map for debugging or other

[bpf-next PATCH v2 1/2] bpf: bpftool, add support for attaching programs to maps

2018-10-15 Thread John Fastabend
Sock map/hash introduce support for attaching programs to maps. To date I have been doing this with custom tooling but this is less than ideal as we shift to using bpftool as the single CLI for our BPF uses. This patch adds new sub commands 'attach' and 'detach' to the 'prog' command to attach

  1   2   >