Re: [PATCH] net/mlx5_core/pagealloc: Remove deprecated create_singlethread_workqueue

2016-08-01 Thread Leon Romanovsky
On Mon, Aug 01, 2016 at 11:11:19AM -0400, Tejun Heo wrote: > > Right now, I'm feeling that I'm participating in soapie where one sends > > patch for every line, waits and sends the same patch for another file. > > It is worth to send one patch set and let us to test it all in once. > > Yeah, I

Re: [PATCH net] qed: Fail driver load in 100g MSI mode.

2016-08-01 Thread David Miller
From: Sudarsana Reddy Kalluru Date: Mon, 1 Aug 2016 09:08:13 -0400 > 100g support is not available in MSI mode. Failing the driver load in this > scenario. > > Please consider applying this to `net'. > > Signed-off-by: Sudarsana Reddy Kalluru

Re: [PATCH v2 00/15] add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread David Miller
From: Peter Chen Date: Mon, 1 Aug 2016 15:02:28 +0800 > This patch set fixes missing of_node_put issue at ethernet driver. > of_node_put needs to be called when the device node which is got > from of_parse_phandle has finished using. > > The compilation test has passed by

[PATCH] vhost: drop vringh dependency

2016-08-01 Thread Michael S. Tsirkin
vringh isn't used by vhost net or scsi - it's used by CAIF only at the moment. Drop the dependency. Signed-off-by: Michael S. Tsirkin --- Including vhost directory twice isn't pretty - we just want it scanned if either of the symbols is defined. Something along the lines of

Re: [PATCH net v2] 8139too: fix system hang when there is a tx timeout event.

2016-08-01 Thread David Miller
From: Chunhao Lin Date: Mon, 1 Aug 2016 13:45:30 +0800 > If tx timeout event occur, kernel will call rtl8139_tx_timeout_task() to reset > hardware. But in this function, driver does not stop tx and rx function before > reset hardware, that will cause system hang. > > In this

Re: [PATCH -next] qed: Fix error return code in qed_resc_alloc()

2016-08-01 Thread David Miller
From: Wei Yongjun Date: Tue, 2 Aug 2016 00:55:34 + > Fix to return error code -EINVAL instead of 0 when EQ elements is > too larger, as done elsewhere in this function. > > Signed-off-by: Wei Yongjun Applied, thanks.

[PATCH net] openvswitch: Remove incorrect WARN_ONCE().

2016-08-01 Thread Jarno Rajahalme
ovs_ct_find_existing() issues a warning if an existing conntrack entry classified as IP_CT_NEW is found, with the premise that this should not happen. However, a newly confirmed, non-expected conntrack entry remains IP_CT_NEW as long as no reply direction traffic is seen. This has resulted into

Re: [PATCH 0/2] HFSC patches

2016-08-01 Thread David Miller
From: Michal Soltys Date: Mon, 1 Aug 2016 01:03:57 +0200 > - first patch: > > This patch syncs virtual times with fair service curve and fixes a very old > subtle bug. > > The detailed explanation is in the commit message. Additionally > I've made an illustration to help

Re: [PATCH v3 3/3] mac80211: mesh: fixed HT ies in beacon template

2016-08-01 Thread Masashi Honma
On 2016年08月01日 19:03, Johannes Berg wrote: But why is that behaviour *correct*? We still support 40 MHz bandwidth things, we just don't use them if we disable HT40. Or do you mean difference between "hardware capability" and "software capability" ? Do you think

[vhost:vhost 6/17] ERROR: "vhost_work_flush" [drivers/vhost/vhost_scsi.ko] undefined!

2016-08-01 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost head: b5311f2ee9cf07e73303ae792bdc363a91b9df0b commit: 6190efb08c16dcd68c64b096a28f47ab33f017d7 [6/17] vhost: drop vringh dependency config: x86_64-randconfig-n0-08020724 (attached as .config) compiler: gcc-4.8 (Debian

Re: [PATCH 1/1] rps: Inspect PPTP encapsulated by GRE to get flow hash

2016-08-01 Thread Philp Prindeville
Inline... Main issue in a nutshell is I don't like using "4" instead of "sizeof(my_32bit_header_field_here)". On 07/28/2016 01:14 AM, f...@48lvckh6395k16k5.yundunddos.com wrote: From: Gao Feng The PPTP is encapsulated by GRE header with that GRE_VERSION bits must contain

Re: [Intel-wired-lan] [PATCH net-next v3 1/2] e1000e: factor out systim sanitization

2016-08-01 Thread Jarod Wilson
On Wed, Jul 27, 2016 at 11:01:55AM -0400, Jarod Wilson wrote: > On Wed, Jul 27, 2016 at 02:09:13PM +, Avargil, Raanan wrote: > >> This is prepatory work for an expanding list of adapter families that have > >> occasional ~10 hour clock jumps when being used for PTP. Factor out the > >>

[PATCH -next] qed: Fix error return code in qed_resc_alloc()

2016-08-01 Thread Wei Yongjun
Fix to return error code -EINVAL instead of 0 when EQ elements is too larger, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/net/ethernet/qlogic/qed/qed_dev.c | 1 + 1 file changed, 1 insertion(+) diff --git

[vhost:vhost 5/15] warning: (VOP && ..) selects VHOST_RING which has unmet direct dependencies (NETDEVICES && ..)

2016-08-01 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost head: ccbb911915b99ba9ff81a1cfa4f07c98f98fff83 commit: 4d167a33a6669c309f8fcdc6cd7143de3b5445b3 [5/15] vhost: drop vringh dependency config: x86_64-randconfig-x005-201631 (attached as .config) compiler: gcc-6 (Debian

Re: 4.6.3, pppoe + shaper workload, skb_panic / skb_push / ppp_start_xmit

2016-08-01 Thread Denys Fedoryshchenko
On 2016-08-01 23:59, Guillaume Nault wrote: On Thu, Jul 28, 2016 at 02:28:23PM +0300, Denys Fedoryshchenko wrote: [ 5449.904989] CPU: 1 PID: 6359 Comm: ip Not tainted 4.7.0-build-0109 #2 [ 5449.905255] Hardware name: Supermicro X10SLM+-LN4F/X10SLM+-LN4F, BIOS 3.0 04/24/2015 [ 5449.905712]

[PATCH net-next v3 3/3] strparser: Documentation

2016-08-01 Thread Tom Herbert
Signed-off-by: Tom Herbert --- Documentation/networking/strparser.txt | 147 + 1 file changed, 147 insertions(+) create mode 100644 Documentation/networking/strparser.txt diff --git a/Documentation/networking/strparser.txt

[PATCH net-next v3 0/3] strp: Stream parser for messages

2016-08-01 Thread Tom Herbert
This patch set introduces a utility for parsing application layer protocol messages in a TCP stream. This is a generalization of the mechanism implemented of Kernel Connection Multiplexor. This patch set adapts KCM to use the strparser. We expect that kTLS can use this mechanism also. RDS would

[PATCH net-next v3 1/3] strparser: Stream parser for messages

2016-08-01 Thread Tom Herbert
This patch introduces a utility for parsing application layer protocol messages in a TCP stream. This is a generalization of the mechanism implemented of Kernel Connection Multiplexor. The API includes a context structure, a set of callbacks, utility functions, and a data ready function. A

[PATCH net-next v3 2/3] kcm: Use stream parser

2016-08-01 Thread Tom Herbert
Adapt KCM to use the stream parser. This mostly involves removing the RX handling and setting up the strparser using the interface. Signed-off-by: Tom Herbert --- include/net/kcm.h | 37 +--- net/ipv6/ila/ila_common.c | 1 - net/kcm/Kconfig | 1 +

Re: [vhost:vhost 11/15] warning: (VOP && ..) selects VHOST_RING which has unmet direct dependencies (NETDEVICES && ..)

2016-08-01 Thread Michael S. Tsirkin
On Tue, Aug 02, 2016 at 04:56:54AM +0800, kbuild test robot wrote: > tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost > head: c4ea43c2c1779f5dde3ff5645b830c90f75ccc15 > commit: efd7eb77f631e1ed3533db7725df157a379c78ef [11/15] VSOCK: Add Makefile > and Kconfig >

Re: 4.6.3, pppoe + shaper workload, skb_panic / skb_push / ppp_start_xmit

2016-08-01 Thread Guillaume Nault
On Thu, Jul 28, 2016 at 02:28:23PM +0300, Denys Fedoryshchenko wrote: > On 2016-07-28 14:09, Guillaume Nault wrote: > > On Tue, Jul 12, 2016 at 10:31:18AM -0700, Cong Wang wrote: > > > On Mon, Jul 11, 2016 at 12:45 PM, wrote: > > > > Hi > > > > > > > > On latest kernel

Re: [PATCH v5 09/11] dtb: xgene: Add MDIO node

2016-08-01 Thread Olof Johansson
Hi, I came across this now when looking at merge conflicts when preparing this. I wish I had seen it sooner. On Thu, Jul 7, 2016 at 4:02 PM, Iyappan Subramanian wrote: > Added mdio node for mdio driver. Also added phy-handle > reference to the ethernet nodes. > > Removed

Re: 4.6.3, pppoe + shaper workload, skb_panic / skb_push / ppp_start_xmit

2016-08-01 Thread Guillaume Nault
On Thu, Jul 28, 2016 at 02:28:23PM +0300, Denys Fedoryshchenko wrote: > [ 5449.904989] CPU: 1 PID: 6359 Comm: ip Not tainted > 4.7.0-build-0109 #2 > [ 5449.905255] Hardware name: Supermicro > X10SLM+-LN4F/X10SLM+-LN4F, BIOS 3.0 04/24/2015 > [ 5449.905712] task: 8803eef4 ti:

[vhost:vhost 11/15] warning: (VOP && ..) selects VHOST_RING which has unmet direct dependencies (NETDEVICES && ..)

2016-08-01 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost head: c4ea43c2c1779f5dde3ff5645b830c90f75ccc15 commit: efd7eb77f631e1ed3533db7725df157a379c78ef [11/15] VSOCK: Add Makefile and Kconfig config: x86_64-randconfig-x005-201631 (attached as .config) compiler: gcc-6 (Debian

Re: [PATCH net-next v2 0/3] strp: Stream parser for messages

2016-08-01 Thread David Miller
From: Tom Herbert Date: Mon, 1 Aug 2016 13:38:46 -0700 > On Mon, Aug 1, 2016 at 1:27 PM, David Miller wrote: >> From: Tom Herbert >> Date: Mon, 1 Aug 2016 09:33:17 -0700 >> >>> - Remove GPL module license from strparser.c >> >>

Re: [PATCH 1/1] net: bcm63xx: avoid possible null pointer dereference

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 10:24:29 +0200 > If dev_get_platdata has failed pd is null. > Do not dereference a null pointer. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH net] sctp: change to use TCP_CLOSE_WAIT as SCTP_SS_CLOSING

2016-08-01 Thread David Miller
From: Xin Long Date: Sat, 30 Jul 2016 20:00:45 +0800 > Prior to this patch, sctp defined TCP_CLOSING as SCTP_SS_CLOSING. > TCP_CLOSING is such a special sk state in TCP that inet common codes > even exclude it. > > For instance, inet_accept thinks the accept sk's state

Re: [PATCH 1/1] net: amd-xgbe: use correct format specifier

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 10:07:18 +0200 > i has been defined as unsigned int. > So use %u for output. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1 v2] dsa: b53: remove redundant if

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 13:42:33 +0200 > For pdata == null the code leaves with an error. > There is no need to check the condition again. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: ethernet: ax88796: avoid null pointer dereference

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 09:58:44 +0200 > If platform_get_resource fails, mem2 is null. > Do not dereference null. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] qed: do not use unitialized variable

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 13:24:52 +0200 > Do not write random bytes from the kernel stack when > calling qed_wr. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: caif: use correct format specifier

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 09:19:50 +0200 > %u is the wrong format specifier for int. > size_t cannot be converted to int without possible > loss of information. > > So leave the result as size_t and use %zu as format specifier. > > cf.

Re: [PATCH net-next v2 0/3] strp: Stream parser for messages

2016-08-01 Thread Tom Herbert
On Mon, Aug 1, 2016 at 1:27 PM, David Miller wrote: > From: Tom Herbert > Date: Mon, 1 Aug 2016 09:33:17 -0700 > >> - Remove GPL module license from strparser.c > > Why? It looked more consistent with other modules. Is GPL module license preferred

Re: [PATCH 1/1] net: s2io: simplify logical constraint

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 11:53:57 +0200 > (!A || (A && B)) is equivalent to (!A || B) > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: bna: use correct type specifications

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 10:53:14 +0200 > addr and len are read with > sscanf(kern_buf, "%x:%x", , ); > and used as arguments for > bna_reg_offset_check. > > So they have to be unsigned. > > Signed-off-by: Heinrich Schuchardt

Re: [PATCH 1/1] wan/fsl_ucc_hdlc: avoid possible NULL pointer dereference

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 13:14:23 +0200 > All assignments to components of priv should only > occur after the check if prif is NULL. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: qlcnic: avoid superfluous assignement

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 12:07:37 +0200 > Assigning NULL to parmeter dcb has no effect outside of the > inlined function. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: qlge: remove superfluous statement

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 12:19:28 +0200 > Variable length is not used after the deleted line. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: enic: use correct type specifier

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 11:10:08 +0200 > i is defined as unsigned. > So print it with %u. > > Signed-off-by: Heinrich Schuchardt Applied.

Re: [PATCH 1/1] net: bna: use correct type specifier (2)

2016-08-01 Thread David Miller
From: Heinrich Schuchardt Date: Sun, 31 Jul 2016 11:01:20 +0200 > add and val are read with > sscanf(kern_buf, "%x:%x", , ); > and used as arguments for bna_reg_offset_check and writel > so they have to be unsigned. > > Signed-off-by: Heinrich Schuchardt

Re: [PATCH net-next v2 0/3] strp: Stream parser for messages

2016-08-01 Thread David Miller
From: Tom Herbert Date: Mon, 1 Aug 2016 09:33:17 -0700 > - Remove GPL module license from strparser.c Why?

[vhost:vhost 14/14] drivers/vhost/vhost.c:915:30: warning: passing argument 2 of 'access_ok' makes pointer from integer without a cast

2016-08-01 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git vhost head: 51ce50c54895044f949129e595ed9a37e4d6c13a commit: 51ce50c54895044f949129e595ed9a37e4d6c13a [14/14] vhost: new device IOTLB API config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-gcc

Re: [v4] Fix to avoid IS_ERR_VALUE and IS_ERR abuses on 64bit systems.

2016-08-01 Thread Scott Wood
On 08/01/2016 02:02 AM, Arnd Bergmann wrote: > On Sunday, July 31, 2016 4:48:44 PM CEST Arvind Yadav wrote: >> IS_ERR_VALUE() assumes that parameter is an unsigned long. >> It can not be used to check if 'unsigned int' is passed insted. >> Which tends to reflect an error. >> >> In 64bit

[PATCH 1/1] virtio-vsock: Fix unbound rx buffer

2016-08-01 Thread ggarcia
From: Gerard Garcia Reset connection and close rx socket when the sender is ignoring our announced available buffer. Signed-off-by: Gerard Garcia --- net/vmw_vsock/virtio_transport_common.c | 22 -- 1 file changed, 16

[PATCH 0/1] virtio-vsock: Fix unbound rx buffer

2016-08-01 Thread ggarcia
From: Gerard Garcia This patch applies over VSOCK RFC v6. Reset connection and close rx socket when the sender is ignoring our announced available buffer. This avoids a DoS. Gerard Garcia (1): Fix unbound rx buffer net/vmw_vsock/virtio_transport_common.c | 22

Re: [PATCH net-next 10/10] Documentation: dtb: xgene: Add rxlos GPIO mapping

2016-08-01 Thread Rob Herring
On Fri, Jul 29, 2016 at 05:34:03PM -0700, Iyappan Subramanian wrote: > Signed-off-by: Quan Nguyen > Signed-off-by: Iyappan Subramanian > Tested-by: Fushen Chen > --- > Documentation/devicetree/bindings/net/apm-xgene-enet.txt | 1 + > 1 file

[PATCH net-next v2 3/3] strparser: Documentation

2016-08-01 Thread Tom Herbert
Signed-off-by: Tom Herbert --- Documentation/networking/strparser.txt | 147 + 1 file changed, 147 insertions(+) create mode 100644 Documentation/networking/strparser.txt diff --git a/Documentation/networking/strparser.txt

[PATCH net-next v2 1/3] strparser: Stream parser for messages

2016-08-01 Thread Tom Herbert
This patch introduces a utility for parsing application layer protocol messages in a TCP stream. This is a generalization of the mechanism implemented of Kernel Connection Multiplexor. The API includes a context structure, a set of callbacks, utility functions, and a data ready function. A

[PATCH net-next v2 2/3] kcm: Use stream parser

2016-08-01 Thread Tom Herbert
Adapt KCM to use the stream parser. This mostly involves removing the RX handling and setting up the strparser using the interface. Signed-off-by: Tom Herbert --- include/net/kcm.h | 37 +--- net/ipv6/ila/ila_common.c | 1 - net/kcm/Kconfig | 1 +

[PATCH net-next v2 0/3] strp: Stream parser for messages

2016-08-01 Thread Tom Herbert
This patch set introduces a utility for parsing application layer protocol messages in a TCP stream. This is a generalization of the mechanism implemented of Kernel Connection Multiplexor. This patch set adapts KCM to use the strparser. We expect that kTLS can use this mechanism also. RDS would

Re: [PATCH 1/1] net: s2io: simplify logical constraint

2016-08-01 Thread Jon Mason
On Sun, Jul 31, 2016 at 5:53 AM, Heinrich Schuchardt wrote: > (!A || (A && B)) is equivalent to (!A || B) Seems logical to me. Acked-by: Jon Mason > Signed-off-by: Heinrich Schuchardt > --- > drivers/net/ethernet/neterion/s2io.c | 2

Re: [PATCH] bpf: fix size of copy_to_user in percpu map.

2016-08-01 Thread William Tu
>> And in my /sys/devices/system/cpu, I have cpu0 and cpu1, >> kernel_max = 63 >> possible = 0-63 >> present = 0-1 > > glibc is doing > ls -d /sys/devices/system/cpu/cpu* > http://osxr.org:8080/glibc/source/sysdeps/unix/sysv/linux/getsysstats.c?v=glibc-2.14#0180 > And

Re: [PATCH] net: thunderx: correct bound check in nic_config_loopback

2016-08-01 Thread Levin, Alexander
On 07/31/2016 12:41 PM, Sunil Kovvuri wrote: > Thanks for finding. > A much better fix would be, > > - if (lbk->vf_id > MAX_LMAC) > + if (lbk->vf_id >= nic->num_vf_en) > return -1; > > where 'num_vf_en' reflects the exact number of physical interfaces or > LMACs on

Re: [PATCH] bpf: fix size of copy_to_user in percpu map.

2016-08-01 Thread Alexei Starovoitov
On Sun, Jul 31, 2016 at 08:25:12AM -0700, William Tu wrote: > >> >>num_possible_cpu == 64 > >> >>num_online_cpu == 2 == sysconf(_SC_NPROCESSORS_CONF) > > ... > >> >> To fix it, I could either > >> >> 1). declare values array based on num_possible_cpu in test_map.c, > >> >>long

Re: [PATCH v2] bpf: silence warnings when building kernel/bpf/core.c with W=1

2016-08-01 Thread Alexei Starovoitov
On Mon, Aug 01, 2016 at 01:18:43AM -0400, valdis.kletni...@vt.edu wrote: > On Sun, 31 Jul 2016 21:42:22 -0700, Alexei Starovoitov said: > > > and at least 2 other such patches for other files... > > Is there a single warning where -Woverride-init was useful? > > May be worth disabling this

Re: [PATCH net-next 2/3] bna: change type of bna_id to atomic_t

2016-08-01 Thread Ivan Vecera
On 1.8.2016 16:20, David Laight wrote: From: Ivan Vecera Sent: 29 July 2016 18:53 Change type of bna_id to atomic_t. The bnad_list_mutex is used to prevent a race when bna_id is incremented. After the change the mutex can be removed in the next step. ... -static u32 bna_id; +static atomic_t

Re: [PATCH] net/mlx5_core/pagealloc: Remove deprecated create_singlethread_workqueue

2016-08-01 Thread Tejun Heo
Hello, Leon. On Sun, Jul 31, 2016 at 09:35:13AM +0300, Leon Romanovsky wrote: > > The conversion uses WQ_MEM_RECLAIM, which is standard for all > > workqueues which can stall packet processing if stalled. The > > requirement comes from nfs or block devices over network. > > The title stays

RE: [PATCH net-next 2/3] bna: change type of bna_id to atomic_t

2016-08-01 Thread David Laight
From: Ivan Vecera > Sent: 29 July 2016 18:53 > Change type of bna_id to atomic_t. The bnad_list_mutex is used to prevent > a race when bna_id is incremented. After the change the mutex can be > removed in the next step. ... > -static u32 bna_id; > +static atomic_t bna_id; ... >

[PATCH net] qed: Fail driver load in 100g MSI mode.

2016-08-01 Thread Sudarsana Reddy Kalluru
100g support is not available in MSI mode. Failing the driver load in this scenario. Please consider applying this to `net'. Signed-off-by: Sudarsana Reddy Kalluru Signed-off-by: Yuval Mintz --- drivers/net/ethernet/qlogic/qed/qed_main.c

Re: [PATCH 1/3] sctp: Export struct sctp_info to userspace

2016-08-01 Thread Phil Sutter
Hi Stephen, On Sun, Jul 31, 2016 at 02:18:24PM -0700, Stephen Hemminger wrote: > On Fri, 29 Jul 2016 18:59:38 +0200 > Phil Sutter wrote: > > > This is required to correctly interpret INET_DIAG_INFO messages exported > > by sctp_diag module. > > > > Signed-off-by: Phil Sutter

e1000e: PHY cann't be initialized correctly on some I218 controllers

2016-08-01 Thread Denis Turischev
There is an error "e1000e: probe of :00:19.0 failed with error -3" that can be observed after power-on on some I218 controllers. Hardware reset and re-initialization fixes the issue. Signed-off-by: Denis Turischev diff -Nru

Re: [PATCH RESEND nf] netfilter: avoid a race between nf_register_hook() and cleanup_net()

2016-08-01 Thread Pablo Neira Ayuso
On Sat, Jul 30, 2016 at 08:24:37AM -0500, Eric W. Biederman wrote: > Michal Kubecek writes: > > > There is a race condition between nf_{,un}register_hook() and > > cleanup_net() which can either trigger WARN check or cause a memory > > leak. The scenario is like this (2a and 2b

Re: [PATCH v3 3/3] mac80211: mesh: fixed HT ies in beacon template

2016-08-01 Thread Masashi Honma
On 2016年08月01日 19:03, Johannes Berg wrote: But why is that behaviour *correct*? We still support 40 MHz bandwidth things, we just don't use them if we disable HT40. I could not fully understand your concern... Do you mean we have 2 bugs about disabling HT40 ? 1) bits in HT capabilities IE

[PATCH v6 8/8] thunderbolt: Adding maintainer entry

2016-08-01 Thread Amir Levy
Add Amir Levy as maintainer for Thunderbolt(TM) ICM driver Signed-off-by: Amir Levy --- MAINTAINERS | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 72f7703..2b6d635 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@

[PATCH v6 3/8] thunderbolt: Kconfig for Thunderbolt(TM) networking

2016-08-01 Thread Amir Levy
Updating the Kconfig Thunderbolt(TM) description. Signed-off-by: Amir Levy --- drivers/thunderbolt/Kconfig | 25 + drivers/thunderbolt/Makefile | 2 +- 2 files changed, 22 insertions(+), 5 deletions(-) diff --git a/drivers/thunderbolt/Kconfig

[PATCH v6 1/8] thunderbolt: Macro rename

2016-08-01 Thread Amir Levy
This first patch updates the registers file to reflect that it isn't only for Cactus Ridge. No functional change intended. Signed-off-by: Amir Levy --- drivers/thunderbolt/nhi_regs.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH v6 2/8] thunderbolt: Updating the register definitions

2016-08-01 Thread Amir Levy
Adding more Thunderbolt(TM) register definitions and some helper macros. Signed-off-by: Amir Levy --- drivers/thunderbolt/nhi_regs.h | 109 + 1 file changed, 109 insertions(+) diff --git a/drivers/thunderbolt/nhi_regs.h

[PATCH v6 5/8] thunderbolt: Networking state machine

2016-08-01 Thread Amir Levy
Negotiation states that a peer goes through in order to establish the communication with the second peer. This includes communication with upper layer and additional infrastructure support to communicate with the second peer through ICM. Signed-off-by: Amir Levy ---

[PATCH v6 7/8] thunderbolt: Networking doc

2016-08-01 Thread Amir Levy
Adding Thunderbolt(TM) networking documentation. Signed-off-by: Amir Levy --- Documentation/00-INDEX | 2 + Documentation/thunderbolt-networking.txt | 135 +++ 2 files changed, 137 insertions(+) create mode 100644

[PATCH v6 0/8] thunderbolt: Introducing Thunderbolt(TM) networking

2016-08-01 Thread Amir Levy
This is version 6 of Thunderbolt(TM) driver for non-Apple hardware. Changes since v5: - Removed the padding of short packets in receive - Replaced RW semaphore with mutex - Cleanup These patches were pushed to GitHub where they can be reviewed more comfortably with green/red highlighting:

[PATCH v6 4/8] thunderbolt: Communication with the ICM (firmware)

2016-08-01 Thread Amir Levy
Firmware-based (a.k.a ICM - Intel Connection Manager) controller is used for establishing and maintaining the Thunderbolt Networking connection. We need to be able to communicate with it. Signed-off-by: Amir Levy --- drivers/thunderbolt/Makefile |1 +

Re: [PATCH net] sctp: change to use TCP_CLOSE_WAIT as SCTP_SS_CLOSING

2016-08-01 Thread marcelo . leitner
On Sat, Jul 30, 2016 at 10:08:03PM -0700, David Miller wrote: > From: Marcelo Ricardo Leitner > Date: Sat, 30 Jul 2016 10:25:35 -0300 > > > On Sat, Jul 30, 2016 at 08:00:45PM +0800, Xin Long wrote: > >> Prior to this patch, sctp defined TCP_CLOSING as SCTP_SS_CLOSING.

RE: [PATCH 14/15] ethernet: stmicro: stmmac: stmmac_platform: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
> >>Hi, >> >>On 07/27/2016 04:20 AM, Peter Chen wrote: >>> of_node_put needs to be called when the device node which is got from >>> of_parse_phandle has finished using. >>> >>> Signed-off-by: Peter Chen >>> --- >>> drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c | 5

Re: [PATCH 2/2] nvme-rdma: Add handling for connecting to IPv6 targets

2016-08-01 Thread Christoph Hellwig
Hi Roland, On Sun, Jul 31, 2016 at 12:27:40AM -0700, Roland Dreier wrote: > From: Roland Dreier > > If a target address does not parse as IPv4, try parsing it as IPv6 > (including handling '%' suffixes for link-local addresses). The code below looks fine to me, but is

RE: [PATCH 1/1] mwifiex: remove superfluous condition

2016-08-01 Thread Amitkumar Karwar
> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > Sent: Sunday, July 31, 2016 4:09 PM > To: Amitkumar Karwar; Nishant Sarmukadam; Kalle Valo > Cc: linux-wirel...@vger.kernel.org; netdev@vger.kernel.org; linux- > ker...@vger.kernel.org; Heinrich Schuchardt > Subject: [PATCH 1/1] mwifiex:

RE: [PATCH 1/1] mwifiex: key_material_v2 remove superfluous condition

2016-08-01 Thread Amitkumar Karwar
> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > Sent: Sunday, July 31, 2016 5:41 PM > To: Amitkumar Karwar; Nishant Sarmukadam; Kalle Valo > Cc: linux-wirel...@vger.kernel.org; netdev@vger.kernel.org; linux- > ker...@vger.kernel.org; Heinrich Schuchardt > Subject: [PATCH 1/1] mwifiex:

[PATCH v6] net: sched: convert qdisc linked list to hashtable

2016-08-01 Thread Jiri Kosina
From: Jiri Kosina Convert the per-device linked list into a hashtable. The primary motivation for this change is that currently, we're not tracking all the qdiscs in hierarchy (e.g. excluding default qdiscs), as the lookup performed over the linked list by

Re: [PATCH v4] net: sched: convert qdisc linked list to hashtable

2016-08-01 Thread Jiri Kosina
On Sun, 31 Jul 2016, Fengguang Wu wrote: > Jiri, I just double checked and find no similar errors related to > qdisc_list_add(). The parent commit 95556a8838 ("dccp: avoid deadlock > in dccp_v4_ctl_send_reset") builds fine without error. You are right, I realized my mistake afterwards. This is

Re: [PATCH 1/4] mac80211: mesh: flush stations before beacons are stopped

2016-08-01 Thread Johannes Berg
On Wed, 2016-07-13 at 10:11 +, Machani, Yaniv wrote: > On Wed, Jun 29, 2016 at 10:14:19, Johannes Berg wrote: > > Cc: Hahn, Maital > > Subject: Re: [PATCH 1/4] mac80211: mesh: flush stations before > > beacons  > > are stopped > > > > On Tue, 2016-06-28 at 14:13 +0300, Yaniv Machani wrote: >

Re: [PATCH v2 1/3] mac80211: mesh: flush stations before beacons are stopped

2016-08-01 Thread Johannes Berg
On Wed, 2016-07-13 at 14:44 +0300, Yaniv Machani wrote: > From: Maital Hahn > > Some drivers (e.g. wl18xx) expect that the last stage in the > de-initialization process will be stopping the beacons, similar to AP > flow. > Update ieee80211_stop_mesh() flow accordingly. > As peers

Re: [PATCH v2 15/15] ethernet: ti: davinci_emac: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Mugunthan V N
+omap ML On Monday 01 August 2016 12:32 PM, Peter Chen wrote: > of_node_put needs to be called when the device node which is got > from of_parse_phandle has finished using. > > Signed-off-by: Peter Chen Reviewed-by: Mugunthan V N Regards Mugunthan V N

Re: [PATCH v2 14/15] ethernet: stmicro: stmmac: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Alexandre Torgue
Hi, On 08/01/2016 09:02 AM, Peter Chen wrote: of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. This commit fixes both local (in stmmac_axi_setup) and global (plat->phy_node) device_node for this issue, and using the correct device node

[RFC PATCH 2/2] net: macb: Add 64 bit addressing support for GEM

2016-08-01 Thread Harini Katakam
This patch adds support for 64 bit addressing and BDs. -> Enable 64 bit addressing in DMACFG register. -> Set DMA mask when design config register shows support for 64 bit addr. -> Add new BD words for higher address when 64 bit DMA support is present. -> Add and update TBQPH and RBQPH for MSB of

Re: [PATCH 1/1] rps: Inspect PPTP encapsulated by GRE to get flow hash

2016-08-01 Thread Feng Gao
Hi Philp & Tom, Thanks your advice. I have send the update patch some days ago. The link is http://www.spinics.net/lists/netdev/msg388083.html. Could you help review it please? Best Regards Feng On Thu, Jul 28, 2016 at 8:46 AM, Philp Prindeville wrote: >

[PATCH v2 03/15] ethernet: arc: emac_main: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/arc/emac_main.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git

[PATCH v2 07/15] ethernet: hisilicon: hns: hns_dsaf_main: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_main.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[RFC PATCH 1/2] net: macb: Correct CAPS mask

2016-08-01 Thread Harini Katakam
USRIO and JUMBO CAPS have the same mask. Fix the same. Signed-off-by: Harini Katakam --- drivers/net/ethernet/cadence/macb.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/cadence/macb.h b/drivers/net/ethernet/cadence/macb.h index

[PATCH v2 15/15] ethernet: ti: davinci_emac: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/ti/davinci_emac.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/ti/davinci_emac.c

[PATCH v2 13/15] ethernet: stmicro: stmmac: dwmac-socfpga: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- .../net/ethernet/stmicro/stmmac/dwmac-socfpga.c| 36 -- 1 file changed, 26 insertions(+), 10 deletions(-) diff

[PATCH v2 12/15] ethernet: renesas: sh_eth: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen Acked-by: Sergei Shtylyov --- drivers/net/ethernet/renesas/sh_eth.c | 1 + 1 file changed, 1 insertion(+)

[PATCH v2 09/15] ethernet: marvell: mvpp2: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/marvell/mvpp2.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/ethernet/marvell/mvpp2.c

[PATCH v2 10/15] ethernet: marvell: pxa168_eth: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/marvell/pxa168_eth.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/marvell/pxa168_eth.c

[PATCH v2 11/15] ethernet: renesas: ravb_main: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen Acked-by: Sergei Shtylyov --- drivers/net/ethernet/renesas/ravb_main.c | 1 + 1 file changed, 1

[PATCH v2 14/15] ethernet: stmicro: stmmac: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. This commit fixes both local (in stmmac_axi_setup) and global (plat->phy_node) device_node for this issue, and using the correct device node when tries to put node at stmmac_probe_config_dt

[PATCH v2 06/15] ethernet: hisilicon: hns: hns_dsaf_mac: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/hisilicon/hns/hns_dsaf_mac.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH v2 04/15] ethernet: aurora: nb8800: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/aurora/nb8800.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/ethernet/aurora/nb8800.c

[PATCH v2 00/15] add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
Hi David, This patch set fixes missing of_node_put issue at ethernet driver. of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. The compilation test has passed by using allmodconfig for drivers/net/ethernet. Thanks. Changes for v2: - If

[PATCH v2 05/15] ethernet: cavium: octeon: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/cavium/octeon/octeon_mgmt.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH v2 01/15] ethernet: altera: add missing of_node_put

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle (or of_node_get) has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/altera/altera_tse_main.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v2 02/15] ethernet: apm: xgene: add missing of_node_put after calling of_parse_phandle

2016-08-01 Thread Peter Chen
of_node_put needs to be called when the device node which is got from of_parse_phandle has finished using. Signed-off-by: Peter Chen --- drivers/net/ethernet/apm/xgene/xgene_enet_hw.c | 1 + 1 file changed, 1 insertion(+) diff --git

  1   2   >