Re: [PATCH net v2] net: sched: cls_flower: validate nested enc_opts_policy to avoid warning

2018-11-09 Thread Jiri Pirko
Sat, Nov 10, 2018 at 06:06:26AM CET, jakub.kicin...@netronome.com wrote: >TCA_FLOWER_KEY_ENC_OPTS and TCA_FLOWER_KEY_ENC_OPTS_MASK can only >currently contain further nested attributes, which are parsed by >hand, so the policy is never actually used resulting in a W=1 >build warning: > >net/sched/c

Re: [net-next PATCH v3] net: sched: cls_flower: Classify packets using port ranges

2018-11-09 Thread Jiri Pirko
Sat, Nov 10, 2018 at 01:11:10AM CET, amritha.namb...@intel.com wrote: [...] >@@ -1026,8 +1122,7 @@ static void fl_init_dissector(struct flow_dissector >*dissector, >FLOW_DISSECTOR_KEY_IPV4_ADDRS, ipv4); > FL_KEY_SET_IF_MASKED(mask, keys, cnt, >

Re: [PATCH net-next] udp6: cleanup stats accounting in recvmsg()

2018-11-09 Thread David Miller
From: Paolo Abeni Date: Fri, 9 Nov 2018 15:52:45 +0100 > In the udp6 code path, we needed multiple tests to select the correct > mib to be updated. Since we touch at least a counter at each iteration, > it's convenient to use the recently introduced __UDPX_MIB() helper once > and remove some cod

Re: [PATCH net-next v2 0/2] dpaa2-eth: defer probe on object allocate

2018-11-09 Thread David Miller
From: Ioana Ciornei Date: Fri, 9 Nov 2018 15:26:45 + > Allocatable objects on the fsl-mc bus may be probed by the fsl_mc_allocator > after the first attempts of other drivers to use them. Defer the probe when > this situation happens. > > Changes in v2: > - proper handling of IS_ERR_OR_NUL

Re: [PATCH net] flow_dissector: do not dissect l4 ports for fragments

2018-11-09 Thread David Miller
From: Eric Dumazet Date: Fri, 9 Nov 2018 16:53:06 -0800 > From: 배석진 > > Only first fragment has the sport/dport information, > not the following ones. > > If we want consistent hash for all fragments, we need to > ignore ports even for first fragment. > > This bug is visible for IPv6 traffic

Re: [net-next PATCH v2] net: sched: cls_flower: Classify packets using port ranges

2018-11-09 Thread Nambiar, Amritha
On 11/9/2018 4:10 AM, Jiri Pirko wrote: > Wed, Nov 07, 2018 at 10:22:42PM CET, amritha.namb...@intel.com wrote: >> Added support in tc flower for filtering based on port ranges. >> >> Example: >> 1. Match on a port range: >> - >> $ tc filter add dev enp4s0 protocol ip parent

Re: [net-next PATCH v2] net: sched: cls_flower: Classify packets using port ranges

2018-11-09 Thread Nambiar, Amritha
On 11/8/2018 3:15 PM, David Miller wrote: > From: Amritha Nambiar > Date: Wed, 07 Nov 2018 13:22:42 -0800 > >> diff --git a/include/uapi/linux/pkt_cls.h b/include/uapi/linux/pkt_cls.h >> index 401d0c1..b63c3cf 100644 >> --- a/include/uapi/linux/pkt_cls.h >> +++ b/include/uapi/linux/pkt_cls.h >> @

Re: [PATCH net] net: sched: cls_flower: validate nested enc_opts_policy to avoid build warning

2018-11-09 Thread Jakub Kicinski
On Fri, 09 Nov 2018 20:40:25 -0800 (PST), David Miller wrote: > From: Jakub Kicinski > Date: Fri, 9 Nov 2018 14:41:22 -0800 > > > TCA_FLOWER_KEY_ENC_OPTS and TCA_FLOWER_KEY_ENC_OPTS_MASK can only > > currently contain further nested attributes, which are parsed by > > hand, so the policy is neve

Re: [PATCH net] net: sched: cls_flower: validate nested enc_opts_policy to avoid build warning

2018-11-09 Thread David Miller
From: Jakub Kicinski Date: Fri, 9 Nov 2018 14:41:22 -0800 > TCA_FLOWER_KEY_ENC_OPTS and TCA_FLOWER_KEY_ENC_OPTS_MASK can only > currently contain further nested attributes, which are parsed by > hand, so the policy is never actually used. Add the validation > anyway to avoid potential bugs when

[PATCH net-next 6/6] nfp: flower: remove unnecessary code in flow lookup

2018-11-09 Thread Jakub Kicinski
From: John Hurley Recent changes to NFP mean that stats updates from fw to driver no longer require a flow lookup and (because egdev offload has been removed) the ingress netdev for a lookup is now always known. Remove obsolete code in a flow lookup that matches on host context and that allows f

[PATCH net-next 0/6] net: sched: indirect tc block cb registration

2018-11-09 Thread Jakub Kicinski
John says: This patchset introduces an alternative to egdev offload by allowing a driver to register for block updates when an external device (e.g. tunnel netdev) is bound to a TC block. Drivers can track new netdevs or register to existing ones to receive information on such events. Based on thi

[PATCH net-next 4/6] nfp: flower: offload tunnel decap rules via indirect TC blocks

2018-11-09 Thread Jakub Kicinski
From: John Hurley Previously, TC block tunnel decap rules were only offloaded when a callback was triggered through registration of the rules egress device. This meant that the driver had no access to the ingress netdev and so could not verify it was the same tunnel type that the rule implied. R

[PATCH net-next 5/6] nfp: flower: remove TC egdev offloads

2018-11-09 Thread Jakub Kicinski
From: John Hurley Previously, only tunnel decap rules required egdev registration for offload in NFP. These are now supported via indirect TC block callbacks. Remove the egdev code from NFP. Signed-off-by: John Hurley Reviewed-by: Jakub Kicinski --- .../net/ethernet/netronome/nfp/flower/main

[net-next PATCH v3] net: sched: cls_flower: Classify packets using port ranges

2018-11-09 Thread Amritha Nambiar
Added support in tc flower for filtering based on port ranges. Example: 1. Match on a port range: - $ tc filter add dev enp4s0 protocol ip parent :\ prio 1 flower ip_proto tcp dst_port range 20-30 skip_hw\ action drop $ tc -s filter show dev enp4s0 parent : fil

Re: [iproute2 PATCH v2] tc: flower: Classify packets based port ranges

2018-11-09 Thread Nambiar, Amritha
On 11/9/2018 12:51 AM, Jiri Pirko wrote: > Wed, Nov 07, 2018 at 10:22:50PM CET, amritha.namb...@intel.com wrote: >> Added support for filtering based on port ranges. >> >> Example: >> 1. Match on a port range: >> - >> $ tc filter add dev enp4s0 protocol ip parent :\ >>

[PATCH] infiniband: nes: Fix more direct skb list accesses.

2018-11-09 Thread David Miller
The following: skb = skb->next; ... if (skb == (struct sk_buff *)queue) is transformed into: skb = skb_peek_next(skb, queue); ... if (!skb) Signed-off-by: David S. Miller --- drivers/infiniband/hw/nes/nes_mgt.c | 4 ++-- 1 file changed, 2 inse

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread David Miller
From: Cong Wang Date: Fri, 9 Nov 2018 11:43:33 -0800 > diff --git a/net/core/datagram.c b/net/core/datagram.c > index 57f3a6fcfc1e..d8f4d55cd6c5 100644 > --- a/net/core/datagram.c > +++ b/net/core/datagram.c > @@ -736,7 +736,7 @@ __sum16 __skb_checksum_complete_head(struct sk_buff *skb, > int l

[PATCH net v2] net: sched: cls_flower: validate nested enc_opts_policy to avoid warning

2018-11-09 Thread Jakub Kicinski
TCA_FLOWER_KEY_ENC_OPTS and TCA_FLOWER_KEY_ENC_OPTS_MASK can only currently contain further nested attributes, which are parsed by hand, so the policy is never actually used resulting in a W=1 build warning: net/sched/cls_flower.c:492:1: warning: ‘enc_opts_policy’ defined but not used [-Wunused-c

Re: [PATCH v2 net-next] net: phy: improve struct phy_device member interrupts handling

2018-11-09 Thread David Miller
From: Heiner Kallweit Date: Fri, 9 Nov 2018 18:35:52 +0100 > As a heritage from the very early days of phylib member interrupts is > defined as u32 even though it's just a flag whether interrupts are > enabled. So we can change it to a bitfield member. In addition change > the code dealing with t

[PATCH net-next 1/6] net: sched: register callbacks for indirect tc block binds

2018-11-09 Thread Jakub Kicinski
From: John Hurley Currently drivers can register to receive TC block bind/unbind callbacks by implementing the setup_tc ndo in any of their given netdevs. However, drivers may also be interested in binds to higher level devices (e.g. tunnel drivers) to potentially offload filters applied to them.

[PATCH net-next 2/6] nfp: flower: allow non repr netdev offload

2018-11-09 Thread Jakub Kicinski
From: John Hurley Previously the offload functions in NFP assumed that the ingress (or egress) netdev passed to them was an nfp repr. Modify the driver to permit the passing of non repr netdevs as the ingress device for an offload rule candidate. This may include devices such as tunnels. The dri

[PATCH net-next 3/6] nfp: flower: increase scope of netdev checking functions

2018-11-09 Thread Jakub Kicinski
From: John Hurley Both the actions and tunnel_conf files contain local functions that check the type of an input netdev. In preparation for re-use with tunnel offload via indirect blocks, move these to static inline functions in a header file. Signed-off-by: John Hurley Reviewed-by: Jakub Kicin

Re: [PATCH net-next] nfp: use the new __netdev_tx_sent_queue() BQL optimisation

2018-11-09 Thread David Miller
From: Jakub Kicinski Date: Fri, 9 Nov 2018 18:50:00 -0800 > __netdev_tx_sent_queue() was added in commit e59020abf0f > ("net: bql: add __netdev_tx_sent_queue()") and allows for > better GSO performance. > > Signed-off-by: Jakub Kicinski > Reviewed-by: Dirk van der Merwe > Reviewed-by: Simon H

Re: [PATCH net] net: qualcomm: rmnet: Fix incorrect assignment of real_dev

2018-11-09 Thread David Miller
From: Subash Abhinov Kasiviswanathan Date: Fri, 9 Nov 2018 18:56:27 -0700 > A null dereference was observed when a sysctl was being set > from userspace and rmnet was stuck trying to complete some actions > in the NETDEV_REGISTER callback. This is because the real_dev is set > only after the dev

Re: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread David Miller
From: Richard Cochran Date: Fri, 9 Nov 2018 17:44:31 -0800 > On Fri, Nov 09, 2018 at 03:28:46PM -0800, David Miller wrote: >> This series looks good to me but I want to give Richard an opportunity to >> review it first. > > The series is good to go. > > Acked-by: Richard Cochran Great, series

[PATCH net-next] nfp: use the new __netdev_tx_sent_queue() BQL optimisation

2018-11-09 Thread Jakub Kicinski
__netdev_tx_sent_queue() was added in commit e59020abf0f ("net: bql: add __netdev_tx_sent_queue()") and allows for better GSO performance. Signed-off-by: Jakub Kicinski Reviewed-by: Dirk van der Merwe Reviewed-by: Simon Horman --- drivers/net/ethernet/netronome/nfp/nfp_net_common.c | 4 +--- 1

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Yunsheng Lin
On 2018/11/10 10:09, Cong Wang wrote: > On Fri, Nov 9, 2018 at 6:02 PM Yunsheng Lin wrote: >> >> On 2018/11/10 9:42, Cong Wang wrote: >>> On Fri, Nov 9, 2018 at 5:39 PM Yunsheng Lin wrote: On 2018/11/10 3:43, Cong Wang wrote: > Currently netdev_rx_csum_fault() only shows a device na

Re: [PATCH] add an initial version of snmp_counter.rst

2018-11-09 Thread Cong Wang
(Cc Randy) On Fri, Nov 9, 2018 at 10:13 AM yupeng wrote: > > The snmp_counter.rst run a set of simple experiments, explains the > meaning of snmp counters depend on the experiments' results. This is > an initial version, only covers a small part of the snmp counters. I don't look into much deta

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Cong Wang
On Fri, Nov 9, 2018 at 6:02 PM Yunsheng Lin wrote: > > On 2018/11/10 9:42, Cong Wang wrote: > > On Fri, Nov 9, 2018 at 5:39 PM Yunsheng Lin wrote: > >> > >> On 2018/11/10 3:43, Cong Wang wrote: > >>> Currently netdev_rx_csum_fault() only shows a device name, > >>> we need more information about t

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Yunsheng Lin
On 2018/11/10 9:42, Cong Wang wrote: > On Fri, Nov 9, 2018 at 5:39 PM Yunsheng Lin wrote: >> >> On 2018/11/10 3:43, Cong Wang wrote: >>> Currently netdev_rx_csum_fault() only shows a device name, >>> we need more information about the skb for debugging. >>> >>> Sample output: >>> >>> ens3: hw csu

Re: [PATCH v4] Wait for running BPF programs when updating map-in-map

2018-11-09 Thread Chenbo Feng
Hi netdev, Could we queue up this patch to stable 4.14 and stable 4.19? I can provide a backport patch if needed. I checked it is a clean cherry-pick for 4.19 but have some minor conflict for 4.14. Thanks Chenbo Feng On Thu, Oct 18, 2018 at 4:36 PM Joel Fernandes wrote: > > On Thu, Oct 18, 2018

[PATCH net] net: qualcomm: rmnet: Fix incorrect assignment of real_dev

2018-11-09 Thread Subash Abhinov Kasiviswanathan
A null dereference was observed when a sysctl was being set from userspace and rmnet was stuck trying to complete some actions in the NETDEV_REGISTER callback. This is because the real_dev is set only after the device registration handler completes. sysctl call stack - <6> Unable to handle kernel

Re: [PATCH net-next 5/8] e1000e: extend PTP gettime function to read system clock

2018-11-09 Thread Jeff Kirsher
On Fri, 2018-11-09 at 11:14 +0100, Miroslav Lichvar wrote: > This adds support for the PTP_SYS_OFFSET_EXTENDED ioctl. > > Cc: Richard Cochran > Cc: Jacob Keller > Cc: Jeff Kirsher > Signed-off-by: Miroslav Lichvar > --- > drivers/net/ethernet/intel/e1000e/e1000.h | 3 ++ > drivers/net/ether

Re: [PATCH net-next 6/8] igb: extend PTP gettime function to read system clock

2018-11-09 Thread Jeff Kirsher
On Fri, 2018-11-09 at 11:14 +0100, Miroslav Lichvar wrote: > This adds support for the PTP_SYS_OFFSET_EXTENDED ioctl. > > Cc: Richard Cochran > Cc: Jacob Keller > Cc: Jeff Kirsher > Signed-off-by: Miroslav Lichvar > --- > drivers/net/ethernet/intel/igb/igb_ptp.c | 65

Re: [PATCH net-next 7/8] ixgbe: extend PTP gettime function to read system clock

2018-11-09 Thread Jeff Kirsher
On Fri, 2018-11-09 at 11:14 +0100, Miroslav Lichvar wrote: > This adds support for the PTP_SYS_OFFSET_EXTENDED ioctl. > > Cc: Richard Cochran > Cc: Jacob Keller > Cc: Jeff Kirsher > Signed-off-by: Miroslav Lichvar > --- > drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c | 54

Re: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread Richard Cochran
On Fri, Nov 09, 2018 at 03:28:46PM -0800, David Miller wrote: > This series looks good to me but I want to give Richard an opportunity to > review it first. The series is good to go. Acked-by: Richard Cochran

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Cong Wang
On Fri, Nov 9, 2018 at 5:39 PM Yunsheng Lin wrote: > > On 2018/11/10 3:43, Cong Wang wrote: > > Currently netdev_rx_csum_fault() only shows a device name, > > we need more information about the skb for debugging. > > > > Sample output: > > > > ens3: hw csum failure > > dev features: 0x00

Re: [Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Yunsheng Lin
On 2018/11/10 3:43, Cong Wang wrote: > Currently netdev_rx_csum_fault() only shows a device name, > we need more information about the skb for debugging. > > Sample output: > > ens3: hw csum failure > dev features: 0x00014b89 > skb len=84 data_len=0 gso_size=0 gso_type=0 ip_summed=0 cs

Re: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread David Miller
From: Jeff Kirsher Date: Fri, 09 Nov 2018 15:33:10 -0800 > On Fri, 2018-11-09 at 15:28 -0800, David Miller wrote: >> From: Miroslav Lichvar >> Date: Fri, 9 Nov 2018 11:14:41 +0100 >> >> > RFC->v1: >> > - added new patches >> > - separated PHC timestamp from ptp_system_timestamp >> > - fixed me

[PATCH net] flow_dissector: do not dissect l4 ports for fragments

2018-11-09 Thread Eric Dumazet
From: 배석진 Only first fragment has the sport/dport information, not the following ones. If we want consistent hash for all fragments, we need to ignore ports even for first fragment. This bug is visible for IPv6 traffic, if incoming fragments do not have a flow label, since skb_get_hash() will g

Re: Kernel 4.19 network performance - forwarding/routing normal users traffic

2018-11-09 Thread David Ahern
On 11/9/18 9:21 AM, David Ahern wrote: >> Is there possible to add only counters from xdp for vlans ? >> This will help me in testing. > I will take a look today at adding counters that you can dump using > bpftool. It will be a temporary solution for this xdp program only. > Same tree, kernel-ta

[PATCH net-next 2/2] net: phy: realtek: use new PHYID matching macros

2018-11-09 Thread Heiner Kallweit
Use new macros for PHYID matching to avoid boilerplate code. Signed-off-by: Heiner Kallweit --- drivers/net/phy/realtek.c | 29 ++--- 1 file changed, 10 insertions(+), 19 deletions(-) diff --git a/drivers/net/phy/realtek.c b/drivers/net/phy/realtek.c index 0f8e5b1c9..c60

[PATCH net-next 1/2] net: phy: add macros for PHYID matching

2018-11-09 Thread Heiner Kallweit
Add macros for PHYID matching to be used in PHY driver configs. By using these macros some boilerplate code can be avoided. Signed-off-by: Heiner Kallweit --- include/linux/phy.h | 4 1 file changed, 4 insertions(+) diff --git a/include/linux/phy.h b/include/linux/phy.h index 17d1f6472..03

Re: [PATCH net 0/5] net: aquantia: 2018-11 bugfixes

2018-11-09 Thread David Miller
From: Igor Russkikh Date: Fri, 9 Nov 2018 11:53:54 + > The patchset fixes a number of bugs found in various areas after > driver validation. Series applied, thank you. Please, when you provide a Fixes: tag, do not separate it with the other Signed-off-by: and Acked-by: etc. tags with an emp

[PATCH net-next 0/2] net: phy: add macros for PHYID matching in PHY driver config

2018-11-09 Thread Heiner Kallweit
Add macros for PHYID matching to be used in PHY driver configs. By using these macros some boilerplate code can be avoided. Use them initially in the Realtek PHY drivers. Heiner Kallweit (2): net: phy: add macros for PHYID matching net: phy: realtek: use new PHYID matching macros drivers/ne

Re: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread Jeff Kirsher
On Fri, 2018-11-09 at 15:28 -0800, David Miller wrote: > From: Miroslav Lichvar > Date: Fri, 9 Nov 2018 11:14:41 +0100 > > > RFC->v1: > > - added new patches > > - separated PHC timestamp from ptp_system_timestamp > > - fixed memory leak in PTP_SYS_OFFSET_EXTENDED > > - changed PTP_SYS_OFFSET_EX

Re: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread David Miller
From: Miroslav Lichvar Date: Fri, 9 Nov 2018 11:14:41 +0100 > RFC->v1: > - added new patches > - separated PHC timestamp from ptp_system_timestamp > - fixed memory leak in PTP_SYS_OFFSET_EXTENDED > - changed PTP_SYS_OFFSET_EXTENDED to work with array of arrays > - fixed PTP_SYS_OFFSET_EXTENDED t

Re: [PATCH net 1/1] bnx2x: Assign unique DMAE channel number for FW DMAE transactions.

2018-11-09 Thread David Miller
From: Sudarsana Reddy Kalluru Date: Fri, 9 Nov 2018 02:10:43 -0800 > +/* Following is the DMAE channel number allocation for the clients. > + * MFW: OCBB/OCSD implementations use DMAE channels 14/15 respectively. > + * Driver: 0-3 and 8-11 (for PF dmae operations) > + * 4 and 12 (fo

Re: [PATCH net-next] cxgb4vf: fix memleak in mac_hlist initialization

2018-11-09 Thread David Miller
From: Arjun Vynipadath Date: Fri, 9 Nov 2018 14:52:01 +0530 > mac_hlist was initialized during adapter_up, which will be called > every time a vf device is first brought up, or every time when device > is brought up again after bringing all devices down. This means our > state of previous list i

Re: [PATCH net-next] cxgb4vf: free mac_hlist properly

2018-11-09 Thread David Miller
From: Arjun Vynipadath Date: Fri, 9 Nov 2018 14:52:53 +0530 > The locally maintained list for tracking hash mac table was > not freed during driver remove. > > Signed-off-by: Arjun Vynipadath > Signed-off-by: Ganesh Goudar Applied.

Re: [PATCH net-next] cxgb4: free mac_hlist properly

2018-11-09 Thread David Miller
From: Arjun Vynipadath Date: Fri, 9 Nov 2018 14:50:25 +0530 > The locally maintained list for tracking hash mac table was > not freed during driver remove. > > Signed-off-by: Arjun Vynipadath > Signed-off-by: Ganesh Goudar Applied.

Re: [PATCH][net-next] net: tcp: remove BUG_ON from tcp_v4_err

2018-11-09 Thread David Miller
From: Li RongQing Date: Fri, 9 Nov 2018 17:04:51 +0800 > if skb is NULL pointer, and the following access of skb's > skb_mstamp_ns will trigger panic, which is same as BUG_ON > > Signed-off-by: Li RongQing Applied.

[PATCH net] net: sched: cls_flower: validate nested enc_opts_policy to avoid build warning

2018-11-09 Thread Jakub Kicinski
TCA_FLOWER_KEY_ENC_OPTS and TCA_FLOWER_KEY_ENC_OPTS_MASK can only currently contain further nested attributes, which are parsed by hand, so the policy is never actually used. Add the validation anyway to avoid potential bugs when other attributes are added and to make the attribute structure sligh

Re: Kernel 4.19 network performance - forwarding/routing normal users traffic

2018-11-09 Thread Paweł Staszewski
W dniu 08.11.2018 o 20:12, Paweł Staszewski pisze: CPU load is lower than for connectx4 - but it looks like bandwidth limit is the same :) But also after reaching 60Gbit/60Gbit  bwm-ng v0.6.1 (probing every 1.000s), press 'h' for help   input: /proc/net/dev type: rate   - iface  

Re: [PATCH net-next 0/4] Remove VLAN_TAG_PRESENT from drivers

2018-11-09 Thread Shiraz Saleem
On Thu, Nov 08, 2018 at 06:44:46PM +0100, Michał Mirosław wrote: > This series removes VLAN_TAG_PRESENT use from network drivers in > preparation to removing its special meaning. > > Michał Mirosław (4): > i40iw: remove use of VLAN_TAG_PRESENT > cnic: remove use of VLAN_TAG_PRESENT > gianfar

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

2018-11-09 Thread Alexei Starovoitov
On 11/9/18 1:28 PM, Edward Cree wrote: > On 09/11/18 21:14, Alexei Starovoitov wrote: >> same link, but i cannot make it right now. >> have to extinguish few fires. >> may be at 2pm (unlikely) or 3pm (more likely) PST? > > Yep I can do either of those, just let me know which when you can. still s

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

2018-11-09 Thread Edward Cree
On 09/11/18 21:14, Alexei Starovoitov wrote: > same link, but i cannot make it right now. > have to extinguish few fires. > may be at 2pm (unlikely) or 3pm (more likely) PST? Yep I can do either of those, just let me know which when you can.

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

2018-11-09 Thread Alexei Starovoitov
On 11/9/18 12:00 PM, Edward Cree wrote: > On 09/11/18 04:35, Alexei Starovoitov wrote: >> On Thu, Nov 08, 2018 at 10:56:55PM +, Edward Cree wrote: >>>  think this question of maps should be discussed in tomorrow's >>>  call, since it is when we start having other kinds of instances >> turned

Re: [PATCH 08/20] octeontx2-af: Alloc and config NPC MCAM entry at a time

2018-11-09 Thread Arnd Bergmann
On Fri, Nov 9, 2018 at 6:13 PM Sunil Kovvuri wrote: > On Fri, Nov 9, 2018 at 4:32 PM Arnd Bergmann wrote: > > On Fri, Nov 9, 2018 at 5:21 AM Sunil Kovvuri > > wrote: > > > > Since b is aligned to four bytes, you get padding between a and b. > > On top of that, you also get padding after c to m

Re: [PATCH net-next v2 1/2] dpaa2-eth: defer probe on object allocate

2018-11-09 Thread Andrew Lunn
On Fri, Nov 09, 2018 at 03:26:45PM +, Ioana Ciornei wrote: > The fsl_mc_object_allocate function can fail because not all allocatable > objects are probed by the fsl_mc_allocator at the call time. Defer the > dpaa2-eth probe when this happens. > > Signed-off-by: Ioana Ciornei > --- > Changes

Re: [PATCH v2 net-next] net: phy: improve struct phy_device member interrupts handling

2018-11-09 Thread Florian Fainelli
On 11/9/18 9:35 AM, Heiner Kallweit wrote: > As a heritage from the very early days of phylib member interrupts is > defined as u32 even though it's just a flag whether interrupts are > enabled. So we can change it to a bitfield member. In addition change > the code dealing with this member in a wa

Re: [PATCH v5 bpf-next 0/7] bpftool: support loading flow dissector

2018-11-09 Thread Jakub Kicinski
On Fri, 9 Nov 2018 08:21:39 -0800, Stanislav Fomichev wrote: > v5 changes: > * FILE -> PATH for load/loadall (can be either file or directory now) > * simpler implementation for __bpf_program__pin_name > * removed p_err for REQ_ARGS checks > * parse_atach_detach_args -> parse_attach_detach_args >

Re: [PATCH v2 net-next] net: phy: improve struct phy_device member interrupts handling

2018-11-09 Thread Andrew Lunn
On Fri, Nov 09, 2018 at 06:35:52PM +0100, Heiner Kallweit wrote: > As a heritage from the very early days of phylib member interrupts is > defined as u32 even though it's just a flag whether interrupts are > enabled. So we can change it to a bitfield member. In addition change > the code dealing wi

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

2018-11-09 Thread Edward Cree
On 09/11/18 04:35, Alexei Starovoitov wrote: > On Thu, Nov 08, 2018 at 10:56:55PM +, Edward Cree wrote: >>  think this question of maps should be discussed in tomorrow's >>  call, since it is when we start having other kinds of instances > turned out most of us have a conflict, so the earliest

Re: Kernel 4.19 network performance - forwarding/routing normal users traffic

2018-11-09 Thread Paweł Staszewski
W dniu 09.11.2018 o 17:21, David Ahern pisze: On 11/9/18 3:20 AM, Paweł Staszewski wrote: I just catch some weird behavior :) All was working fine for about 20k packets Then after xdp start to forward every 10 packets Interesting. Any counter showing drops? nothing that will fit NIC stati

Re: [PATCH iproute] ss: Actually print left delimiter for columns

2018-11-09 Thread Stefano Brivio
On Fri, 9 Nov 2018 09:05:46 -0800 Stephen Hemminger wrote: > On Mon, 29 Oct 2018 23:04:25 +0100 > Stefano Brivio wrote: > > > While rendering columns, we use a local variable to keep track of the > > field currently being printed, without touching current_field, which is > > used for buffering.

Re: [PATCH net] ip: hash fragments consistently

2018-11-09 Thread Eric Dumazet
On 07/23/2018 09:26 AM, Eric Dumazet wrote: > > > On 07/23/2018 07:50 AM, Paolo Abeni wrote: >> The skb hash for locally generated ip[v6] fragments belonging >> to the same datagram can vary in several circumstances: >> * for connected UDP[v6] sockets, the first fragment get its hash >> via

[Patch net-next] net: dump more useful information in netdev_rx_csum_fault()

2018-11-09 Thread Cong Wang
Currently netdev_rx_csum_fault() only shows a device name, we need more information about the skb for debugging. Sample output: ens3: hw csum failure dev features: 0x00014b89 skb len=84 data_len=0 gso_size=0 gso_type=0 ip_summed=0 csum=0, csum_complete_sw=0, csum_valid=0 Signed-off-b

Re: [PATCH bpf-next v2 2/3] bpf: Support socket lookup in CGROUP_SOCK_ADDR progs

2018-11-09 Thread Martin Lau
On Fri, Nov 09, 2018 at 10:54:01AM -0800, Andrey Ignatov wrote: > Make bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers > available in programs of type BPF_PROG_TYPE_CGROUP_SOCK_ADDR. > > Such programs operate on sockets and have access to socket and struct > sockaddr passed by user

[PATCH bpf-next v2 2/3] bpf: Support socket lookup in CGROUP_SOCK_ADDR progs

2018-11-09 Thread Andrey Ignatov
Make bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers available in programs of type BPF_PROG_TYPE_CGROUP_SOCK_ADDR. Such programs operate on sockets and have access to socket and struct sockaddr passed by user to system calls such as sys_bind, sys_connect, sys_sendmsg. It's useful

[PATCH bpf-next v2 3/3] selftest/bpf: Use bpf_sk_lookup_{tcp,udp} in test_sock_addr

2018-11-09 Thread Andrey Ignatov
Use bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers from test_sock_addr programs to make sure they're available and can lookup and release socket properly for IPv4/IPv4, TCP/UDP. Reading from a few fields of returned struct bpf_sock is also tested. Signed-off-by: Andrey Ignatov A

[PATCH bpf-next v2 1/3] bpf: Fix IPv6 dport byte order in bpf_sk_lookup_udp

2018-11-09 Thread Andrey Ignatov
Lookup functions in sk_lookup have different expectations about byte order of provided arguments. Specifically __inet_lookup, __udp4_lib_lookup and __udp6_lib_lookup expect dport to be in network byte order and do ntohs(dport) internally. At the same time __inet6_lookup expects dport to be in hos

[PATCH bpf-next v2 0/3] bpf: Support socket lookup in CGROUP_SOCK_ADDR progs

2018-11-09 Thread Andrey Ignatov
This patch set makes bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers available in programs of type BPF_PROG_TYPE_CGROUP_SOCK_ADDR. Patch 1 is a fix for bpf_sk_lookup_udp that was already merged to bpf (stable) tree. Here it's prerequisite for patch 3. Patch 2 is the main patch in

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

2018-11-09 Thread Martin Lau
On Thu, Nov 08, 2018 at 04:11:37PM +0100, 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 > --- > include/uapi/linux/bpf.h | 3 ++ > n

Re: bring back IPX and NCPFS, please!

2018-11-09 Thread Willy Tarreau
On Fri, Nov 09, 2018 at 06:30:14PM +0100, Johannes C. Schulz wrote: > Hello Willy, hello Stephen > > Thankyou for your reply. > But I'm not able to maintain or code these modules. I'm just a bloody > user/webdev. That's what we've all claimed before taking over something many years ago you know :

[PATCH bpf-next] selftests/bpf: Fix uninitialized duration warning

2018-11-09 Thread Joe Stringer
Daniel Borkmann reports: test_progs.c: In function ‘main’: test_progs.c:81:3: warning: ‘duration’ may be used uninitialized in this function [-Wmaybe-uninitialized] printf("%s:PASS:%s %d nsec\n", __func__, tag, duration);\ ^~ test_progs.c:1706:8: note: ‘duration’ was declared here __u

RE: [PATCH net-next 7/8] ixgbe: extend PTP gettime function to read system clock

2018-11-09 Thread Keller, Jacob E
> -Original Message- > From: Miroslav Lichvar [mailto:mlich...@redhat.com] > Sent: Friday, November 09, 2018 2:15 AM > To: netdev@vger.kernel.org > Cc: Richard Cochran ; Keller, Jacob E > ; Miroslav Lichvar ; Kirsher, > Jeffrey T > Subject: [PATCH net-next 7/8] ixgbe: extend PTP gettime fu

[PATCH] add an initial version of snmp_counter.rst

2018-11-09 Thread yupeng
The snmp_counter.rst run a set of simple experiments, explains the meaning of snmp counters depend on the experiments' results. This is an initial version, only covers a small part of the snmp counters. Signed-off-by: yupeng --- Documentation/networking/index.rst| 1 + Documentation/ne

RE: [PATCH net-next 0/8] More accurate PHC<->system clock synchronization

2018-11-09 Thread Keller, Jacob E
> -Original Message- > From: Miroslav Lichvar [mailto:mlich...@redhat.com] > Sent: Friday, November 09, 2018 2:15 AM > To: netdev@vger.kernel.org > Cc: Richard Cochran ; Keller, Jacob E > ; Miroslav Lichvar ; Marcelo > Tosatti ; Kirsher, Jeffrey T > ; > Michael Chan > Subject: [PATCH net-

[PATCH net-next 3/3] net: phy: improve and inline phy_change

2018-11-09 Thread Heiner Kallweit
Now that phy_mac_interrupt() doesn't call phy_change() any longer it's called from phy_interrupt() only. Therefore phy_interrupt_is_valid() returns true always and the check can be removed. In case of PHY_HALTED phy_interrupt() bails out immediately, therefore the second check for PHY_HALTED includ

[PATCH net-next 2/3] net: phy: simplify phy_mac_interrupt and related functions

2018-11-09 Thread Heiner Kallweit
When using phy_mac_interrupt() the irq number is set to PHY_IGNORE_INTERRUPT, therefore phy_interrupt_is_valid() returns false. As a result phy_change() effectively just calls phy_trigger_machine() when called from phy_mac_interrupt() via phy_change_work(). So we can call phy_trigger_machine() from

[PATCH net-next 1/3] net: phy: don't set state PHY_CHANGELINK in phy_change

2018-11-09 Thread Heiner Kallweit
State PHY_CHANGELINK isn't needed here, we can call the state machine directly. We just have to remove the check for phy_polling_mode() to make this work also in interrupt mode. Removing this check doesn't cause any overhead because when not polling the state machine is called only if required by s

[PATCH net-next 0/3] net: phy: further phylib simplifications after recent changes to the state machine

2018-11-09 Thread Heiner Kallweit
After the recent changes to the state machine phylib can be further simplified (w/o having to make any assumptions). Heiner Kallweit (3): net: phy: don't set state PHY_CHANGELINK in phy_change net: phy: simplify phy_mac_interrupt and related functions net: phy: improve and inline phy_change

Re: [PATCH v5 bpf-next 0/7] bpftool: support loading flow dissector

2018-11-09 Thread Quentin Monnet
2018-11-09 08:21 UTC-0800 ~ Stanislav Fomichev > v5 changes: > * FILE -> PATH for load/loadall (can be either file or directory now) > * simpler implementation for __bpf_program__pin_name > * removed p_err for REQ_ARGS checks > * parse_atach_detach_args -> parse_attach_detach_args > * for -> while

[PATCH v2 net-next] net: phy: improve struct phy_device member interrupts handling

2018-11-09 Thread Heiner Kallweit
As a heritage from the very early days of phylib member interrupts is defined as u32 even though it's just a flag whether interrupts are enabled. So we can change it to a bitfield member. In addition change the code dealing with this member in a way that it's clear we're dealing with a bool value.

Re: [PATCH bpf-next 2/4] bpf: Split bpf_sk_lookup

2018-11-09 Thread Andrey Ignatov
Martin Lau [Fri, 2018-11-09 09:19 -0800]: > On Thu, Nov 08, 2018 at 08:54:23AM -0800, Andrey Ignatov wrote: > > Split bpf_sk_lookup to separate core functionality, that can be reused > > to make socket lookup available to more program types, from > > functionality specific to program types that ha

Re: bring back IPX and NCPFS, please!

2018-11-09 Thread Johannes C. Schulz
Hello Willy, hello Stephen Thankyou for your reply. But I'm not able to maintain or code these modules. I'm just a bloody user/webdev. It would be really nice if these modules will find a good maintainer! Best regards Johannes Am Fr., 9. Nov. 2018 um 17:09 Uhr schrieb Willy Tarreau : > > On Fri,

Re: [PATCH bpf-next 4/4] selftest/bpf: Use bpf_sk_lookup_{tcp,udp} in test_sock_addr

2018-11-09 Thread Martin Lau
On Thu, Nov 08, 2018 at 08:54:25AM -0800, Andrey Ignatov wrote: > Use bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers from > test_sock_addr programs to make sure they're available and can lookup > and release socket properly for IPv4/IPv4, TCP/UDP. > > Reading from a few fields of

Re: [PATCH][RFC] udp: cache sock to avoid searching it twice

2018-11-09 Thread Eric Dumazet
On 11/08/2018 10:21 PM, Li RongQing wrote: > GRO for UDP needs to lookup socket twice, first is in gro receive, > second is gro complete, so if store sock to skb to avoid looking up > twice, this can give small performance boost > > netperf -t UDP_RR -l 10 > > Before: > Rate per sec: 287

Re: [PATCH bpf-next 3/4] bpf: Support socket lookup in CGROUP_SOCK_ADDR progs

2018-11-09 Thread Martin Lau
On Thu, Nov 08, 2018 at 08:54:24AM -0800, Andrey Ignatov wrote: > Make bpf_sk_lookup_tcp, bpf_sk_lookup_udp and bpf_sk_release helpers > available in programs of type BPF_PROG_TYPE_CGROUP_SOCK_ADDR. > > Such programs operate on sockets and have access to socket and struct > sockaddr passed by user

Re: [PATCH bpf-next 2/4] bpf: Split bpf_sk_lookup

2018-11-09 Thread Martin Lau
On Thu, Nov 08, 2018 at 08:54:23AM -0800, Andrey Ignatov wrote: > Split bpf_sk_lookup to separate core functionality, that can be reused > to make socket lookup available to more program types, from > functionality specific to program types that have access to skb. > > Core functionality is placed

Re: [PATCH 08/20] octeontx2-af: Alloc and config NPC MCAM entry at a time

2018-11-09 Thread Sunil Kovvuri
On Fri, Nov 9, 2018 at 4:32 PM Arnd Bergmann wrote: > > On Fri, Nov 9, 2018 at 5:21 AM Sunil Kovvuri wrote: > > > > On Fri, Nov 9, 2018 at 2:13 AM Arnd Bergmann wrote: > > > > > > On Thu, Nov 8, 2018 at 7:37 PM wrote: > > > > @@ -666,4 +668,20 @@ struct npc_mcam_unmap_counter_req { > > > >

Re: [PATCH][net-next] net: tcp: remove BUG_ON from tcp_v4_err

2018-11-09 Thread Eric Dumazet
On 11/09/2018 01:04 AM, Li RongQing wrote: > if skb is NULL pointer, and the following access of skb's > skb_mstamp_ns will trigger panic, which is same as BUG_ON > > Signed-off-by: Li RongQing > --- > net/ipv4/tcp_ipv4.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/net/ipv4/tcp_i

Re: [PATCH iproute] ss: Actually print left delimiter for columns

2018-11-09 Thread Stephen Hemminger
On Mon, 29 Oct 2018 23:04:25 +0100 Stefano Brivio wrote: > While rendering columns, we use a local variable to keep track of the > field currently being printed, without touching current_field, which is > used for buffering. > > Use the right pointer to access the left delimiter for the current

Re: [PATCH 11/20] octeontx2-af: Add support for stripping STAG/CTAG

2018-11-09 Thread Sunil Kovvuri
On Fri, Nov 9, 2018 at 4:42 PM Arnd Bergmann wrote: > > On Fri, Nov 9, 2018 at 5:29 AM Sunil Kovvuri wrote: > > On Fri, Nov 9, 2018 at 2:17 AM Arnd Bergmann wrote: > > > On Thu, Nov 8, 2018 at 7:37 PM wrote: > > > > > > > Here is another instance of bitfields in an interface structure. As > > >

Re: [PATCH mlx5-next 08/10] IB/mlx5: Call PAGE_FAULT_RESUME command asynchronously

2018-11-09 Thread Jason Gunthorpe
On Fri, Nov 09, 2018 at 06:26:22PM +0200, Leon Romanovsky wrote: > On Thu, Nov 08, 2018 at 07:49:03PM +, Jason Gunthorpe wrote: > > On Thu, Nov 08, 2018 at 09:10:15PM +0200, Leon Romanovsky wrote: > > > From: Moni Shoua > > > > > > Telling the HCA that page fault handling is done and QP can re

Re: [PATCH iproute2 net-next v2 0/2] Add DF configuration for VXLAN and GENEVE link types

2018-11-09 Thread David Ahern
On 11/8/18 4:21 AM, Stefano Brivio wrote: > This series adds configuration of the DF bit in outgoing IPv4 packets for > VXLAN and GENEVE link types. > > Stefano Brivio (2): > iplink_vxlan: Add DF configuration > iplink_geneve: Add DF configuration > > include/uapi/linux/if_link.h | 18 ++

Re: Should the bridge learn from frames with link local destination MAC address?

2018-11-09 Thread nikolay
On 9 November 2018 18:24:18 EET, Roopa Prabhu wrote: >On Fri, Nov 9, 2018 at 8:00 AM Stephen Hemminger > wrote: >> >> On Fri, 9 Nov 2018 04:24:43 +0100 >> Andrew Lunn wrote: >> >> > Hi Roopa, Nikolay >> > >> > br_handle_frame() looks out for frames with a destination MAC >> > addresses with is Et

[PATCH iproute2] testsuite: correctly use CC macros for generate_nlmsg

2018-11-09 Thread Luca Boccassi
It's $(QUIET_CC)$(CC) not $(QUIET_CC), copy-paste error. CI does verbose build so it slipped through. Fixes: 6e7d347aabbb ("testsuite: build generate_nlmsg with QUIET_CC") Signed-off-by: Luca Boccassi --- testsuite/tools/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH mlx5-next 08/10] IB/mlx5: Call PAGE_FAULT_RESUME command asynchronously

2018-11-09 Thread Leon Romanovsky
On Thu, Nov 08, 2018 at 07:49:03PM +, Jason Gunthorpe wrote: > On Thu, Nov 08, 2018 at 09:10:15PM +0200, Leon Romanovsky wrote: > > From: Moni Shoua > > > > Telling the HCA that page fault handling is done and QP can resume > > its flow is done in the context of the page fault handler. This bl

  1   2   >