[PATCH net] stmmac: dwmac-sunxi: Call exit cleanup function in probe error path

2015-12-11 Thread Chen-Yu Tsai
dwmac-sunxi has 2 callbacks that were called from stmmac_platform as part of the probe and remove sequences. Ater the conversion of dwmac-sunxi into a standalone platform driver, the .init function is called before calling into the stmmac driver core, but .exit is not called to clean up if stmmac

Re: [PATCH] net/macb: add support for resetting PHY using GPIO

2015-12-11 Thread Sascha Hauer
On Fri, Dec 11, 2015 at 10:40:22AM +0100, Gregory CLEMENT wrote: > Hi Sascha, > > On ven., déc. 11 2015, Sascha Hauer wrote: > > > On Thu, Dec 10, 2015 at 04:08:08PM +0100, Gregory CLEMENT wrote: > >> Hi Sascha, > >> > >> On jeu., déc. 10 2015, Sascha Hauer

[PATCH v2] net/macb: add support for resetting PHY using GPIO

2015-12-11 Thread Gregory CLEMENT
With device tree it is no more possible to reset the PHY at board level. Furthermore, doing in the driver allow to power down the PHY when the network interface is no more used. This reset can't be done at the PHY driver level. The PHY must be able to answer the to the mii bus scan to let the

[PATCH net-next 7/8] qeth: repair SBAL elements calculation

2015-12-11 Thread Ursula Braun
When sending skbs, qeth determines the number of qdio SBAL elements required. If qeth sends a fragmented skb, the SBAL element number calculation is wrong, because the fragmented data part is added twice in qeth_l3_tso_elements(). This patch makes sure fragmented data is handled in

Re: [v2] wlcore/wl12xx: spi: fix oops on firmware load

2015-12-11 Thread Kalle Valo
> The maximum chunks used by the function is > (SPI_AGGR_BUFFER_SIZE / WSPI_MAX_CHUNK_SIZE + 1). > The original commands array had space for > (SPI_AGGR_BUFFER_SIZE / WSPI_MAX_CHUNK_SIZE) commands. > When the last chunk is used (len > 4 * WSPI_MAX_CHUNK_SIZE), the last > command is stored outside

Re: [PATCH net-next v4 4/4] ila: Add generic ILA translation facility

2015-12-11 Thread Florian Westphal
Tom Herbert wrote: > +static struct nf_hook_ops ila_nf_hook_ops[] __read_mostly = { > + { > + .hook = ila_nf_input, > + .pf = NFPROTO_IPV6, > + .hooknum = NF_INET_PRE_ROUTING, > + .priority = -1, [..] > +int

[PATCH net-next 1/8] iucv: prevent information leak in iucv_message

2015-12-11 Thread Ursula Braun
From: Eugene Crosser Initialize storage for the future IUCV header that will be included in the transmitted packet. Some of the header fields are unused with HiperSockets transport, and will contain data left from some other functions. Signed-off-by: Eugene Crosser

[PATCH net-next 5/8] qeth use common function qeth_get_setassparms_cmd

2015-12-11 Thread Ursula Braun
From: Thomas Richter There have been 2 identical versions of function qeth_get_setassparms_cmd() for layer 2 and layer 3. Remove the layer 3 function qeth_l3_get_setassparms_cmd() and call the common one named qeth_get_setassparms_cmd() located in qeth_core_main.c

[PATCH net-next 8/8] qeth: get rid of redundant 0-termination

2015-12-11 Thread Ursula Braun
From: Rasmus Villemoes 0-termination is redundant, since sprintf has done that. Signed-off-by: Rasmus Villemoes Signed-off-by: Ursula Braun --- drivers/s390/net/qeth_core_main.c | 2 -- 1 file changed, 2

[PATCH net-next 6/8] qeth: initialize net_device with carrier off

2015-12-11 Thread Ursula Braun
/sys/class/net//operstate for an active qeth network interface offen shows "unknown", which translates to "state UNKNOWN in output of "ip link show". It is caused by a missing initialization of the __LINK_STATE_NOCARRIER bit in the net_device state field. This patch adds a netif_carrier_off()

[PATCH net-next 4/8] s390-ctcm: Delete unnecessary checks before the function call "channel_remove"

2015-12-11 Thread Ursula Braun
From: Markus Elfring The channel_remove() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH net-next 0/8] s390 network patches

2015-12-11 Thread Ursula Braun
Hi Dave, here are some s390 related patches for net-next. Most important is the skb_linearize af_iucv patch from Eugene solving traffic problems in certain scenarios. The remaining patches are minor improvements. Thanks, Ursula Eugene Crosser (2): iucv: prevent information leak in

[PATCH net-next 2/8] iucv: call skb_linearize() when needed

2015-12-11 Thread Ursula Braun
From: Eugene Crosser When the linear buffer of the received sk_buff is shorter than the header, use skb_linearize(). sk_buffs with short linear buffer happen on the sending side under high traffic, and some kernel configurations, when allocated buffer starts just

[PATCH net-next 3/8] MAINTAINERS: switch to alternate IBM mail address

2015-12-11 Thread Ursula Braun
Signed-off-by: Ursula Braun --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8eba565..ab79736 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9125,7 +9125,7 @@ F:drivers/s390/block/dasd* F:

Re: prism54: fix compare_const_fl.cocci warnings

2015-12-11 Thread Kalle Valo
> Move constants to the right of binary operators. > > Generated by: scripts/coccinelle/misc/compare_const_fl.cocci > > Signed-off-by: Fengguang Wu > Signed-off-by: Julia Lawall Thanks, applied to wireless-drivers-next.git. Kalle Valo -- To

Re: [PATCH] net/macb: add support for resetting PHY using GPIO

2015-12-11 Thread Gregory CLEMENT
Hi Sascha, On ven., déc. 11 2015, Sascha Hauer wrote: > On Thu, Dec 10, 2015 at 04:08:08PM +0100, Gregory CLEMENT wrote: >> Hi Sascha, >> >> On jeu., déc. 10 2015, Sascha Hauer wrote: >> >> > Hi Gregory, >> > >> > On Wed, Dec 09, 2015 at

[PATCH net-next 2/2] igb: Unpair the queues when changing the number of queues

2015-12-11 Thread Shota Suzuki
By the commit 72ddef0506da ("igb: Fix oops caused by missing queue pairing"), the IGB_FLAG_QUEUE_PAIRS flag can now be set when changing the number of queues by "ethtool -L", but it is never cleared unless the igb driver is reloaded. This patch clears it if queue pairing becomes unnecessary as a

[PATCH 1/7] iucv: prevent information leak in iucv_message

2015-12-11 Thread Ursula Braun
From: Eugene Crosser Initialize storage for the future IUCV header that will be included in the transmitted packet. Some of the header fields are unused with HiperSockets transport, and will contain data left from some other functions. Signed-off-by: Eugene Crosser

[PATCH 6/7] qeth: repair SBAL elements calculation

2015-12-11 Thread Ursula Braun
When sending skbs, qeth determines the number of qdio SBAL elements required. If qeth sends a fragmented skb, the SBAL element number calculation is wrong, because the fragmented data part is added twice in qeth_l3_tso_elements(). This patch makes sure fragmented data is handled in

[PATCH 3/7] switch to alternate IBM mail address

2015-12-11 Thread Ursula Braun
Signed-off-by: Ursula Braun --- MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 8eba565..f62648e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9125,7 +9125,7 @@ F:drivers/s390/block/dasd* F:

[PATCH 2/7] iucv: call skb_linearize() when needed

2015-12-11 Thread Ursula Braun
From: Eugene Crosser When the linear buffer of the received sk_buff is shorter than the header, use skb_linearize(). sk_buffs with short linear buffer happen on the sending side under high traffic, and some kernel configurations, when allocated buffer starts just

[PATCH net-next 0/7] s390 network patches

2015-12-11 Thread Ursula Braun
Hi Dave, here are some s390 related patches for net-next. Most important is the skb_linearize af_iucv patch from Eugene solving traffic problems in certain scenarios. The remaining patches are minor improvements. Thanks, Ursula Eugene Crosser (2): iucv: prevent information leak in

[PATCH 4/7] s390-ctcm: Delete unnecessary checks before the function call "channel_remove"

2015-12-11 Thread Ursula Braun
From: Markus Elfring The channel_remove() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

Re: mwifiex: fix semicolon.cocci warnings

2015-12-11 Thread Kalle Valo
> Remove unneeded semicolon. > > Generated by: scripts/coccinelle/misc/semicolon.cocci > > Signed-off-by: Fengguang Wu > Signed-off-by: Julia Lawall Thanks, applied to wireless-drivers-next.git. Kalle Valo -- To unsubscribe from this list: send

[iproute PATCH] ip{,6}tunnel: have a shared stats parser/printer

2015-12-11 Thread Phil Sutter
This has a slight side-effect of not aborting when /proc/net/dev is malformed, but OTOH stats are not parsed for uninteresting interfaces. Signed-off-by: Phil Sutter --- ip/ip6tunnel.c | 21 ++--- ip/iptunnel.c | 21 ++--- ip/tunnel.c| 28

Re: [PATCH RFC 10/26] phy: add I2C mdio bus

2015-12-11 Thread Russell King - ARM Linux
On Tue, Dec 08, 2015 at 10:15:08AM -0800, Florian Fainelli wrote: > On 07/12/15 09:38, Russell King wrote: > > Add an I2C MDIO bus bridge library, to allow phylib to access PHYs which > > are connected to an I2C bus instead of the more conventional MDIO bus. > > Such PHYs can be found in SFP

[PATCH ]net: sched/cls_flow.c : allow nfct-* keys work on ingress interfaces

2015-12-11 Thread Igor Gavrilov
Improved CTTUPLE macro with code from sched/act_connmark.c, so it be able to get unNATed addresses from nf_conntrack. cls_flow.patch Description: Binary data

Re: brcm80211: fix compare_const_fl.cocci warnings

2015-12-11 Thread Kalle Valo
> Move constants to the right of binary operators. > > Generated by: scripts/coccinelle/misc/compare_const_fl.cocci > > Signed-off-by: Fengguang Wu > Signed-off-by: Julia Lawall Thanks, applied to wireless-drivers-next.git. Kalle Valo -- To

Re: iwlegacy: mark il_adjust_beacon_interval as noinline

2015-12-11 Thread Kalle Valo
> With the new optimized do_div() code, some versions of gcc > produce obviously incorrect code that leads to a link error > in iwlegacy/common.o: > > drivers/built-in.o: In function `il_send_rxon_timing': > :(.text+0xa6b4d4): undefined reference to `ilog2_NaN' > :(.text+0xa6b4f0): undefined

[PATCH net v2] sfc: only use RSS filters if we're using RSS

2015-12-11 Thread Bert Kenward
Without this, filter insertion on a VF would fail if only one channel was in use. This would include the unicast station filter and therefore no traffic would be received. Signed-off-by: Bert Kenward --- v2 - fix indenting as requested drivers/net/ethernet/sfc/ef10.c

Re: [PATCH v2] r8169: Don't claim WoL works if LanWake flag is not set

2015-12-11 Thread Corinna Vinschen
On Dec 11 01:06, Francois Romieu wrote: > Corinna Vinschen : > [...] > > It's still a bit weird. On the machines I tested this on, if I disable > > LanWake and shutdown the machine, I can send, e.g., MagicPackets as much > > as I like, the machined don't come up. Isn't it a

[PATCH net-next 1/2] igb: Remove unnecessary flag setting in igb_set_flag_queue_pairs()

2015-12-11 Thread Shota Suzuki
If VFs are enabled (max_vfs >= 1), both max_rss_queues and adapter->rss_queues are set to 2 in the case of e1000_82576. In this case, IGB_FLAG_QUEUE_PAIRS is always set in the default block as a result of fall-through, thus setting it in the e1000_82576 block is not necessary. Signed-off-by:

Re: brcmfmac: only lock and unlock fws if fws is not null

2015-12-11 Thread Kalle Valo
> From: Colin Ian King > > There is a null ptr check for fws to set bcmc_credit_check, however, > there a lock and unlock on fws should only performed if fwts is > also not null to also avoid a potential null pointer deference. > > Signed-off-by: Colin Ian King

Re: [PATCH] net/macb: add support for resetting PHY using GPIO

2015-12-11 Thread Russell King - ARM Linux
On Fri, Dec 11, 2015 at 11:21:30AM +0100, Sascha Hauer wrote: > On Fri, Dec 11, 2015 at 10:40:22AM +0100, Gregory CLEMENT wrote: > > So I see 2 options: > > > > - leaving the phy-reset-gpios property in the ethernet node. Thanks to > > this we can use the gpiod functions, and it is still

[PATCH 7/7] qeth: get rid of redundant 0-termination

2015-12-11 Thread Ursula Braun
From: Rasmus Villemoes 0-termination is redundant, since sprintf has done that. Signed-off-by: Rasmus Villemoes Signed-off-by: Ursula Braun --- drivers/s390/net/qeth_core_main.c | 2 -- 1 file changed, 2

[PATCH 5/7] qeth: initialize net_device with carrier off

2015-12-11 Thread Ursula Braun
/sys/class/net//operstate for an active qeth network interface offen shows "unknown", which translates to "state UNKNOWN in output of "ip link show". It is caused by a missing initialization of the __LINK_STATE_NOCARRIER bit in the net_device state field. This patch adds a netif_carrier_off()

Re: [v2] rtlwifi: fix memory leak for USB device

2015-12-11 Thread Kalle Valo
> Free skb for received frames with a wrong checksum. This can happen > pretty rapidly, exhausting all memory. > > This fixes a memleak (detected with kmemleak). Originally found while > using monitor mode, but it also appears during managed mode (once the > link is up). > > Cc:

Re: [PATCH] ath9k: fix inconsistent indenting on return statement

2015-12-11 Thread Kalle Valo
Colin King writes: > From: Colin Ian King > > minor change, indenting is one tab out. > > Signed-off-by: Colin Ian King Applied to ath.git, thanks. -- Kalle Valo -- To unsubscribe from this list: send the line

Another pppoe-related crash

2015-12-11 Thread Andrew
I've got another pppoe-related crash on one PPPoE BRAS. Kernel is 4.1.13 with patch "pppoe: fix memory corruption in padt work structure" Crash place: (gdb) list *unregister_sysctl_table+0x1a 0xc118f26a is in unregister_sysctl_table

Re: ieee802154-atusb: Delete an unnecessary check before the function call "kfree_skb"

2015-12-11 Thread Stefan Schmidt
Hello. On 11/12/15 14:07, SF Markus Elfring wrote: this slipped through, but now it does no longer apply. Applying: ieee802154-atusb: Delete an unnecessary check before the function call "kfree_skb" error: patch failed: drivers/net/ieee802154/atusb.c:310 error: drivers/net/ieee802154/atusb.c:

Re: [PATCH v3 3/4] VSOCK: Introduce vhost-vsock.ko

2015-12-11 Thread Alex Bennée
Stefan Hajnoczi writes: > From: Asias He > > VM sockets vhost transport implementation. This module runs in host > kernel. As per previous checkpatch comments. > > Signed-off-by: Asias He > Signed-off-by: Stefan Hajnoczi

Re: [PATCH] wireless: change cfg80211 regulatory domain info as debug messages

2015-12-11 Thread Johannes Berg
On Mon, 2015-11-23 at 09:37 +0800, Dave Young wrote: > Seems there're a lot of other wireless messages. Should we refactor > them as well? I still did not get chance to see where is the code. > (My wireless driver being used is iwlwifi) Most are probably from net/mac80211/. > # dmesg|grep

[PATCH v2] ixgbe: on recv increment rx.ring->stats.yields

2015-12-11 Thread Pavel Tikhomirov
It seem to be non intentionaly changed to tx in commit adc810900a70 ("ixgbe: Refactor busy poll socket code to address multiple issues") Lock is taken from ixgbe_low_latency_recv, and there under this lock we use ixgbe_clean_rx_irq so it looks wrong for me to increment tx counter. Yield stats

Re: ieee802154-atusb: Delete an unnecessary check before the function call "kfree_skb"

2015-12-11 Thread SF Markus Elfring
oes not apply > Patch failed at 0001 ieee802154-atusb: Delete an unnecessary check before the > function call "kfree_skb" How can this hiccup happen? A command like the following is still working for me on source files for the software "Linux next-20151211". elfring@Sonne:~/

Re: use-after-free in sctp_do_sm

2015-12-11 Thread Dmitry Vyukov
On Wed, Dec 9, 2015 at 5:41 PM, Marcelo Ricardo Leitner wrote: > On Wed, Dec 09, 2015 at 01:03:56PM -0200, Marcelo Ricardo Leitner wrote: >> On Wed, Dec 09, 2015 at 03:41:29PM +0100, Dmitry Vyukov wrote: >> > On Tue, Dec 8, 2015 at 8:22 PM, Dmitry Vyukov

Re: use-after-free in sctp_do_sm

2015-12-11 Thread Marcelo Ricardo Leitner
Em 11-12-2015 11:35, Dmitry Vyukov escreveu: On Wed, Dec 9, 2015 at 5:41 PM, Marcelo Ricardo Leitner wrote: On Wed, Dec 09, 2015 at 01:03:56PM -0200, Marcelo Ricardo Leitner wrote: On Wed, Dec 09, 2015 at 03:41:29PM +0100, Dmitry Vyukov wrote: On Tue, Dec 8, 2015

Re: Double free of dst_entry in ipv4_dst_destroy()

2015-12-11 Thread Eric Dumazet
On Sun, 2015-12-06 at 17:58 -0800, Eric Dumazet wrote: > On Sun, 2015-12-06 at 13:03 -0800, Eric Dumazet wrote: > > > But then when later we promote a skb->dst to a refctounted one > > (skb_dst_force(), we might make sure we abort the operation if __refcnt > > == 0 ( and DST_NOCACHE is in

Re: [PATCH net-next] hv_netvsc: Fix race condition on Multi-Send Data field

2015-12-11 Thread Vitaly Kuznetsov
Haiyang Zhang writes: > In commit 2a04ae8acb14 ("hv_netvsc: remove locking in netvsc_send()"), the > locking for MSD (Multi-Send Data) field was removed. This could cause a > race condition between RNDIS control messages and data packets processing, > because these two

Re: [PATCH net-next] hv_netvsc: Fix race condition on Multi-Send Data field

2015-12-11 Thread Vitaly Kuznetsov
Vitaly Kuznetsov writes: > Haiyang Zhang writes: > >> In commit 2a04ae8acb14 ("hv_netvsc: remove locking in netvsc_send()"), the >> locking for MSD (Multi-Send Data) field was removed. This could cause a >> race condition between RNDIS control

Re: use-after-free in sctp_do_sm

2015-12-11 Thread Marcelo Ricardo Leitner
On Fri, Dec 11, 2015 at 11:51:21AM -0200, Marcelo Ricardo Leitner wrote: > Em 11-12-2015 11:35, Dmitry Vyukov escreveu: > >On Wed, Dec 9, 2015 at 5:41 PM, Marcelo Ricardo Leitner > > wrote: > >>On Wed, Dec 09, 2015 at 01:03:56PM -0200, Marcelo Ricardo Leitner wrote: >

Re: [linux-next] iwl_mvm_get_key_sta_id() suspicious RCU usage

2015-12-11 Thread Johannes Berg
On Fri, 2015-12-11 at 22:44 +0900, Sergey Senozhatsky wrote: > [ 6385.246300] drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 > suspicious rcu_dereference_protected() usage! > Funny, Laura Abbott also reported this bug earlier today :) I've sent out a fix, you can see it here:

Re: use-after-free in sctp_do_sm

2015-12-11 Thread Dmitry Vyukov
On Fri, Dec 11, 2015 at 3:03 PM, Marcelo Ricardo Leitner wrote: > On Fri, Dec 11, 2015 at 11:51:21AM -0200, Marcelo Ricardo Leitner wrote: >> Em 11-12-2015 11:35, Dmitry Vyukov escreveu: >> >On Wed, Dec 9, 2015 at 5:41 PM, Marcelo Ricardo Leitner >>

Re: [PATCH] wireless: change cfg80211 regulatory domain info as debug messages

2015-12-11 Thread Johannes Berg
On Sun, 2015-11-15 at 15:31 +0800, Dave Young wrote: > cfg80211 module prints a lot of messages like below. Actually > printing once is acceptable but sometimes it will print again and > again, it looks very annoying. It is better to change these detail > messages to debugging only. > Despite

Re: Double free of dst_entry in ipv4_dst_destroy()

2015-12-11 Thread Eric Dumazet
On Fri, 2015-12-11 at 07:48 -0800, Eric Dumazet wrote: > On Fri, 2015-12-11 at 06:23 -0800, Eric Dumazet wrote: > > On Sun, 2015-12-06 at 17:58 -0800, Eric Dumazet wrote: > > > On Sun, 2015-12-06 at 13:03 -0800, Eric Dumazet wrote: > > > > > > > But then when later we promote a skb->dst to a

rhashtable concurrent writers issue, 4.2+

2015-12-11 Thread Colin Ian King
Hi, After testing with commit 7def0f952eccdd0edb3c504f4dab35ee0d3aba1f ("lib: fix data race in rhashtable_rehash_one") on the current 4.4-rc4 kernel, I believe I am still seeing race conditions that seem to bite concurrent readers. The reproducer is found in my stress-ng system stress tool:

Re: [RFCv3 bluetooth-next 4/4] 6lowpan: iphc: add support for stateful compression

2015-12-11 Thread Alexander Aring
On Fri, Dec 11, 2015 at 06:05:41PM +0100, Alexander Aring wrote: > Hi, > > status update here. > > I implemented successful basic 6CO RA messages at sending and > processing inside radvd. > > There might also the question if processing 6CO inside kernel or > userspace? I put the basic

Re: [RFCv3 bluetooth-next 4/4] 6lowpan: iphc: add support for stateful compression

2015-12-11 Thread Alexander Aring
Hi, status update here. I implemented successful basic 6CO RA messages at sending and processing inside radvd. There might also the question if processing 6CO inside kernel or userspace? I put the basic functionality inside userspace for now. I will send patches for this in the next days.

RE: [PATCH net-next] hv_netvsc: Fix race condition on Multi-Send Data field

2015-12-11 Thread Haiyang Zhang
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Friday, December 11, 2015 8:53 AM > To: Haiyang Zhang > Cc: da...@davemloft.net; netdev@vger.kernel.org; o...@aepfle.de; > jasow...@redhat.com;

RE: [PATCH net-next] hv_netvsc: Fix race condition on Multi-Send Data field

2015-12-11 Thread KY Srinivasan
> -Original Message- > From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On > Behalf Of Vitaly Kuznetsov > Sent: Friday, December 11, 2015 5:53 AM > To: Haiyang Zhang > Cc: o...@aepfle.de; netdev@vger.kernel.org; jasow...@redhat.com; >

Re: Double free of dst_entry in ipv4_dst_destroy()

2015-12-11 Thread Eric Dumazet
On Fri, 2015-12-11 at 06:23 -0800, Eric Dumazet wrote: > On Sun, 2015-12-06 at 17:58 -0800, Eric Dumazet wrote: > > On Sun, 2015-12-06 at 13:03 -0800, Eric Dumazet wrote: > > > > > But then when later we promote a skb->dst to a refctounted one > > > (skb_dst_force(), we might make sure we abort

Another pppoe-related crash

2015-12-11 Thread Andrew
I've got another pppoe-related crash on one PPPoE BRAS. Kernel is 4.1.13 with patch "pppoe: fix memory corruption in padt work structure" Crash place: (gdb) list *unregister_sysctl_table+0x1a 0xc118f26a is in unregister_sysctl_table

Re: use-after-free in sctp_do_sm

2015-12-11 Thread Marcelo Ricardo Leitner
Em 11-12-2015 12:30, Dmitry Vyukov escreveu: On Fri, Dec 11, 2015 at 3:03 PM, Marcelo Ricardo Leitner wrote: On Fri, Dec 11, 2015 at 11:51:21AM -0200, Marcelo Ricardo Leitner wrote: Em 11-12-2015 11:35, Dmitry Vyukov escreveu: On Wed, Dec 9, 2015 at 5:41 PM,

Re: [PATCH net 1/2] xfrm: add rcu grace period in xfrm_policy_destroy()

2015-12-11 Thread David Miller
From: Steffen Klassert Date: Thu, 10 Dec 2015 13:34:22 +0100 > On Tue, Dec 08, 2015 at 10:58:29PM -0500, David Miller wrote: >> From: Eric Dumazet >> Date: Tue, 8 Dec 2015 07:22:01 -0800 >> >> > We will soon switch sk->sk_policy[] to RCU

Re: [PATCH] iwlwifi: mvm: protect RCU dereference in iwl_mvm_get_key_sta_id

2015-12-11 Thread Laura Abbott
On 12/11/2015 12:13 AM, Johannes Berg wrote: From: Johannes Berg Properly protect the RCU dereference in iwl_mvm_get_key_sta_id() when coming from iwl_mvm_update_tkip_key() which cannot hold the mvm->mutex by moving the call into the RCU critical section. Modify the

Re: [PATCH net 1/4] mpls: validate L2 via address length

2015-12-11 Thread roopa
On 12/10/15, 11:30 AM, Robert Shearman wrote: > If an L2 via address for an mpls nexthop is specified, the length of > the L2 address must match that expected by the output device, > otherwise it could access memory beyond the end of the via address > buffer in the route. > > This check was

Re: [linux-next] iwl_mvm_get_key_sta_id() suspicious RCU usage

2015-12-11 Thread Sergey Senozhatsky
On (12/12/15 09:05), Sergey Senozhatsky wrote: > On (12/11/15 14:49), Johannes Berg wrote: > > On Fri, 2015-12-11 at 22:44 +0900, Sergey Senozhatsky wrote: > > > > > [ 6385.246300] drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 > > > suspicious rcu_dereference_protected() usage! > > > > > >

Re: Checksum offload queries

2015-12-11 Thread Tom Herbert
>> If your device is trying do offload more than one checksum on its own > >> accord without being asked to do so by the stack it is doing the > wrong >> thing! > From the stack's perspective: yes, it is doing the wrong thing. (I've > been discussing with colleagues today how we could change

Re: [PATCH net 1/4] mpls: validate L2 via address length

2015-12-11 Thread roopa
On 12/11/15, 2:51 PM, roopa wrote: > On 12/10/15, 11:30 AM, Robert Shearman wrote: >> If an L2 via address for an mpls nexthop is specified, the length of >> the L2 address must match that expected by the output device, >> otherwise it could access memory beyond the end of the via address >>

Re: [linux-next] iwl_mvm_get_key_sta_id() suspicious RCU usage

2015-12-11 Thread Sergey Senozhatsky
On (12/11/15 14:49), Johannes Berg wrote: > On Fri, 2015-12-11 at 22:44 +0900, Sergey Senozhatsky wrote: > > > [ 6385.246300] drivers/net/wireless/intel/iwlwifi/mvm/sta.c:1226 > > suspicious rcu_dereference_protected() usage! > > > > Funny, Laura Abbott also reported this bug earlier today :) >

Re: [RFC 1/2] net/ethtool: Add new coalescing parameter for queue

2015-12-11 Thread David Miller
From: Ben Hutchings Date: Tue, 08 Dec 2015 19:00:52 + > On Tue, 2015-12-08 at 10:02 -0800, Shannon Nelson wrote: >> On Mon, Dec 7, 2015 at 8:42 PM,   wrote: >> > From: Kan Liang >> > >> > Intrdouce "queue" option for coalesce

Re: [PATCH net-next v2] Driver for IBM System i/p VNIC protocol

2015-12-11 Thread David Miller
From: Thomas Falcon Date: Tue, 8 Dec 2015 11:52:19 -0600 > +static long h_reg_sub_crq(unsigned long unit_address, unsigned long token, > + unsigned long length, unsigned long *number, > + unsigned long *irq) > +{ > +

Re: [PATCH net v2 0/4] bnxt_en: Bug fix and add tx timeout recovery.

2015-12-11 Thread David Miller
From: Michael Chan Date: Wed, 9 Dec 2015 19:35:40 -0500 > Fix a bitmap declaration bug and add missing tx timeout recovery. > > v2: Fixed white space error. Thanks Dave. Series applied, thanks Michael. -- To unsubscribe from this list: send the line "unsubscribe netdev" in

Re: [PATCH net v2] sfc: only use RSS filters if we're using RSS

2015-12-11 Thread David Miller
From: Bert Kenward Date: Fri, 11 Dec 2015 09:39:32 + > Without this, filter insertion on a VF would fail if only one channel > was in use. This would include the unicast station filter and therefore > no traffic would be received. > > Signed-off-by: Bert Kenward

Re: [PATCH v3 4/4] geneve: Add geneve_get_rx_port support

2015-12-11 Thread David Miller
From: Anjali Singhai Jain Date: Tue, 8 Dec 2015 10:12:14 -0800 > This patch adds an op that the drivers can call into to get existing > geneve ports. > > Signed-off-by: Anjali Singhai Jain Add the new geneve infrastructure in one patch,

Re: [PATCH net-next] cxgb4: Handle clip return values

2015-12-11 Thread David Miller
From: Hariprasad Shenai Date: Wed, 9 Dec 2015 17:16:35 +0530 > Add a warn message when clip table overflows. If clip table isn't > allocated, return from cxgb4_clip_release() to avoid panic. > Disable offload if clip isn't enabled in the hardware. > > Signed-off-by:

Re: [PATCH net-next] mpls_iptunnel: add static qualifier to mpls_output

2015-12-11 Thread David Miller
From: Roopa Prabhu Date: Wed, 9 Dec 2015 06:56:41 -0800 > From: Roopa Prabhu > > This gets rid of the following compile warn: > net/mpls/mpls_iptunnel.c:40:5: warning: no previous prototype for > mpls_output [-Wmissing-prototypes] > >

Re: [PATCH net 1/2] openvswitch: Fix helper reference leak

2015-12-11 Thread David Miller
From: Joe Stringer Date: Wed, 9 Dec 2015 14:07:39 -0800 > If the actions (re)allocation fails, or the actions list is larger than the > maximum size, and the conntrack action is the last action when these > problems are hit, then references to helper modules may be leaked. Fix >

Re: [PATCH net 2/2] openvswitch: Respect conntrack zone even if invalid

2015-12-11 Thread David Miller
From: Joe Stringer Date: Wed, 9 Dec 2015 14:07:40 -0800 > If userspace executes ct(zone=1), and the connection tracker determines > that the packet is invalid, then the ct_zone flow key field is populated > with the default zone rather than the zone that was specified. Even >

Re: [patch -next] mlxsw: core: remove an unneeded condition

2015-12-11 Thread David Miller
From: Dan Carpenter Date: Wed, 9 Dec 2015 13:41:29 +0300 > We already know "err" is zero so there is no need to check. > > Signed-off-by: Dan Carpenter Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body

Re: [patch -next] mlxsw: spectrum: fix some error handling

2015-12-11 Thread David Miller
From: Dan Carpenter Date: Wed, 9 Dec 2015 13:33:51 +0300 > The "err = " assignment is missing here. > > Fixes: 0d65fc13042f ('mlxsw: spectrum: Implement LAG port join/leave') > Signed-off-by: Dan Carpenter Applied. -- To unsubscribe from

Re: [PATCH net-next V1 0/9] mlx5 improved flow steering management

2015-12-11 Thread David Miller
From: Saeed Mahameed Date: Thu, 10 Dec 2015 17:12:36 +0200 > First two patches fixes some minor issues in recently > introduced SRIOV code. > > The other seven patches modifies the driver's code that > manages flow steering rules with Connectx-4 devices. Series applied,

Re: [PATCH net 0/4] mpls: fixes for nexthops without via addresses

2015-12-11 Thread David Miller
From: Robert Shearman Date: Thu, 10 Dec 2015 19:30:47 + > These four fixes all apply to the case of having an mpls route with an > output device, but without a nexthop. Series applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the

Re: [PATCH (net-next.git) 16/18] stmmac: do not perform zero-copy for rx frames

2015-12-11 Thread David Miller
From: Giuseppe Cavallaro Date: Wed, 9 Dec 2015 09:37:52 +0100 > @@ -98,6 +98,10 @@ static int buf_sz = DEFAULT_BUFSIZE; > module_param(buf_sz, int, S_IRUGO | S_IWUSR); > MODULE_PARM_DESC(buf_sz, "DMA buffer size"); > > +static int minrx = 256; > +module_param(minrx,

Re: [PATCH v3 1/4] geneve: Add geneve udp port offload for ethernet devices

2015-12-11 Thread Tom Herbert
On Tue, Dec 8, 2015 at 10:12 AM, Anjali Singhai Jain wrote: > Add ndo_ops to add/del UDP ports to a device that supports geneve > offload. > > v3: Add some more comments about the use of the new ndo ops. > > Signed-off-by: Anjali Singhai Jain > Signed-off-by: Kiran

Re: [PATCH v2 0/2] net: thunderx: Support for pass-2 hw features

2015-12-11 Thread David Miller
From: Sunil Goutham Date: Thu, 10 Dec 2015 13:25:18 +0530 > From: Sunil Goutham > > This patch set adds support for new features added in pass-2 revision > of hardware like TSO and count based interrupt coalescing. > > Changes from v1: > -

Re: [PATCH net-next v3 0/4] mvneta: Introduce RSS support and XPS configuration

2015-12-11 Thread David Miller
From: Gregory CLEMENT Date: Wed, 9 Dec 2015 18:23:47 +0100 > this series is the first step add RSS support on mvneta. Series applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org

Re: [RFC 1/1] drivers/net: fix eisa_driver probe section mismatch

2015-12-11 Thread David Miller
From: Fabian Frederick Date: Tue, 8 Dec 2015 18:36:25 +0100 > Some eisa_driver structures use __init probe functions which generates > a warning and could crash if function is called after being deleted. I can't apply this to my networking tree because you included SCSI changes

Re: [PATCH v3 1/4] geneve: Add geneve udp port offload for ethernet devices

2015-12-11 Thread David Miller
From: Anjali Singhai Jain Date: Tue, 8 Dec 2015 10:12:11 -0800 > @@ -1013,6 +1013,20 @@ typedef u16 (*select_queue_fallback_t)(struct > net_device *dev, > * a new port starts listening. The operation is protected by the > * vxlan_net->sock_lock. > * > + *

Re: [PATCHv2 net-next] ipv6: allow routes to be configured with expire values

2015-12-11 Thread David Miller
From: Xin Long Date: Thu, 10 Dec 2015 00:27:53 +0800 > Add the support for adding expire value to routes, requested by > Tom Gundersen for systemd-networkd, and NetworkManager > wants it too. > > add it by using the field rta_expires of rta_cacheinfo > >

Re: [PATCH 1/1] net: bonding: remove redudant brackets

2015-12-11 Thread David Miller
From: Date: Wed, 9 Dec 2015 11:30:42 +0800 > From: yzhu1 > > It is not necessary to use two brackets. As such, the redudant brackets > are removed. > > CC: Jay Vosburgh > CC: Veaceslav Falico > CC: Andy

Re: [PATCH net] ipv6: sctp: clone options to avoid use after free

2015-12-11 Thread David Miller
From: Eric Dumazet Date: Wed, 09 Dec 2015 07:25:06 -0800 > From: Eric Dumazet > > SCTP is lacking proper np->opt cloning at accept() time. > > TCP and DCCP use ipv6_dup_options() helper, do the same > in SCTP. > > We might later factorize this

Re: [PATCH net] uapi: export ila.h

2015-12-11 Thread David Miller
From: Stephen Hemminger Date: Thu, 10 Dec 2015 09:14:20 -0800 > The file ila.h used for lightweight tunnels is being used by iproute2 > but is not exported yet. > > Signed-off-by: Stephen Hemminger Applied and queued up for -stable,

Re: [patch net-next] mlxsw: reg: Fix max temperature getting

2015-12-11 Thread David Miller
From: Jiri Pirko Date: Fri, 11 Dec 2015 16:10:39 +0100 > From: Jiri Pirko > > Fix copy & paste error in MTPM unpack helper. > > Fixes: 85926f877040 ("mlxsw: reg: Add definition of temperature management > registers") > Signed-off-by: Jiri Pirko

Re: [patch net-next] mlxsw: core: Fix temperature sensor index during initialization

2015-12-11 Thread David Miller
From: Jiri Pirko Date: Fri, 11 Dec 2015 18:26:00 +0100 > From: Jiri Pirko > > Sensor index should be passed instead of 0. For now, this does not make > a difference, since there is so far only one temperature sensor > exposed by HW. > > Fixes: 89309da39

[patch net-next] mlxsw: reg: Fix max temperature getting

2015-12-11 Thread Jiri Pirko
From: Jiri Pirko Fix copy & paste error in MTPM unpack helper. Fixes: 85926f877040 ("mlxsw: reg: Add definition of temperature management registers") Signed-off-by: Jiri Pirko --- drivers/net/ethernet/mellanox/mlxsw/reg.h | 2 +- 1 file changed, 1

[PATCH 1/1] net: sctp: dynamically enable or disable pf state

2015-12-11 Thread zyjzyj2000
From: Zhu Yanjun As we all know, the vale of pf_retrans >= max_retrans_path can disable pf state. The variables of pf_retrans and max_retrans_path can be changed by the user space application. Sometimes the user expects to disable pf state while the 2 variables are changed

Re: [PATCH v5 1/4] stmmac: create of compatible mdio bus for stmacc driver

2015-12-11 Thread Giuseppe CAVALLARO
Hello Phil On 12/11/2015 2:44 AM, Phil Reid wrote: G'day Giuseppe, On 11/12/2015 1:16 AM, Giuseppe CAVALLARO wrote: Hi also pls fix this typo stmmac: create of compatible mdio bus for stmacc driver

Re: [RFCv3 bluetooth-next 4/4] 6lowpan: iphc: add support for stateful compression

2015-12-11 Thread Alexander Aring
On Fri, Dec 11, 2015 at 08:11:42PM +0100, Stefan Schmidt wrote: > Hello. > > On 11/12/15 18:05, Alexander Aring wrote: > >Hi, > > > >status update here. > > > >I implemented successful basic 6CO RA messages at sending and > >processing inside radvd. > > > >There might also the question if

Re: [PATCH v3 4/4] VSOCK: Add Makefile and Kconfig

2015-12-11 Thread Alex Bennée
Stefan Hajnoczi writes: > From: Asias He > > Enable virtio-vsock and vhost-vsock. > > Signed-off-by: Asias He > Signed-off-by: Stefan Hajnoczi > --- > v3: > * Don't put vhost vsock driver into staging > * Add

[patch net-next] mlxsw: core: Fix temperature sensor index during initialization

2015-12-11 Thread Jiri Pirko
From: Jiri Pirko Sensor index should be passed instead of 0. For now, this does not make a difference, since there is so far only one temperature sensor exposed by HW. Fixes: 89309da39 ("mlxsw: core: Implement temperature hwmon interface") Signed-off-by: Jiri Pirko

  1   2   >