Re: [PATCH net-next v3 6/7] vmxnet3: introduce command to register memory region

2016-06-13 Thread Or Gerlitz
On Tue, Jun 14, 2016 at 4:50 AM, Shrikrishna Khare wrote: > +++ b/drivers/net/vmxnet3/vmxnet3_defs.h > @@ -81,6 +81,7 @@ enum { > VMXNET3_CMD_RESERVED2, > VMXNET3_CMD_RESERVED3, > VMXNET3_CMD_SET_COALESCE, > + VMXNET3_CMD_REGISTER_MEMREGS, > >

Re: [PATCH net-next v3 0/7] vmxnet3: upgrade to version 3

2016-06-13 Thread Or Gerlitz
On Tue, Jun 14, 2016 at 4:50 AM, Shrikrishna Khare wrote: > This patchset upgrades vmxnet3 to version 3. Except for one patch, all the rest lack change-log, which makes it somehow needlessly harder to review and maintain > Shrikrishna Khare (7): > vmxnet3: prepare for

Re: [PATCH net-next] tun: fix csum generation for tap devices

2016-06-13 Thread Mike Rapoport
On Tue, Jun 14, 2016 at 12:00:04AM +0200, Paolo Abeni wrote: > The commit 34166093639b ("tuntap: use common code for virtio_net_hdr > and skb GSO conversion") replaced the tun code for header manipulation > with the generic helpers. While doing so, it implictly moved the > skb_partial_csum_set()

[PATCH net-next] virtio_net: fix csum generation for virtio-net devices

2016-06-13 Thread Mike Rapoport
The commit e858fae2b0b8 ("virtio_net: use common code for virtio_net_hdr and skb GSO conversion") replaced the tun code for header manipulation with the generic helpers. While doing so, it implictly moved the skb_partial_csum_set() invocation after eth_type_trans(), which invalidate the current

[PATCH net] htb: call qdisc_root with rcu read lock held

2016-06-13 Thread Florian Westphal
saw a debug splat: net/include/net/sch_generic.h:287 suspicious rcu_dereference_check() usage! other info that might help us debug this: rcu_scheduler_active = 1, debug_locks = 0 2 locks held by kworker/2:1/710: #0: ("events"){.+.+.+}, at: [] #1: ((>work)){+.+...}, at: []

Re: [ethtool PATCH v7 1/2] ethtool.c: add support for ETHTOOL_xLINKSETTINGS ioctls

2016-06-13 Thread Vidya Sagar Ravipati
Acked-by:Vidya Sagar Ravipati Validated this patch with various modules. Replied from wrong email alias earlier. On Thu, Jun 9, 2016 at 5:55 PM, David Decotigny wrote: > From: David Decotigny > > More info with kernel commit

Re: [ethtool PATCH v7 1/2] ethtool.c: add support for ETHTOOL_xLINKSETTINGS ioctls

2016-06-13 Thread Vidya Chowdary
Acked-by:Vidya Sagar Ravipati Validated this patch with various modules. On Thu, Jun 9, 2016 at 5:55 PM, David Decotigny wrote: > From: David Decotigny > > More info with kernel commit 8d3f2806f8fb ("Merge branch >

[PATCH net-next 10/10] net_sched: sch_fq: defer skb freeing

2016-06-13 Thread Eric Dumazet
sfq_reset() can use rtnl_kfree_skbs() instead of kfree_skb() Signed-off-by: Eric Dumazet --- net/sched/sch_sfq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/sched/sch_sfq.c b/net/sched/sch_sfq.c index a2e0b855d1c8..57d118b41cad 100644 ---

[PATCH net-next 09/10] net_sched: sch_pie: defer skb freeing

2016-06-13 Thread Eric Dumazet
pie_change() can use rtnl_qdisc_drop() to benefit from deferred freeing. pie_reset() is already using qdisc_reset_queue() Signed-off-by: Eric Dumazet --- net/sched/sch_pie.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/sched/sch_pie.c

[PATCH net-next 07/10] net_sched: sch_htb: defer skb freeing

2016-06-13 Thread Eric Dumazet
Both htb_reset() and htb_destroy() can use __qdisc_reset_queue() instead of __skb_queue_purge() to defer skb freeing of internal queues. Signed-off-by: Eric Dumazet --- net/sched/sch_htb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH net-next 06/10] net_sched: sch_hhf: defer skb freeing

2016-06-13 Thread Eric Dumazet
Both hhf_reset() and hhf_change() can use rtnl_kfree_skbs() Signed-off-by: Eric Dumazet --- net/sched/sch_hhf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/sched/sch_hhf.c b/net/sched/sch_hhf.c index c51791848a38..c44593b8e65a 100644 ---

[PATCH net-next 08/10] net_sched: sch_netem: defer skb freeing

2016-06-13 Thread Eric Dumazet
rtnl_kfree_skbs() can be used in tfifo_reset() It would be nice if we could iterate through rb tree instead of removing one skb at a time, and build a single skb chain. But this is left for a future patch. Signed-off-by: Eric Dumazet --- net/sched/sch_netem.c | 4 +--- 1

[PATCH net-next 03/10] net_sched: sch_codel: defer skb freeing in codel_change()

2016-06-13 Thread Eric Dumazet
codel_change() can use rtnl_qdisc_drop() to defer expensive skb freeing after locks are released. codel_reset() already has support for deferred skb freeing because it uses qdisc_reset_queue() Signed-off-by: Eric Dumazet --- net/sched/sch_codel.c | 2 +- 1 file changed, 1

[PATCH net-next 04/10] net_sched: sch_fq: defer skb freeing

2016-06-13 Thread Eric Dumazet
Both fq_change() and fq_reset() can use rtnl_kfree_skbs() Signed-off-by: Eric Dumazet --- net/sched/sch_fq.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/net/sched/sch_fq.c b/net/sched/sch_fq.c index f49c81e91acd..6eb06674f778

[PATCH net-next 02/10] net_sched: sch_choke: defer skb freeing

2016-06-13 Thread Eric Dumazet
choke_reset() and choke_change() can use rtnl_qdisc_drop() to defer expensive skb freeing after locks are released. Signed-off-by: Eric Dumazet --- net/sched/sch_choke.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/net/sched/sch_choke.c

[PATCH net-next 05/10] net_sched: fq_codel: defer skb freeing

2016-06-13 Thread Eric Dumazet
Both fq_codel_change() and fq_codel_reset() can use rtnl_kfree_skbs() Signed-off-by: Eric Dumazet --- net/sched/sch_fq_codel.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/net/sched/sch_fq_codel.c b/net/sched/sch_fq_codel.c index

[PATCH net-next 01/10] net_sched: add the ability to defer skb freeing

2016-06-13 Thread Eric Dumazet
qdisc are changed under RTNL protection and often while blocking BH and root qdisc spinlock. When lots of skbs need to be dropped, we free them under these locks causing TX/RX freezes, and more generally latency spikes. This commit adds rtnl_kfree_skbs(), used to queue skbs for deferred freeing.

[PATCH net-next 00/10] net_sched: defer skb freeing while changing qdiscs

2016-06-13 Thread Eric Dumazet
qdiscs/classes are changed under RTNL protection and often while blocking BH and root qdisc spinlock. When lots of skbs need to be dropped, we free them under these locks causing TX/RX freezes, and more generally latency spikes. I saw spikes of 50+ ms on quite fast hardware... This patch series

Re: [PATCH 4/5] ipvs: Lock socket before setting SK_CAN_REUSE

2016-06-13 Thread Eric Dumazet
On Mon, 2016-06-13 at 20:12 -0700, Eric Dumazet wrote: > > Have you tested this patch ? Hmm, please ignore this question ;)

Re: [PATCH 4/5] ipvs: Lock socket before setting SK_CAN_REUSE

2016-06-13 Thread Eric Dumazet
On Tue, 2016-06-14 at 02:43 +0100, Quentin Armitage wrote: > When other settings are changed in the socket it is locked, so > lock the socket before setting SK_CAN_REUSE. > > Signed-off-by: Quentin Armitage > --- > net/netfilter/ipvs/ip_vs_sync.c |2 ++ > 1 files

Re: [PATCH -next] sctp: fix error return code in sctp_init()

2016-06-13 Thread Xin Long
On Mon, Jun 13, 2016 at 11:08 PM, wrote: > From: Wei Yongjun > > Fix to return a negative error code from the error handling > case instead of 0, as done elsewhere in this function. > > Signed-off-by: Wei Yongjun >

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 5:28 PM, Tom Herbert wrote: > On Mon, Jun 13, 2016 at 4:12 PM, Alexander Duyck > wrote: >> On Mon, Jun 13, 2016 at 3:17 PM, Tom Herbert wrote: >>> On Mon, Jun 13, 2016 at 2:51 PM, Alexander Duyck >>>

Re: [PATCH 2/5] ipvs: Stop calling __dev_get_by_name() repeatedly when starting sync daemon

2016-06-13 Thread kbuild test robot
Hi, [auto build test ERROR on ipvs-next/master] [also build test ERROR on v4.7-rc3 next-20160609] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH net-next v3 1/7] vmxnet3: prepare for version 3 changes

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/Makefile | 4 ++-- drivers/net/vmxnet3/upt1_defs.h | 4 ++-- drivers/net/vmxnet3/vmxnet3_defs.h| 9 ++--- drivers/net/vmxnet3/vmxnet3_drv.c | 22 +-

[PATCH net-next v3 5/7] vmxnet3: add support for get_coalesce, set_coalesce ethtool operations

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Keyong Sun Signed-off-by: Manoj Tammali Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_defs.h| 33 ++- drivers/net/vmxnet3/vmxnet3_drv.c | 54

[PATCH net-next v3 4/7] vmxnet3: add receive data ring support

2016-06-13 Thread Shrikrishna Khare
Receive Data Ring buffer length is configurable via ethtool -G ethX rx-mini Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_defs.h| 14 +++- drivers/net/vmxnet3/vmxnet3_drv.c | 153 +++---

[PATCH net-next v3 7/7] vmxnet3: update to version 3

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_drv.c | 7 ++- drivers/net/vmxnet3/vmxnet3_int.h | 4 ++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/vmxnet3/vmxnet3_drv.c b/drivers/net/vmxnet3/vmxnet3_drv.c index

[PATCH net-next v3 6/7] vmxnet3: introduce command to register memory region

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Guolin Yang Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_defs.h | 17 + 1 file changed, 17 insertions(+) diff --git a/drivers/net/vmxnet3/vmxnet3_defs.h b/drivers/net/vmxnet3/vmxnet3_defs.h index

[PATCH 0/5] ipvs: fix backup sync daemon with IPv6, and minor updates

2016-06-13 Thread Quentin Armitage
This series of patches arise from discovering that: ipvsadm --start-daemon backup --mcast-group IPv6_address ... would always fail. The first patch resolves the problem. The second and third patches are optimizations that were noticed while investigating the original problem. The fourth patch

[PATCH 5/5] ipvs: log additional sync daemon parameters

2016-06-13 Thread Quentin Armitage
Add new multicast parameters to log messages when sync daemons start. Commits ("ipvs: add sync_maxlen parameter for the sync daemon") and ("ipvs: add more mcast parameters for the sync daemon") added additional multicast parameters, but didn't add them to the log messages when the sync daemons

[PATCH 3/5] ipvs: Don't check result < 0 after setting result = 0

2016-06-13 Thread Quentin Armitage
Move the block testing result < 0 to avoid the test immediately after setting result = 0 Signed-off-by: Quentin Armitage --- net/netfilter/ipvs/ip_vs_sync.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/net/netfilter/ipvs/ip_vs_sync.c

[PATCH 4/5] ipvs: Lock socket before setting SK_CAN_REUSE

2016-06-13 Thread Quentin Armitage
When other settings are changed in the socket it is locked, so lock the socket before setting SK_CAN_REUSE. Signed-off-by: Quentin Armitage --- net/netfilter/ipvs/ip_vs_sync.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git

[PATCH 2/5] ipvs: Stop calling __dev_get_by_name() repeatedly when starting sync daemon

2016-06-13 Thread Quentin Armitage
Optimise starting sync daemons by using the result of the first call to __dev_get_by_name() and pass the result or ifindex to subsequent functions to avoid them having to call __dev_get_by_name() again. Signed-off-by: Quentin Armitage --- net/netfilter/ipvs/ip_vs_sync.c

[PATCH 1/5] ipvs: Enable setting IPv6 multicast address for ipvs sync daemon backup

2016-06-13 Thread Quentin Armitage
When using HEAD from https://git.kernel.org/cgit/utils/kernel/ipvsadm/ipvsadm.git/, the command: ipvsadm --start-daemon backup --mcast-interface eth0.60 --mcast-group ff01::1:81 fails with the error message: Argument list too long whereas both: ipvsadm --start-daemon master --mcast-interface

[PATCH net-next v3 2/7] vmxnet3: introduce generic command interface to configure the device

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Guolin Yang Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_defs.h | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/drivers/net/vmxnet3/vmxnet3_defs.h

[PATCH net-next v3 3/7] vmxnet3: allow variable length transmit data ring buffer

2016-06-13 Thread Shrikrishna Khare
Signed-off-by: Sriram Rangarajan Signed-off-by: Shrikrishna Khare --- drivers/net/vmxnet3/vmxnet3_defs.h| 12 +++- drivers/net/vmxnet3/vmxnet3_drv.c | 55 ++- drivers/net/vmxnet3/vmxnet3_ethtool.c | 9 +++---

[PATCH net-next v3 0/7] vmxnet3: upgrade to version 3

2016-06-13 Thread Shrikrishna Khare
This patchset upgrades vmxnet3 to version 3. Changes in v2: - Following patch is updated. See that patch for details: Driver: Vmxnet3: Add support for get_coalesce, set_coalesce ethtool Changes in v3: - fix subject line to use vmxnet3: instead of Driver:Vmxnet3 - resubmit when net-next is

[PATCH net-next 1/1] tipc: add neighbor monitoring framework

2016-06-13 Thread Jon Maloy
TIPC based clusters are by default set up with full-mesh link connectivity between all nodes. Those links are expected to provide a short failure detection time, by default set to 1500 ms. Because of this, the background load for neighbor monitoring in an N-node cluster increases with a factor N

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Tom Herbert
On Mon, Jun 13, 2016 at 4:12 PM, Alexander Duyck wrote: > On Mon, Jun 13, 2016 at 3:17 PM, Tom Herbert wrote: >> On Mon, Jun 13, 2016 at 2:51 PM, Alexander Duyck >> wrote: >>> On Mon, Jun 13, 2016 at 1:36 PM, Tom

[PATCH net-next] net: vrf: Update flags and features settings

2016-06-13 Thread David Ahern
1. Default VRF devices to not having a qdisc (IFF_NO_QUEUE). Users can add one as desired. 2. Disable adding a VLAN to a VRF device. 3. Enable offloads and hardware features similar to other logical devices (e.g., dummy, veth) Change provides a significant boost in TCP stream Tx

[ethtool PATCH v1 0/2] Add support for QSFP+/QSFP28 Diagnostics

2016-06-13 Thread Vidya Sagar Ravipati
From: Vidya Sagar Ravipati This patch series provides following support a) Reorganized fields based out of SFF-8024 fields i.e. Identifier/ Encoding/Connector types which are common across SFP/SFP+ (SFF-8472) and QSFP+/QSFP28 (SFF-8436/SFF-8636) modules into

[ethtool PATCH v1 2/2] ethtool:QSFP Plus/QSFP28 Diagnostics Information Support

2016-06-13 Thread Vidya Sagar Ravipati
From: Vidya Sagar Ravipati This patch series provides following support a) Reorganized fields based out of SFF-8024 fields i.e. Identifier/ Encoding/Connector types which are common across SFP/SFP+ (SFF-8472) and QSFP+/QSFP28 (SFF-8436/SFF-8636) modules into

[ethtool PATCH v1 1/2] ethtool-copy.h:sync with net

2016-06-13 Thread Vidya Sagar Ravipati
From: Vidya Sagar Ravipati This covers kernel changes upto: commit 3851112e4737cd52aaeda0ce8d084be9ee128106 Author: Vidya Sagar Ravipati Date: Sun May 22 23:59:00 2016 -0700 ethtool: add support for 25G/50G/100G speed modes This

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 3:17 PM, Tom Herbert wrote: > On Mon, Jun 13, 2016 at 2:51 PM, Alexander Duyck > wrote: >> On Mon, Jun 13, 2016 at 1:36 PM, Tom Herbert wrote: >>> On Mon, Jun 13, 2016 at 1:24 PM, Alexander Duyck >>>

Re: [PATCH net 1/1] net sched actions: bug fix dumping actions directly didnt produce NLMSG_DONE

2016-06-13 Thread Cong Wang
On Mon, Jun 13, 2016 at 3:08 PM, Jamal Hadi Salim wrote: > From: Jamal Hadi Salim > > This refers to commands to direct action access as follows: > > sudo tc actions add action drop index 12 > sudo tc actions add action pipe index 10 > > And then dumping

Re: [PATCH net-next] tcp: use RFC6298 compliant TCP RTO calculation

2016-06-13 Thread Yuchung Cheng
On Mon, Jun 13, 2016 at 1:45 PM, Daniel Metz wrote: > This patch adjusts Linux RTO calculation to be RFC6298 Standard > compliant. MinRTO is no longer added to the computed RTO, RTO damping > and overestimation are decreased. > > In RFC 6298 Standard TCP Retransmission Timeout

Re: [PATCH RFC 3/3] ipv4: kill RTO_ONLINK, RT_CONN_FLAGS and RT_CONN_FLAGS_TOS

2016-06-13 Thread David Ahern
On 6/13/16 2:47 PM, Hannes Frederic Sowa wrote: Unfortunately because of bisectability I cannot split this patch. :( Signed-off-by: Hannes Frederic Sowa --- drivers/net/bonding/bond_main.c | 3 +-- drivers/net/gtp.c | 3 ++- drivers/net/ppp/pptp.c

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Tom Herbert
On Mon, Jun 13, 2016 at 2:51 PM, Alexander Duyck wrote: > On Mon, Jun 13, 2016 at 1:36 PM, Tom Herbert wrote: >> On Mon, Jun 13, 2016 at 1:24 PM, Alexander Duyck >> wrote: >>> On Mon, Jun 13, 2016 at 12:55 PM, Tom

[PATCH net 1/1] net sched actions: bug fix dumping actions directly didnt produce NLMSG_DONE

2016-06-13 Thread Jamal Hadi Salim
From: Jamal Hadi Salim This refers to commands to direct action access as follows: sudo tc actions add action drop index 12 sudo tc actions add action pipe index 10 And then dumping them like so: sudo tc actions ls action gact iproute2 worked because it depended on absence

Re: [PATCH RFC 2/3] ipv4: make flow functions more grepable by removing flowi4_update_output

2016-06-13 Thread David Ahern
On 6/13/16 2:47 PM, Hannes Frederic Sowa wrote: Signed-off-by: Hannes Frederic Sowa --- include/net/flow.h | 11 --- include/net/route.h | 8 net/netfilter/ipvs/ip_vs_xmit.c | 9 +++-- net/sctp/protocol.c |

Re: [PATCH RFC 1/3] ipv4: make flow functions more grepable by removing flowi4_init_output

2016-06-13 Thread David Ahern
On 6/13/16 2:47 PM, Hannes Frederic Sowa wrote: No functional changes. Signed-off-by: Hannes Frederic Sowa --- include/net/flow.h | 21 - include/net/route.h | 33 +++--

[PATCH net-next] tun: fix csum generation for tap devices

2016-06-13 Thread Paolo Abeni
The commit 34166093639b ("tuntap: use common code for virtio_net_hdr and skb GSO conversion") replaced the tun code for header manipulation with the generic helpers. While doing so, it implictly moved the skb_partial_csum_set() invocation after eth_type_trans(), which invalidate the current gso

Re: [net-next PATCH 02/15] net: Merge VXLAN and GENEVE push notifiers into a single notifier

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 2:08 PM, Hannes Frederic Sowa wrote: > On 13.06.2016 21:47, Alexander Duyck wrote: >> On Mon, Jun 13, 2016 at 10:57 AM, Hannes Frederic Sowa >> wrote: >>> Hi Alex, >>> >>> very cool series! >>> >>> On 13.06.2016 19:48, Alexander Duyck

Re: [Patch net-next] net_sched: make tcf_hash_check() boolean

2016-06-13 Thread Jamal Hadi Salim
On 16-06-13 04:46 PM, Cong Wang wrote: Cc: Jamal Hadi Salim Signed-off-by: Cong Wang Acked-by: Jamal Hadi Salim cheers, jamal

Re: [Patch net] act_ipt: fix a bind refcnt leak

2016-06-13 Thread Jamal Hadi Salim
On 16-06-13 04:44 PM, Cong Wang wrote: And avoid calling tcf_hash_check() twice. Fixes: a57f19d30b2d ("net sched: ipt action fix late binding") Cc: Jamal Hadi Salim Signed-off-by: Cong Wang Good catch. Acked-by: Jamal Hadi Salim

Re: [Patch net-next] act_police: rename tcf_act_police_locate() to tcf_act_police_init()

2016-06-13 Thread Jamal Hadi Salim
On 16-06-13 01:47 PM, Cong Wang wrote: This function is just ->init(), rename it to make it obvious. Cc: Jamal Hadi Salim Signed-off-by: Cong Wang Acked-by: Jamal Hadi Salim cheers, jamal

Re: [Patch net-next] net_sched: remove internal use of TC_POLICE_*

2016-06-13 Thread Jamal Hadi Salim
On 16-06-13 01:47 PM, Cong Wang wrote: These should be gone when we removed CONFIG_NET_CLS_POLICE. We can not totally remove them since they are exposed to userspace. Cc: Jamal Hadi Salim Signed-off-by: Cong Wang Acked-by: Jamal Hadi Salim

Re: [net-next PATCH v2 1/1] net sched actions: skbedit add support for mod-ing skb pkt_type

2016-06-13 Thread Jamal Hadi Salim
On 16-06-13 08:21 AM, Daniel Borkmann wrote: Looks like nft_meta_set_eval() can already do that, but restricted via pkt_type_ok(), so we can't play bad games with PACKET_LOOPBACK et al. Wasn't aware of that. Hm, so if you want to make use of this from tc as well, probably it makes sense to add

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 1:36 PM, Tom Herbert wrote: > On Mon, Jun 13, 2016 at 1:24 PM, Alexander Duyck > wrote: >> On Mon, Jun 13, 2016 at 12:55 PM, Tom Herbert wrote: >>> On Mon, Jun 13, 2016 at 10:47 AM, Alexander Duyck

Re: [PATCH net] fib_rules: don't break ECN with TOS rules

2016-06-13 Thread Hannes Frederic Sowa
On Mon, Jun 13, 2016, at 00:18, David Ahern wrote: > On 6/11/16 3:10 PM, Hannes Frederic Sowa wrote: > > @@ -215,6 +216,9 @@ static int fib4_rule_configure(struct fib_rule *rule, > > struct sk_buff *skb, > > rule4->dst_len = frh->dst_len; > > rule4->dstmask =

Re: [RESEND PATCH 1/3] rfkill: Create "rfkill-airplane-mode" LED trigger

2016-06-13 Thread Pavel Machek
On Mon 2016-06-13 17:10:02, João Paulo Rechi Vita wrote: > On 13 June 2016 at 17:01, Pavel Machek wrote: > > On Mon 2016-06-13 15:59:35, João Paulo Rechi Vita wrote: > >> On 13 June 2016 at 15:00, Pavel Machek wrote: > >> > Hi! > >> > > >> >> > João, that means you

Re: [PATCH v2 net] net_sched: prio: insure proper transactional behavior

2016-06-13 Thread Cong Wang
On Mon, Jun 13, 2016 at 11:33 AM, Eric Dumazet wrote: > From: Eric Dumazet > > Now prio_init() can return -ENOMEM, it also has to make sure > any allocated qdiscs are freed, since the caller (qdisc_create()) wont > call ->destroy() handler for us. > >

Re: [PATCH net-next] tcp: use RFC6298 compliant TCP RTO calculation

2016-06-13 Thread Eric Dumazet
On Mon, 2016-06-13 at 22:45 +0200, Daniel Metz wrote: > This patch adjusts Linux RTO calculation to be RFC6298 Standard > compliant. MinRTO is no longer added to the computed RTO, RTO damping > and overestimation are decreased. > > In RFC 6298 Standard TCP Retransmission Timeout (RTO) calculation

Re: [RESEND PATCH 1/3] rfkill: Create "rfkill-airplane-mode" LED trigger

2016-06-13 Thread João Paulo Rechi Vita
On 13 June 2016 at 17:01, Pavel Machek wrote: > On Mon 2016-06-13 15:59:35, João Paulo Rechi Vita wrote: >> On 13 June 2016 at 15:00, Pavel Machek wrote: >> > Hi! >> > >> >> > João, that means you should send a patch to add the ::rfkill suffix. >> >> > >> >> >> >> IMO

Re: [net-next PATCH 02/15] net: Merge VXLAN and GENEVE push notifiers into a single notifier

2016-06-13 Thread Hannes Frederic Sowa
On 13.06.2016 21:47, Alexander Duyck wrote: > On Mon, Jun 13, 2016 at 10:57 AM, Hannes Frederic Sowa > wrote: >> Hi Alex, >> >> very cool series! >> >> On 13.06.2016 19:48, Alexander Duyck wrote: >>> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h >>> index

Re: [RESEND PATCH 1/3] rfkill: Create "rfkill-airplane-mode" LED trigger

2016-06-13 Thread Pavel Machek
On Mon 2016-06-13 15:59:35, João Paulo Rechi Vita wrote: > On 13 June 2016 at 15:00, Pavel Machek wrote: > > Hi! > > > >> > João, that means you should send a patch to add the ::rfkill suffix. > >> > > >> > >> IMO "airplane" (or maybe "airplane-mode") is a better suffix, as it > >>

[PATCH v8 1/5] ptr_ring: array based FIFO for pointers

2016-06-13 Thread Michael S. Tsirkin
A simple array based FIFO of pointers. Intended for net stack which commonly has a single consumer/producer. Signed-off-by: Michael S. Tsirkin --- include/linux/ptr_ring.h | 264 +++ 1 file changed, 264 insertions(+) create mode

[PATCH v8 3/5] skb_array: array based FIFO for skbs

2016-06-13 Thread Michael S. Tsirkin
A simple array based FIFO of pointers. Intended for net stack so uses skbs for type safety. Implemented as a set of wrappers around ptr_ring. Signed-off-by: Michael S. Tsirkin --- include/linux/skb_array.h | 144 ++ 1 file changed,

[PATCH v8 2/5] ptr_ring: ring test

2016-06-13 Thread Michael S. Tsirkin
Add ringtest based unit test for ptr ring. Signed-off-by: Michael S. Tsirkin --- tools/virtio/ringtest/ptr_ring.c | 192 +++ tools/virtio/ringtest/Makefile | 5 +- 2 files changed, 196 insertions(+), 1 deletion(-) create mode 100644

[PATCH v8 0/5] skb_array: array based FIFO for skbs

2016-06-13 Thread Michael S. Tsirkin
This is in response to the proposal by Jason to make tun rx packet queue lockless using a circular buffer. My testing seems to show that at least for the common usecase in networking, which isn't lockless, circular buffer with indices does not perform that well, because each index access causes a

[PATCH v8 4/5] ptr_ring: resize support

2016-06-13 Thread Michael S. Tsirkin
This adds ring resize support. Seems to be necessary as users such as tun allow userspace control over queue size. If resize is used, this costs us ability to peek at queue without consumer lock - should not be a big deal as peek and consumer are usually run on the same CPU. If ring is made

[PATCH v8 5/5] skb_array: resize support

2016-06-13 Thread Michael S. Tsirkin
Update skb_array after ptr_ring API changes. Signed-off-by: Michael S. Tsirkin --- include/linux/skb_array.h | 33 + 1 file changed, 29 insertions(+), 4 deletions(-) diff --git a/include/linux/skb_array.h b/include/linux/skb_array.h index

[PATCH RFC 3/3] ipv4: kill RTO_ONLINK, RT_CONN_FLAGS and RT_CONN_FLAGS_TOS

2016-06-13 Thread Hannes Frederic Sowa
Unfortunately because of bisectability I cannot split this patch. :( Signed-off-by: Hannes Frederic Sowa --- drivers/net/bonding/bond_main.c | 3 +-- drivers/net/gtp.c | 3 ++- drivers/net/ppp/pptp.c | 2 +- drivers/net/vrf.c |

[PATCH RFC 0/3] ipv4: preparation work for making ipv4 routing ecn robust

2016-06-13 Thread Hannes Frederic Sowa
Probably out of laziness we encoded the RTO_ONLINK flag into the lowest ToS bit to easier pass it around in the flowi structure (we need one less argument to do so). This series cleans it up and uses flowi4_scope correctly from nearly the top-level, thus removing the need to encode the RTO_ONLINK

[PATCH RFC 2/3] ipv4: make flow functions more grepable by removing flowi4_update_output

2016-06-13 Thread Hannes Frederic Sowa
Signed-off-by: Hannes Frederic Sowa --- include/net/flow.h | 11 --- include/net/route.h | 8 net/netfilter/ipvs/ip_vs_xmit.c | 9 +++-- net/sctp/protocol.c | 10 -- 4 files changed, 15 insertions(+),

[PATCH RFC 1/3] ipv4: make flow functions more grepable by removing flowi4_init_output

2016-06-13 Thread Hannes Frederic Sowa
No functional changes. Signed-off-by: Hannes Frederic Sowa --- include/net/flow.h | 21 - include/net/route.h | 33 +++-- net/ipv4/inet_connection_sock.c | 41

[Patch net-next] net_sched: make tcf_hash_check() boolean

2016-06-13 Thread Cong Wang
Cc: Jamal Hadi Salim Signed-off-by: Cong Wang --- include/net/act_api.h | 4 ++-- net/sched/act_api.c | 8 net/sched/act_ife.c | 3 ++- net/sched/act_ipt.c | 3 ++- net/sched/act_mirred.c | 3 ++- net/sched/act_simple.c | 3

[PATCH net-next] tcp: use RFC6298 compliant TCP RTO calculation

2016-06-13 Thread Daniel Metz
This patch adjusts Linux RTO calculation to be RFC6298 Standard compliant. MinRTO is no longer added to the computed RTO, RTO damping and overestimation are decreased. In RFC 6298 Standard TCP Retransmission Timeout (RTO) calculation the calculated RTO is rounded up to the Minimum RTO (MinRTO),

[PATCH net-next 1/3] net: l3mdev: Remove const from flowi6 arg to get_rt6_dst

2016-06-13 Thread David Ahern
Allow drivers to pass flow arg to functions where the arg is not const and allow the driver to make updates as needed (eg., setting oif). Signed-off-by: David Ahern --- drivers/net/vrf.c| 2 +- include/net/l3mdev.h | 6 +++--- net/l3mdev/l3mdev.c | 2 +- 3 files

[PATCH net-next 0/3] net: vrf: Handle ipv6 multicast and link-local addresses

2016-06-13 Thread David Ahern
IPv6 multicast and link-local addresses require special handling by the VRF driver. Rather than using the VRF device index and full FIB lookups, packets to/from these addresses should use direct FIB lookups based on the VRF device table. Multicast routes do not make sense for the L3 master device

[PATCH net-next 2/3] net: ipv6: Do not add multicast route for l3 master devices

2016-06-13 Thread David Ahern
L3 master devices are virtual devices similar to the loopback device. Link local and multicast routes for these devices do not make sense. The ipv6 addrconf code already skips adding a linklocal address; do the same for the mcast route. Signed-off-by: David Ahern ---

[Patch net] act_ipt: fix a bind refcnt leak

2016-06-13 Thread Cong Wang
And avoid calling tcf_hash_check() twice. Fixes: a57f19d30b2d ("net sched: ipt action fix late binding") Cc: Jamal Hadi Salim Signed-off-by: Cong Wang --- net/sched/act_ipt.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git

[PATCH net-next 3/3] net: vrf: Handle ipv6 multicast and link-local addresses

2016-06-13 Thread David Ahern
IPv6 multicast and link-local addresses require special handling by the VRF driver: 1. Rather than using the VRF device index and full FIB lookups, packets to/from these addresses should use direct FIB lookups based on the VRF device table. 2. fail sends/receives on a VRF device to/from a

Re: [PATCH net-next] net: ena: Add a driver for Amazon Elastic Network Adapters (ENA)

2016-06-13 Thread Matt Wilson
On Mon, Jun 13, 2016 at 11:46:13AM +0300, Netanel Belgazal wrote: > This is a driver for the forthcoming ENA family of networking devices. Reviewed-by: Matt Wilson > Signed-off-by: Netanel Belgazal > --- > Documentation/networking/00-INDEX

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Tom Herbert
On Mon, Jun 13, 2016 at 1:24 PM, Alexander Duyck wrote: > On Mon, Jun 13, 2016 at 12:55 PM, Tom Herbert wrote: >> On Mon, Jun 13, 2016 at 10:47 AM, Alexander Duyck >> wrote: >>> This patch merges the GENEVE and VXLAN code so

Re: [PATCH] brcmfmac: rework function picking free BSS index

2016-06-13 Thread Rafał Miłecki
On 13 June 2016 at 21:30, Arend van Spriel wrote: > On 09-06-16 21:16, Arend van Spriel wrote: >> On 26-05-16 01:44, Rafał Miłecki wrote: >>> The old implementation was overcomplicated and slightly bugged in some >>> corner cases. >>> > > [...] > >>> New code is

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 12:55 PM, Tom Herbert wrote: > On Mon, Jun 13, 2016 at 10:47 AM, Alexander Duyck wrote: >> This patch merges the GENEVE and VXLAN code so that both functions pass >> through a shared code path. This way we can start the effort

Re: [Intel-wired-lan] [net-next PATCH 02/15] net: Merge VXLAN and GENEVE push notifiers into a single notifier

2016-06-13 Thread kbuild test robot
Hi, [auto build test ERROR on net/master] [also build test ERROR on v4.7-rc3 next-20160609] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [RESEND PATCH 1/3] rfkill: Create "rfkill-airplane-mode" LED trigger

2016-06-13 Thread João Paulo Rechi Vita
On 13 June 2016 at 15:00, Pavel Machek wrote: > Hi! > >> > João, that means you should send a patch to add the ::rfkill suffix. >> > >> >> IMO "airplane" (or maybe "airplane-mode") is a better suffix, as it >> reflects the label on the machine's chassis. I'll name it >>

Re: udp failures traced to e858fae2b0b8

2016-06-13 Thread David Ahern
On 6/13/16 12:40 PM, Mike Rapoport wrote: diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 9af0a98..9f8be96 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -482,10 +482,6 @@ static void receive_buf(struct virtnet_info *vi, struct receive_queue *rq,

Re: [net-next PATCH 01/15] net: Combine GENEVE and VXLAN port offload notifiers into single functions

2016-06-13 Thread Tom Herbert
On Mon, Jun 13, 2016 at 10:47 AM, Alexander Duyck wrote: > This patch merges the GENEVE and VXLAN code so that both functions pass > through a shared code path. This way we can start the effort of using a > single function on the network device drivers to handle both of

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-13 Thread Richard Cochran
On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > 3. ALSA support for tunable AD/DA clocks. The rate of the Listener's >DA clock must match that of the Talker and the other Listeners. >Either you adjust it in HW using a VCO or similar, or you do >adaptive sample rate

Re: [Intel-wired-lan] [net-next PATCH 04/15] bnxt: Replace ndo_add/del_vxlan_port with ndo_add/del_udp_enc_port

2016-06-13 Thread kbuild test robot
Hi, [auto build test ERROR on net/master] [also build test ERROR on v4.7-rc3 next-20160609] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [net-next PATCH 02/15] net: Merge VXLAN and GENEVE push notifiers into a single notifier

2016-06-13 Thread Alexander Duyck
On Mon, Jun 13, 2016 at 10:57 AM, Hannes Frederic Sowa wrote: > Hi Alex, > > very cool series! > > On 13.06.2016 19:48, Alexander Duyck wrote: >> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h >> index d101e4d904ba..e959b6348f91 100644 >> ---

Re: [PATCH net v2 0/4] ovs: fix rtnl notifications on interface deletion

2016-06-13 Thread pravin shelar
On Mon, Jun 13, 2016 at 1:31 AM, Nicolas Dichtel wrote: > > There was no rtnl notifications for interfaces (gre, vxlan, geneve) created > by ovs. This problem is fixed by adjusting the creation path. > > v1 -> v2: > - add patch #1 and #4 > - rework error handling in

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-13 Thread Richard Cochran
On Mon, Jun 13, 2016 at 03:00:59PM +0200, Henrik Austad wrote: > On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > > People have been asking me about TSN and Linux, and we've made some > > thoughts about it. The interest is there, and so I am glad to see > > discussion on this

Re: [Intel-wired-lan] [net-next PATCH 04/15] bnxt: Replace ndo_add/del_vxlan_port with ndo_add/del_udp_enc_port

2016-06-13 Thread kbuild test robot
Hi, [auto build test WARNING on net/master] [also build test WARNING on v4.7-rc3 next-20160609] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [very-RFC 0/8] TSN driver for the kernel

2016-06-13 Thread Richard Cochran
On Mon, Jun 13, 2016 at 03:00:59PM +0200, Henrik Austad wrote: > On Mon, Jun 13, 2016 at 01:47:13PM +0200, Richard Cochran wrote: > > Which driver is that? > > drivers/net/ethernet/renesas/ That driver is merely a PTP capable MAC driver, nothing more. Although AVB is in the device name, the

Re: [net-next PATCH 02/15] net: Merge VXLAN and GENEVE push notifiers into a single notifier

2016-06-13 Thread Tom Herbert
On Mon, Jun 13, 2016 at 10:57 AM, Hannes Frederic Sowa wrote: > Hi Alex, > > very cool series! > > On 13.06.2016 19:48, Alexander Duyck wrote: >> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h >> index d101e4d904ba..e959b6348f91 100644 >> ---

Re: [PATCH] brcmfmac: rework function picking free BSS index

2016-06-13 Thread Arend van Spriel
On 09-06-16 21:16, Arend van Spriel wrote: > On 26-05-16 01:44, Rafał Miłecki wrote: >> The old implementation was overcomplicated and slightly bugged in some >> corner cases. >> [...] >> New code is simpler, placed in file where it's really used, handles >> running out of free BSS-es and

  1   2   3   >