[PATCH] stmmac: ipq806x: Return error values instead of pointers

2015-12-02 Thread Stephen Boyd
Typically we return error pointers when we want to use those pointers in the non-error case, but this function is just returning error pointers or NULL for success. Change the style to plain int to follow normal kernel coding styles. Cc: Joachim Eastwood Signed-off-by: Stephen Boyd --- .../net/

Re: [PATCH v2 1/1] atm: solos-pci: Replace simple_strtol by kstrtoint

2015-12-02 Thread LABBE Corentin
On Wed, Dec 02, 2015 at 05:02:19PM +0300, Sergei Shtylyov wrote: > Hello. > > On 12/2/2015 3:54 PM, LABBE Corentin wrote: > > > The simple_strtol function is obsolete. > > This patch replace it by kstrtoint. > > This will simplify code, since some error case not handled by > > simple_strtol are h

Re: [PATCH] Improve Atheros ethernet driver not to do order 4 GFP_ATOMIC allocation

2015-12-02 Thread Pavel Machek
On Wed 2015-12-02 22:43:31, Chris Snook wrote: > On Tue, Dec 1, 2015 at 12:35 PM David Miller wrote: > > > From: Michal Hocko > > Date: Mon, 30 Nov 2015 14:21:29 +0100 > > > > > On Sat 28-11-15 15:51:13, Pavel Machek wrote: > > >> > > >> atl1c driver is doing order-4 allocation with GFP_ATOMIC >

Re: [patch net-next v2 01/28] net: dev: Check CHANGEUPPER notifier return value

2015-12-02 Thread Or Gerlitz
On Thu, Dec 3, 2015 at 1:20 AM, John Fastabend wrote: > On 15-12-02 12:07 PM, Jiri Pirko wrote: >> From: Ido Schimmel >> >> switchdev drivers reflect the newly requested topology to hardware when >> CHANGEUPPER is received, after software links were already formed. >> However, the operation can f

Re: [PATCH net] ipv6: kill sk_dst_lock

2015-12-02 Thread Eric Dumazet
On Wed, 2015-12-02 at 23:42 -0500, David Miller wrote: > Please update the commit log message with the details you > provided in a reply to this thread. Sure, I sent a v2. Thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.k

[PATCH v2 net] ipv6: kill sk_dst_lock

2015-12-02 Thread Eric Dumazet
From: Eric Dumazet While testing the np->opt RCU conversion, I found that UDP/IPv6 was using a mixture of xchg() and sk_dst_lock to protect concurrent changes to sk->sk_dst_cache, leading to possible corruptions and crashes. ip6_sk_dst_lookup_flow() uses sk_dst_check() anyway, so the simplest wa

[PATCH net] ipv6: sctp: add rcu protection around np->opt

2015-12-02 Thread Eric Dumazet
From: Eric Dumazet This patch completes the work I did in commit 45f6fad84cc3 ("ipv6: add complete rcu protection around np->opt"), as I missed sctp part. This simply makes sure np->opt is used with proper RCU locking and accessors. Signed-off-by: Eric Dumazet --- net/sctp/ipv6.c | 13 +

Re: [PATCH net] ipv6: add complete rcu protection around np->opt

2015-12-02 Thread Eric Dumazet
On Wed, 2015-12-02 at 23:38 -0500, David Miller wrote: > From: Eric Dumazet > Date: Sun, 29 Nov 2015 19:37:57 -0800 > > > From: Eric Dumazet > > > > This patch addresses multiple problems : > > > > UDP/RAW sendmsg() need to get a stable struct ipv6_txoptions > > while socket is not locked : Ot

Re: [PATCH] net/neighbour: fix crash at dumping device-agnostic proxy entries

2015-12-02 Thread David Miller
From: Konstantin Khlebnikov Date: Tue, 01 Dec 2015 01:14:48 +0300 > Proxy entries could have null pointer to net-device. > > Signed-off-by: Konstantin Khlebnikov > Fixes: 84920c1420e2 ("net: Allow ipv6 proxies and arp proxies be shown with > iproute2") Applied and queued up for -stable. -- To

Re: [PATCH net-next] sfc: use ALIGN macro for aligning frame sizes

2015-12-02 Thread David Miller
From: Jarod Wilson Date: Mon, 30 Nov 2015 17:12:21 -0500 > Don't open-code it. > > CC: Solarflare linux maintainers > CC: Shradha Shah > CC: netdev@vger.kernel.org > Signed-off-by: Jarod Wilson Applied, thank you. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the bo

Re: [PATCH] net: smc911x: convert pxa dma to dmaengine

2015-12-02 Thread David Miller
From: Robert Jarzmik Date: Mon, 30 Nov 2015 22:40:28 +0100 > Convert the dma transfers to be dmaengine based, now pxa has a dmaengine > slave driver. This makes this driver a bit more PXA agnostic. > > The driver was only compile tested. The risk is quite small as no > current PXA platform I'm a

Re: [PATCH net] bridge: Only call /sbin/bridge-stp for the initial network namespace

2015-12-02 Thread David Miller
From: ebied...@xmission.com (Eric W. Biederman) Date: Mon, 30 Nov 2015 15:38:15 -0600 > + if (dev_net(br->dev) == &init_net) Please respin this using net_eq() as Hannes pointed out. Thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to major

Re: [PATCH v2 net-next] tcp: suppress too verbose messages in tcp_send_ack()

2015-12-02 Thread David Miller
From: Eric Dumazet Date: Mon, 30 Nov 2015 08:57:28 -0800 > From: Eric Dumazet > > If tcp_send_ack() can not allocate skb, we properly handle this > and setup a timer to try later. > > Use __GFP_NOWARN to avoid polluting syslog in the case host is > under memory pressure, so that pertinent mess

Re: [PATCH net] ipv6: kill sk_dst_lock

2015-12-02 Thread David Miller
From: Eric Dumazet Date: Mon, 30 Nov 2015 08:35:15 -0800 > From: Eric Dumazet > > While testing the np->opt RCU conversion, I found that UDP/IPv6 was > using a mixture of xchg() and sk_dst_lock to protect concurrent changes > to sk->sk_dst_cache, leading to possible corruptions and crashes. >

Re: [PATCH] sctp: convert sack_needed and sack_generation to bits

2015-12-02 Thread David Miller
From: Marcelo Ricardo Leitner Date: Mon, 30 Nov 2015 12:17:06 -0200 > They don't need to be any bigger than that and with this we start a new > bitfield for tracking association runtime stuff, like zero window > situation. > > Signed-off-by: Marcelo Ricardo Leitner Applied. -- To unsubscribe f

Re: [PATCH] sctp: use GFP_USER for user-controlled kmalloc

2015-12-02 Thread David Miller
From: Marcelo Ricardo Leitner Date: Mon, 30 Nov 2015 14:32:54 -0200 > Dmitry Vyukov reported that the user could trigger a kernel warning by > using a large len value for getsockopt SCTP_GET_LOCAL_ADDRS, as that > value directly affects the value used as a kmalloc() parameter. > > This patch thu

Re: [PATCH net] ipv6: add complete rcu protection around np->opt

2015-12-02 Thread David Miller
From: Eric Dumazet Date: Sun, 29 Nov 2015 19:37:57 -0800 > From: Eric Dumazet > > This patch addresses multiple problems : > > UDP/RAW sendmsg() need to get a stable struct ipv6_txoptions > while socket is not locked : Other threads can change np->opt > concurrently. Dmitry posted a syzkaller

Re: [PATCH net] bpf: fix allocation warnings in bpf maps and integer overflow

2015-12-02 Thread David Miller
From: Alexei Starovoitov Date: Sun, 29 Nov 2015 16:59:35 -0800 > From: Alexei Starovoitov > > For large map->value_size the user space can trigger memory allocation > warnings like: ... > To avoid never succeeding kmalloc with order >= MAX_ORDER check that > elem->value_size and computed elem

Re: BUG: soft lockup happened for ixgbevf

2015-12-02 Thread Ding Tianhong
On 2015/12/2 1:25, Alexander Duyck wrote: > On 11/30/2015 11:12 PM, Ding Tianhong wrote: >> Hi Everyone: >> >> I found this problem when using the Testgine to send package to the 82599 >> ethernet: >> 1.wait for the speed to 10G/bit, it's ok. >> 2.then down the eth and then up, loop for several ti

Ethernet MII still negotiating status?

2015-12-02 Thread Phillip Susi
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 I'm looking into a problem with the mediatomb package not starting up correctly on recent Ubuntu releases, and I see that the problem is that the service depends on a network up event generated by network-manager to start. While it actually depends

Re: [PATCH v1 1/6] net: Generalize udp based tunnel offload

2015-12-02 Thread Alexei Starovoitov
On Wed, Dec 02, 2015 at 03:35:53PM -0800, John Fastabend wrote: > [...] > > BPF. Implementing protocol generic offloads are not just a HW concern > > either, adding kernel GRO code for every possible protocol that comes > > along doesn't scale well. This becomes especially obvious when we > > consi

Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2015-12-02

2015-12-02 Thread Jeff Kirsher
On Wed, 2015-12-02 at 16:45 -0500, David Miller wrote: > From: Jeff Kirsher > Date: Wed,  2 Dec 2015 04:30:53 -0800 > > > This series contains updates to ixgbe and ixgbevf only. > > I've marked this "changed requested" pending resolution of the ipv6 > header parsing feedback Alexander gave. v2

Re: ipsec impact on performance

2015-12-02 Thread Rick Jones
On 12/02/2015 03:56 AM, David Laight wrote: From: Sowmini Varadhan Sent: 01 December 2015 18:37 ... I was using esp-null merely to not have the crypto itself perturb the numbers (i.e., just focus on the s/w overhead for now), but here are the numbers for the stock linux kernel stack

Re: [net-next 13/15] ixgbe: Handle extended IPv6 headers in tx path

2015-12-02 Thread Rustad, Mark D
Alexander Duyck wrote: > This doesn't look right. How come this doesn't match the implementation you > did for the ixgbevf driver? If I am not mistaken this approach had issues > where it could spin forever didn't it? Yes, this is the original version of the patch instead of the V4 of the pa

Re: [PATCH v1 1/6] net: Generalize udp based tunnel offload

2015-12-02 Thread Tom Herbert
On Wed, Dec 2, 2015 at 3:35 PM, John Fastabend wrote: > [...] > >>> >>> I wonder why we need protocol generic offloads? I know there are >>> currently a lot of overlay encapsulation protocols. Are there many more >>> coming? >>> >> Yes, and assume that there are more coming with an unbounded limit

Re: gigaset: freeing an active object

2015-12-02 Thread Peter Hurley
On 11/30/2015 01:01 PM, Paul Bolle wrote: > On ma, 2015-11-30 at 00:23 +0100, Paul Bolle wrote: >> Relevant part of dmesg attached at the end of this message. This >> should give me (and Tilman too?) an entry to get to bottom of this. >> Since this is relevant for anyone with just the ser-gigaset

Re: [PATCH v1 1/6] net: Generalize udp based tunnel offload

2015-12-02 Thread John Fastabend
[...] >> >> I wonder why we need protocol generic offloads? I know there are >> currently a lot of overlay encapsulation protocols. Are there many more >> coming? >> > Yes, and assume that there are more coming with an unbounded limit > (for instance I just noticed today that there is a netdev1.1

Re: [patch net-next v2 00/28] bonding/team offload + mlxsw implementation

2015-12-02 Thread David Miller
From: Jiri Pirko Date: Wed, 2 Dec 2015 23:53:53 +0100 > Dave, why this was marked "Changes Requested" in patchwork? Because you gotta kill that BUG_ON() as per our discussion. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.o

Re: [patch net-next v2 4/6] mlxsw: core: Implement temperature hwmon interface

2015-12-02 Thread David Miller
From: Jiri Pirko Date: Wed, 2 Dec 2015 23:35:31 +0100 > Why to continue to work if we know that we are making things wrong > (incorrect structure initialization in this case) ? What am I missing? The user might be able to continue to have connectivity and report the problem. BUG_ON() is really

Re: [patch net-next v2 01/28] net: dev: Check CHANGEUPPER notifier return value

2015-12-02 Thread John Fastabend
On 15-12-02 12:07 PM, Jiri Pirko wrote: > From: Ido Schimmel > > switchdev drivers reflect the newly requested topology to hardware when > CHANGEUPPER is received, after software links were already formed. > However, the operation can fail and user will not be notified, as the > return value of t

[PATCH 3.13.y-ckt 43/86] can: Use correct type in sizeof() in nla_put()

2015-12-02 Thread Kamal Mostafa
3.13.11-ckt31 -stable review patch. If anyone has any objections, please let me know. -- From: Marek Vasut commit 562b103a21974c2f9cd67514d110f918bb3e1796 upstream. The sizeof() is invoked on an incorrect variable, likely due to some copy-paste error, and this might result in

Re: [PATCH net-next 0/3] vxlan: IPv6 fill_metadata_dst support

2015-12-02 Thread Pravin Shelar
On Wed, Dec 2, 2015 at 10:12 AM, Jiri Benc wrote: > This adds IPv6 support to ndo_fill_metadata_dst in vxlan together with > restructuring to avoid duplicate code. > We need the ndo_fill_metadata_dst IPv6 support in net branch, since OVS IPv6 tunnel support is there already. Is there reason you

Re: [PATCH net] openvswitch: fix hangup on vxlan/gre/geneve device deletion

2015-12-02 Thread Pravin Shelar
On Tue, Dec 1, 2015 at 9:33 AM, Paolo Abeni wrote: > Each openvswitch tunnel vport (vxlan,gre,geneve) holds a reference > to the underlying tunnel device, but never released it when such > device is deleted. > Deleting the underlying device via the ip tool cause the kernel to > hangup in the netde

Re: [PATCH 00/13] mvneta Buffer Management and enhancements

2015-12-02 Thread Gregory CLEMENT
Hi Marcin, On mer., déc. 02 2015, Marcin Wojtas wrote: >>> 2. Change condition in mvebu_mbus_get_dram_win_info to: if (cs->base <= phyaddr && phyaddr <= (cs->base + cs->size -1)) >>> >>> I think it would be the best solution. >> >> So I applied the following patch: >> --- a/drivers/bu

Re: [patch net-next v2 00/28] bonding/team offload + mlxsw implementation

2015-12-02 Thread Jiri Pirko
Dave, why this was marked "Changes Requested" in patchwork? Thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[3.13.y-ckt stable] Patch "can: Use correct type in sizeof() in nla_put()" has been added to staging queue

2015-12-02 Thread Kamal Mostafa
This is a note to let you know that I have just added a patch titled can: Use correct type in sizeof() in nla_put() to the linux-3.13.y-queue branch of the 3.13.y-ckt extended stable tree which can be found at: http://kernel.ubuntu.com/git/ubuntu/linux.git/log/?h=linux-3.13.y-queue Thi

Re: [patch net-next v2 4/6] mlxsw: core: Implement temperature hwmon interface

2015-12-02 Thread Jiri Pirko
Wed, Dec 02, 2015 at 11:28:47PM CET, da...@davemloft.net wrote: >From: Jiri Pirko >Date: Wed, 2 Dec 2015 23:25:52 +0100 > >> Again, there is no possible way to achieve this bugon, other than a >> stack corruption. I don't think we should try to cope with a stack >> corruption. > >Then remove the a

Re: [patch net-next v2 4/6] mlxsw: core: Implement temperature hwmon interface

2015-12-02 Thread David Miller
From: Jiri Pirko Date: Wed, 2 Dec 2015 23:25:52 +0100 > Again, there is no possible way to achieve this bugon, other than a > stack corruption. I don't think we should try to cope with a stack > corruption. Then remove the assertion. -- To unsubscribe from this list: send the line "unsubscribe n

Re: [patch net-next v2 4/6] mlxsw: core: Implement temperature hwmon interface

2015-12-02 Thread Jiri Pirko
Wed, Dec 02, 2015 at 11:05:48PM CET, gerlitz...@gmail.com wrote: >On Sun, Nov 29, 2015 at 12:49 PM, Jiri Pirko wrote: >> Sun, Nov 29, 2015 at 10:04:07AM CET, gerlitz...@gmail.com wrote: >>>[..] +enum mlxsw_hwmon_attr_type { + MLXSW_HWMON_ATTR_TYPE_TEMP, + MLXSW_HWMON_ATT

Re: ipsec impact on performance

2015-12-02 Thread Eric Dumazet
On Wed, 2015-12-02 at 16:12 -0500, Sowmini Varadhan wrote: > IPv6 would be an interesting academic exercise Really, you made my day ! -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.ke

Re: [PATCH 00/13] mvneta Buffer Management and enhancements

2015-12-02 Thread Marcin Wojtas
Hi Gregory, 2015-12-02 17:21 GMT+01:00 Gregory CLEMENT : > Hi, > > On mer., déc. 02 2015, Gregory CLEMENT > wrote: > >>> >>> So far the issue may have been not noticed, because in every IO driver >>> using mvebu_mbus_dram_info for configuring MBUS windows, there's >>> following substraction: >>

[PATCH net-next v3 4/5] ila: Add generic ILA translation facility

2015-12-02 Thread Tom Herbert
This patch implements an ILA tanslation table. This table can be configured with identifier to locator mappings, and can be be queried to resolve a mapping. Queries can be parameterized based on interface, direction (incoming or outoing), and matching locator. The table is implemented using rhasht

[PATCH net-next v3 0/5] ila: Optimization to preserve value of early demux

2015-12-02 Thread Tom Herbert
In the current implementation of ILA, LWT is used to perform translation on both the input and output paths. This is functional, however there is a big performance hit in the receive path. Early demux occurs before the routing lookup (a hit actually obviates the route lookup). Therefore the stack c

[PATCH net-next v3 2/5] rhashtable: add function to replace an element

2015-12-02 Thread Tom Herbert
Add the rhashtable_replace_fast function. This replaces one object in the table with another atomically. The hashes of the new and old objects must be equal. Signed-off-by: Tom Herbert --- include/linux/rhashtable.h | 82 ++ 1 file changed, 82 insertio

[PATCH net-next v3 1/5] ila: Create net/ipv6/ila directory

2015-12-02 Thread Tom Herbert
Create ila directory in preparation for supporting other hooks in the kernel than LWT for doing ILA. This includes: - Moving ila.c to ila/ila_lwt.c - Splitting out some common functions into ila_common.c Signed-off-by: Tom Herbert --- net/ipv6/Makefile | 2 +- net/ipv6/ila.c

[PATCH net-next v3 3/5] netlink: add a start callback for starting a netlink dump

2015-12-02 Thread Tom Herbert
The start callback allows the caller to set up a context for the dump callbacks. Presumably, the context can then be destroyed in the done callback. Signed-off-by: Tom Herbert --- include/linux/netlink.h | 2 ++ include/net/genetlink.h | 2 ++ net/netlink/af_netlink.c | 4 net/netlink/

[PATCH net-next v3 5/5] net: ILA iptables target

2015-12-02 Thread Tom Herbert
Add two target ILAIN and ILAOUT which hook into the ILA module. Signed-off-by: Tom Herbert --- net/netfilter/Kconfig | 12 net/netfilter/Makefile | 1 + net/netfilter/xt_ILA.c | 82 ++ 3 files changed, 95 insertions(+) create mode 10064

Re: [net-next v4 4/8] dpaa_eth: add driver's Tx queue selection

2015-12-02 Thread Scott Wood
On Mon, 2015-11-02 at 19:31 +0200, Madalin Bucur wrote: > Allow the selection of the transmission queue based on the CPU id. Explain why. > > Signed-off-by: Madalin Bucur > --- > drivers/net/ethernet/freescale/dpaa/Kconfig | 10 ++ > drivers/net/ethernet/freescale/dpaa/dpaa_e

Re: ipsec impact on performance

2015-12-02 Thread Sowmini Varadhan
On (12/02/15 14:01), Tom Herbert wrote: > No, please don't persist is this myopic "we'll get to IPv6 later" > model! IPv6 is a real protocol, it has significant deployment of the > Internet, and there are now whole data centers that are IPv6 only > (e.g. FB), and there are plenty of use cases of IP

Re: [patch net-next v2 4/6] mlxsw: core: Implement temperature hwmon interface

2015-12-02 Thread Or Gerlitz
On Sun, Nov 29, 2015 at 12:49 PM, Jiri Pirko wrote: > Sun, Nov 29, 2015 at 10:04:07AM CET, gerlitz...@gmail.com wrote: >>[..] >>> +enum mlxsw_hwmon_attr_type { >>> + MLXSW_HWMON_ATTR_TYPE_TEMP, >>> + MLXSW_HWMON_ATTR_TYPE_TEMP_MAX, >>> +}; >>> + >>> +static void mlxsw_hwmon_attr_add(st

Re: ipsec impact on performance

2015-12-02 Thread Tom Herbert
On Wed, Dec 2, 2015 at 1:47 PM, Sowmini Varadhan wrote: > On (12/02/15 13:44), Tom Herbert wrote: >> > IPv6 would be an interesting academic exercise, but it's going >> > to be a while before we get RDS-TCP to go over IPv6. >> > >> Huh? Who said anything about RDS-TCP? I thought you were trying to

pull-request: mac80211 2015-12-02

2015-12-02 Thread Johannes Berg
> Please add them and send a new pull request, I'll just skip over your > existing one. Ok, here we go, below a new pull request. I've added the two commits I spoke about, fixing scanning and an uninitialized cookie variable that lead to problems if the value happened to be 0. Let me know if th

Re: ipsec impact on performance

2015-12-02 Thread Sowmini Varadhan
On (12/02/15 13:44), Tom Herbert wrote: > > IPv6 would be an interesting academic exercise, but it's going > > to be a while before we get RDS-TCP to go over IPv6. > > > Huh? Who said anything about RDS-TCP? I thought you were trying to > improve IPsec performance... yes, and it would be nice to f

Re: [net-next 00/15][pull request] Intel Wired LAN Driver Updates 2015-12-02

2015-12-02 Thread David Miller
From: Jeff Kirsher Date: Wed, 2 Dec 2015 04:30:53 -0800 > This series contains updates to ixgbe and ixgbevf only. I've marked this "changed requested" pending resolution of the ipv6 header parsing feedback Alexander gave. -- To unsubscribe from this list: send the line "unsubscribe netdev" in t

Re: ipsec impact on performance

2015-12-02 Thread Tom Herbert
On Wed, Dec 2, 2015 at 1:12 PM, Sowmini Varadhan wrote: > On (12/02/15 13:07), Tom Herbert wrote: >> That's easy enough to add to flow dissector, but is SPI really >> intended to be used an L4 entropy value? We would need to consider the > > yes. To quote https://en.wikipedia.org/wiki/Security_Par

Re: ipsec impact on performance

2015-12-02 Thread Sowmini Varadhan
On (12/02/15 13:07), Tom Herbert wrote: > That's easy enough to add to flow dissector, but is SPI really > intended to be used an L4 entropy value? We would need to consider the yes. To quote https://en.wikipedia.org/wiki/Security_Parameter_Index "This works like port numbers in TCP and UDP connec

[P.A. Semi] Does the ethernet interface work on your Electra, Chitra, Nemo, and Athena board?

2015-12-02 Thread Christian Zigotzky
Hi Michael, Thanks a lot for your answer. We tested some vanilla mainline 4.3 kernels on an Electra reference board. Additionally we tested some patched 4.3 kernels on our Nemo boards. It would be really fantastic, if you could fix the problem. Cheers, Christian On 02 December 2015 at 09:1

Re: [PATCH] i40e: Fix i40e_print_features() VEB mode output

2015-12-02 Thread Joe Perches
On Wed, 2015-12-02 at 15:48 -0500, David Miller wrote: > From: Joe Perches 02 Dec 2015 02:12:29 -0800 > > On Wed, 2015-12-02 at 01:56 -0800, Jeff Kirsher wrote: > >> On Wed, 2015-12-02 at 00:38 -0800, Joe Perches wrote: > >> > Noticed-by: Sergei Shtylyov   > >> Don't you mean Reported-by?  I am no

Re: ipsec impact on performance

2015-12-02 Thread Tom Herbert
On Wed, Dec 2, 2015 at 12:50 PM, Sowmini Varadhan wrote: > On (12/02/15 12:41), David Laight wrote: >> You are getting 0.7 Gbps with ass-ccm-a-128, scale the esp-null back to >> that and it would use 7/18*71 = 27% of the cpu. >> So 69% of the cpu in the a-128 case is probably caused by the >> encr

Re: [PATCH v2 4/5] net: thunderx: Switchon carrier only upon interface link up

2015-12-02 Thread David Miller
From: Pavel Fedin Date: Wed, 02 Dec 2015 14:30:37 +0300 > Hello again! > >> Subject: RE: [PATCH v2 4/5] net: thunderx: Switchon carrier only upon >> interface link up >> >> Just a reminder, we have issue with this one too, which is not addressed >> yet. > > I have examined the problem tho

Re: ipsec impact on performance

2015-12-02 Thread Sowmini Varadhan
On (12/02/15 12:41), David Laight wrote: > You are getting 0.7 Gbps with ass-ccm-a-128, scale the esp-null back to > that and it would use 7/18*71 = 27% of the cpu. > So 69% of the cpu in the a-128 case is probably caused by the > encryption itself. > Even if the rest of the code cost nothing you'd

Re: [PATCH v2 0/5] net: thunderx: Miscellaneous fixes

2015-12-02 Thread David Miller
I'm not even looking at this patch series while it still causes unresolved bugs. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] i40e: Fix i40e_print_features() VEB mode output

2015-12-02 Thread David Miller
From: Joe Perches Date: Wed, 02 Dec 2015 02:12:29 -0800 > On Wed, 2015-12-02 at 01:56 -0800, Jeff Kirsher wrote: >> On Wed, 2015-12-02 at 00:38 -0800, Joe Perches wrote: >> > Commit 7fd89545f337 ("i40e: remove BUG_ON from feature string >> > building") >> > added defective output when I40E_FLAG_V

Re: pull-request: mac80211 2015-11-26

2015-12-02 Thread David Miller
From: Johannes Berg Date: Wed, 02 Dec 2015 10:46:50 +0100 >> >> A small set of fixes for 4.4: >>  * remove NL80211_FEATURE_FULL_AP_CLIENT_STATE again, it >>    was broken and needs more work, we'll enable it for 4.5 >>  * fix call_r

Re: [RESEND PATCH] arm64: bpf: add 'store immediate' instruction

2015-12-02 Thread David Miller
From: Will Deacon Date: Wed, 2 Dec 2015 09:15:18 + > On Tue, Dec 01, 2015 at 02:20:40PM -0800, Shi, Yang wrote: >> On 11/30/2015 2:24 PM, Yang Shi wrote: >> >aarch64 doesn't have native store immediate instruction, such operation >> >has to be implemented by the below instruction sequence: >>

Re: [net-next 00/17][pull request] Intel Wired LAN Driver Updates 2015-12-01

2015-12-02 Thread David Miller
From: Jeff Kirsher Date: Wed, 2 Dec 2015 00:25:18 -0800 > This series contains updates to i40e and i40evf only. Pulled, thanks Jeff. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.ker

Re: [PATCH v4 net 0/6] Marvell Armada XP/370/38X Neta fixes

2015-12-02 Thread David Miller
From: Gregory CLEMENT Date: Wed, 02 Dec 2015 09:16:06 +0100 > Hi David, > > On mer., déc. 02 2015, David Miller wrote: > >> From: Marcin Wojtas >> Date: Mon, 30 Nov 2015 13:27:40 +0100 >> >>> I'm sending v4 with corrected commit log of the last patch, in order >>> to avoid possible conflict

Re: [patch net-next v2 12/28] bonding: fill-up LAG changeupper info struct and pass it along

2015-12-02 Thread Nikolay Aleksandrov
On 12/02/2015 09:07 PM, Jiri Pirko wrote: > From: Jiri Pirko > > Initialize netdev_lag_upper_info structure by TX type according to > current bonding mode and pass it along via netdev_master_upper_dev_link. > > Signed-off-by: Jiri Pirko > --- > v1->v2: > -added mode broadcast suggested by Nik >

Re: [patch net-next v2 01/28] net: dev: Check CHANGEUPPER notifier return value

2015-12-02 Thread Nikolay Aleksandrov
On 12/02/2015 09:07 PM, Jiri Pirko wrote: > From: Ido Schimmel > > switchdev drivers reflect the newly requested topology to hardware when > CHANGEUPPER is received, after software links were already formed. > However, the operation can fail and user will not be notified, as the > return value of

Re: [patch net-next v2 02/28] net: Add support for CHANGEUPPER notifier error injection

2015-12-02 Thread Nikolay Aleksandrov
On 12/02/2015 09:07 PM, Jiri Pirko wrote: > From: Ido Schimmel > > Since CHANGEUPPER can now fail, add support for it in the newly > introduced netdev notifier error injection infrastructure. > > Signed-off-by: Ido Schimmel > Signed-off-by: Jiri Pirko > --- > v1->v2: > -new patch > --- > Docu

[PATCH net-next 1/1] tipc: fix node reference count bug

2015-12-02 Thread Jon Maloy
Commit 5405ff6e15f40f2f ("tipc: convert node lock to rwlock") introduced a bug to the node reference counter handling. When a message is successfully sent in the function tipc_node_xmit(), we return directly after releasing the node lock, instead of continuing and decrementing the node reference co

[patch net-next v2 02/28] net: Add support for CHANGEUPPER notifier error injection

2015-12-02 Thread Jiri Pirko
From: Ido Schimmel Since CHANGEUPPER can now fail, add support for it in the newly introduced netdev notifier error injection infrastructure. Signed-off-by: Ido Schimmel Signed-off-by: Jiri Pirko --- v1->v2: -new patch --- Documentation/fault-injection/notifier-error-inject.txt | 1 + lib/net

[patch net-next v2 27/28] mlxsw: spectrum: Implement FDB add/remove/dump for LAG

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Implement FDB offloading for lagged ports, including learning LAG FDB entries, adding/removing static FDB entries and dumping existing LAG FDB entries. Signed-off-by: Jiri Pirko --- .../ethernet/mellanox/mlxsw/spectrum_switchdev.c | 182 + 1 file changed,

[patch net-next v2 15/28] team: rtnl_lock for options set

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko During options set, there will be needed to hold rtnl_mutex in order to safely call netdev notifiers. Signed-off-by: Jiri Pirko --- drivers/net/team/team.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/team/team.c b/drivers/net/tea

[patch net-next v2 19/28] bonding: set inactive flags on release

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Be correct and symmetric to enslave and set inactive flags during release. That gives LAG offload drivers - lower state change listeners - possibility to do proper cleanup. Signed-off-by: Jiri Pirko --- drivers/net/bonding/bond_main.c | 2 ++ 1 file changed, 2 insertions(+) d

[patch net-next v2 17/28] bonding: allow notifications for bond_set_slave_link_state

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Similar to state notifications. We allow caller to indicate if the notification should happen now or later, depending on if he holds rtnl mutex or not. Introduce bond_slave_link_notify function (similar to bond_slave_state_notify) which is later on called with rtnl mutex and goe

[patch net-next v2 10/28] net: add info struct for LAG changeupper

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko This struct will be shared by bonding and team to pass internal information to notifier listeners. Signed-off-by: Jiri Pirko --- include/linux/netdevice.h | 13 + 1 file changed, 13 insertions(+) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h i

[patch net-next v2 13/28] net: introduce change lower state notifier

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko When lower device like bonding slave, team/bridge port, etc changes its state, it is useful for others to notice this change. Currently this is implemented specificly for bonding as NETDEV_BONDING_INFO notifier. This patch aims to replace this specific usage and make this more ge

[patch net-next v2 21/28] mlxsw: core: Add support for packets received from LAG port

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Lower layer (pci) has information if the packet is received via LAG port. If that is the case, it fills up rx_info accordingly. However upper layer does not care about lag_id/port_index for received packets so convert it to local_port before passing it up. For that conversion, la

[patch net-next v2 26/28] mlxsw: spectrum: Implement LAG port join/leave

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Implement basic procedures for joining/leaving port to/from LAG. That includes HW setup of collector, core LAG mapping setup. Signed-off-by: Jiri Pirko --- v1->v2: - return NOTIFY_BAD in case join/leave fails as requested by Andy - change BUG_ON to WARN_ON as suggested by Or an

[patch net-next v2 24/28] mlxsw: reg: Add definition of LAG unicast record for SFD register

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko LAG-related records have specific format in SFD register. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/reg.h | 79 +++ 1 file changed, 71 insertions(+), 8 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlxsw/reg.h b/d

[patch net-next v2 23/28] mlxsw: reg: Add link aggregation configuration registers definitions

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Add definitions of SLDR, SLCR2, SLCOR registers that are used to configure LAG. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/reg.h | 293 ++ 1 file changed, 293 insertions(+) diff --git a/drivers/net/ethernet/mellanox/mlxsw/reg

[patch net-next v2 07/28] net: add netif_is_lag_port helper

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Some code does not mind if a device is bond slave or team port and treats them the same, as generic LAG ports. Signed-off-by: Jiri Pirko --- include/linux/netdevice.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h

[patch net-next v2 03/28] bonding: add 802.3ad support for 100G speeds

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Similar to other speeds, add 100G to bonding 802.3ad code. Signed-off-by: Jiri Pirko --- drivers/net/bonding/bond_3ad.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c index 940e2eb

[patch net-next v2 11/28] team: fill-up LAG changeupper info struct and pass it along

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Initialize netdev_lag_upper_info structure by TX type according to current team mode and pass it along via netdev_master_upper_dev_link. Signed-off-by: Jiri Pirko --- drivers/net/team/team.c | 23 --- drivers/net/team/team_mode_activebacku

[patch net-next v2 25/28] mlxsw: reg: Add definition of LAG unicast record for SFN register

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko LAG-related records have specific format in SFN register. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/reg.h | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/mellanox/mlxsw/reg.h b/drivers/net

[patch net-next v2 06/28] net: add netif_is_lag_master helper

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Some code does not mind if the master is bond or team and treats them the same, as generic LAG. Signed-off-by: Jiri Pirko --- include/linux/netdevice.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index e7eca8f.

[patch net-next v2 22/28] mlxsw: pci: Implement LAG processing for received packets

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Completion queue element for receive queue provides information if the packet was received via LAG port. Extract this info and pass it along to core. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/pci.c | 19 --- drivers/net/ethernet/mellanox

[patch net-next v2 28/28] mlxsw: spectrum: Implement LAG tx enabled lower state change

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Enabling/disabling TX on a LAG port means enabling/disabling distribution in our HW. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 81 +- 1 file changed, 79 insertions(+), 2 deletions(-) diff --git a/drivers/net/etherne

[patch net-next v2 08/28] net: propagate upper priv via netdev_master_upper_dev_link

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Eliminate netdev_master_upper_dev_link_private and pass priv directly as a parameter of netdev_master_upper_dev_link. Signed-off-by: Jiri Pirko --- drivers/net/bonding/bond_main.c | 2 +- drivers/net/team/team.c | 2 +- drivers/net/vrf.c | 2 +- includ

[patch net-next v2 20/28] mlxsw: spectrum: Add set_rx_mode ndo stub

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Add just a stub for now. This allows to pass check in dev_ifsioc, SIOCADDMULTI and SIOCDELMULTI cases. Teamd is using these to add LACP slow MAC. Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 5 + 1 file changed, 5 insertions(+) diff --git

[patch net-next v2 05/28] net: add netif_is_team_port helper

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Similar to other helpers, caller can use this to find out if device is team port. Signed-off-by: Jiri Pirko --- include/linux/netdevice.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index cfb56e0..e7eca8f 10064

[patch net-next v2 12/28] bonding: fill-up LAG changeupper info struct and pass it along

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Initialize netdev_lag_upper_info structure by TX type according to current bonding mode and pass it along via netdev_master_upper_dev_link. Signed-off-by: Jiri Pirko --- v1->v2: -added mode broadcast suggested by Nik --- drivers/net/bonding/bond_main.c | 45 +++

[patch net-next v2 18/28] bonding: implement lower state change propagation

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Let netdev notifier listeners know about link and slave state change. Signed-off-by: Jiri Pirko --- drivers/net/bonding/bond_main.c | 10 ++ include/net/bonding.h | 7 +++ 2 files changed, 17 insertions(+) diff --git a/drivers/net/bonding/bond_main.c b/

[patch net-next v2 14/28] net: introduce lower state changed info structure for LAG lowers

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko This is shared info structure for bonding and team. Serves to pass down info about link state and port activity to notification listeners. Signed-off-by: Jiri Pirko --- include/linux/netdevice.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/netdevice.h

[patch net-next v2 00/28] bonding/team offload + mlxsw implementation

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko This patchset introduces needed infrastructure for link aggregation offload - for both team and bonding. It also implements the offload in mlxsw driver. Particulary, this patchset introduces possibility for upper driver (bond/team/bridge/..) to pass type-specific info down to no

[patch net-next v2 01/28] net: dev: Check CHANGEUPPER notifier return value

2015-12-02 Thread Jiri Pirko
From: Ido Schimmel switchdev drivers reflect the newly requested topology to hardware when CHANGEUPPER is received, after software links were already formed. However, the operation can fail and user will not be notified, as the return value of the notifier is not checked. Add this check and roll

[patch net-next v2 16/28] team: implement lower state change propagation

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Let netdev notifier listeners know about link-up and port-enable state changes. Signed-off-by: Jiri Pirko --- drivers/net/team/team.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c ind

[patch net-next v2 04/28] net: add netif_is_team_master helper

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Similar to other helpers, caller can use this to find out if device is team master. Signed-off-by: Jiri Pirko --- drivers/net/team/team.c | 1 + include/linux/netdevice.h | 8 2 files changed, 9 insertions(+) diff --git a/drivers/net/team/team.c b/drivers/net/team/

[patch net-next v2 09/28] net: add possibility to pass information about upper device via notifier

2015-12-02 Thread Jiri Pirko
From: Jiri Pirko Sometimes the drivers and other code would find it handy to know some internal information about upper device being changed. So allow upper-code to pass information down to notifier listeners during linking. Signed-off-by: Jiri Pirko --- v1->v2: - change "is upper info" to "upp

  1   2   3   >