Re: macvlan: Fix device ref leak when purging bc_queue

2017-04-25 Thread Joe.Ghalam
From: David Miller Sent: Tuesday, April 25, 2017 7:42 AM To: herb...@gondor.apana.org.au Cc: Ghalam, Joe; Wichmann, Clifford; netdev@vger.kernel.org Subject: Re: macvlan: Fix device ref leak when purging bc_queue > Applied and queued up for -stable, thanks Herbert. Herbert and David, Glad to rep

Re: [PATCH net] sfc: tx ring can only have 2048 entries for all EF10 NICs

2017-04-25 Thread David Miller
From: Bert Kenward Date: Tue, 25 Apr 2017 13:44:54 +0100 > Fixes: dd248f1bc65b ("sfc: Add PCI ID for Solarflare 8000 series 10/40G NIC") > Reported-by: Patrick Talbert > Signed-off-by: Bert Kenward Applied, thanks.

Re: [PATCH v4 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread David Miller
John this isn't how it works. When you submit new versions of a patch that are part of a patch series, you must resubmit the entire series not just the patches which are changing. Thanks.

Re: [PATCH 2/2] ARM: dts: Add the ethernet and ethernet PHY to the cygnus core DT.

2017-04-25 Thread Jon Mason
On Tue, Apr 25, 2017 at 5:40 AM, Sergei Shtylyov wrote: > Hello. > > On 4/25/2017 12:50 AM, Eric Anholt wrote: > >> Cygnus has a single amac controller connected to the B53 switch with 2 >> PHYs. On the BCM911360_EP platform, those two PHYs are connected to >> the external ethernet jacks. > > >

Re: [PATCH] macsec: avoid heap overflow in skb_to_sgvec

2017-04-25 Thread Jason A. Donenfeld
On Tue, Apr 25, 2017 at 5:12 PM, Sabrina Dubroca wrote: >> https://patchwork.ozlabs.org/patch/754861/ > > Yes, that prevents the overflow, but now you're just dropping > packets. Right, it's a so-called "defense-in-depth" measure. > I'll review that later, let's fix the overflow without > breaki

Re: [PATCH] macsec: avoid heap overflow in skb_to_sgvec

2017-04-25 Thread Jason A. Donenfeld
Hi Sabrina, On Tue, Apr 25, 2017 at 4:53 PM, Sabrina Dubroca wrote: > Ugh, good catch :/ > > AFAICT this patch doesn't really help, because NETIF_F_FRAGLIST > doesn't get tested in paths that can lead to triggering this. You're right. This fixes the xmit() path, but not the receive path, which a

Re: [Intel-wired-lan] [PATCH] ixgbe: initialize u64_stats_sync structures early at ixgbe_probe

2017-04-25 Thread Alexander Duyck
On Mon, Apr 24, 2017 at 4:00 PM, Singh, Krishneil K wrote: > > > > -Original Message- > From: Intel-wired-lan [mailto:intel-wired-lan-boun...@lists.osuosl.org] On > Behalf Of Liwei Song > Sent: Sunday, December 4, 2016 7:41 PM > To: Kirsher, Jeffrey T > Cc: netdev@vger.kernel.org; intel-

[PATCH v4 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread Jason A. Donenfeld
This is a defense-in-depth measure in response to bugs like 4d6fa57b4dab ("macsec: avoid heap overflow in skb_to_sgvec") Signed-off-by: Jason A. Donenfeld --- v4 fixes the commit message and moves the check into the inner-most if. net/core/skbuff.c | 12 +++- 1 file changed, 11 insertio

[PATCHv2 net] bridge: move bridge multicast cleanup to ndo_uninit

2017-04-25 Thread Xin Long
During removing a bridge device, if the bridge is still up, a new mdb entry still can be added in br_multicast_add_group() after all mdb entries are removed in br_multicast_dev_del(). Like the path: mld_ifc_timer_expire -> mld_sendpack -> ... br_multicast_rcv -> br_multicast_ad

Re: [RFC 0/4] xdp: use netlink extended ACK reporting

2017-04-25 Thread David Ahern
On 4/25/17 2:06 AM, Jakub Kicinski wrote: > Also - is anyone working on adding proper extack support to iproute2? > The code I have right now is a bit of a hack... This is what I have done: https://github.com/dsahern/iproute2/commits/ext-ack Basically, added the parsing code and then a new r

Re: [PATCH] macsec: avoid heap overflow in skb_to_sgvec

2017-04-25 Thread Sabrina Dubroca
2017-04-21, 23:14:48 +0200, Jason A. Donenfeld wrote: > While this may appear as a humdrum one line change, it's actually quite > important. An sk_buff stores data in three places: > > 1. A linear chunk of allocated memory in skb->data. This is the easiest >one to work with, but it precludes u

Re: pull-request: can-next 2017-04-25,pull-request: can-next 2017-04-25

2017-04-25 Thread David Miller
From: Marc Kleine-Budde Date: Tue, 25 Apr 2017 10:44:26 +0200 > this is a pull request of 21 patches for net-next/master. > > There are 4 patches by Stephane Grosjean for the PEAK PCAN-PCIe FD > CAN-FD boards. The next 7 patches are by Mario Huettel, which add > support for M_CAN IP version >= v

[vhost:vhost 6/19] drivers/net/virtio_net.c:2089:19: error: assignment of read-only location '*(ctx + (sizetype)rxq2vq(i))'

2017-04-25 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost head: 674c124665ca2ff1bcf81b1b92a207f71a326742 commit: e43eed6b8068f1c570551fe33bed12ef840c956b [6/19] virtio_net: allow specifying context for rx config: x86_64-acpi-redef (attached as .config) compiler: gcc-6 (Debian 6

Re: [PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread David Miller
From: Florian Westphal Date: Tue, 25 Apr 2017 16:17:49 +0200 > I'd have less of an issue with this if we'd be talking about > something computationally expensive, but this is about storing > an extra value inside a struct just to avoid one "shr" in insert path... Agreed, this shift is probably f

Re: [PATCH 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread David Miller
From: "Jason A. Donenfeld" Date: Tue, 25 Apr 2017 16:08:05 +0200 > This is a defense-in-depth measure in response to bugs like > 4d6fa57b4dab0d77f4d8e9d9c73d1e63f6fe8fee. > > Signed-off-by: Jason A. Donenfeld Please refer to commits in the form: $(SHA1_ID) ("Commit header line.") That is, 12

Re: [PATCH net-next] ipvlan: use pernet operations and restrict l3s hooks to master netns

2017-04-25 Thread David Miller
From: Florian Westphal Date: Thu, 20 Apr 2017 18:08:15 +0200 > commit 4fbae7d83c98c30efc ("ipvlan: Introduce l3s mode") added > registration of netfilter hooks via nf_register_hooks(). > > This API provides the illusion of 'global' netfilter hooks by placing the > hooks in all current and future

Re: macvlan: Fix device ref leak when purging bc_queue

2017-04-25 Thread David Miller
From: Herbert Xu Date: Thu, 20 Apr 2017 20:55:12 +0800 > When a parent macvlan device is destroyed we end up purging its > broadcast queue without dropping the device reference count on > the packet source device. This causes the source device to linger. > > This patch drops that reference coun

Re: [PATCH v3 07/29] x86: bpf_jit, use ENTRY+ENDPROC

2017-04-25 Thread Jiri Slaby
On 04/24/2017, 08:24 PM, David Miller wrote: > From: Jiri Slaby > Date: Mon, 24 Apr 2017 19:51:54 +0200 > >> For example what's the point of making the sk_load_word_positive_offset >> label a global, callable function? Note that this is exactly the reason >> why this particular two hunks look wei

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Or Gerlitz
On Tue, Apr 25, 2017 at 2:43 PM, Erez Shitrit wrote: > On Tue, Apr 25, 2017 at 2:14 PM, Or Gerlitz wrote: >> thanks for the info. Is this bug there since ipoib/bonding day one (and >> hence my bug...) >> or was indeed introduced later? if later, can you explain how >> fc791b633515 introduced th

Re: [PATCH v1] net: phy: fix auto-negotiation stall due to unavailable interrupt

2017-04-25 Thread David Miller
From: Alexander Kochetkov Date: Thu, 20 Apr 2017 14:00:04 +0300 > The Ethernet link on an interrupt driven PHY was not coming up if the Ethernet > cable was plugged before the Ethernet interface was brought up. > > The patch trigger PHY state machine to update link state if PHY was requested >

Re: [PATCH net-next v3 2/5] virtio-net: transmit napi

2017-04-25 Thread Willem de Bruijn
On Tue, Apr 25, 2017 at 4:36 AM, Jason Wang wrote: > > > On 2017年04月25日 01:49, Willem de Bruijn wrote: >> >> @@ -1371,8 +1419,10 @@ static int virtnet_close(struct net_device *dev) >> /* Make sure refill_work doesn't re-enable napi! */ >> cancel_delayed_work_sync(&vi->refill); >>

Re: [PATCH] usb: plusb: Add support for PL-27A1

2017-04-25 Thread David Miller
From: Roman Spychała Date: Thu, 20 Apr 2017 12:04:10 +0200 > From: Roman Spychała > > This patch adds support for the PL-27A1 by adding the appropriate > USB ID's. This chip is used in the goobay Active USB 3.0 Data Link > and Unitek Y-3501 cables. > > Signed-off-by: Roman Spychała Applied,

RE: [PATCH net-next 3/3] samples/bpf: check before defining offsetof

2017-04-25 Thread David Laight
From: Daniel Borkmann > Sent: 24 April 2017 15:41 > To: Alexander Alemayhu; netdev@vger.kernel.org > Cc: a...@fb.com > Subject: Re: [PATCH net-next 3/3] samples/bpf: check before defining offsetof > > On 04/24/2017 03:31 PM, Alexander Alemayhu wrote: > > Fixes the following warning > > > > samples

Re: Network cooling device and how to control NIC speed on thermal condition

2017-04-25 Thread Alan Cox
> I am looking on Linux thermal framework and on how to cool down the > system effectively when it hits thermal condition. Already existing > cooling methods cpu_cooling and clock_cooling are good. However, I > wanted to go further and dynamically control also a switch ports' > speed based on therm

[PATCH v3 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread Jason A. Donenfeld
This is a defense-in-depth measure in response to bugs like 4d6fa57b4dab0d77f4d8e9d9c73d1e63f6fe8fee. Signed-off-by: Jason A. Donenfeld --- Sorry for the completely stupid amount of churn - v1,v2,v3 in the span of two minutes. It's just that after noticing first that nsg needs to be checked, I al

Re: [RFC 3/4] nfp: make use of extended ack message reporting

2017-04-25 Thread David Miller
From: Jamal Hadi Salim Date: Tue, 25 Apr 2017 08:42:32 -0400 > So are we going to standardize these strings? No. > i.e what if some user has written a bash script that depends on this > string and it gets changed later. They can't do that. It's free form extra information an application may o

Re: [PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread Florian Westphal
Herbert Xu wrote: > On Tue, Apr 25, 2017 at 01:23:56PM +0200, Florian Westphal wrote: > > > > What extra cost? > > > > The only change is that ht->nelems has to be right-shifted by one, > > I don't think that warrants extra space in struct rhashtable, its > > already way too large (I think we can

[PATCH v2 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread Jason A. Donenfeld
This is a defense-in-depth measure in response to bugs like 4d6fa57b4dab0d77f4d8e9d9c73d1e63f6fe8fee. Signed-off-by: Jason A. Donenfeld --- net/core/skbuff.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index f86bf69cfb8d

admin

2017-04-25 Thread administrador
ATENCIÓN; Su buzón ha superado el límite de almacenamiento, que es de 5 GB definidos por el administrador, quien actualmente está ejecutando en 10.9GB, no puede ser capaz de enviar o recibir correo nuevo hasta que vuelva a validar su buzón de correo electrónico. Para revalidar su buzón de corre

[PATCH 4/5] macsec: check return value of skb_to_sgvec always

2017-04-25 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld --- drivers/net/macsec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/net/macsec.c b/drivers/net/macsec.c index dbab05afcdbe..d846f42b99ec 100644 --- a/drivers/net/macsec.c +++ b/drivers/net/macsec.c @@ -733,7 +733,12

[PATCH 2/5] ipsec: check return value of skb_to_sgvec always

2017-04-25 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld --- net/ipv4/ah4.c | 8 ++-- net/ipv4/esp4.c | 30 -- net/ipv6/ah6.c | 8 ++-- net/ipv6/esp6.c | 31 +-- 4 files changed, 53 insertions(+), 24 deletions(-) diff --git a/net/ipv4/ah4.c b/net/ipv4

[PATCH 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-04-25 Thread Jason A. Donenfeld
This is a defense-in-depth measure in response to bugs like 4d6fa57b4dab0d77f4d8e9d9c73d1e63f6fe8fee. Signed-off-by: Jason A. Donenfeld --- net/core/skbuff.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index f86bf69cfb8d..3c

[PATCH 3/5] rxrpc: check return value of skb_to_sgvec always

2017-04-25 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld --- net/rxrpc/rxkad.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/net/rxrpc/rxkad.c b/net/rxrpc/rxkad.c index 4374e7b9c7bf..dcf46c9c3ece 100644 --- a/net/rxrpc/rxkad.c +++ b/net/rxrpc/rxkad.c @@ -229,7 +229,9 @@ static int rxkad

[PATCH 5/5] virtio_net: check return value of skb_to_sgvec always

2017-04-25 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld --- drivers/net/virtio_net.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index f36584616e7d..1709fd0b4bf7 100644 --- a/drivers/net/virtio_net.c +++ b/drivers/net/virtio_net.c @@ -1081,

Re: [PATCH] net: bridge: suppress broadcast when multicast flood is disabled

2017-04-25 Thread Nikolay Aleksandrov
On 25/04/17 16:32, Mike Manning wrote: > On 24/04/17 20:52, Nikolay Aleksandrov wrote: >> On 24/04/17 17:09, Mike Manning wrote: >>> Flood suppression for packets that are not unicast needs to be handled >>> consistently by also not flooding broadcast packets. As broadcast is a >>> special case of

[PATCH net] ipv6: fix source routing

2017-04-25 Thread Sabrina Dubroca
Commit a149e7c7ce81 ("ipv6: sr: add support for SRH injection through setsockopt") introduced handling of IPV6_SRCRT_TYPE_4, but at the same time restricted it to only IPV6_SRCRT_TYPE_0 and IPV6_SRCRT_TYPE_4. Previously, ipv6_push_exthdr() and fl6_update_dst() would also handle other values (ie STR

Re: [RFC PATCH 3/7] net: add option to get information about timestamped packets

2017-04-25 Thread Miroslav Lichvar
On Mon, Apr 24, 2017 at 11:18:13AM -0400, Willem de Bruijn wrote: > On Mon, Apr 24, 2017 at 5:00 AM, Miroslav Lichvar wrote: > > Would "skb->data - skb->head - > > skb->mac_header + skb->len" always work as the L2 length for received > > packets at the time when the cmsg is prepared? > > (skb->da

Re: [B.A.T.M.A.N.] [PATCH net] net: batman-adv: Fix possible memleaks when fail to register_netdevice

2017-04-25 Thread Sven Eckelmann
On Dienstag, 25. April 2017 20:03:20 CEST gfree.w...@foxmail.com wrote: > From: Gao Feng > > Because the func batadv_softif_init_late allocate some resources and > it would be invoked in register_netdevice. So we need to invoke the > func batadv_softif_free instead of free_netdev to cleanup when

RE: [PATCH net-next] qed: fix invalid use of sizeof in qed_alloc_qm_data()

2017-04-25 Thread Mintz, Yuval
> sizeof() when applied to a pointer typed expression gives the size of the > pointer, not that of the pointed data. > > Signed-off-by: Wei Yongjun Thanks! Acked-by: Yuval Mintz I'd also mention that - Fixes: b5a9ee7cf3be ("qed: Revise QM configuration")

Re: [PATCH] net: bridge: suppress broadcast when multicast flood is disabled

2017-04-25 Thread Mike Manning
On 24/04/17 20:52, Nikolay Aleksandrov wrote: > On 24/04/17 17:09, Mike Manning wrote: >> Flood suppression for packets that are not unicast needs to be handled >> consistently by also not flooding broadcast packets. As broadcast is a >> special case of multicast, the same kernel parameter should b

Re: [PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread Herbert Xu
On Tue, Apr 25, 2017 at 01:23:56PM +0200, Florian Westphal wrote: > > What extra cost? > > The only change is that ht->nelems has to be right-shifted by one, > I don't think that warrants extra space in struct rhashtable, its > already way too large (I think we can reduce its size further). I see

[PATCH] ipv6: ensure message length for raw socket is at least sizeof(ipv6hdr)

2017-04-25 Thread Alexander Potapenko
rawv6_send_hdrinc() expects that the buffer copied from the userspace contains the IPv6 header, so if too few bytes are copied parts of the header may remain uninitialized. This bug has been detected with KMSAN. Signed-off-by: Alexander Potapenko --- For the record, the KMSAN report: ==

Re: Network cooling device and how to control NIC speed on thermal condition

2017-04-25 Thread Andrew Lunn
On Tue, Apr 25, 2017 at 10:36:28AM +0200, Waldemar Rymarkiewicz wrote: > Hi, > > I am not much aware of linux networking architecture so I'd like to > ask first before will start to dig into the code. Appreciate any > feedback. > > I am looking on Linux thermal framework and on how to cool down t

Re: [PATCH net-next v3 0/5] virtio-net tx napi

2017-04-25 Thread David Miller
From: Willem de Bruijn Date: Mon, 24 Apr 2017 13:49:25 -0400 > Add napi for virtio-net transmit completion processing. Series applied, thanks.

Re: [PATCH net-next v8 2/3] net sched actions: dump more than TCA_ACT_MAX_PRIO actions per batch

2017-04-25 Thread Jamal Hadi Salim
On 17-04-25 08:13 AM, Jiri Pirko wrote: Tue, Apr 25, 2017 at 01:54:06PM CEST, j...@mojatatu.com wrote: [..] -#define TCAA_MAX 1 +/* tcamsg flags stored in attribute TCA_ROOT_FLAGS + * + * TCA_FLAG_LARGE_DUMP_ON user->kernel to request for larger than TCA_ACT_MAX_PRIO + * actions in a dump.

RE: [PATCH net] driver/net: Fix possible memleaks when fail to register_netdevice

2017-04-25 Thread Gao Feng
> From: Gao Feng > > These drivers allocate kinds of resources in init routine, and free some > resources in the destructor of net_device. It may cause memleak when some > errors happen after register_netdevice invokes the init callback. Because only > the uninit callback is invoked in the error

Re: [PATCH v3 net] net: ipv6: regenerate host route if moved to gc list

2017-04-25 Thread Andrey Konovalov
On Mon, Apr 24, 2017 at 5:09 PM, David Ahern wrote: > Taking down the loopback device wreaks havoc on IPv6 routing. By > extension, taking down a VRF device wreaks havoc on its table. > > Dmitry and Andrey both reported heap out-of-bounds reports in the IPv6 > FIB code while running syzkaller fuzz

Re: [PATCH net-next 0/2] flower: add MPLS matching support

2017-04-25 Thread Jamal Hadi Salim
On 17-04-25 07:55 AM, Simon Horman wrote: [..] I agree something should be done wrt BOS. If the LABEL and TC are to be left as-is then I think a similar treatment of BOS - that is masking it - makes sense. I also agree with statements made earlier in the thread that it is unlikely that the unus

[PATCH net] sfc: tx ring can only have 2048 entries for all EF10 NICs

2017-04-25 Thread Bert Kenward
Fixes: dd248f1bc65b ("sfc: Add PCI ID for Solarflare 8000 series 10/40G NIC") Reported-by: Patrick Talbert Signed-off-by: Bert Kenward --- drivers/net/ethernet/sfc/efx.h | 5 - drivers/net/ethernet/sfc/workarounds.h | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a

Re: [RFC 3/4] nfp: make use of extended ack message reporting

2017-04-25 Thread Jamal Hadi Salim
Good stuff. Question below: On 17-04-25 04:06 AM, Jakub Kicinski wrote: Try to carry error messages to the user via the netlink extended [..] +nfp_net_check_config(struct nfp_net *nn, struct nfp_net_dp *dp, +struct netlink_ext_ack *extack) { /* XDP-enabled tests *

Re: net/ipv6: slab-out-of-bounds in ip6_tnl_xmit

2017-04-25 Thread Andrey Konovalov
On Tue, Apr 25, 2017 at 7:04 AM, Cong Wang wrote: > On Mon, Apr 24, 2017 at 9:47 AM, Cong Wang wrote: >> >> We use ipv4 dst in ip6_tunnel and cast an IPv4 neigh key as an >> IPv6 address... >> >> >> neigh = dst_neigh_lookup(skb_dst(skb), >>

net: pcnet32: [BUG] IRQ disabled when resizing ring buffer

2017-04-25 Thread Corentin Labbe
Hello When resizing ring buffer on qemu/pcnet32 with linux-next and CONFIG_DMA_API_DEBUG=y, I got the following trace: [8862.793779] [ cut here ] [ 8862.793784] WARNING: CPU: 0 PID: 26592 at /linux-next/include/linux/dma-mapping.h:505 pcnet32_set_ringparam+0xd55/0xda0 [ 8

[PATCH 2/3] can: usb: Kconfig: Add PCAN-USB X6 device in help text

2017-04-25 Thread Marc Kleine-Budde
From: Stephane Grosjean This patch adds a text line in the help section of the CAN_PEAK_USB config item describing the support of the PCAN-USB X6 adapter, which is already included in the Kernel since 4.9. Signed-off-by: Stephane Grosjean Signed-off-by: Marc Kleine-Budde --- drivers/net/can/u

[PATCH 3/3] net: can: usb: gs_usb: Fix buffer on stack

2017-04-25 Thread Marc Kleine-Budde
From: Maksim Salau Allocate buffers on HEAP instead of STACK for local structures that are to be sent using usb_control_msg(). Signed-off-by: Maksim Salau Cc: linux-stable # >= v4.8 Signed-off-by: Marc Kleine-Budde --- drivers/net/can/usb/gs_usb.c | 17 - 1 file changed, 12 i

[PATCH 1/3] can: usb: Add support of PCAN-Chip USB stamp module

2017-04-25 Thread Marc Kleine-Budde
From: Stephane Grosjean This patch adds the support of the PCAN-Chip USB, a stamp module for customer hardware designs, which communicates via USB 2.0 with the hardware. The integrated CAN controller supports the protocols CAN 2.0 A/B as well as CAN FD. The physical CAN connection is determined b

pull-request: can 2017-04-25

2017-04-25 Thread Marc Kleine-Budde
following changes since commit 38a98bceaf5f786b931d16826fbb46e73280849b: Merge branch 'dsa-b53-58xx-fixes' (2017-04-24 18:29:11 -0400) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/mkl/linux-can.git tags/linux-can-fixes-for-4.11-20170425 for yo

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Jan Kiszka
On 2017-04-25 13:42, Andy Shevchenko wrote: > On Tue, Apr 25, 2017 at 1:09 PM, Jan Kiszka wrote: >> On 2017-04-25 12:07, Jan Kiszka wrote: >>> On 2017-04-25 11:46, Andy Shevchenko wrote: On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: > On 2017-04-25 09:30, Andy Shevchenko wrot

Re: [PATCH net-next v8 2/3] net sched actions: dump more than TCA_ACT_MAX_PRIO actions per batch

2017-04-25 Thread Jiri Pirko
Tue, Apr 25, 2017 at 01:54:06PM CEST, j...@mojatatu.com wrote: >From: Jamal Hadi Salim > >When you dump hundreds of thousands of actions, getting only 32 per >dump batch even when the socket buffer and memory allocations allow >is inefficient. > >With this change, the user will get as many as poss

[PATCH net] net: batman-adv: Fix possible memleaks when fail to register_netdevice

2017-04-25 Thread gfree . wind
From: Gao Feng Because the func batadv_softif_init_late allocate some resources and it would be invoked in register_netdevice. So we need to invoke the func batadv_softif_free instead of free_netdev to cleanup when fail to register_netdevice. Signed-off-by: Gao Feng --- net/batman-adv/soft-int

[PATCH net] driver/net: Fix possible memleaks when fail to register_netdevice

2017-04-25 Thread gfree . wind
From: Gao Feng These drivers allocate kinds of resources in init routine, and free some resources in the destructor of net_device. It may cause memleak when some errors happen after register_netdevice invokes the init callback. Because only the uninit callback is invoked in the error handler of r

[PATCH net-next v7 1/3] net sched actions: Use proper root attribute table for actions

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim Bug fix for an issue which has been around for about a decade. We got away with it because the enumeration was larger than needed. Fixes: 7ba699c604ab ("[NET_SCHED]: Convert actions from rtnetlink to new netlink API") Suggested-by: Jiri Pirko Signed-off-by: Jamal Hadi Sa

Re: [PATCH net-next 0/2] flower: add MPLS matching support

2017-04-25 Thread Simon Horman
On Mon, Apr 24, 2017 at 07:07:43PM -0700, Jakub Kicinski wrote: > On Mon, 24 Apr 2017 22:06:08 -0400, Jamal Hadi Salim wrote: > > On 17-04-24 10:00 PM, Jamal Hadi Salim wrote: > > > On 17-04-24 09:48 PM, Jamal Hadi Salim wrote: > > > > > > > > Hrm. maybe I am wrong. > > > Lets say user sets all

[PATCH net-next v8 3/3] net sched actions: add time filter for action dumping

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim This adds support for filtering based on time since last used. When we are dumping a large number of actions it is useful to have the option of filtering based on when the action was last used to reduce the amount of data crossing to user space. With this patch the user sp

[PATCH net-next v8 0/3] net sched actions: improve dump performance

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim Changes since v7: - Jamal: Patch 1 went out twice. Resend without two copies of patch 1 changes since v6: - 1) DaveM: New rules for netlink messages. From now on we are going to start checking for bits that are not used and rejecting anyth

[PATCH net-next v8 2/3] net sched actions: dump more than TCA_ACT_MAX_PRIO actions per batch

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim When you dump hundreds of thousands of actions, getting only 32 per dump batch even when the socket buffer and memory allocations allow is inefficient. With this change, the user will get as many as possibly fitting within the given constraints available to the kernel. Th

[PATCH net-next v8 1/3] net sched actions: Use proper root attribute table for actions

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim Bug fix for an issue which has been around for about a decade. We got away with it because the enumeration was larger than needed. Fixes: 7ba699c604ab ("[NET_SCHED]: Convert actions from rtnetlink to new netlink API") Suggested-by: Jiri Pirko Signed-off-by: Jamal Hadi Sa

Re: [PATCH net-next v7 0/3] net sched actions: improve dump performance

2017-04-25 Thread Jamal Hadi Salim
Grr. I gitta git my git-foo right. Patch 1 repeated twice - I'll send v8. cheers, jamal

[PATCH net-next v7 1/3] net sched actions: Use proper root attribute table for actions

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim Bug fix for an issue which has been around for about a decade. We got away with it because the enumeration was larger than needed. Fixes: 7ba699c604ab ("[NET_SCHED]: Convert actions from rtnetlink to new netlink API") Suggested-by: Jiri Pirko Signed-off-by: Jamal Hadi Sa

[PATCH net-next v7 2/3] net sched actions: dump more than TCA_ACT_MAX_PRIO actions per batch

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim When you dump hundreds of thousands of actions, getting only 32 per dump batch even when the socket buffer and memory allocations allow is inefficient. With this change, the user will get as many as possibly fitting within the given constraints available to the kernel. Th

[PATCH net-next v7 3/3] net sched actions: add time filter for action dumping

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim This adds support for filtering based on time since last used. When we are dumping a large number of actions it is useful to have the option of filtering based on when the action was last used to reduce the amount of data crossing to user space. With this patch the user sp

[PATCH net-next v7 0/3] net sched actions: improve dump performance

2017-04-25 Thread Jamal Hadi Salim
From: Jamal Hadi Salim changes since v6: - 1) DaveM: New rules for netlink messages. From now on we are going to start checking for bits that are not used and rejecting anything we dont understand. In the future this is going to require major changes to user space code (tc etc).

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Erez Shitrit
On Tue, Apr 25, 2017 at 2:14 PM, Or Gerlitz wrote: > On Tue, Apr 25, 2017 at 2:11 PM, Erez Shitrit > wrote: >> On Tue, Apr 25, 2017 at 1:32 PM, Or Gerlitz wrote: >>> On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI wrote: From: Honggang Li Minimal hard_header_len set by bond_compu

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Andy Shevchenko
On Tue, Apr 25, 2017 at 1:09 PM, Jan Kiszka wrote: > On 2017-04-25 12:07, Jan Kiszka wrote: >> On 2017-04-25 11:46, Andy Shevchenko wrote: >>> On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: On 2017-04-25 09:30, Andy Shevchenko wrote: > On Tue, Apr 25, 2017 at 8:44 AM, Jan Kiszka >

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Andy Shevchenko
On Tue, Apr 25, 2017 at 1:07 PM, Jan Kiszka wrote: > On 2017-04-25 11:46, Andy Shevchenko wrote: >> On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: >>> On 2017-04-25 09:30, Andy Shevchenko wrote: On Tue, Apr 25, 2017 at 8:44 AM, Jan Kiszka wrote: > On 2017-04-24 23:27, Andy Shevchen

[PATCH net-next] drivers: net: xgene-v2: Fix error return code in xge_mdio_config()

2017-04-25 Thread Wei Yongjun
From: Wei Yongjun Fix to return error code -ENODEV from the no PHY found error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/net/ethernet/apm/xgene-v2/mdio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/apm/xg

Re: [PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread Florian Westphal
Herbert Xu wrote: > Florian Westphal wrote: > > no users in the tree, insecure_max_entries is always set to > > ht->p.max_size * 2 in rhtashtable_init(). > > > > Replace only spot that uses it with a ht->p.max_size check. > > I'd suggest that as this needs to be computed every time we insert >

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Or Gerlitz
On Tue, Apr 25, 2017 at 2:11 PM, Erez Shitrit wrote: > On Tue, Apr 25, 2017 at 1:32 PM, Or Gerlitz wrote: >> On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI wrote: >>> From: Honggang Li >>> >>> Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which >>> is smaller than IPOIB_HARD_

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Erez Shitrit
On Tue, Apr 25, 2017 at 1:32 PM, Or Gerlitz wrote: > On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI wrote: >> From: Honggang Li >> >> Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which >> is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the >> size of headroom t

Re: [PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread Herbert Xu
Florian Westphal wrote: > no users in the tree, insecure_max_entries is always set to > ht->p.max_size * 2 in rhtashtable_init(). > > Replace only spot that uses it with a ht->p.max_size check. I'd suggest that as this needs to be computed every time we insert an element that you keep the value

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Honggang LI
On Tue, Apr 25, 2017 at 01:32:59PM +0300, Or Gerlitz wrote: > On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI wrote: > > From: Honggang Li > > > > Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which > > is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the > > size

Re: [PATCH net-next 2/2] l2tp: define "l2tpeth" device type

2017-04-25 Thread James Chapman
On 24/04/17 13:16, Guillaume Nault wrote: > Export type of l2tpeth interfaces to userspace > (/sys/class/net//uevent). > > Signed-off-by: Guillaume Nault Acked-by: James Chapman

Re: [PATCH net-next 1/2] l2tp: set name_assign_type for devices created by l2tp_eth.c

2017-04-25 Thread James Chapman
On 24/04/17 13:16, Guillaume Nault wrote: > Export naming scheme used when creating l2tpeth interfaces > (/sys/class/net//name_assign_type). This let userspace know if > the device's name has been generated automatically or defined manually. > > Signed-off-by: Guillaume Nault Acked-by: James Chapm

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Or Gerlitz
On Tue, Apr 25, 2017 at 12:55 PM, Honggang LI wrote: > From: Honggang Li > > Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which > is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the > size of headroom to avoid skb_under_panic. sounds terrible, ipoib bonding is

Re: [PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Yuval Shaia
On Tue, Apr 25, 2017 at 05:55:55PM +0800, Honggang LI wrote: > From: Honggang Li > > Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which > is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the > size of headroom to avoid skb_under_panic. > > [ 122.871493] ipoib_

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Jan Kiszka
On 2017-04-25 12:07, Jan Kiszka wrote: > On 2017-04-25 11:46, Andy Shevchenko wrote: >> On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: >>> On 2017-04-25 09:30, Andy Shevchenko wrote: On Tue, Apr 25, 2017 at 8:44 AM, Jan Kiszka wrote: > On 2017-04-24 23:27, Andy Shevchenko wrote: >>>

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Jan Kiszka
On 2017-04-25 11:46, Andy Shevchenko wrote: > On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: >> On 2017-04-25 09:30, Andy Shevchenko wrote: >>> On Tue, Apr 25, 2017 at 8:44 AM, Jan Kiszka wrote: On 2017-04-24 23:27, Andy Shevchenko wrote: > On Mon, Apr 24, 2017 at 10:27 PM, Jan Kisz

Re: [PATCH] net: hso: fix module unloading

2017-04-25 Thread Johan Hovold
On Mon, Apr 24, 2017 at 09:18:39PM +0200, Andreas Kemnade wrote: > keep tty driver until usb driver is unregistered > rmmod hso > produces traces like this without that: Yeah, a blatant use-after-free. > Signed-off-by: Andreas Kemnade Reviewed-by: Johan Hovold > --- > drivers/net/usb/hso.c |

[PATCH] IB/IPoIB: Check the headroom size

2017-04-25 Thread Honggang LI
From: Honggang Li Minimal hard_header_len set by bond_compute_features is ETH_HLEN, which is smaller than IPOIB_HARD_LEN. ipoib_hard_header should check the size of headroom to avoid skb_under_panic. [ 122.871493] ipoib_hard_header: skb->head= 8808179d9400, skb->data= 8808179d9420, skb

Re: [PATCH] stmmac: Add support for SIMATIC IOT2000 platform

2017-04-25 Thread Andy Shevchenko
On Tue, Apr 25, 2017 at 12:00 PM, Jan Kiszka wrote: > On 2017-04-25 09:30, Andy Shevchenko wrote: >> On Tue, Apr 25, 2017 at 8:44 AM, Jan Kiszka wrote: >>> On 2017-04-24 23:27, Andy Shevchenko wrote: On Mon, Apr 24, 2017 at 10:27 PM, Jan Kiszka wrote: > The IOT2000 is industrial c

Re: [PATCH 2/2] ARM: dts: Add the ethernet and ethernet PHY to the cygnus core DT.

2017-04-25 Thread Sergei Shtylyov
On 4/25/2017 12:50 AM, Eric Anholt wrote: Cygnus has a single amac controller connected to the B53 switch with 2 PHYs. On the BCM911360_EP platform, those two PHYs are connected to the external ethernet jacks. Signed-off-by: Eric Anholt --- arch/arm/boot/dts/bcm-cygnus.dtsi | 60 +++

[PATCH net-next] rhashtable: remove insecure_max_entries param

2017-04-25 Thread Florian Westphal
no users in the tree, insecure_max_entries is always set to ht->p.max_size * 2 in rhtashtable_init(). Replace only spot that uses it with a ht->p.max_size check. Signed-off-by: Florian Westphal --- include/linux/rhashtable.h | 6 ++ lib/rhashtable.c | 6 -- 2 files changed, 2

Re: [PATCH 2/2] ARM: dts: Add the ethernet and ethernet PHY to the cygnus core DT.

2017-04-25 Thread Sergei Shtylyov
Hello. On 4/25/2017 12:50 AM, Eric Anholt wrote: Cygnus has a single amac controller connected to the B53 switch with 2 PHYs. On the BCM911360_EP platform, those two PHYs are connected to the external ethernet jacks. My spell checker trips on "amac" and "ethernet" -- perhaps they need ca

Re: [PATCH 1/2] net: dsa: b53: Add compatible strings for the Cygnus-family BCM11360.

2017-04-25 Thread Sergei Shtylyov
Hello! On 4/25/2017 12:50 AM, Eric Anholt wrote: Cygnus is a small family of SoCs, of which we currently have devicetree for BCM11360 and BCM58300. The 11360's B53 is mostly the same as 58xx, just requiring a tiny bit of setup that was previously missing. Signed-off-by: Eric Anholt --- Docu

Re: xdp_redirect ifindex vs port. Was: best API for returning/setting egress port?

2017-04-25 Thread Jesper Dangaard Brouer
On Thu, 20 Apr 2017 10:10:08 -0700 Alexei Starovoitov wrote: > On Thu, Apr 20, 2017 at 08:10:51AM +0200, Jesper Dangaard Brouer wrote: > > On Wed, 19 Apr 2017 19:56:13 -0700 > > Alexei Starovoitov wrote: > > > > > On Thu, Apr 20, 2017 at 12:51:31AM +0200, Daniel Borkmann wrote: > > > > > >

[PATCH v3] brcmfmac: Make skb header writable before use

2017-04-25 Thread James Hughes
The driver was making changes to the skb_header without ensuring it was writable (i.e. uncloned). This patch also removes some boiler plate header size checking/adjustment code as that is also handled by the skb_cow_header function used to make header writable. Please apply to 4.12, important fix.

Re: [RFC 1/4] netlink: make extended ACK setting NULL-friendly

2017-04-25 Thread Daniel Borkmann
On 04/25/2017 10:06 AM, Jakub Kicinski wrote: As we propagate extended ack reporting throughout various paths in the kernel it may happen that the same function is called with the extended ack parameter passed as NULL. Make the NL_SET_ERR_MSG() macro simply print the message to the logs if that

Re: [PATCH net v3] bridge: ebtables: fix reception of frames DNAT-ed to bridge device/port

2017-04-25 Thread Pablo Neira Ayuso
On Wed, Apr 19, 2017 at 09:47:33PM +0200, Linus Lüssing wrote: > When trying to redirect bridged frames to the bridge device itself or > a bridge port (brouting) via the dnat target then this currently fails: > > The ethernet destination of the frame is dnat'ed to the MAC address of > the bridge d

Re: [Intel-wired-lan] [PATCH 1/2] e1000e: Don't return uninitialized stats

2017-04-25 Thread Jeff Kirsher
On Tue, 2017-04-25 at 07:10 +, Brown, Aaron F wrote: > > From: Intel-wired-lan [mailto:intel-wired-lan-bounces@lists.osuosl. > > org] On > > Behalf Of Benjamin Poirier > > Sent: Monday, April 24, 2017 12:10 PM > > To: Neftin, Sasha > > Cc: kirs...@f1.synalogic.ca; Stefan Priebe ; > > netdev@vg

Re: [PATCH v2] cpsw: ethtool: add support for getting/setting EEE registers

2017-04-25 Thread Niklas Cassel
On 04/18/2017 06:40 PM, Florian Fainelli wrote: > On 04/18/2017 06:23 AM, Niklas Cassel wrote: >> On 01/04/2017 03:33 PM, Florian Fainelli wrote: >>> On 12/02/2016 09:48 AM, Florian Fainelli wrote: >> Peppe, any thoughts on this? > > I share what you say. > > In sum, the EEE m

Re: [RFC 0/4] xdp: use netlink extended ACK reporting

2017-04-25 Thread Daniel Borkmann
On 04/25/2017 10:06 AM, Jakub Kicinski wrote: Hi! This series is an attempt to make XDP more user friendly by enabling exploiting the recently added netlink extended ACK reporting to carry messages to user space. I made iproute2 parse the extended messages and have it showing the errors like th

<    1   2   3   4   >