Re: Bug report (with fix) for DEC Tulip driver (de2104x.c)

2019-09-17 Thread Helge Deller
On 18.09.19 00:51, John David Anglin wrote: > On 2019-09-17 5:36 p.m., Arlie Davis wrote: >> Likewise, I'm at a loss for testing with real hardware. It's hard to >> find such things, now. > How does de2104x compare to ds2142/43?  I have a c3750 with ds2142/43 tulip.  > Helge > or some others might

Re: [patch iproute2-next v2] devlink: add reload failed indication

2019-09-17 Thread David Ahern
On 9/17/19 12:36 PM, Jiri Pirko wrote: > Tue, Sep 17, 2019 at 06:46:31PM CEST, dsah...@gmail.com wrote: >> On 9/16/19 3:44 AM, Jiri Pirko wrote: >>> From: Jiri Pirko >>> >>> Add indication about previous failed devlink reload. >>> >>> Example outputs: >>> >>> $ devlink dev >>> netdevsim/netdevsim1

Re: Bug report (with fix) for DEC Tulip driver (de2104x.c)

2019-09-17 Thread John David Anglin
On 2019-09-17 5:36 p.m., Arlie Davis wrote: > Likewise, I'm at a loss for testing with real hardware. It's hard to > find such things, now. How does de2104x compare to ds2142/43?  I have a c3750 with ds2142/43 tulip.  Helge or some others might have a machine with a de2104x. Dave -- John David

Re: [PATCH net-next] net: dsa: mv88e6xxx: Add support for port mirroring

2019-09-17 Thread Florian Fainelli
On 9/17/19 3:32 PM, Iwan R Timmer wrote: > On Tue, Sep 17, 2019 at 10:55:05PM +0200, Andrew Lunn wrote: >> On Tue, Sep 17, 2019 at 10:23:01PM +0200, Iwan R Timmer wrote: >>> Add support for configuring port mirroring through the cls_matchall >>> classifier. We do a full ingress and/or egress captur

Re: [PATCH net-next] net: dsa: mv88e6xxx: Add support for port mirroring

2019-09-17 Thread Iwan R Timmer
On Tue, Sep 17, 2019 at 10:55:05PM +0200, Andrew Lunn wrote: > On Tue, Sep 17, 2019 at 10:23:01PM +0200, Iwan R Timmer wrote: > > Add support for configuring port mirroring through the cls_matchall > > classifier. We do a full ingress and/or egress capture towards the > > capture port, configured w

Re: [PATCH net] ipv4: Revert removal of rt_uses_gateway

2019-09-17 Thread Julian Anastasov
Hello, On Tue, 17 Sep 2019, David Ahern wrote: > On 9/17/19 12:50 PM, Julian Anastasov wrote: > > > > Looks good to me, thanks! > > > > Reviewed-by: Julian Anastasov > > > > BTW, do you have any tests for the rt_uses_gateway paths - showing why > it is needed? No speci

Re: Bug report (with fix) for DEC Tulip driver (de2104x.c)

2019-09-17 Thread Arlie Davis
I checked QEMU v3.1, and I don't see any Tulip implementation for it. I haven't checked any other emulators. A few cursory searches didn't turn up anything. I checked the FreeBSD driver for the same device. It just treats the control field as a write-only field; the driver just uses its own intern

Re: Bug report (with fix) for DEC Tulip driver (de2104x.c)

2019-09-17 Thread Andrew Lunn
On Mon, Sep 16, 2019 at 02:50:53PM -0700, Arlie Davis wrote: > Hello. I'm a developer on GCE, Google's virtual machine platform. As > part of my work, we needed to emulate a DEC Tulip 2104x NIC, so I > implemented a basic virtual device for it. > > While doing so, I believe I found a bug in the Li

Re: [PATCH V2 net 1/1] net/tls(TLS_SW): Fix list_del double free caused by a race condition in tls_tx_records

2019-09-17 Thread Pooja Trivedi
Hello, I see that the 'State' of the initially submitted patch has changed to "Changes Requested" but I have not seen a follow-up email regarding what needs to be changed. I also realized that the tabs had not been preserved in my initial submission, hence the resubmission. Thanks, Pooja. On T

[PATCH V2 net 1/1] net/tls(TLS_SW): Fix list_del double free caused by a race condition in tls_tx_records

2019-09-17 Thread Pooja Trivedi
From: Pooja Trivedi Enclosing tls_tx_records within lock_sock/release_sock pair to ensure write-synchronization is not sufficient because socket lock gets released under memory pressure situation by sk_wait_event while it sleeps waiting for memory, allowing another writer into tls_tx_records. Thi

Re: [PATCH net-next] net: dsa: mv88e6xxx: Add support for port mirroring

2019-09-17 Thread Andrew Lunn
On Tue, Sep 17, 2019 at 10:23:01PM +0200, Iwan R Timmer wrote: > Add support for configuring port mirroring through the cls_matchall > classifier. We do a full ingress and/or egress capture towards the > capture port, configured with set_egress_port. Hi Iwan This looks good as far as it goes. Ha

[PATCH net-next] net: dsa: mv88e6xxx: Add support for port mirroring

2019-09-17 Thread Iwan R Timmer
Add support for configuring port mirroring through the cls_matchall classifier. We do a full ingress and/or egress capture towards the capture port, configured with set_egress_port. Signed-off-by: Iwan R Timmer --- drivers/net/dsa/mv88e6xxx/chip.c | 36 drivers/n

udp sendmsg ENOBUFS clarification

2019-09-17 Thread Josh Hunt
I was running some tests recently with the udpgso_bench_tx benchmark in selftests and noticed that in some configurations it reported sending more than line rate! Looking into it more I found that I was overflowing the qdisc queue and so it was sending back NET_XMIT_DROP however this error did

Re: [PATCH v2] ftgmac100: Disable HW checksum generation on AST2500

2019-09-17 Thread Vijay Khemka
Please review below patch and provide your valuable feedback. Regards -Vijay On 9/11/19, 1:05 PM, "Vijay Khemka" wrote: HW checksum generation is not working for AST2500, specially with IPV6 over NCSI. All TCP packets with IPv6 get dropped. By disabling this it works perfectly fine

Re: [PATCH bpf 0/2] bpf: BTF fixes

2019-09-17 Thread Andrii Nakryiko
On Tue, Sep 17, 2019 at 12:27 PM Alexei Starovoitov wrote: > > Two trivial BTF fixes. > > Alexei Starovoitov (2): > bpf: fix BTF verification of enums > bpf: fix BTF limits > > include/uapi/linux/btf.h | 4 ++-- > kernel/bpf/btf.c | 5 ++--- > 2 files changed, 4 insertions(+), 5 delet

Re: [PATCH net] ipv4: Revert removal of rt_uses_gateway

2019-09-17 Thread David Ahern
On 9/17/19 12:50 PM, Julian Anastasov wrote: > > Looks good to me, thanks! > > Reviewed-by: Julian Anastasov > BTW, do you have any tests for the rt_uses_gateway paths - showing why it is needed? All of the pmtu, redirect, fib tests, etc worked fine without the special flag. Sure, the '

Re: [PATCH net] ipv4: Revert removal of rt_uses_gateway

2019-09-17 Thread Julian Anastasov
Hello, On Tue, 17 Sep 2019, David Ahern wrote: > From: David Ahern > > Julian noted that rt_uses_gateway has a more subtle use than 'is gateway > set': > > https://lore.kernel.org/netdev/alpine.lfd.2.21.1909151104060.2...@ja.home.ssi.bg/ > > Revert that part of the commit refere

Re: [patch iproute2-next v2] devlink: add reload failed indication

2019-09-17 Thread Jiri Pirko
Tue, Sep 17, 2019 at 06:46:31PM CEST, dsah...@gmail.com wrote: >On 9/16/19 3:44 AM, Jiri Pirko wrote: >> From: Jiri Pirko >> >> Add indication about previous failed devlink reload. >> >> Example outputs: >> >> $ devlink dev >> netdevsim/netdevsim10: reload_failed true > >odd output to user. Why

Re: [PATCH v2] tcp: Add TCP_INFO counter for packets received out-of-order

2019-09-17 Thread Neal Cardwell
On Tue, Sep 17, 2019 at 1:22 PM Eric Dumazet wrote: > > Tue, Sep 17, 2019 at 10:13 AM Jason Baron wrote: > > > > > > Hi, > > > > I was interested in adding a field to tcp_info around the TFO state of a > > socket. So for the server side it would indicate if TFO was used to > > create the socket

[PATCH bpf 2/2] bpf: fix BTF limits

2019-09-17 Thread Alexei Starovoitov
vmlinux BTF has more than 64k types. Its string section is also at the offset larger than 64k. Adjust both limits to make in-kernel BTF verifier successfully parse in-kernel BTF. Fixes: 69b693f0aefa ("bpf: btf: Introduce BPF Type Format (BTF)") Signed-off-by: Alexei Starovoitov Acked-by: Martin

[PATCH bpf 0/2] bpf: BTF fixes

2019-09-17 Thread Alexei Starovoitov
Two trivial BTF fixes. Alexei Starovoitov (2): bpf: fix BTF verification of enums bpf: fix BTF limits include/uapi/linux/btf.h | 4 ++-- kernel/bpf/btf.c | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) -- 2.20.0

[PATCH bpf 1/2] bpf: fix BTF verification of enums

2019-09-17 Thread Alexei Starovoitov
vmlinux BTF has enums that are 8 byte and 1 byte in size. 2 byte enum is a valid construct as well. Fix BTF enum verification to accept those sizes. Fixes: 69b693f0aefa ("bpf: btf: Introduce BPF Type Format (BTF)") Signed-off-by: Alexei Starovoitov Acked-by: Martin KaFai Lau --- kernel/bpf/btf.

[PATCH net] ipv4: Revert removal of rt_uses_gateway

2019-09-17 Thread David Ahern
From: David Ahern Julian noted that rt_uses_gateway has a more subtle use than 'is gateway set': https://lore.kernel.org/netdev/alpine.lfd.2.21.1909151104060.2...@ja.home.ssi.bg/ Revert that part of the commit referenced in the Fixes tag. Currently, there are no u8 holes in 'struct rtable'

[PATCH net] selftests: Update fib_tests to handle missing ping6

2019-09-17 Thread David Ahern
From: David Ahern Some distributions (e.g., debian buster) do not install ping6. Re-use the hook in pmtu.sh to detect this and fallback to ping. Fixes: a0e11da78f48 ("fib_tests: Add tests for metrics on routes") Signed-off-by: David Ahern --- tools/testing/selftests/net/fib_tests.sh | 4 +++-

[PATCH] selftests: Update fib_nexthop_multiprefix to handle missing ping6

2019-09-17 Thread David Ahern
From: David Ahern Some distributions (e.g., debian buster) do not install ping6. Re-use the hook in pmtu.sh to detect this and fallback to ping. Fixes: 735ab2f65dce ("selftests: Add test with multiple prefixes using single nexthop") Signed-off-by: David Ahern --- tools/testing/selftests/net/f

Re: -Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread Lendacky, Thomas
On 9/17/19 2:32 AM, Nathan Chancellor wrote: > Hi all, > > Clang recently added a new diagnostic in r371605, -Wsizeof-array-div, > that tries to warn when sizeof(X) / sizeof(Y) does not compute the > number of elements in an array X (i.e., sizeof(Y) is wrong). See that > commit for more details: >

Re: [PATCH v2] tcp: Add TCP_INFO counter for packets received out-of-order

2019-09-17 Thread Eric Dumazet
Tue, Sep 17, 2019 at 10:13 AM Jason Baron wrote: > > > Hi, > > I was interested in adding a field to tcp_info around the TFO state of a > socket. So for the server side it would indicate if TFO was used to > create the socket and on the client side it would report whether TFO > worked and if not

Re: [PATCH v2] tcp: Add TCP_INFO counter for packets received out-of-order

2019-09-17 Thread Jason Baron
On 9/10/19 4:38 PM, Eric Dumazet wrote: > On Tue, Sep 10, 2019 at 10:11 PM Thomas Higdon wrote: >> >> > ... >> Because an additional 32-bit member in struct tcp_info would cause >> a hole on 64-bit systems, we reserve a struct member '_reserved'. > ... >> diff --git a/include/uapi/linux/tcp.h b

Re: [PATCH] selftests: Add test cases for `ip nexthop flush proto XX`

2019-09-17 Thread David Ahern
On 9/16/19 6:26 AM, Donald Sharp wrote: > Add some test cases to allow the fib_nexthops.sh test code > to test the flushing of nexthops based upon the proto passed > in upon creation of the nexthop group. > > Signed-off-by: Donald Sharp > --- > tools/testing/selftests/net/fib_nexthops.sh | 14 ++

Re: [patch iproute2-next v2] devlink: add reload failed indication

2019-09-17 Thread David Ahern
On 9/16/19 3:44 AM, Jiri Pirko wrote: > From: Jiri Pirko > > Add indication about previous failed devlink reload. > > Example outputs: > > $ devlink dev > netdevsim/netdevsim10: reload_failed true odd output to user. Why not just "reload failed"?

Re: -Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread Nick Desaulniers
On Tue, Sep 17, 2019 at 6:27 AM Jose Abreu wrote: > > From: David Laight > Date: Sep/17/2019, 11:36:21 (UTC+00:00) > > > From: Jose Abreu > > > Sent: 17 September 2019 08:59 > > > From: Nathan Chancellor > > > Date: Sep/17/2019, 08:32:32 (UTC+00:00) > > > > > > > Hi all, > > > > > > > > Clang re

Re: [PATCH net] net/rds: Check laddr_check before calling it

2019-09-17 Thread santosh . shilimkar
On 9/17/19 8:29 AM, Ka-Cheong Poon wrote: In rds_bind(), laddr_check is called without checking if it is NULL or not. And rs_transport should be reset if rds_add_bound() fails. Reported-by: syzbot+fae39afd2101a17ec...@syzkaller.appspotmail.com Signed-off-by: Ka-Cheong Poon --- Thanks Ka-Cheon

[PATCH net] net/rds: Check laddr_check before calling it

2019-09-17 Thread Ka-Cheong Poon
In rds_bind(), laddr_check is called without checking if it is NULL or not. And rs_transport should be reset if rds_add_bound() fails. Reported-by: syzbot+fae39afd2101a17ec...@syzkaller.appspotmail.com Signed-off-by: Ka-Cheong Poon --- net/rds/bind.c | 5 - 1 file changed, 4 insertions(+),

Re: [PATCH iproute2 4/4] devlink: Fix devlink health set command

2019-09-17 Thread Stephen Hemminger
On Thu, 5 Sep 2019 15:43:07 +0300 Tariq Toukan wrote: > From: Aya Levin > > Prior to this patch both the reporter's name and the grace period > attributes shared the same bit. This caused zeroing grace period when > setting auto recovery. Let each parameter has its own bit. > > Fixes: b18d891

Re: [PATCH] netfilter: bridge: drop a broken include

2019-09-17 Thread Jeremy Sowden
On 2019-09-17, at 07:09:46 +0200, Pablo Neira Ayuso wrote: > On Mon, Sep 16, 2019 at 02:08:12PM +0100, Jeremy Sowden wrote: > > On 2019-09-16, at 02:05:16 +0200, Adam Borowski wrote: > > > This caused a build failure if CONFIG_NF_CONNTRACK_BRIDGE is set > > > but CONFIG_NF_TABLES=n -- and appears t

RE: -Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread Jose Abreu
From: David Laight Date: Sep/17/2019, 11:36:21 (UTC+00:00) > From: Jose Abreu > > Sent: 17 September 2019 08:59 > > From: Nathan Chancellor > > Date: Sep/17/2019, 08:32:32 (UTC+00:00) > > > > > Hi all, > > > > > > Clang recently added a new diagnostic in r371605, -Wsizeof-array-div, > > > that

Re: [PATCH ipsec-next v2 6/6] xfrm: add espintcp (RFC 8229)

2019-09-17 Thread Sabrina Dubroca
2019-09-17, 14:04:13 +0200, Steffen Klassert wrote: > On Tue, Sep 17, 2019 at 01:57:43PM +0200, Sabrina Dubroca wrote: > > 2019-09-17, 13:26:49 +0200, Steffen Klassert wrote: > > > On Wed, Sep 11, 2019 at 04:13:07PM +0200, Sabrina Dubroca wrote: > > > ... > > > > diff --git a/net/xfrm/Kconfig b/net

Re: [PATCH ipsec-next v2 6/6] xfrm: add espintcp (RFC 8229)

2019-09-17 Thread Steffen Klassert
On Tue, Sep 17, 2019 at 01:57:43PM +0200, Sabrina Dubroca wrote: > 2019-09-17, 13:26:49 +0200, Steffen Klassert wrote: > > On Wed, Sep 11, 2019 at 04:13:07PM +0200, Sabrina Dubroca wrote: > > ... > > > diff --git a/net/xfrm/Kconfig b/net/xfrm/Kconfig > > > index 51bb6018f3bf..e67044527fb7 100644 >

Re: [PATCH ipsec-next v2 6/6] xfrm: add espintcp (RFC 8229)

2019-09-17 Thread Sabrina Dubroca
2019-09-17, 13:26:49 +0200, Steffen Klassert wrote: > On Wed, Sep 11, 2019 at 04:13:07PM +0200, Sabrina Dubroca wrote: > ... > > diff --git a/net/xfrm/Kconfig b/net/xfrm/Kconfig > > index 51bb6018f3bf..e67044527fb7 100644 > > --- a/net/xfrm/Kconfig > > +++ b/net/xfrm/Kconfig > > @@ -73,6 +73,16 @@

Re: [PATCH net] net/sched: act_sample: don't push mac header on ip6gre ingress

2019-09-17 Thread yotam gigi
On Tue, Sep 17, 2019 at 1:02 PM Davide Caratti wrote: > > current 'sample' action doesn't push the mac header of ingress packets if > they are received by a layer 3 tunnel (like gre or sit); but it forgot to > check for gre over ipv6, so the following script: > > # tc q a dev $d clsact > # tc f

Re: [PATCH ipsec-next v2 6/6] xfrm: add espintcp (RFC 8229)

2019-09-17 Thread Steffen Klassert
On Wed, Sep 11, 2019 at 04:13:07PM +0200, Sabrina Dubroca wrote: ... > diff --git a/net/xfrm/Kconfig b/net/xfrm/Kconfig > index 51bb6018f3bf..e67044527fb7 100644 > --- a/net/xfrm/Kconfig > +++ b/net/xfrm/Kconfig > @@ -73,6 +73,16 @@ config XFRM_IPCOMP > select CRYPTO > select CRYPTO_DEF

Re: [PATCH bpf-next] i40e: fix xdp handle calculations

2019-09-17 Thread Björn Töpel
On 2019-09-17 12:48, Maxim Mikityanskiy wrote: Hi, it looks to me that headroom is still broken after this commit. i40e_run_xdp_zc adds it a second time, i.e.: Indeed, but Ciara fixed that in this series [1]. Thanks for reviewing the patch! Cheers, Björn [1] https://lore.kernel.org/bpf/20190

Re: [PATCH bpf-next] i40e: fix xdp handle calculations

2019-09-17 Thread Maxim Mikityanskiy
On Thu, Sep 5, 2019 at 4:11 AM Kevin Laatz wrote: > > Currently, we don't add headroom to the handle in i40e_zca_free, > i40e_alloc_buffer_slow_zc and i40e_alloc_buffer_zc. The addition of the > headroom to the handle was removed in > commit 2f86c806a8a8 ("i40e: modify driver for handling offsets"

RE: -Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread David Laight
From: Jose Abreu > Sent: 17 September 2019 08:59 > From: Nathan Chancellor > Date: Sep/17/2019, 08:32:32 (UTC+00:00) > > > Hi all, > > > > Clang recently added a new diagnostic in r371605, -Wsizeof-array-div, > > that tries to warn when sizeof(X) / sizeof(Y) does not compute the > > number of ele

[PATCH net] net/sched: act_sample: don't push mac header on ip6gre ingress

2019-09-17 Thread Davide Caratti
current 'sample' action doesn't push the mac header of ingress packets if they are received by a layer 3 tunnel (like gre or sit); but it forgot to check for gre over ipv6, so the following script: # tc q a dev $d clsact # tc f a dev $d ingress protocol ip flower ip_proto icmp action sample \ >

Re: [PATCHv2 ipsec-next] xfrm: remove the unnecessary .net_exit for xfrmi

2019-09-17 Thread Steffen Klassert
On Mon, Sep 09, 2019 at 04:30:10PM +0800, Xin Long wrote: > The xfrm_if(s) on each netns can be deleted when its xfrmi dev is > deleted. xfrmi dev's removal can happen when: > > a. netns is being removed and all xfrmi devs will be deleted. > > b. rtnl_link_unregister(&xfrmi_link_ops) in xfrmi

Re: [PATCH] netfilter: bridge: drop a broken include

2019-09-17 Thread Jeremy Sowden
On 2019-09-17, at 07:09:46 +0200, Pablo Neira Ayuso wrote: > On Mon, Sep 16, 2019 at 02:08:12PM +0100, Jeremy Sowden wrote: > > On 2019-09-16, at 02:05:16 +0200, Adam Borowski wrote: > > > This caused a build failure if CONFIG_NF_CONNTRACK_BRIDGE is set but > > > CONFIG_NF_TABLES=n -- and appears t

RE: -Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread Jose Abreu
From: Nathan Chancellor Date: Sep/17/2019, 08:32:32 (UTC+00:00) > Hi all, > > Clang recently added a new diagnostic in r371605, -Wsizeof-array-div, > that tries to warn when sizeof(X) / sizeof(Y) does not compute the > number of elements in an array X (i.e., sizeof(Y) is wrong). See that > commi

-Wsizeof-array-div warnings in ethernet drivers

2019-09-17 Thread Nathan Chancellor
Hi all, Clang recently added a new diagnostic in r371605, -Wsizeof-array-div, that tries to warn when sizeof(X) / sizeof(Y) does not compute the number of elements in an array X (i.e., sizeof(Y) is wrong). See that commit for more details: https://github.com/llvm/llvm-project/commit/3240ad4ced0d3

AW: big ICMP requests get disrupted on IPSec tunnel activation

2019-09-17 Thread Bartschies, Thomas
Hello, thanks for the suggestion. Running pmtu.sh with kernel versions 4.19, 4.20 and even 5.2.13 made no difference. All tests were successful every time. Although my external ping tests still failing with the newer kernels. I've ran the script after triggering my problem, to make sure all pos