Re: [PATCH bpf-next v2 08/12] tools: libbpf: add extended attributes version of bpf_object__open()

2018-07-09 Thread Andrey Ignatov
Jakub Kicinski [Mon, 2018-07-09 11:01 -0700]: > Similarly to bpf_prog_load() users of bpf_object__open() may need > to specify the expected program type. Program type is needed at > open to avoid the kernel version check for program types which don't > require it. > > Signed-off-by: Jakub Kicins

Re: [PATCH bpf-next v2 05/12] tools: libbpf: expose the prog type guessing from section name logic

2018-07-09 Thread Andrey Ignatov
Jakub Kicinski [Mon, 2018-07-09 11:01 -0700]: > libbpf can guess program type based on ELF section names. As libbpf > becomes more popular its association between section name strings and > types becomes more of a standard. Allow libbpf users to use the same > logic for matching strings to types

Re: [net-next,v2] tcp: Improve setsockopt() TCP_USER_TIMEOUT accuracy

2018-07-09 Thread Eric Dumazet
On 07/04/2018 04:34 PM, Jonathan Maxwell wrote: > Let's wait for Eric to review. Then I'll put together the next version. Sorry for the delay (I was travelling last week) , please respin a v3, thanks !

Re: [PATCH bpf-next v2 11/12] tools: libbpf: allow map reuse

2018-07-09 Thread Andrey Ignatov
Jakub Kicinski [Mon, 2018-07-09 19:49 -0700]: > On Mon, 9 Jul 2018 13:22:54 -0700, Andrey Ignatov wrote: > > Jakub Kicinski [Mon, 2018-07-09 11:01 -0700]: > > > More advanced applications may want to only replace programs without > > > destroying associated maps. Allow libbpf users to achieve th

Re: [PATCH bpf-next v2 11/12] tools: libbpf: allow map reuse

2018-07-09 Thread Jakub Kicinski
On Mon, 9 Jul 2018 13:22:54 -0700, Andrey Ignatov wrote: > Jakub Kicinski [Mon, 2018-07-09 11:01 -0700]: > > More advanced applications may want to only replace programs without > > destroying associated maps. Allow libbpf users to achieve that. > > Instead of always creating all of the maps at l

Re: [PATCH bpf-next v2 12/12] tools: bpftool: allow reuse of maps with bpftool prog load

2018-07-09 Thread Jakub Kicinski
On Mon, 9 Jul 2018 12:48:20 -0700, Alexei Starovoitov wrote: > On Mon, Jul 09, 2018 at 10:59:44AM -0700, Jakub Kicinski wrote: > > Add map parameter to prog load which will allow reuse of existing > > maps instead of creating new ones. > > > > We need feature detection and compat code for realloca

[PATCH net-next] macvlan: Change status when lower device goes down

2018-07-09 Thread Travis Brown
Today macvlan ignores the notification when a lower device goes administratively down, preventing the lack of connectivity from bubbling up. Processing NETDEV_DOWN results in a macvlan state of LOWERLAYERDOWN with NO-CARRIER which should be easy to interpret in userspace. 2: lower: mtu 1500 qdis

[PATCH v4 iproute2-next 2/3] tc: Add support for the ETF Qdisc

2018-07-09 Thread Jesus Sanchez-Palencia
From: Vinicius Costa Gomes The "Earliest TxTime First" (ETF) queueing discipline allows precise control of the transmission time of packets by providing a sorted time-based scheduling of packets. The syntax is: tc qdisc add dev DEV parent NODE etf delta clockid [offload]

[PATCH v4 iproute2-next 0/3] Add support for ETF qdisc

2018-07-09 Thread Jesus Sanchez-Palencia
fixes since v3: - Add support for clock names with the "CLOCK_" prefix; - Print clock name on print_opt(); - Use strcasecmp() instead of strncasecmp(). The ETF (earliest txtime first) qdisc was recently merged into net-next [1], so this patchset adds support for it through the tc command line

[PATCH v4 iproute2-next 3/3] man: Add initial manpage for tc-etf(8)

2018-07-09 Thread Jesus Sanchez-Palencia
Add an initial manpage for tc-etf covering all config options, basic concepts and operation modes. Signed-off-by: Jesus Sanchez-Palencia --- man/man8/tc-etf.8 | 141 ++ 1 file changed, 141 insertions(+) create mode 100644 man/man8/tc-etf.8 diff --git

[PATCH v4 iproute2-next 1/3] uapi pkt_sched: Add etf info - DO NOT COMMIT

2018-07-09 Thread Jesus Sanchez-Palencia
This should come from the next uapi headers update. Sending it now just as a convenience so anyone can build tc with etf and taprio support. Signed-off-by: Jesus Sanchez-Palencia --- include/uapi/linux/pkt_sched.h | 21 + 1 file changed, 21 insertions(+) diff --git a/include

Re: [PATCH v2 net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread David Miller
From: Jesus Sanchez-Palencia Date: Mon, 9 Jul 2018 16:20:56 -0700 > We are not supposed to use u32 in uapi, so change the flags member of > struct sock_txtime from u32 to __u32 instead. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit > time") > Reported-by: Eric Du

Re: [PATCH v2] rhashtable: add restart routine in rhashtable_free_and_destroy()

2018-07-09 Thread David Miller
From: Taehee Yoo Date: Sun, 8 Jul 2018 11:55:51 +0900 > rhashtable_free_and_destroy() cancels re-hash deferred work > then walks and destroys elements. at this moment, some elements can be > still in future_tbl. that elements are not destroyed. > > test case: > nft_rhash_destroy() calls rhashta

Re: [PATCH v2 net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread Eric Dumazet
On 07/09/2018 04:20 PM, Jesus Sanchez-Palencia wrote: > We are not supposed to use u32 in uapi, so change the flags member of > struct sock_txtime from u32 to __u32 instead. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit > time") > Reported-by: Eric Dumazet > Sig

Re: [PATCH net 0/6] bnxt_en: Bug fixes.

2018-07-09 Thread David Miller
From: Michael Chan Date: Mon, 9 Jul 2018 02:24:46 -0400 > These are bug fixes in error code paths, TC Flower VLAN TCI flow > checking bug fix, proper filtering of Broadcast packets if IFF_BROADCAST > is not set, and a bug fix in bnxt_get_max_rings() to return 0 ring > parameters when the return

Re: [PATCH net-next 00/12] mlxsw: More Spectrum-2 preparations

2018-07-09 Thread David Miller
From: Ido Schimmel Date: Sun, 8 Jul 2018 23:51:15 +0300 > This is the second and last set of preparations towards initial > Spectrum-2 support in mlxsw. It mainly re-arranges parts of the code > that need to work with both ASICs, but somewhat differ. > > The first three patches allow different

[PATCH v2 net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread Jesus Sanchez-Palencia
We are not supposed to use u32 in uapi, so change the flags member of struct sock_txtime from u32 to __u32 instead. Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time") Reported-by: Eric Dumazet Signed-off-by: Jesus Sanchez-Palencia --- include/uapi/linux/net_tstamp.h

Re: [PATCH net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread Jesus Sanchez-Palencia
On 07/09/2018 04:18 PM, Eric Dumazet wrote: > > > On 07/09/2018 04:08 PM, Jesus Sanchez-Palencia wrote: >> We are not supposed to use u32 in uapi, so change the flags member of >> struct sock_txtime from u32 to __u32 instead. >> >> Fixes: 80b14dee2bea ("net: Add a new socket option for a futur

Re: [PATCH v2 net-next 00/18] TLS offload rx, netdev & mlx5

2018-07-09 Thread David Miller
From: Boris Pismenny Date: Sun, 8 Jul 2018 12:36:14 +0300 > The following series provides TLS TX inline crypto offload. I think this patch series provides RX inline offload not TX inline offload.

Re: [PATCH v3 iproute2 2/3] tc: Add support for the ETF Qdisc

2018-07-09 Thread Jesus Sanchez-Palencia
On 07/09/2018 10:32 AM, David Ahern wrote: > On 7/9/18 9:48 AM, Jesus Sanchez-Palencia wrote: >> Hi David, >> >> >> On 07/06/2018 08:58 AM, David Ahern wrote: >>> On 7/5/18 4:42 PM, Jesus Sanchez-Palencia wrote: >>> +static int get_clockid(__s32 *val, const char *arg) +{ + const

Re: [PATCH net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread Eric Dumazet
On 07/09/2018 04:08 PM, Jesus Sanchez-Palencia wrote: > We are not supposed to use u32 in uapi, so change the flags member of > struct sock_txtime from u32 to __u32 instead. > > Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit > time") > Signed-off-by: Jesus Sanchez-Pal

[PATCH net-next] net: Use __u32 in uapi net_stamp.h

2018-07-09 Thread Jesus Sanchez-Palencia
We are not supposed to use u32 in uapi, so change the flags member of struct sock_txtime from u32 to __u32 instead. Fixes: 80b14dee2bea ("net: Add a new socket option for a future transmit time") Signed-off-by: Jesus Sanchez-Palencia --- include/uapi/linux/net_tstamp.h | 2 +- 1 file changed, 1

Re: [PATCH net-next v2] tcp: remove SG-related comment in tcp_sendmsg()

2018-07-09 Thread David Miller
From: Julian Wiedmann Date: Mon, 9 Jul 2018 09:45:14 +0200 > Since commit 74d4a8f8d378 ("tcp: remove sk_can_gso() use"), the code > doesn't care whether the interface supports SG. > > Signed-off-by: Julian Wiedmann > --- > v2: remove the whole comment (Eric) Applied.

[PATCH bpf] bpf: fix ldx in ld_abs rewrite for large offsets

2018-07-09 Thread Daniel Borkmann
Mark reported that syzkaller triggered a KASAN detected slab-out-of-bounds bug in ___bpf_prog_run() with a BPF_LD | BPF_ABS word load at offset 0x8001. After further investigation it became clear that the issue was the BPF_LDX_MEM() which takes offset as an argument whereas it cannot encode larger

Re: [BUG] mlx5 have problems with ipv4-ipv6 tunnels in linux 4.4

2018-07-09 Thread Saeed Mahameed
On Tue, Jul 3, 2018 at 10:45 PM, Konstantin Khlebnikov wrote: > I'm seeing problems with tunnelled traffic with Mellanox Technologies > MT27710 Family [ConnectX-4 Lx] using vanilla driver from linux 4.4.y > > Packets with payload bigger than 116 bytes are not exmited. > Smaller packets and normal

Re: [PATCH v2 net-next 02/14] net: Add a new socket option for a future transmit time.

2018-07-09 Thread Jesus Sanchez-Palencia
On 07/07/2018 05:44 PM, Eric Dumazet wrote: > > > On 07/03/2018 03:42 PM, Jesus Sanchez-Palencia wrote: >> From: Richard Cochran >> >> This patch introduces SO_TXTIME. User space enables this option in >> order to pass a desired future transmit time in a CMSG when calling >> sendmsg(2). The a

[net-next 2/7] net: Add support for subordinate device traffic classes

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This patch is meant to provide the basic tools needed to allow us to create subordinate device traffic classes. The general idea here is to allow subdividing the queues of a device into queue groups accessible through an upper device such as a macvlan. The idea here is to e

[net-next 1/7] net-sysfs: Drop support for XPS and traffic_class on single queue device

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This patch makes it so that we do not report the traffic class or allow XPS configuration on single queue devices. This is mostly to avoid unnecessary complexity with changes I have planned that will allow us to reuse the unused tc_to_txq and XPS configuration on a single qu

[net-next 4/7] net: Add support for subordinate traffic classes to netdev_pick_tx

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This change makes it so that we can support the concept of subordinate device traffic classes to the core networking code. In doing this we can start pulling out the driver specific bits needed to support selecting a queue based on an upper device. The solution at is curren

[net-next 7/7] net: allow fallback function to pass netdev

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck For most of these calls we can just pass NULL through to the fallback function as the sb_dev. The only cases where we cannot are the cases where we might be dealing with either an upper device or a driver that would have configured things to support an sb_dev itself. The on

[net-next 6/7] net: allow ndo_select_queue to pass netdev

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This patch makes it so that instead of passing a void pointer as the accel_priv we instead pass a net_device pointer as sb_dev. Making this change allows us to pass the subordinate device through to the fallback function eventually so that we can keep the actual code in the

[net-next 3/7] ixgbe: Add code to populate and use macvlan TC to Tx queue map

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This patch makes it so that we use the tc_to_txq mapping in the macvlan device in order to select the Tx queue for outgoing packets. The idea here is to try and move away from using ixgbe_select_queue and to come up with a generic way to make this work for devices going for

[net-next 0/7][pull request] L2 Fwd Offload & 10GbE Intel Driver Updates 2018-07-09

2018-07-09 Thread Jeff Kirsher
This patch series is meant to allow support for the L2 forward offload, aka MACVLAN offload without the need for using ndo_select_queue. The existing solution currently requires that we use ndo_select_queue in the transmit path if we want to associate specific Tx queues with a given MACVLAN interf

[net-next 5/7] net: Add generic ndo_select_queue functions

2018-07-09 Thread Jeff Kirsher
From: Alexander Duyck This patch adds a generic version of the ndo_select_queue functions for either returning 0 or selecting a queue based on the processor ID. This is generally meant to just reduce the number of functions we have to change in the future when we have to deal with ndo_select_queu

Re: [PATCH v3 net-next 0/3] fix use-after-free bugs in skb list processing

2018-07-09 Thread David Miller
From: Edward Cree Date: Mon, 9 Jul 2018 18:08:38 +0100 > A couple of bugs in skb list handling were spotted by Dan Carpenter, with > the help of Smatch; following up on them I found a couple more similar > cases. This series fixes them by changing the relevant loops to use the > dequeue-enque

Re: [PATCH v3 net-next] net/sched: add skbprio scheduler

2018-07-09 Thread Marcelo Ricardo Leitner
On Mon, Jul 09, 2018 at 05:03:31PM -0400, Michel Machado wrote: > On 07/09/2018 03:53 PM, Marcelo Ricardo Leitner wrote: > > On Mon, Jul 09, 2018 at 02:18:33PM -0400, Michel Machado wrote: > > > On 07/09/2018 11:44 AM, Marcelo Ricardo Leitner wrote: > > > > On Sat, Jul 07, 2018 at 03:43:55PM +0530,

Re: [PATCH net-next 07/10] r8169: migrate speed_down function to phylib

2018-07-09 Thread Heiner Kallweit
On 03.07.2018 18:48, Florian Fainelli wrote: > > > On 07/02/2018 02:31 PM, Heiner Kallweit wrote: >> On 02.07.2018 23:20, Andrew Lunn wrote: >>> On Mon, Jul 02, 2018 at 09:37:08PM +0200, Heiner Kallweit wrote: Change rtl_speed_down() to use phylib. Signed-off-by: Heiner Kallweit

Re: [PATCH v3 net-next] net/sched: add skbprio scheduler

2018-07-09 Thread Michel Machado
On 07/09/2018 03:53 PM, Marcelo Ricardo Leitner wrote: On Mon, Jul 09, 2018 at 02:18:33PM -0400, Michel Machado wrote: On 07/09/2018 11:44 AM, Marcelo Ricardo Leitner wrote: On Sat, Jul 07, 2018 at 03:43:55PM +0530, Nishanth Devarajan wrote: net/sched: add skbprio scheduer Skbprio (SKB Priori

Re: [PATCH net-next] net: sched: fix unprotected access to rcu cookie pointer

2018-07-09 Thread Eric Dumazet
On 07/09/2018 01:34 PM, Marcelo Ricardo Leitner wrote: > I am not sure if this is enough to fix the entire issue. Now it will > fetch the length correctly but, what guarantees that when it tries to > actually copy the key (tcf_action_dump_1), the same act_cookie pointer > will be used? As in, c

Re: [PATCH net-next] net: sched: fix unprotected access to rcu cookie pointer

2018-07-09 Thread Vlad Buslov
On Mon 09 Jul 2018 at 20:34, Marcelo Ricardo Leitner wrote: > On Mon, Jul 09, 2018 at 08:26:47PM +0300, Vlad Buslov wrote: >> Fix action attribute size calculation function to take rcu read lock and >> access act_cookie pointer with rcu dereference. >> >> Fixes: eec94fdb0480 ("net: sched: use

RE: [PATCH v1 net-next 9/9] lan743x: Add PTP support

2018-07-09 Thread Bryan.Whitehead
Thanks Richard, I'll add it in my next revision. > -Original Message- > From: Richard Cochran [mailto:richardcoch...@gmail.com] > Sent: Friday, July 6, 2018 5:25 PM > To: Bryan Whitehead - C21958 > Cc: da...@davemloft.net; netdev@vger.kernel.org; UNGLinuxDriver > > Subject: Re: [PATCH v1

Re: [PATCH net-next] net: sched: fix unprotected access to rcu cookie pointer

2018-07-09 Thread Marcelo Ricardo Leitner
On Mon, Jul 09, 2018 at 08:26:47PM +0300, Vlad Buslov wrote: > Fix action attribute size calculation function to take rcu read lock and > access act_cookie pointer with rcu dereference. > > Fixes: eec94fdb0480 ("net: sched: use rcu for action cookie update") > Reported-by: Marcelo Ricardo Leitner

Re: [PATCH bpf-next v2 11/12] tools: libbpf: allow map reuse

2018-07-09 Thread Andrey Ignatov
Jakub Kicinski [Mon, 2018-07-09 11:01 -0700]: > More advanced applications may want to only replace programs without > destroying associated maps. Allow libbpf users to achieve that. > Instead of always creating all of the maps at load time, expose to > users an API to reconstruct the map object

Re: [PATCH v3 net-next] net/sched: add skbprio scheduler

2018-07-09 Thread Marcelo Ricardo Leitner
On Mon, Jul 09, 2018 at 02:18:33PM -0400, Michel Machado wrote: > On 07/09/2018 11:44 AM, Marcelo Ricardo Leitner wrote: > > On Sat, Jul 07, 2018 at 03:43:55PM +0530, Nishanth Devarajan wrote: > > > net/sched: add skbprio scheduer > > > > > > Skbprio (SKB Priority Queue) is a queueing discipline t

[PATCH iproute2-next 3/9] tc: convert stats print to json

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Convert compatiablity statistics to print in JSON. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 32 ++-- 1 file changed, 22 insertions(+), 10 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index e8e1241d083d..05b6c97563b3 10064

[PATCH iproute2-next 5/9] tc/util: remove print_rate

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger This function is not used, only sprint_rate is used. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 10 +++--- tc/tc_util.h | 1 - 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index 05b6c97563b3..f5ffe3443892 10064

[PATCH iproute2-next 7/9] tc/util: remove unused print_time

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Only sprint_time is used. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 8 ++-- tc/tc_util.h | 1 - 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index 01e131b5c5d7..95cb49b98612 100644 --- a/tc/tc_util.c +++ b/tc/t

[PATCH iproute2-next 1/9] tc: use JSON in error handling

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger If option or qdisc is unknown, still output in JSON. Signed-off-by: Stephen Hemminger --- tc/tc.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/tc/tc.c b/tc/tc.c index 0d223281ba25..b6d2a6151793 100644 --- a/tc/tc.c +++ b/tc/tc

[PATCH iproute2-next 6/9] tc/util: remove unused print_size

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Only sprint_size is used, so fold it in. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 7 ++- tc/tc_util.h | 1 - 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index f5ffe3443892..01e131b5c5d7 100644 --- a/tc/tc_uti

[PATCH iproute2-next 8/9] tc/util: add print helpers for JSON

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Add a helper to print rate and size in numeric or pretty format based on JSON flag. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 59 ++-- tc/tc_util.h | 2 ++ 2 files changed, 31 insertions(+), 30 deletions(-) dif

[PATCH iproute2-next 2/9] tc: use const char in util

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger For prefix and path, can use const char. Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 8 +--- tc/tc_util.h | 6 +++--- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index e0c96291ade0..e8e1241d083d 100644 --- a/tc/

Re: [PATCH bpf-next v2 12/12] tools: bpftool: allow reuse of maps with bpftool prog load

2018-07-09 Thread Alexei Starovoitov
On Mon, Jul 09, 2018 at 10:59:44AM -0700, Jakub Kicinski wrote: > Add map parameter to prog load which will allow reuse of existing > maps instead of creating new ones. > > We need feature detection and compat code for reallocarray, since > it's not available in many libc versions. > > Signed-off

[PATCH iproute2-next 4/9] tc/cbq: use sprint_rate

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger All other places in tc use sprint_rate. Signed-off-by: Stephen Hemminger --- tc/q_cbq.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/tc/q_cbq.c b/tc/q_cbq.c index e7f1a3bfaf5d..ad0170c41858 100644 --- a/tc/q_cbq.c +++ b/tc/q_cbq.c

[PATCH iproute2-next 9/9] tc/sfq: add json support

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Convert to use JSON Signed-off-by: Stephen Hemminger --- tc/q_sfq.c | 65 -- 1 file changed, 39 insertions(+), 26 deletions(-) diff --git a/tc/q_sfq.c b/tc/q_sfq.c index 6a1d853b7c93..cc8ce0dddf7e 100644 --- a/tc/q_sf

[PATCH iproute2-next 0/9] TC more JSON support

2018-07-09 Thread Stephen Hemminger
From: Stephen Hemminger Update core of TC command and library to do more JSON. Most of this patch set is about getting tc utility functions to be more friendly to the json_print infrastructure. Stephen Hemminger (9): tc: use JSON in error handling tc: use const char in util tc: convert sta

Re: [PATCH bpf-next v2 09/12] tools: bpftool: reimplement bpf_prog_load() for prog load

2018-07-09 Thread Alexei Starovoitov
On Mon, Jul 09, 2018 at 10:59:41AM -0700, Jakub Kicinski wrote: > bpf_prog_load() is a very useful helper but it doesn't give us full > flexibility of modifying the BPF objects before loading. Open code > bpf_prog_load() in bpftool so we can add extra logic in following > commits. > > Signed-off-

Re: [PATCH net-next v3 0/2] tcp: fix high tail latencies in DCTCP

2018-07-09 Thread Lawrence Brakmo
On 7/9/18, 12:32 PM, "Yuchung Cheng" wrote: On Sat, Jul 7, 2018 at 7:07 AM, Neal Cardwell wrote: > On Sat, Jul 7, 2018 at 7:15 AM David Miller wrote: >> >> From: Lawrence Brakmo >> Date: Tue, 3 Jul 2018 09:26:13 -0700 >> >> > When have observed high tail latencies w

Re: [PATCH v3 net-next] net/sched: add skbprio scheduler

2018-07-09 Thread Michel Machado
On 07/09/2018 11:44 AM, Marcelo Ricardo Leitner wrote: On Sat, Jul 07, 2018 at 03:43:55PM +0530, Nishanth Devarajan wrote: net/sched: add skbprio scheduer Skbprio (SKB Priority Queue) is a queueing discipline that prioritizes packets according to their skb->priority field. Under congestion, alr

Re: [PATCH net-next] tcp: expose both send and receive intervals for rate sample

2018-07-09 Thread Eric Dumazet
On 07/09/2018 10:53 AM, Deepti Raghavan wrote: > Congestion control algorithms, which access the rate sample > through the tcp_cong_control function, only have access to the maximum > of the send and receive interval, for cases where the acknowledgment > rate may be inaccurate due to ACK compres

Re: [PATCH net-next] tcp: expose both send and receive intervals for rate sample

2018-07-09 Thread Neal Cardwell
On Mon, Jul 9, 2018 at 1:58 PM Deepti Raghavan wrote: > > Congestion control algorithms, which access the rate sample > through the tcp_cong_control function, only have access to the maximum > of the send and receive interval, for cases where the acknowledgment > rate may be inaccurate due to ACK

[PATCH bpf-next v2 07/12] tools: libbpf: recognize offload neutral maps

2018-07-09 Thread Jakub Kicinski
Add helper to libbpf for recognizing maps which should not have ifindex set when program is loaded. These maps only contain host metadata and therefore are not marked for offload, e.g. the perf event map. Use this helper in bpf_prog_load_xattr(). Signed-off-by: Jakub Kicinski Reviewed-by: Quent

[PATCH bpf-next v2 04/12] tools: bpftool: add support for loading programs for offload

2018-07-09 Thread Jakub Kicinski
Extend the bpftool prog load command to also accept "dev" parameter, which will allow us to load programs onto devices. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- .../bpftool/Documentation/bpftool-prog.rst| 6 ++-- tools/bpf/bpftool/bash-completion/bpftool | 23 +

[PATCH bpf-next v2 06/12] tools: bpftool: allow users to specify program type for prog load

2018-07-09 Thread Jakub Kicinski
Sometimes program section names don't match with libbpf's expectation. In particular XDP's default section names differ between libbpf and iproute2. Allow users to pass program type on command line. Name the types like the libbpf expected section names. Signed-off-by: Jakub Kicinski Reviewed-by

[PATCH bpf-next v2 05/12] tools: libbpf: expose the prog type guessing from section name logic

2018-07-09 Thread Jakub Kicinski
libbpf can guess program type based on ELF section names. As libbpf becomes more popular its association between section name strings and types becomes more of a standard. Allow libbpf users to use the same logic for matching strings to types, e.g. when the string originates from command line. S

[PATCH bpf-next v2 10/12] tools: bpf: make use of reallocarray

2018-07-09 Thread Jakub Kicinski
reallocarray() is a safer variant of realloc which checks for multiplication overflow in case of array allocation. Since it's not available in Glibc < 2.26 import kernel's overflow.h and add a static inline implementation when needed. Use feature detection to probe for existence of reallocarray.

[PATCH bpf-next v2 11/12] tools: libbpf: allow map reuse

2018-07-09 Thread Jakub Kicinski
More advanced applications may want to only replace programs without destroying associated maps. Allow libbpf users to achieve that. Instead of always creating all of the maps at load time, expose to users an API to reconstruct the map object from already existing map. The map parameters are read

[PATCH bpf-next v2 03/12] tools: bpftool: refactor argument parsing for prog load

2018-07-09 Thread Jakub Kicinski
Add a new macro for printing more informative message than straight usage() when parameters are missing, and use it for prog do_load(). Save the object and pin path argument to variables for clarity. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/bpf/bpftool/main.h | 15 +++

[PATCH bpf-next v2 08/12] tools: libbpf: add extended attributes version of bpf_object__open()

2018-07-09 Thread Jakub Kicinski
Similarly to bpf_prog_load() users of bpf_object__open() may need to specify the expected program type. Program type is needed at open to avoid the kernel version check for program types which don't require it. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/lib/bpf/libbpf.

[PATCH bpf-next v2 12/12] tools: bpftool: allow reuse of maps with bpftool prog load

2018-07-09 Thread Jakub Kicinski
Add map parameter to prog load which will allow reuse of existing maps instead of creating new ones. We need feature detection and compat code for reallocarray, since it's not available in many libc versions. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- .../bpftool/Documentati

[PATCH bpf-next v2 09/12] tools: bpftool: reimplement bpf_prog_load() for prog load

2018-07-09 Thread Jakub Kicinski
bpf_prog_load() is a very useful helper but it doesn't give us full flexibility of modifying the BPF objects before loading. Open code bpf_prog_load() in bpftool so we can add extra logic in following commits. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/bpf/bpftool/prog

[PATCH bpf-next v2 01/12] selftests/bpf: remove duplicated word from test offloads

2018-07-09 Thread Jakub Kicinski
Trivial removal of duplicated "mode" in error message. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/testing/selftests/bpf/test_offload.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/bpf/test_offload.py b/tools/testing/selft

[PATCH bpf-next v2 00/12] tools: bpf: extend bpftool prog load

2018-07-09 Thread Jakub Kicinski
Hi! This series starts with two minor clean ups to test_offload.py selftest script. The next 9 patches extend the abilities of bpftool prog load beyond the simple cgroup use cases. Three new parameters are added: - type - allows specifying program type, independent of how code sections are

[PATCH bpf-next v2 02/12] selftests/bpf: add Error: prefix in check_extack helper

2018-07-09 Thread Jakub Kicinski
Currently the test only checks errors, not warnings, so save typing and prefix the extack messages with "Error:" inside the check helper. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/testing/selftests/bpf/test_offload.py | 8 1 file changed, 4 insertions(+), 4 de

[PATCH net-next] tcp: expose both send and receive intervals for rate sample

2018-07-09 Thread Deepti Raghavan
Congestion control algorithms, which access the rate sample through the tcp_cong_control function, only have access to the maximum of the send and receive interval, for cases where the acknowledgment rate may be inaccurate due to ACK compression or decimation. Algorithms may want to use send rates

Re: [PATCH v3 iproute2 2/3] tc: Add support for the ETF Qdisc

2018-07-09 Thread David Ahern
On 7/9/18 9:48 AM, Jesus Sanchez-Palencia wrote: > Hi David, > > > On 07/06/2018 08:58 AM, David Ahern wrote: >> On 7/5/18 4:42 PM, Jesus Sanchez-Palencia wrote: >> >>> +static int get_clockid(__s32 *val, const char *arg) >>> +{ >>> + const struct static_clockid { >>> + const char *na

Re: [PATCH] net: sched: Fix warnings from xchg() on RCU'd cookie pointer.

2018-07-09 Thread Vlad Buslov
On Mon 09 Jul 2018 at 15:30, Marcelo Ricardo Leitner wrote: > On Sun, Jul 08, 2018 at 05:03:58PM +0900, David Miller wrote: >> >> The kbuild test robot reports: >> >> >> net/sched/act_api.c:71:15: sparse: incorrect type in initializer >> >> (different address spaces) @@expected struct tc

[PATCH net-next] net: sched: fix unprotected access to rcu cookie pointer

2018-07-09 Thread Vlad Buslov
Fix action attribute size calculation function to take rcu read lock and access act_cookie pointer with rcu dereference. Fixes: eec94fdb0480 ("net: sched: use rcu for action cookie update") Reported-by: Marcelo Ricardo Leitner Signed-off-by: Vlad Buslov --- net/sched/act_api.c | 9 +++-- 1

Re: [PATCH net v2 4/5] net/ipv6: propagate net.ipv6.conf.all.addr_gen_mode to devices

2018-07-09 Thread David Ahern
On 7/9/18 4:25 AM, Sabrina Dubroca wrote: > This aligns the addr_gen_mode sysctl with the expected behavior of the > "all" variant. > > Fixes: d35a00b8e33d ("net/ipv6: allow sysctl to change link-local address > generation mode") > Suggested-by: David Ahern > Signed-off-by: Sabrina Dubroca > --

Re: [PATCH net v2 5/5] Documentation: ip-sysctl.txt: document addr_gen_mode

2018-07-09 Thread David Ahern
On 7/9/18 4:25 AM, Sabrina Dubroca wrote: > addr_gen_mode was introduced in without documentation, add it now. > > Fixes: d35a00b8e33d ("net/ipv6: allow sysctl to change link-local address > generation mode") > Signed-off-by: Sabrina Dubroca > --- > Documentation/networking/ip-sysctl.txt | 9 ++

Re: [PATCH net v2 1/5] net/ipv6: fix addrconf_sysctl_addr_gen_mode

2018-07-09 Thread David Ahern
On 7/9/18 4:25 AM, Sabrina Dubroca wrote: > addrconf_sysctl_addr_gen_mode() has multiple problems. First, it ignores > the errors returned by proc_dointvec(). > > addrconf_sysctl_addr_gen_mode() calls proc_dointvec() directly, which > writes the value to memory, and then checks if it's valid and m

[PATCH v3 net-next 3/3] net: core: fix use-after-free in __netif_receive_skb_list_core

2018-07-09 Thread Edward Cree
__netif_receive_skb_core can free the skb, so we have to use the dequeue- enqueue model when calling it from __netif_receive_skb_list_core. Fixes: 88eb1944e18c ("net: core: propagate SKB lists through packet_type lookup") Signed-off-by: Edward Cree --- net/core/dev.c | 9 +++-- 1 file chan

[PATCH v3 net-next 2/3] netfilter: fix use-after-free in NF_HOOK_LIST

2018-07-09 Thread Edward Cree
nf_hook() can free the skb, so we need to remove it from the list before calling, and add passed skbs to a sublist afterwards. Fixes: 17266ee93984 ("net: ipv4: listified version of ip_rcv") Reported-by: Dan Carpenter Signed-off-by: Edward Cree --- include/linux/netfilter.h | 10 +++--- 1 f

[PATCH v3 net-next 1/3] net: core: fix uses-after-free in list processing

2018-07-09 Thread Edward Cree
In netif_receive_skb_list_internal(), all of skb_defer_rx_timestamp(), do_xdp_generic() and enqueue_to_backlog() can lead to kfree(skb). Thus, we cannot wait until after they return to remove the skb from the list; instead, we remove it first and, in the pass case, add it to a sublist afterwar

[PATCH v3 net-next 0/3] fix use-after-free bugs in skb list processing

2018-07-09 Thread Edward Cree
A couple of bugs in skb list handling were spotted by Dan Carpenter, with the help of Smatch; following up on them I found a couple more similar cases. This series fixes them by changing the relevant loops to use the dequeue-enqueue model (rather than in-place list modification). v3: fixed ano

Re: [PATCH net-next] tcp: expose both send and receive intervals for rate sample

2018-07-09 Thread Eric Dumazet
On 07/09/2018 09:23 AM, Yuchung Cheng wrote: > On Mon, Jul 9, 2018 at 9:05 AM, Deepti Raghavan wrote: >> Congestion control algorithms, which access the rate sample >> through the tcp_cong_control function, only have access to the maximum >> of the send and receive interval, for cases where the

[PATCH iproute2] tc: don't double print rate

2018-07-09 Thread Stephen Hemminger
Conversion to print stats in JSON forgot to remove existing fprintf. Fixes: 4fcec7f3665b ("tc: jsonify stats2") Signed-off-by: Stephen Hemminger --- tc/tc_util.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/tc/tc_util.c b/tc/tc_util.c index e0c96291ade0..d7578528a31b 100644 --- a/tc/tc_u

Re: [PATCH net-next] qed: Add srq core support for RoCE and iWARP

2018-07-09 Thread Jason Gunthorpe
On Wed, May 30, 2018 at 04:11:37PM +0300, Yuval Bason wrote: > This patch adds support for configuring SRQ and provides the necessary > APIs for rdma upper layer driver (qedr) to enable the SRQ feature. > > Signed-off-by: Michal Kalderon > Signed-off-by: Ariel Elior > Signed-off-by: Yuval Bason

Re: [PATCH net-next v3 0/2] tcp: fix high tail latencies in DCTCP

2018-07-09 Thread Yuchung Cheng
On Sat, Jul 7, 2018 at 7:07 AM, Neal Cardwell wrote: > On Sat, Jul 7, 2018 at 7:15 AM David Miller wrote: >> >> From: Lawrence Brakmo >> Date: Tue, 3 Jul 2018 09:26:13 -0700 >> >> > When have observed high tail latencies when using DCTCP for RPCs as >> > compared to using Cubic. For example, in

[jkirsher/next-queue PATCH v2 0/7] Add support for L2 Fwd Offload w/o ndo_select_queue

2018-07-09 Thread Alexander Duyck
This patch series is meant to allow support for the L2 forward offload, aka MACVLAN offload without the need for using ndo_select_queue. The existing solution currently requires that we use ndo_select_queue in the transmit path if we want to associate specific Tx queues with a given MACVLAN interf

[jkirsher/next-queue PATCH v2 5/7] net: Add generic ndo_select_queue functions

2018-07-09 Thread Alexander Duyck
This patch adds a generic version of the ndo_select_queue functions for either returning 0 or selecting a queue based on the processor ID. This is generally meant to just reduce the number of functions we have to change in the future when we have to deal with ndo_select_queue changes. Signed-off-b

RE: [EXT] [PATCH net-next] net: mvpp2: explicitly include linux/interrupt.h

2018-07-09 Thread Yan Markman
+10 Yan Markman Tel. 05-44732819 -Original Message- From: Antoine Tenart [mailto:antoine.ten...@bootlin.com] Sent: Monday, July 09, 2018 6:01 PM To: da...@davemloft.net Cc: Antoine Tenart ; netdev@vger.kernel.org; linux-ker...@vger.kernel.org; thomas.petazz...@bootlin.com; maxime.chev

[jkirsher/next-queue PATCH v2 6/7] net: allow ndo_select_queue to pass netdev

2018-07-09 Thread Alexander Duyck
This patch makes it so that instead of passing a void pointer as the accel_priv we instead pass a net_device pointer as sb_dev. Making this change allows us to pass the subordinate device through to the fallback function eventually so that we can keep the actual code in the ndo_select_queue call as

[jkirsher/next-queue PATCH v2 3/7] ixgbe: Add code to populate and use macvlan tc to Tx queue map

2018-07-09 Thread Alexander Duyck
This patch makes it so that we use the tc_to_txq mapping in the macvlan device in order to select the Tx queue for outgoing packets. The idea here is to try and move away from using ixgbe_select_queue and to come up with a generic way to make this work for devices going forward. By encoding this i

[jkirsher/next-queue PATCH v2 7/7] net: allow fallback function to pass netdev

2018-07-09 Thread Alexander Duyck
For most of these calls we can just pass NULL through to the fallback function as the sb_dev. The only cases where we cannot are the cases where we might be dealing with either an upper device or a driver that would have configured things to support an sb_dev itself. The only driver that has any s

[jkirsher/next-queue PATCH v2 4/7] net: Add support for subordinate traffic classes to netdev_pick_tx

2018-07-09 Thread Alexander Duyck
This change makes it so that we can support the concept of subordinate device traffic classes to the core networking code. In doing this we can start pulling out the driver specific bits needed to support selecting a queue based on an upper device. The solution at is currently stands is only parti

[jkirsher/next-queue PATCH v2 1/7] net-sysfs: Drop support for XPS and traffic_class on single queue device

2018-07-09 Thread Alexander Duyck
This patch makes it so that we do not report the traffic class or allow XPS configuration on single queue devices. This is mostly to avoid unnecessary complexity with changes I have planned that will allow us to reuse the unused tc_to_txq and XPS configuration on a single queue device to allow it t

[jkirsher/next-queue PATCH v2 2/7] net: Add support for subordinate device traffic classes

2018-07-09 Thread Alexander Duyck
This patch is meant to provide the basic tools needed to allow us to create subordinate device traffic classes. The general idea here is to allow subdividing the queues of a device into queue groups accessible through an upper device such as a macvlan. The idea here is to enforce the idea that an

Hello Dear

2018-07-09 Thread Smadar Barber-Tsadik
My name is Smadar Barber-Tsadik, I'm the Chief Executive Officer (C.P.A) of the First International Bank of Israel (FIBI). I'm getting in touch with you in regards to a very important and urgent matter. Kindly respond back at your earliest convinience so I can provide you the details. Faithfull

Re: [PATCH net-next] tcp: expose both send and receive intervals for rate sample

2018-07-09 Thread Yuchung Cheng
On Mon, Jul 9, 2018 at 9:05 AM, Deepti Raghavan wrote: > Congestion control algorithms, which access the rate sample > through the tcp_cong_control function, only have access to the maximum > of the send and receive interval, for cases where the acknowledgment > rate may be inaccurate due to ACK c

Re: [PATCH net] net: diag: Don't double-free TCP_NEW_SYN_RECV sockets in tcp_abort

2018-07-09 Thread Eric Dumazet
On 07/09/2018 09:14 AM, David Ahern wrote: > Perhaps it is something with my config, settings, ss version or test > program, but I do not see it on 4.11: > Maybe some vrf issue, I dunno.

  1   2   >