[PATCH bpf-next] selftests/bpf: add netdevsim to config

2017-12-18 Thread Jakub Kicinski
BPF offload tests (test_offload.py) will require netdevsim to be built, add it to config. Signed-off-by: Jakub Kicinski Reviewed-by: Quentin Monnet --- tools/testing/selftests/bpf/config | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/selftests/bpf/config b/tools/testing/self

[PATCH ethtool v2] ethtool: Support for FEC encoding control

2017-12-18 Thread Jakub Kicinski
From: Dustin Byford As FEC settings and different FEC modes are mandatory and configurable across various interfaces of 25G/50G/100G/40G, the lack of FEC encoding control and reporting today is a source for interoperability issues for many vendors set-fec/show-fec option(s) are designed to provi

[PATCHv3 net-next 02/14] net: sched: sch_api: handle generic qdisc errors

2017-12-18 Thread Alexander Aring
This patch adds extack support for generic qdisc handling. The extack will be set deeper to each called function which is not part of netdev core api. Cc: David Ahern Signed-off-by: Alexander Aring --- net/sched/sch_api.c | 148 +--- 1 file change

[PATCHv3 net-next 07/14] net: sched: sch: add extack for graft callback

2017-12-18 Thread Alexander Aring
This patch adds extack support for graft callback to prepare per-qdisc specific changes for extack. Cc: David Ahern Signed-off-by: Alexander Aring --- include/net/sch_generic.h | 3 ++- net/sched/sch_api.c | 3 ++- net/sched/sch_atm.c | 3 ++- net/sched/sch_cbq.c | 2 +- net/s

[PATCHv3 net-next 08/14] net: sch: api: add extack support in qdisc_get_rtab

2017-12-18 Thread Alexander Aring
This patch adds extack support for the function qdisc_get_rtab which is a common used function in the tc subsystem. Callers which are interested in the receiving error can assign extack to get a more detailed information why qdisc_get_rtab failed. Cc: David Ahern Signed-off-by: Alexander Aring -

[PATCHv3 net-next 09/14] net: sch: api: add extack support in tcf_block_get

2017-12-18 Thread Alexander Aring
This patch adds extack support for the function tcf_block_get which is a common used function in the tc subsystem. Callers which are interested in the receiving error can assign extack to get a more detailed information why tcf_block_get failed. Cc: David Ahern Signed-off-by: Alexander Aring ---

[PATCHv3 net-next 11/14] net: sch: api: add extack support in qdisc_create_dflt

2017-12-18 Thread Alexander Aring
This patch adds extack support for the function qdisc_create_dflt which is a common used function in the tc subsystem. Callers which are interested in the receiving error can assign extack to get a more detailed information why qdisc_create_dflt failed. The function qdisc_create_dflt will also call

[PATCHv3 net-next 12/14] net: sch: sch_cbq: add extack support

2017-12-18 Thread Alexander Aring
This patch adds extack support for the cbq qdisc implementation by adding NL_SET_ERR_MSG in validation of user input. Also it serves to illustrate a use case of how the infrastructure ops api changes are to be used by individual qdiscs. Cc: David Ahern Signed-off-by: Alexander Aring --- net/sch

[PATCHv3 net-next 14/14] net: sch: sch_drr: add extack support

2017-12-18 Thread Alexander Aring
This patch adds extack support for the drr qdisc implementation by adding NL_SET_ERR_MSG in validation of user input. Also it serves to illustrate a use case of how the infrastructure ops api changes are to be used by individual qdiscs. Cc: David Ahern Signed-off-by: Alexander Aring --- net/sch

[PATCHv3 net-next 13/14] net: sch: sch_cbs: add extack support

2017-12-18 Thread Alexander Aring
This patch adds extack support for the cbs qdisc implementation by adding NL_SET_ERR_MSG in validation of user input. Also it serves to illustrate a use case of how the infrastructure ops api changes are to be used by individual qdiscs. Cc: David Ahern Signed-off-by: Alexander Aring --- net/sch

[PATCHv3 net-next 04/14] net: sched: sch: add extack for change qdisc ops

2017-12-18 Thread Alexander Aring
This patch adds extack support for change callback for qdisc ops structtur to prepare per-qdisc specific changes for extack. Cc: David Ahern Signed-off-by: Alexander Aring --- include/net/sch_generic.h | 3 ++- net/sched/sch_api.c | 2 +- net/sched/sch_cbs.c | 5 +++-- net/sched/

[PATCHv3 net-next 05/14] net: sched: sch: add extack to change class

2017-12-18 Thread Alexander Aring
This patch adds extack support for class change callback api. This prepares to handle extack support inside each specific class implementation. Cc: David Ahern Signed-off-by: Alexander Aring --- include/net/sch_generic.h | 3 ++- net/sched/sch_api.c | 2 +- net/sched/sch_atm.c | 3 +

[PATCHv3 net-next 06/14] net: sched: sch: add extack for block callback

2017-12-18 Thread Alexander Aring
This patch adds extack support for block callback to prepare per-qdisc specific changes for extack. Cc: David Ahern Signed-off-by: Alexander Aring --- include/net/sch_generic.h | 3 ++- net/sched/cls_api.c | 4 ++-- net/sched/sch_api.c | 2 +- net/sched/sch_atm.c | 3 ++- net/

[PATCHv3 net-next 10/14] net: sch: api: add extack support in qdisc_alloc

2017-12-18 Thread Alexander Aring
This patch adds extack support for the function qdisc_alloc which is a common used function in the tc subsystem. Callers which are interested in the receiving error can assign extack to get a more detailed information why qdisc_alloc failed. Cc: David Ahern Signed-off-by: Alexander Aring --- in

[PATCHv3 net-next 03/14] net: sched: sch: add extack for init callback

2017-12-18 Thread Alexander Aring
This patch adds extack support for init callback to prepare per-qdisc specific changes for extack. Cc: David Ahern Signed-off-by: Alexander Aring --- include/net/sch_generic.h | 3 ++- net/sched/sch_api.c | 2 +- net/sched/sch_atm.c | 3 ++- net/sched/sch_cbq.c | 3 ++- ne

[PATCHv3 net-next 00/14] net: sched: sch: introduce extack support

2017-12-18 Thread Alexander Aring
Hi, this patch series basically add support for extack in common qdisc handling. Additional it adds extack pointer to common qdisc callback handling this offers per qdisc implementation to setting the extack message for each failure over netlink. The extack message will be set deeper in qdisc fun

[PATCHv3 net-next 01/14] net: sched: fix coding style issues

2017-12-18 Thread Alexander Aring
This patch fix checkpatch issues for upcomming patches according to the sched api file. It changes mostly how to check on null pointer. Signed-off-by: Alexander Aring --- net/sched/sch_api.c| 2 +- net/sched/sch_cbq.c| 12 ++-- net/sched/sch_gred.c | 7 --- net/sched/sch_

Re: [PATCH 1/3] kallsyms: don't leak address when symbol not found

2017-12-18 Thread Tobin C. Harding
On Mon, Dec 18, 2017, at 20:55, Felix Fietkau wrote: > On 2017-12-18 00:53, Tobin C. Harding wrote: > > Currently if kallsyms_lookup() fails to find the symbol then the address > > is printed. This potentially leaks sensitive information. Instead of > > printing the address we can return an error,

[PATCH 1/2] net: stmmac: Fix TX timestamp calculation

2017-12-18 Thread Fredrik Hallenberg
When using GMAC4 the value written in PTP_SSIR should be shifted however the shifted value is also used in subsequent calculations which results in a bad timestamp value. Signed-off-by: Fredrik Hallenberg --- drivers/net/ethernet/stmicro/stmmac/stmmac_hwtstamp.c | 6 -- 1 file changed, 4 ins

[PATCH 2/2] net: stmmac: Fix bad RX timestamp extraction

2017-12-18 Thread Fredrik Hallenberg
As noted in dwmac4_wrback_get_rx_timestamp_status the timestamp is found in the context descriptor following the current descriptor. However the current code looks for the context descriptor in the current descriptor, which will always fail. Signed-off-by: Fredrik Hallenberg --- drivers/net/ethe

Re: [PATCH 3/3] trace: print address if symbol not found

2017-12-18 Thread Tobin C. Harding
On Mon, Dec 18, 2017 at 11:49:47AM -0500, Steven Rostedt wrote: > On Mon, 18 Dec 2017 10:53:32 +1100 > "Tobin C. Harding" wrote: > > > Fixes behaviour modified by: commit bd6b239cdbb2 ("kallsyms: don't leak > > address when symbol not found") > > > > Previous patch changed behaviour of kallsyms

[Patch net-next] net_sched: properly check for empty skb array on error path

2017-12-18 Thread Cong Wang
First, the check of &q->ring.queue against NULL is wrong, it is always false. We should check the value rather than the address. Secondly, we need the same check in pfifo_fast_reset() too, as both ->reset() and ->destroy() are called in qdisc_destroy(). Fixes: c5ad119fb6c0 ("net: sched: pfifo_fas

Re: r8169 regression: UDP packets dropped intermittantly

2017-12-18 Thread Jonathan Woithe
Hi Holger On Mon, Dec 18, 2017 at 02:38:53PM +0100, Holger Hoffstätte wrote: > On 12/18/17 06:49, Jonathan Woithe wrote: > > Resend to netdev. LKML CCed in case anyone in the wider kernel community > > can suggest a way forward. Please CC responses if replying only to LKML. > > > > It seems tha

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Dave Chinner
On Sun, Dec 17, 2017 at 04:28:44PM -0800, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for the

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Alexandre Belloni
On 17/12/2017 at 16:28:44 -0800, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for these modifi

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Darrick J. Wong
On Sun, Dec 17, 2017 at 04:28:44PM -0800, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for the

Re: [PATCH] net: thunderx: add support for rgmii internal delay

2017-12-18 Thread Tim Harvey
On Thu, Dec 14, 2017 at 12:45 AM, Andrew Lunn wrote: > On Wed, Dec 13, 2017 at 03:28:33PM -0800, Tim Harvey wrote: >> On Wed, Dec 13, 2017 at 3:10 AM, Andrew Lunn wrote: >> >> +void xcv_init_hw(int phy_mode) >> >> { >> >> u64 cfg; >> >> >> >> @@ -81,12 +81,31 @@ void xcv_init_hw(void) >>

[RFC 12/14] tcp_md5: Use tcp_extra_options in output path

2017-12-18 Thread Christoph Paasch
This patch starts making use of the extra_option framework for TCP_MD5. One tricky part is that extra_options are called at the end of the tcp_syn_options(), while TCP_MD5 is called at the beginning. TCP_MD5 is called at the beginning because it wants to disable TCP-timestamps (for option-space r

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Alex Deucher
On Sun, Dec 17, 2017 at 7:28 PM, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for these modifie

[RFC 05/14] tcp: Register handlers for extra TCP options

2017-12-18 Thread Christoph Paasch
From: Mat Martineau Allow additional TCP options to be handled by registered hook functions. Registered options have a priority that determines the order in which options are prepared and written. Lower priority numbers are handled first. Option parsing will call the provided 'parse' function w

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Paul Moore
On Sun, Dec 17, 2017 at 7:28 PM, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for these modifie

Re: thunderx sgmii interface hang

2017-12-18 Thread Tim Harvey
On Wed, Dec 13, 2017 at 11:43 AM, Andrew Lunn wrote: >> The nic appears to work fine (pings, TCP etc) up until a performance >> test is attempted. >> When an iperf bandwidth test is attempted the nic ends up in a state >> where truncated-ip packets are being sent out (per a tcpdump from >> another

[RFC 06/14] tcp_smc: Make SMC use TCP extra-option framework

2017-12-18 Thread Christoph Paasch
Adopt the extra-option framework for SMC. It allows us to entirely remove SMC-code out of the TCP-stack. The static key is gone, as this is now covered by the static key of the extra-option framework. We allocate state (struct tcp_smc_opt) that indicates whether SMC was successfully negotiated or

[RFC 04/14] tcp_smc: Make smc_parse_options return 1 on success

2017-12-18 Thread Christoph Paasch
As we allow a generic TCP-option parser that also parses experimental TCP options, we need to add a return-value to smc_parse_options() that indicates whether the option actually matched or not. Cc: Ursula Braun Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- net/ipv4/tcp_input.

[RFC 13/14] tcp_md5: Cleanup TCP-code

2017-12-18 Thread Christoph Paasch
Now that we have consolidated the TCP_MD5 output path, we can cleanup TCP and its callbacks to MD5. These callbacks are solely there to handle the different address-familiese (v4, v6 and v4mapped). Now that we have isolated the TCP_MD5-code it is acceptable to add a bit more complexity inside tcp

[RFC 11/14] tcp_md5: Move TCP-MD5 code out of TCP itself

2017-12-18 Thread Christoph Paasch
This is all just copy-pasting the TCP_MD5-code into functions that are placed in net/ipv4/tcp_md5.c. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- include/linux/inet_diag.h |1 + include/linux/tcp_md5.h | 138 ++ include/net/tcp.h | 77 net/ipv4/Makefi

[RFC 08/14] tcp_md5: Detect key inside tcp_v4_send_ack instead of passing it as an argument

2017-12-18 Thread Christoph Paasch
This will simplify to consolidate the TCP_MD5-code into a single place. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- net/ipv4/tcp_ipv4.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/net/ipv4/tcp_ipv4.c b/net/ipv4/tcp_ipv4.c index dee2

[RFC 10/14] tcp_md5: Check for TCP_MD5 after TCP Timestamps in tcp_established_options

2017-12-18 Thread Christoph Paasch
It really does not matter, because we never use TCP timestamps when TCP_MD5 is enabled (see tcp_syn_options). Moving TCP_MD5 a bit lower allows for easier adoption of the tcp_extra_option framework. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- net/ipv4/tcp_output.c | 14 +

[RFC 02/14] tcp: Pass sock and skb to tcp_options_write

2017-12-18 Thread Christoph Paasch
An upcoming patch adds a configurable, per-socket list of TCP options to populate in the TCP header. This requires tcp_options_write() to know the socket (to use the options list) and the skb (to provide visibility to the packet data for options like TCP_MD5SIG). Signed-off-by: Christoph Paasch R

[RFC 14/14] tcp_md5: Use TCP extra-options on the input path

2017-12-18 Thread Christoph Paasch
The checks are now being done through the extra-option framework. For TCP MD5 this means that the check happens a bit later than usual. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- include/linux/tcp_md5.h | 23 +-- net/ipv4/tcp_input.c| 8 net

[RFC 09/14] tcp_md5: Detect key inside tcp_v6_send_response instead of passing it as an argument

2017-12-18 Thread Christoph Paasch
We want to move all the TCP-MD5 code to a single place which enables us to factor the TCP-MD5 code out of the TCP-stack into the extra-option framework. Detection of whether or not to drop the segment (as done in tcp_v6_send_reset()) has now been moved to tcp_v6_send_response(). So we needed to ad

[RFC 03/14] tcp: Allow tcp_fast_parse_options to drop segments

2017-12-18 Thread Christoph Paasch
After parsing the TCP-options, some option-kinds might trigger a drop of the segment (e.g., as is the case for TCP_MD5). As we are moving to consolidate the TCP_MD5-code in follow-up patches, we need to add the capability to drop a segment right after parsing the options in tcp_fast_parse_options()

[RFC 07/14] tcp_md5: Don't pass along md5-key

2017-12-18 Thread Christoph Paasch
It is much cleaner to store the key-pointer in tcp_out_options. It allows to remove some MD5-specific code out of the function-arguments and paves the way to adopting the TCP-option framework with TCP-MD5. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martineau --- include/linux/tcp.h | 1

[RFC 00/14] Generic TCP-option framework and adoption for TCP-SMC and TCP-MD5

2017-12-18 Thread Christoph Paasch
This patchset introduces a generic framework for handling TCP-options. TCP-options like TCP_MD5 and SMC are rather rare use-cases, but their implementation is rather intrusive to the TCP-stack. Other, more recent TCP extensions like TCP-crypt, MPTCP or TCP-AO would make this situation even worse.

[RFC 01/14] tcp: Write options after the header has been fully done

2017-12-18 Thread Christoph Paasch
The generic TCP-option framework will need to have access to the full TCP-header (e.g., if we want to compute a checksum for TCP-MD5). Thus, we move the call to tcp_options_write() to after all the fields in the header have been filled out. Signed-off-by: Christoph Paasch Reviewed-by: Mat Martin

Re: [PATCH v7 2/3] sock: Move the socket inuse to namespace.

2017-12-18 Thread Cong Wang
On Mon, Dec 18, 2017 at 11:30 AM, David Miller wrote: > From: Tonghao Zhang > Date: Thu, 14 Dec 2017 05:51:58 -0800 > >> In some case, we want to know how many sockets are in use in >> different _net_ namespaces. It's a key resource metric. > > Useful or not, you're not exporting this value. > >

[PATCH net-next] dev: Correctly get length of alias string in dev_set_alias()

2017-12-18 Thread Serhey Popovych
We supply number of bytes available in @alias via @len parameter to dev_set_alias() which is not the same as zero terminated string length that can be shorter. Both dev_set_alias() users (rtnetlink and sysfs) can submit number of bytes up to IFALIASZ with actual string length slightly shorter by p

Re: [PATCH iproute2 1/3] iplink: Improve index parameter handling

2017-12-18 Thread Serhey Popovich
Stephen Hemminger wrote: > On Mon, 18 Dec 2017 23:02:07 +0200 > Serhey Popovich wrote: > >> Stephen Hemminger wrote: >>> On Mon, 18 Dec 2017 20:54:06 +0200 >>> Serhey Popovych wrote: >>> diff --git a/ip/iplink.c b/ip/iplink.c index 1e685cc..4f9c169 100644 --- a/ip/iplink.c

Re: [PATCH iproute2 1/3] iplink: Improve index parameter handling

2017-12-18 Thread Stephen Hemminger
On Mon, 18 Dec 2017 23:02:07 +0200 Serhey Popovich wrote: > Stephen Hemminger wrote: > > On Mon, 18 Dec 2017 20:54:06 +0200 > > Serhey Popovych wrote: > > > >> diff --git a/ip/iplink.c b/ip/iplink.c > >> index 1e685cc..4f9c169 100644 > >> --- a/ip/iplink.c > >> +++ b/ip/iplink.c > >> @@ -586,

Re: [PATCH 3/3] trace: print address if symbol not found

2017-12-18 Thread Tobin C. Harding
On Mon, Dec 18, 2017 at 11:49:47AM -0500, Steven Rostedt wrote: > On Mon, 18 Dec 2017 10:53:32 +1100 > "Tobin C. Harding" wrote: > > > Fixes behaviour modified by: commit bd6b239cdbb2 ("kallsyms: don't leak > > address when symbol not found") > > > > Previous patch changed behaviour of kallsyms

Re: [PATCH iproute2 1/3] iplink: Improve index parameter handling

2017-12-18 Thread Serhey Popovich
Stephen Hemminger wrote: > On Mon, 18 Dec 2017 20:54:06 +0200 > Serhey Popovych wrote: > >> diff --git a/ip/iplink.c b/ip/iplink.c >> index 1e685cc..4f9c169 100644 >> --- a/ip/iplink.c >> +++ b/ip/iplink.c >> @@ -586,8 +586,10 @@ int iplink_parse(int argc, char **argv, struct >> iplink_req *req,

Re: [PATCH net-next 2/2 v9] net: ethernet: Add a driver for Gemini gigabit ethernet

2017-12-18 Thread Linus Walleij
On Mon, Dec 18, 2017 at 3:54 PM, Russell King - ARM Linux wrote: > On Mon, Dec 18, 2017 at 03:48:17PM +0100, Michał Mirosław wrote: >> On Mon, Dec 18, 2017 at 02:57:37PM +0100, Linus Walleij wrote: >> > On Sat, Dec 16, 2017 at 8:39 PM, Linus Walleij >> > wrote: >> > >> > > The Gemini ethernet ha

linux-next: Signed-off-by missing for commits in the net-next tree

2017-12-18 Thread Stephen Rothwell
Hi all, Commits d8d8ccf27741 ("sfc: update EF10 register definitions") 0bc959a95e8c ("sfc: populate the timer reload field") are missing a Signed-off-by from their author. -- Cheers, Stephen Rothwell

Re: [PATCH] qed: Remove unused QED_RDMA_DEV_CAP_* symbols and dev->dev_caps

2017-12-18 Thread David Miller
From: David Miller Date: Mon, 18 Dec 2017 15:13:54 -0500 (EST) > From: Bjorn Helgaas > Date: Fri, 15 Dec 2017 17:03:01 -0600 > >> From: Bjorn Helgaas >> >> The QED_RDMA_DEV_CAP_* symbols are only used to set bits in dev->dev_caps. >> Nobody ever looks at those bits. Remove the symbols and de

Re: [bpf-next V1-RFC PATCH 08/14] nfp: setup xdp_rxq_info

2017-12-18 Thread Jesper Dangaard Brouer
On Wed, 13 Dec 2017 18:34:27 -0800 Jakub Kicinski wrote: > On Wed, 13 Dec 2017 12:20:01 +0100, Jesper Dangaard Brouer wrote: > > Driver hook points for xdp_rxq_info: > > * init+reg: nfp_net_rx_ring_alloc > > * unreg : nfp_net_rx_ring_free > > > > In struct nfp_net_rx_ring moved member @size

Re: [PATCH] qed: Remove unused QED_RDMA_DEV_CAP_* symbols and dev->dev_caps

2017-12-18 Thread David Miller
From: Bjorn Helgaas Date: Fri, 15 Dec 2017 17:03:01 -0600 > From: Bjorn Helgaas > > The QED_RDMA_DEV_CAP_* symbols are only used to set bits in dev->dev_caps. > Nobody ever looks at those bits. Remove the symbols and dev_caps itself. > > Note that if these are ever used and added back, it loo

Re: [PATCH] cxgb4: Simplify PCIe Completion Timeout setting

2017-12-18 Thread David Miller
From: Bjorn Helgaas Date: Fri, 15 Dec 2017 17:01:50 -0600 > From: Bjorn Helgaas > > Simplify PCIe Completion Timeout setting by using the > pcie_capability_clear_and_set_word() interface. No functional change > intended. > > Signed-off-by: Bjorn Helgaas Applied to net-next.

Re: [PATCH net-next 0/2] net: erspan: a couple fixes

2017-12-18 Thread David Miller
From: William Tu Date: Fri, 15 Dec 2017 14:27:42 -0800 > Haishuang Yan reports a couple of issues (wrong return value, > pskb_may_pull) on erspan V1. Since erspan V2 is in net-next, > this series fix the similar issues on v2. Series applied, thank you.

Re: [PATCH] net: phy: xgene: disable clk on error paths

2017-12-18 Thread David Miller
From: Alexey Khoroshilov Date: Sat, 16 Dec 2017 00:52:39 +0300 > There are several error paths in xgene_mdio_probe(), > where clk is left undisabled. The patch fixes them. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov Applied, thank y

Re: [PATCH v16 3/4] hinic: Replace PCI pool old API

2017-12-18 Thread David Miller
From: Romain Perier Date: Fri, 15 Dec 2017 20:31:22 +0100 > From: Romain Perier > > The PCI pool API is deprecated. This commit replaces the PCI pool old > API by the appropriate function with the DMA pool API. > > Signed-off-by: Romain Perier Acked-by: David S. Miller

Re: [PATCH v16 2/4] net: e100: Replace PCI pool old API

2017-12-18 Thread David Miller
From: Romain Perier Date: Fri, 15 Dec 2017 20:31:21 +0100 > From: Romain Perier > > The PCI pool API is deprecated. This commit replaces the PCI pool old > API by the appropriate function with the DMA pool API. > > Signed-off-by: Romain Perier > Acked-by: Peter Senna Tschudin > Acked-by: Jef

Re: [PATCH] net: phy: marvell: avoid pause mode on SGMII-to-Copper for 88e151x

2017-12-18 Thread David Miller
From: Russell King Date: Fri, 15 Dec 2017 16:10:20 + > Observed on the 88e1512 in SGMII-to-Copper mode, negotiating pause > is unreliable. While the pause bits can be set in the advertisment > register, they clear shortly after negotiation with a link partner > commences irrespective of the

Re: [PATCH 0/3] More SFP/phylink fixes

2017-12-18 Thread David Miller
From: Russell King - ARM Linux Date: Fri, 15 Dec 2017 16:03:44 + > This series fixes a few more bits with sfp/phylink, particularly > confusion with the right way to test for the RTNL mutex being > held, a change in 2016 to the mdiobus_scan() behaviour that wasn't > noticed, and a fix for rea

Re: [PATCH] net: arc_emac: restart stalled EMAC

2017-12-18 Thread David Miller
From: Alexander Kochetkov Date: Fri, 15 Dec 2017 14:12:51 +0300 > Under certain conditions EMAC stop reception of incoming packets and > continuously increment R_MISS register instead of saving data into > provided buffer. The commit implement workaround for such situation. > Then the stall detec

Re: [PATCH v3 net-next 6/6] tls: Add generic NIC offload infrastructure.

2017-12-18 Thread Marcelo Ricardo Leitner
On Mon, Dec 18, 2017 at 01:10:33PM +0200, Ilya Lesokhin wrote: > This patch adds a generic infrastructure to offload TLS crypto to a > network devices. It enables the kernel TLS socket to skip encryption > and authentication operations on the transmit side of the data path. > Leaving those computat

Re: [PATCH net-next] net/ncsi: Don't take any action on HNCDSC AEN

2017-12-18 Thread David Miller
From: Samuel Mendoza-Jonas Date: Fri, 15 Dec 2017 16:16:40 +1100 > The current HNCDSC handler takes the status flag from the AEN packet and > will update or change the current channel based on this flag and the > current channel status. > > However the flag from the HNCDSC packet merely represen

Re: [PATCH v2 4/5] rds: Add runchecks.cfg for net/rds

2017-12-18 Thread Santosh Shilimkar
On 12/16/2017 6:02 PM, Knut Omang wrote: On Sat, 2017-12-16 at 12:00 -0800, santosh.shilim...@oracle.com wrote: On 12/16/17 10:24 AM, Joe Perches wrote: [...] Most of these existing messages from checkpatch should probably be inspected and corrected where possible to minimize the style differ

Re: [RFC PATCH 0/9] ethtool netlink interface (WiP)

2017-12-18 Thread David Miller
From: "John W. Linville" Date: Thu, 14 Dec 2017 16:07:56 -0500 > Even without considering the ioctl problesms, the current ethtool > API seems a bit crufty. It has been a catch-all, "where else would it > go?" dumping ground for a long time, and it has accrued a number of > not-entirely-related b

Re: [PATCH v7 2/3] sock: Move the socket inuse to namespace.

2017-12-18 Thread David Miller
From: Tonghao Zhang Date: Thu, 14 Dec 2017 05:51:58 -0800 > In some case, we want to know how many sockets are in use in > different _net_ namespaces. It's a key resource metric. Useful or not, you're not exporting this value. All this patch series does is convert the existing export of the glo

Re: [PATCH] Staging: irda: Do not check for NOT NULL before kfree()

2017-12-18 Thread Shreeya Patel
On Mon, 2017-12-18 at 11:20 -0800, Stephen Hemminger wrote: > On Tue, 19 Dec 2017 00:41:30 +0530 > Shreeya Patel wrote: > > > > > Do not check for NOT NULL before calling kfree because if the > > pointer is NULL, no action occurs. > > Done using the following semantic patch by coccinelle. > > >

Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program

2017-12-18 Thread Leon Romanovsky
On Mon, Dec 18, 2017 at 07:39:50PM +0100, Knut Omang wrote: > On Mon, 2017-12-18 at 17:56 +, Bart Van Assche wrote: > > On Mon, 2017-12-18 at 10:46 -0700, Jason Gunthorpe wrote: > > > On Sun, Dec 17, 2017 at 10:00:17PM -0800, Joe Perches wrote: > > > > > > > > Today when we run checkers we get

Re: [PATCH iproute2 1/3] iplink: Improve index parameter handling

2017-12-18 Thread Stephen Hemminger
On Mon, 18 Dec 2017 20:54:06 +0200 Serhey Popovych wrote: > diff --git a/ip/iplink.c b/ip/iplink.c > index 1e685cc..4f9c169 100644 > --- a/ip/iplink.c > +++ b/ip/iplink.c > @@ -586,8 +586,10 @@ int iplink_parse(int argc, char **argv, struct > iplink_req *req, > *name = *arg

Re: [net-next] phylib: Add device reset GPIO support causes DSA MT7530 acquires reset-gpios fails

2017-12-18 Thread Florian Fainelli
On 12/18/2017 12:01 AM, Andrew Lunn wrote: > Hi Sean > >> It probably can't. Because before the GPIO line is manipulated to reset, >> certain power control should be handled such as power sources from >> external PMIC to let devices actually enter the proper state. >> >> So, I thought the kind of

Re: [PATCH] Staging: irda: Do not check for NOT NULL before kfree()

2017-12-18 Thread Stephen Hemminger
On Tue, 19 Dec 2017 00:41:30 +0530 Shreeya Patel wrote: > Do not check for NOT NULL before calling kfree because if the > pointer is NULL, no action occurs. > Done using the following semantic patch by coccinelle. > > @@ > expression ptr; > @@ > > - if (ptr != NULL) { > kfree(ptr); > ptr =

Re: [PATCH v3 net-next 3/6] net: Add SW fallback infrastructure for offloaded sockets

2017-12-18 Thread Marcelo Ricardo Leitner
On Mon, Dec 18, 2017 at 01:10:30PM +0200, Ilya Lesokhin wrote: > Offloaded sockets rely on the netdev to transform the transmitted > packets before sending them over the network. > When a packet from an offloaded socket is looped back or > rerouted to a different device we need to detect it and > d

[PATCH] utils: fix makeargs stack overflow

2017-12-18 Thread Stephen Hemminger
The makeargs() function did not handle end of string correctly and would reference past end of string. Signed-off-by: Stephen Hemminger --- lib/utils.c | 23 --- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/lib/utils.c b/lib/utils.c index 7ced8c061cb0..df1f3

[PATCH] Staging: irda: Do not check for NOT NULL before kfree()

2017-12-18 Thread Shreeya Patel
Do not check for NOT NULL before calling kfree because if the pointer is NULL, no action occurs. Done using the following semantic patch by coccinelle. @@ expression ptr; @@ - if (ptr != NULL) { kfree(ptr); ptr = NULL; - } The semantic patch has the effect of adding an assignment of ptr to N

[net 1/1] tipc: fix list sorting bug in function tipc_group_update_member()

2017-12-18 Thread Jon Maloy
When, during a join operation, or during message transmission, a group member needs to be added to the group's 'congested' list, we sort it into the list in ascending order, according to its current advertised window size. However, we miss the case when the member is already on that list. This will

Re: [PATCH ethtool] ethtool: Support for FEC encoding control

2017-12-18 Thread John W. Linville
On Fri, Dec 15, 2017 at 04:35:17PM -0800, Jakub Kicinski wrote: > From: Dustin Byford > > As FEC settings and different FEC modes are mandatory > and configurable across various interfaces of 25G/50G/100G/40G, > the lack of FEC encoding control and reporting today is a source > for interoperabili

[PATCH iproute2 0/3] Improve iplink index, alias and name parameters handling

2017-12-18 Thread Serhey Popovych
In this series I present following improvements: 1) Check index is greather than zero and forbid specifying it multiple times in iplink_parse(). Use 0 instead of -1 as special value in iplink_modify(). 2) Do not stop parameters processing after alias given. Check alias length d

[PATCH iproute2 1/3] iplink: Improve index parameter handling

2017-12-18 Thread Serhey Popovych
Correctly check for valid network device index supplied on command line: indexes are always greather than zero. Check for duplicate "index" argument. Initialize @index to 0 to simplify handling it in iplink_modify(). Other callers (link_veth.c, iplink_vxcan.c) already did so. No need to initializ

[PATCH iproute2 3/3] iplink: Kill redundant network device name checks

2017-12-18 Thread Serhey Popovych
Since commit 625df645b703 (Check user supplied interface name lengths) iplink_parse() validates network device name using check_ifname() helpers. Remove redundant "name" length checks from iplink_parse() callers. Signed-off-by: Serhey Popovych --- ip/iplink_vxcan.c |8 +++- ip/link_veth

[PATCH iproute2 2/3] iplink: Process "alias" parameter correctly

2017-12-18 Thread Serhey Popovych
Do not stop parameters processing after "alias" parameter: it might not be a last one. Seems copy pasted from "type" parameter code. Check it's length does not exceed IFALIASZ - 1. Better we warn than get RTNL error. Signed-off-by: Serhey Popovych --- ip/iplink.c |7 --- 1 file changed,

[PATCH 1/2 net-next] ibmvnic: Rename IBMVNIC_MAX_TX_QUEUES to IBMVNIC_MAX_QUEUES

2017-12-18 Thread Thomas Falcon
This value denotes the maximum number of TX queues but is used to allocate both RX and TX queues. Signed-off-by: Thomas Falcon --- drivers/net/ethernet/ibm/ibmvnic.c | 2 +- drivers/net/ethernet/ibm/ibmvnic.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethe

[PATCH net-next] ibmvnic: Include header descriptor support for ARP packets

2017-12-18 Thread Thomas Falcon
In recent tests with new adapters, it was discovered that ARP packets were not being properly processed. This patch adds support for ARP packet headers to be passed to backing adapters, if necessary. Signed-off-by: Thomas Falcon --- drivers/net/ethernet/ibm/ibmvnic.c | 7 ++- 1 file changed,

[PATCH 2/2 net-next] ibmvnic: Increase maximum number of RX/TX queues

2017-12-18 Thread Thomas Falcon
Increase the number of queues allocated to accommodate recent network adapter inclusions on the IBM vNIC platform. Signed-off-by: Thomas Falcon --- drivers/net/ethernet/ibm/ibmvnic.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/ibm/ibmvnic.h b/drivers

[PATCH 0/2 net-next] ibmvnic: Fix and increase maximum TX/RX queues

2017-12-18 Thread Thomas Falcon
This series renames IBMVNIC_MAX_TX_QUEUES to IBMVNIC_MAX_QUEUES since it is used to allocate both RX and TX queues. The value is also increased to accommodate newer hardware. Thomas Falcon (2): ibmvnic: Rename IBMVNIC_MAX_TX_QUEUES to IBMVNIC_MAX_QUEUES ibmvnic: Increase maximum number of RX/T

Re: [PATCH] ethtool: fix MFLCN register dump for 82599 and newer

2017-12-18 Thread John W. Linville
On Fri, Dec 15, 2017 at 09:56:51AM +0800, Zhang Kang wrote: > Use MFLCN for 82599 and X540 HW instead of FCTRL. > > Signed-off-by: Zhang Kang > Signed-off-by: Gao Wayne > Signed-off-by: Wei Net Thanks, applied. -- John W. LinvilleSomeday the world will need a hero, and you li

Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program

2017-12-18 Thread Knut Omang
On Mon, 2017-12-18 at 17:56 +, Bart Van Assche wrote: > On Mon, 2017-12-18 at 10:46 -0700, Jason Gunthorpe wrote: > > On Sun, Dec 17, 2017 at 10:00:17PM -0800, Joe Perches wrote: > > > > > > Today when we run checkers we get so many warnings it is too hard to > > > > make any sense of it. > >

Re: [PATCH bpf-next] bpf: arm64: fix uninitialized variable

2017-12-18 Thread Alexei Starovoitov
On 12/18/17 10:19 AM, Daniel Borkmann wrote: On 12/18/2017 07:09 PM, Alexei Starovoitov wrote: From: Alexei Starovoitov fix the following issue: arch/arm64/net/bpf_jit_comp.c: In function 'bpf_int_jit_compile': arch/arm64/net/bpf_jit_comp.c:982:18: error: 'image_size' may be used uninitialized

Re: [net-next: PATCH 0/8] Armada 7k/8k PP2 ACPI support

2017-12-18 Thread Marcin Wojtas
Hi Ard 2017-12-18 10:40 GMT+01:00 Ard Biesheuvel : > On 18 December 2017 at 10:17, Marcin Wojtas wrote: >> Hi, >> >> This patchset introduces ACPI support in mvpp2 and mvmdio drivers. >> First three patches introduce fwnode helpers for obtaining PHY >> information from nodes and also MDIO fwnode

Re: pull-request: bpf 2017-12-17

2017-12-18 Thread David Miller
From: Daniel Borkmann Date: Sun, 17 Dec 2017 21:06:26 +0100 > The following pull-request contains BPF updates for your *net* tree. > > The main changes are: > > 1) Fix a corner case in generic XDP where we have non-linear skbs >but enough tailroom in the skb to not miss to linearizing there

Re: [PATCH net v2] net: bridge: fix early call to br_stp_change_bridge_id and plug newlink leaks

2017-12-18 Thread David Miller
From: Nikolay Aleksandrov Date: Mon, 18 Dec 2017 17:35:09 +0200 > The early call to br_stp_change_bridge_id in bridge's newlink can cause > a memory leak if an error occurs during the newlink because the fdb > entries are not cleaned up if a different lladdr was specified, also > another minor is

Re: [Intel-wired-lan] v4.15-rc2 on thinkpad x60: ethernet stopped working

2017-12-18 Thread Neftin, Sasha
On 12/18/2017 13:58, Pavel Machek wrote: On Mon 2017-12-18 13:24:40, Neftin, Sasha wrote: On 12/18/2017 12:26, Pavel Machek wrote: Hi! In v4.15-rc2+, network manager can not see my ethernet card, and manual attempts to ifconfig it up did not really help, either. Card is: 02:00.0 Ethernet co

Re: [PATCH net-next v3 0/7] net: phy: meson-gxl: clean-up and improvements

2017-12-18 Thread David Miller
From: Jerome Brunet Date: Mon, 18 Dec 2017 10:44:39 +0100 > This patchset adds defines for the control registers and helpers to access > the banked registers. The goal being to make it easier to understand what > the driver actually does. > Then CONFIG_A6 settings is removed since this statement

Re: [PATCH net] sctp: add SCTP_CID_RECONF conversion in sctp_cname

2017-12-18 Thread David Miller
From: Xin Long Date: Mon, 18 Dec 2017 14:13:17 +0800 > Whenever a new type of chunk is added, the corresp conversion in > sctp_cname should be added. Otherwise, in some places, pr_debug > will print it as "unknown chunk". > > Fixes: cc16f00f6529 ("sctp: add support for generating stream reconf s

Re: [PATCH net] sctp: fix the issue that a __u16 variable may overflow in sctp_ulpq_renege

2017-12-18 Thread David Miller
From: Xin Long Date: Mon, 18 Dec 2017 14:07:25 +0800 > Now when reneging events in sctp_ulpq_renege(), the variable freed > could be increased by a __u16 value twice while freed is of __u16 > type. It means freed may overflow at the second addition. > > This patch is to fix it by using __u32 typ

Re: [PATCH] net: qcom/emac: Change the order of mac up and sgmii open

2017-12-18 Thread David Miller
From: Hemanth Puranik Date: Mon, 18 Dec 2017 11:27:47 +0530 > This patch fixes the order of mac_up and sgmii_open for the > reasons noted below: > > - If open takes more time(if the SGMII block is not responding or > if we want to do some delay based task) in this situation we > will hit NET

Re: [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Rafael J. Wysocki
On Monday, December 18, 2017 1:28:44 AM CET Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for t

Re: [PATCH bpf-next] bpf: arm64: fix uninitialized variable

2017-12-18 Thread Daniel Borkmann
On 12/18/2017 07:09 PM, Alexei Starovoitov wrote: > From: Alexei Starovoitov > > fix the following issue: > arch/arm64/net/bpf_jit_comp.c: In function 'bpf_int_jit_compile': > arch/arm64/net/bpf_jit_comp.c:982:18: error: 'image_size' may be used > uninitialized in this function [-Werror=maybe-uni

<    1   2   3   4   >