Re: [PATCH v5 2/2] ip_tunnel: add mpls over gre encapsulation

2017-09-30 Thread kbuild test robot
Hi Amine, [auto build test ERROR on net/master] [also build test ERROR on v4.14-rc2 next-20170929] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH net-next 0/3] support changing steering policies in tuntap

2017-09-30 Thread Michael S. Tsirkin
On Thu, Sep 28, 2017 at 12:09:05PM -0400, Willem de Bruijn wrote: > Programming from the guest is > indeed different. I don't fully understand that use case. Generally programming host BPF from guest is a clear win - think DOS protection. Guest runs logic to detect dos attacks, then passes the

Re: [kbuild-all] [PATCH net-next] vhost_net: do not stall on zerocopy depletion

2017-09-30 Thread Fengguang Wu
On Sun, Oct 01, 2017 at 06:20:49AM +0300, Michael S. Tsirkin wrote: On Sun, Oct 01, 2017 at 08:09:30AM +0800, kbuild test robot wrote: Hi Willem, [auto build test WARNING on net-next/master] url:

[PATCH v2 iproute2] tc: fix ipv6 filter selector attribute for some prefix lengths

2017-09-30 Thread Yulia Kartseva
Wrong TCA_U32_SEL attribute packing if prefixLen AND 0x1f equals 0x1f. These are /31, /63, /95 and /127 prefix lengths. Example: ip6 dst face:b00f::/31 filter parent b: protocol ipv6 pref 2307 u32 filter parent b: protocol ipv6 pref 2307 u32 fh 800: ht divisor 1 filter parent b: protocol ipv6

Re: [PATCH net-next] vhost_net: do not stall on zerocopy depletion

2017-09-30 Thread Michael S. Tsirkin
On Sun, Oct 01, 2017 at 08:09:30AM +0800, kbuild test robot wrote: > Hi Willem, > > [auto build test WARNING on net-next/master] > > url: > https://github.com/0day-ci/linux/commits/Willem-de-Bruijn/vhost_net-do-not-stall-on-zerocopy-depletion/20171001-054709 > reproduce: > # apt-get

Re: [PATCH net-next v2 0/7] net: dsa: change dsa_ptr for a dsa_port

2017-09-30 Thread David Miller
From: Vivien Didelot Date: Fri, 29 Sep 2017 17:19:14 -0400 > With DSA, a master net_device is physically wired to a dedicated CPU > switch port. For interaction with the DSA layer, the struct net_device > contains a dsa_ptr, which currently points to a

Re: [PATCH] net: hns3: fix null pointer dereference before null check

2017-09-30 Thread David Miller
From: Colin King Date: Fri, 29 Sep 2017 20:51:23 +0100 > From: Colin Ian King > > pointer ndev is being dereferenced with the call to netif_running > before it is being null checked. Re-order the code to only dereference > ndev after it has

Re: [PATCH net-next] bpf: Fix compiler warning on info.map_ids for 32bit platform

2017-09-30 Thread David Miller
From: Martin KaFai Lau Date: Fri, 29 Sep 2017 10:52:17 -0700 > This patch uses u64_to_user_ptr() to cast info.map_ids to a userspace ptr. > It also tags the user_map_ids with '__user' for sparse check. > > Fixes: cb4d2b3f03d8 ("bpf: Add name, load_time, uid and map_ids to >

Re: [PATCH net-next] hv_netvsc: report stop_queue and wake_queue

2017-09-30 Thread David Miller
From: Simon Xiao Date: Fri, 29 Sep 2017 11:39:46 -0700 > Report the numbers of events for stop_queue and wake_queue in > ethtool stats. > > Example: > ethtool -S eth0 > NIC statistics: > ... > stop_queue: 7 > wake_queue: 7 > ... > > Signed-off-by:

Re: [PATCH][net-next] net_sched: remove redundant assignment to ret

2017-09-30 Thread David Miller
From: Colin King Date: Fri, 29 Sep 2017 15:01:16 +0100 > From: Colin Ian King > > The assignment of -EINVAL to variable ret is redundant as it > is being overwritten on the following error exit paths or > to the return value from the

Re: [PATCH][net-next] net: ipmr: make function ipmr_notifier_init static

2017-09-30 Thread David Miller
From: Colin King Date: Fri, 29 Sep 2017 14:34:22 +0100 > From: Colin Ian King > > The function ipmr_notifier_init is local to the source and does > not need to be in global scope, so make it static. > > Cleans up sparse warning: > warning:

Re: [PATCH net v1 1/1] tipc: use only positive error codes in messages

2017-09-30 Thread David Miller
From: Parthasarathy Bhuvaragan Date: Fri, 29 Sep 2017 10:02:54 +0200 > In commit e3a77561e7d32 ("tipc: split up function tipc_msg_eval()"), > we have updated the function tipc_msg_lookup_dest() to set the error > codes to negative values at destination

Re: [PATCH net-next] ibmvnic: Set state UP

2017-09-30 Thread David Miller
From: Mick Tarsel Date: Thu, 28 Sep 2017 13:53:18 -0700 > State is initially reported as UNKNOWN. Before register call > netif_carrier_off(). Once the device is opened, call netif_carrier_on() in > order to set the state to UP. > > Signed-off-by: Mick Tarsel

Re: [PATCH net-next] Revert "net: dsa: bcm_sf2: Defer port enabling to calling port_enable"

2017-09-30 Thread David Miller
From: Florian Fainelli Date: Thu, 28 Sep 2017 11:19:06 -0700 > This reverts commit e85ec74ace29 ("net: dsa: bcm_sf2: Defer port > enabling to calling port_enable") because this now makes an unbind > followed by a bind to fail connecting to the ingrated PHY. > > What this

Re: [PATCH net] ppp: fix __percpu annotation

2017-09-30 Thread David Miller
From: Guillaume Nault Date: Thu, 28 Sep 2017 17:57:58 +0200 > Move sparse annotation right after pointer type. > > Fixes sparse warning: > drivers/net/ppp/ppp_generic.c:1422:13: warning: incorrect type in > initializer (different address spaces) >

Re: [PATCH net 0/2] udp: fix early demux for mcast packets

2017-09-30 Thread David Miller
From: Paolo Abeni Date: Thu, 28 Sep 2017 15:51:35 +0200 > Currently the early demux callbacks do not perform source address validation. > This is not an issue for TCP or UDP unicast, where the early demux > is only allowed for connected sockets and the source address is

Re: [PATCH net] ip_gre: ipgre_tap device should keep dst

2017-09-30 Thread David Miller
From: Xin Long Date: Thu, 28 Sep 2017 13:23:31 +0800 > Without keeping dst, the tunnel will not update any mtu/pmtu info, > since it does not have a dst on the skb. > > Reproducer: > client(ipgre_tap1 - eth1) <-> (eth1 - ipgre_tap1)server > > After reducing eth1's

Re: [PATCH net] ip6_tunnel: update mtu properly for ARPHRD_ETHER tunnel device in tx path

2017-09-30 Thread David Miller
From: Xin Long Date: Thu, 28 Sep 2017 13:24:07 +0800 > Now when updating mtu in tx path, it doesn't consider ARPHRD_ETHER tunnel > device, like ip6gre_tap tunnel, for which it should also subtract ether > header to get the correct mtu. > > Signed-off-by: Xin Long

Re: [PATCH net] ip6_gre: ip6gre_tap device should keep dst

2017-09-30 Thread David Miller
From: Xin Long Date: Thu, 28 Sep 2017 13:23:50 +0800 > The patch 'ip_gre: ipgre_tap device should keep dst' fixed > a issue that ipgre_tap mtu couldn't be updated in tx path. > > The same fix is needed for ip6gre_tap as well. > > Signed-off-by: Xin Long

Re: [net-next 00/15][pull request] 40GbE Intel Wired LAN Driver Updates 2017-09-29

2017-09-30 Thread David Miller
From: Jeff Kirsher Date: Fri, 29 Sep 2017 17:44:52 -0700 > This series contains updates to i40e and i40evf only. Pulled, thanks a lot Jeff.

Re: [next-queue PATCH v2 3/5] net/sched: Introduce the user API for the CBS shaper

2017-09-30 Thread Cong Wang
On Fri, Sep 29, 2017 at 5:26 PM, Vinicius Costa Gomes wrote: > Export the API necessary for configuring the CBS shaper (implemented > in the next patch) via the tc tool. This one can be folded into patch 4/5.

Re: [next-queue PATCH v2 2/5] net/sched: Fix accessing invalid dev_queue

2017-09-30 Thread Cong Wang
On Fri, Sep 29, 2017 at 5:26 PM, Vinicius Costa Gomes wrote: > From: Jesus Sanchez-Palencia > > In qdisc_alloc() the dev_queue pointer was used without any checks being > performed. If qdisc_create() gets a null dev_queue pointer, it

Re: [PATCH v4 2/2] ip_tunnel: add mpls over gre encapsulation

2017-09-30 Thread kbuild test robot
Hi Amine, [auto build test ERROR on net/master] [also build test ERROR on v4.14-rc2 next-20170929] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [net-next V2 PATCH 2/5] bpf: XDP_REDIRECT enable use of cpumap

2017-09-30 Thread kbuild test robot
Hi Jesper, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Jesper-Dangaard-Brouer/New-bpf-cpumap-type-for-XDP_REDIRECT/20171001-064716 config: i386-randconfig-i0-201740 (attached as .config) compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4 reproduce:

Re: [PATCH net-next] vhost_net: do not stall on zerocopy depletion

2017-09-30 Thread kbuild test robot
Hi Willem, [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Willem-de-Bruijn/vhost_net-do-not-stall-on-zerocopy-depletion/20171001-054709 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1

Dear Talented

2017-09-30 Thread Kim Sharma
Dear Talented, I am Talent Scout For BLUE SKY FILM STUDIO, Present Blue sky Studio a Film Corporation Located in the United State, is Soliciting for the Right to use Your Photo/Face and Personality as One of the Semi -Major Role/ Character in our Upcoming ANIMATED Stereoscope 3D Movie-The Story

Re: [PATCH v4 2/2] ip_tunnel: add mpls over gre encapsulation

2017-09-30 Thread kbuild test robot
Hi Amine, [auto build test ERROR on net/master] [also build test ERROR on v4.14-rc2 next-20170929] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH net-next] vhost_net: do not stall on zerocopy depletion

2017-09-30 Thread kbuild test robot
Hi Willem, [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Willem-de-Bruijn/vhost_net-do-not-stall-on-zerocopy-depletion/20171001-054709 config: tile-allyesconfig (attached as .config) compiler: tilegx-linux-gcc (GCC) 4.6.2 reproduce: wget

Re: [PATCH net-next] vhost_net: do not stall on zerocopy depletion

2017-09-30 Thread kbuild test robot
Hi Willem, [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Willem-de-Bruijn/vhost_net-do-not-stall-on-zerocopy-depletion/20171001-054709 config: x86_64-randconfig-x002-201740 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901

Re: [PATCH v3 net-next 1/8] flow_dissector: Change skbuf argument to be non const

2017-09-30 Thread kbuild test robot
Hi Tom, [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Tom-Herbert/flow_dissector-Change-skbuf-argument-to-be-non-const/20171001-052131 config: x86_64-randconfig-x010-201740 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901

Re: [PATCH v2] lib: fix multiple strlcpy definition

2017-09-30 Thread Baruch Siach
Hi Stephen, On Fri, Sep 29, 2017 at 11:58:58AM -0700, Stephen Hemminger wrote: > On Thu, 28 Sep 2017 21:02:11 +0300 > Baruch Siach wrote: > > > Some C libraries, like uClibc and musl, provide BSD compatible > > strlcpy(). Add check_strlcpy() to configure, and avoid defining

Re: [PATCH 2/4] ravb: Add optional PHY reset during system resume

2017-09-30 Thread Sergei Shtylyov
Hello! On 09/28/2017 10:21 PM, Florian Fainelli wrote: If the optional "reset-gpios" property is specified in DT, the generic MDIO bus code takes care of resetting the PHY during device probe. However, the PHY may still have to be reset explicitly after system resume. This allows to restore

Re: WARNING: kernel stack frame pointer at ffff880156a5fea0 in bash:2103 has bad value 00007ffec7d87e50

2017-09-30 Thread Richard Weinberger
Josh, Am Mittwoch, 27. September 2017, 16:14:30 CEST schrieb Josh Poimboeuf: > On Wed, Sep 27, 2017 at 08:51:22AM +0200, Richard Weinberger wrote: > > Am Mittwoch, 27. September 2017, 00:42:46 CEST schrieb Josh Poimboeuf: > > > > Here is another variant of the warning, it matches the attached

Re: [PATCH net-next] bpf: Fix compiler warning on info.map_ids for 32bit platform

2017-09-30 Thread Daniel Borkmann
On 09/29/2017 07:52 PM, Martin KaFai Lau wrote: This patch uses u64_to_user_ptr() to cast info.map_ids to a userspace ptr. It also tags the user_map_ids with '__user' for sparse check. Fixes: cb4d2b3f03d8 ("bpf: Add name, load_time, uid and map_ids to bpf_prog_info") Signed-off-by: Martin

Re: [PATCH net-next 00/10] Introduce SCTP Stream Schedulers

2017-09-30 Thread Xin Long
On Fri, Sep 29, 2017 at 4:25 AM, Marcelo Ricardo Leitner wrote: > This patchset introduces the SCTP Stream Schedulers are defined by > https://tools.ietf.org/html/draft-ietf-tsvwg-sctp-ndata-13 > > It provides 3 schedulers at the moment: FCFS, Priority and Round Robin.

Re: [PATCH v2] netlink: do not proceed if dump's start() errs

2017-09-30 Thread David Miller
From: Johannes Berg Date: Sat, 30 Sep 2017 08:56:10 +0200 > On Thu, 2017-09-28 at 00:41 +0200, Jason A. Donenfeld wrote: >> Drivers that use the start method for netlink dumping rely on dumpit >> not >> being called if start fails. For example, ila_xlat.c allocates

Re: [PATCHv4 iproute2 1/2] lib/libnetlink: re malloc buff if size is not enough

2017-09-30 Thread Hangbin Liu
Hi Stephen, On Fri, Sep 29, 2017 at 10:54:40AM -0700, Stephen Hemminger wrote: > > Doubling the number of system calls per message is not going to make > users with 5,000,000 routes or 1000 vlans, or 10,000 tunnels happy. > Please rethink this. I tried to add 2500 vlans and 70,000 routes. Then

Re: [PATCH v2 1/2] net: phonet: mark header_ops as const

2017-09-30 Thread RĂ©mi Denis-Courmont
On samedi 30 septembre 2017 17:39:36 EEST Lin Zhang wrote: > Signed-off-by: Lin Zhang > --- > net/phonet/af_phonet.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/phonet/af_phonet.c b/net/phonet/af_phonet.c > index f925753..b12142e 100644 >

[PATCH v2] netfilter: SYNPROXY: fix process non tcp packet bug in {ipv4,ipv6}_synproxy_hook

2017-09-30 Thread Lin Zhang
In function {ipv4,ipv6}_synproxy_hook we expect a normal tcp packet, but the real server maybe reply an icmp error packet related to the exist tcp conntrack, so we will access wrong tcp data. For fix it, check for the protocol field and only process tcp traffic. Signed-off-by: Lin Zhang

[PATCH] net: stmmac: dwmac-rk: Add RK3128 GMAC support

2017-09-30 Thread David Wu
Add constants and callback functions for the dwmac on rk3128 soc. As can be seen, the base structure is the same, only registers and the bits in them moved slightly. Signed-off-by: David Wu --- .../devicetree/bindings/net/rockchip-dwmac.txt | 1 +

[PATCH v2 2/2] net: phonet: mark phonet_protocol as const

2017-09-30 Thread Lin Zhang
The phonet_protocol structs don't need to be written by anyone and so can be marked as const. Signed-off-by: Lin Zhang --- changelog: v1 -> v2: * remove __read_mostly tag --- include/net/phonet/phonet.h | 6 -- net/phonet/af_phonet.c | 15

[PATCH v2 1/2] net: phonet: mark header_ops as const

2017-09-30 Thread Lin Zhang
Signed-off-by: Lin Zhang --- net/phonet/af_phonet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/phonet/af_phonet.c b/net/phonet/af_phonet.c index f925753..b12142e 100644 --- a/net/phonet/af_phonet.c +++ b/net/phonet/af_phonet.c @@ -149,7 +149,7

[PATCH 2/2] net: phonet: mark phonet_protocol as const

2017-09-30 Thread Lin Zhang
The phonet_protocol structs don't need to be written by anyone and so can be marked as const. Signed-off-by: Lin Zhang --- include/net/phonet/phonet.h | 6 -- net/phonet/af_phonet.c | 15 --- net/phonet/datagram.c | 2 +- net/phonet/pep.c

[PATCH 1/2] net: phonet: mark header_ops as const

2017-09-30 Thread Lin Zhang
Signed-off-by: Lin Zhang --- net/phonet/af_phonet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/phonet/af_phonet.c b/net/phonet/af_phonet.c index f925753..b12142e 100644 --- a/net/phonet/af_phonet.c +++ b/net/phonet/af_phonet.c @@ -149,7 +149,7

[PATCH net] net: fib_rules: Fix fib_rules_ops->compare implementations to support exact match

2017-09-30 Thread Shmulik Ladkani
From: Shmulik Ladkani Commit 153380ec4b9b ("fib_rules: Added NLM_F_EXCL support to fib_nl_newrule") added a check to 'fib_nl_newrule' that tests whether the suggested rule already exists (i.e. has same properties). The check uses fib_rules_ops->compare() method to

Re: [PATCH v2] netlink: do not proceed if dump's start() errs

2017-09-30 Thread Johannes Berg
On Thu, 2017-09-28 at 00:41 +0200, Jason A. Donenfeld wrote: > Drivers that use the start method for netlink dumping rely on dumpit > not > being called if start fails. For example, ila_xlat.c allocates memory > and assigns it to cb->args[0] in its start() function. It might fail > to > do that

Re: tc-ipt v0.2: Extension does not know id 1504083504

2017-09-30 Thread Corey Hickey
On 2017-09-29 23:34, Sergey K. wrote: Hello. I have to apply this patch https://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git/commit/?h=v4.10.0=97a02cabefb2e2dcfe27f89943709afa84be5525 to my version of iproute2. But now I have a new information message, when I'm using

tc-ipt v0.2: Extension does not know id 1504083504

2017-09-30 Thread Sergey K.
Hello. I have to apply this patch https://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git/commit/?h=v4.10.0=97a02cabefb2e2dcfe27f89943709afa84be5525 to my version of iproute2. But now I have a new information message, when I'm using construction like this: tc filter add dev

Re: [PATCH v2] netlink: do not proceed if dump's start() errs

2017-09-30 Thread David Miller
From: "Jason A. Donenfeld" Date: Thu, 28 Sep 2017 00:41:44 +0200 > Drivers that use the start method for netlink dumping rely on dumpit not > being called if start fails. For example, ila_xlat.c allocates memory > and assigns it to cb->args[0] in its start() function. It might

Re: [PATCH] net: hns3: Fix an error handling path in 'hclge_rss_init_hw()'

2017-09-30 Thread Yunsheng Lin
Hi, Christophe On 2017/9/30 13:34, Christophe JAILLET wrote: > If this sanity check fails, we must free 'rss_indir'. Otherwise there is a > memory leak. > 'goto err' as done in the other error handling paths to fix it. Thanks for fixing. > > Fixes: 46a3df9f9718 ("net: hns3: Fix for setting