Re: [PATCH] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread Julia Lawall
On Tue, 19 Feb 2019, Mao Wenan wrote: > There is no need to have the 'int first' static > since new value always be assigned before use it. > The old codes of first dereferencing this variable have > been commented out. So it is useless with 'static int first'. > /* > if (!first) { > ... > } >

Re: [PATCH 0/2] ARM: dts: am335x-evm/evmsk: Fix PHY mode for ethernet

2019-02-19 Thread Peter Ujfalusi
Hi Tony, On 18/02/2019 18.26, Tony Lindgren wrote: > * Peter Ujfalusi [190218 16:22]: >> >> >> On 18/02/2019 16.44, Tony Lindgren wrote: >>> * Peter Ujfalusi [190218 14:36]: Hi, cd28d1d6e52e: ("net: phy: at803x: Disable phy delay for RGMII mode") broke the ethernet netw

[PATCH net-next] net: rose: add missing dev_put() on error in rose_bind

2019-02-19 Thread YueHaibing
when capable check failed, dev_put should be call before return -EACCES. Signed-off-by: YueHaibing --- net/rose/af_rose.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/rose/af_rose.c b/net/rose/af_rose.c index d00a0ef..c96f63f 100644 --- a/net/rose/af_rose.c +++ b/ne

Re: [PATCH] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread maowenan
On 2019/2/19 16:01, Julia Lawall wrote: > > > On Tue, 19 Feb 2019, Mao Wenan wrote: > >> There is no need to have the 'int first' static >> since new value always be assigned before use it. >> The old codes of first dereferencing this variable have >> been commented out. So it is useless with

Re: [PATCH net-next v3 0/2] net: phy: at803x: Update delays for RGMII modes

2019-02-19 Thread Peter Ujfalusi
Hi Vinod, On 19/02/2019 8.17, Vinod Koul wrote: > Peter[1] reported that patch cd28d1d6e52e: ("net: phy: at803x: Disable > phy delay for RGMII mode") caused regression on am335x-evmsk board. > This board expects the Phy delay to be enabled but specified RGMII mode > which refers to delays being di

Re: [PATCH] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread Julia Lawall
On Tue, 19 Feb 2019, maowenan wrote: > > > On 2019/2/19 16:01, Julia Lawall wrote: > > > > > > On Tue, 19 Feb 2019, Mao Wenan wrote: > > > >> There is no need to have the 'int first' static > >> since new value always be assigned before use it. > >> The old codes of first dereferencing this var

Re: [PATCH] net/mlx4_en: fix spelling mistake: "quiting" -> "quitting"

2019-02-19 Thread Tariq Toukan
On 2/18/2019 10:08 PM, David Miller wrote: > From: Colin King > Date: Sun, 17 Feb 2019 23:03:31 + > >> From: Colin Ian King >> >> There is a spelling mistake in a en_err error message. Fix it. >> >> Signed-off-by: Colin Ian King > > Applied, thanks Colin. > > And I agree that this doesn

Regarding CLNP driver

2019-02-19 Thread Shreeya Patel
Hi All, My goal is to have the CLNP ( Connectionless network protocol ) driver working on the latest Ubuntu version / any Linux distros. But I see that CLNP driver was developed by a group of people during 2.6 kernel and now it doesn't exist anymore. Here is the source code... https://github.com/c

[PATCH v2] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread Mao Wenan
There is no need to have the 'int first' static since new value always be assigned before use it. This patch cleans up the codes in ns83820_probe_phy() as well. Signed-off-by: Mao Wenan --- v1->v2: delete unused codes. drivers/net/ethernet/natsemi/ns83820.c | 17 + 1 file chang

Re: [PATCH v2] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread Julia Lawall
On Tue, 19 Feb 2019, Mao Wenan wrote: > There is no need to have the 'int first' static > since new value always be assigned before use it. > > This patch cleans up the codes in ns83820_probe_phy() as well. As mentioned previously, there is no use of first in the function at all, other than se

Re: [PATCH v2] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread maowenan
On 2019/2/19 16:31, Julia Lawall wrote: > > > On Tue, 19 Feb 2019, Mao Wenan wrote: > >> There is no need to have the 'int first' static >> since new value always be assigned before use it. >> >> This patch cleans up the codes in ns83820_probe_phy() as well. > > As mentioned previously, ther

Re: stmmac / meson8b-dwmac

2019-02-19 Thread Jose Abreu
Hi Simon, On 2/18/2019 6:05 PM, Simon Huelck wrote: > disabling EEE doesnt help ( did it via the entry in the .dtb / .dts ), > the results are the same. I can confirm the LPI counters are zero or one > only after the test. It's interesting to see that you have a lot of RX packets but few RX i

Re: [PATCH v3 perf,bpf 05/11] perf, bpf: save bpf_prog_info in a rbtree in perf_env

2019-02-19 Thread Jiri Olsa
On Tue, Feb 19, 2019 at 05:52:20AM +, Song Liu wrote: > > > > On Feb 17, 2019, at 3:05 PM, Jiri Olsa wrote: > > > > On Fri, Feb 15, 2019 at 01:53:48PM -0800, Song Liu wrote: > > > > SNIP > > > >>info_linear = bpf_program__get_prog_info_linear(fd, arrays); > >>if (IS_ERR_OR_NULL(in

Re: [RFC v2] net: bridge: don't flood known multicast traffic when snooping is enabled

2019-02-19 Thread Ido Schimmel
Hi Nik, On Mon, Feb 18, 2019 at 02:21:07PM +0200, Nikolay Aleksandrov wrote: > This is v2 of the RFC patch which aims to forward packets to known > mdsts' ports only (the no querier case). After v1 I've kept > the previous behaviour when it comes to unregistered traffic or when > a querier is pres

[PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread Mao Wenan
This patch is to do code cleanup for ns83820_probe_phy(). It deletes unused variable 'first' and commented out code. Signed-off-by: Mao Wenan --- v2->v3: delte unused variable 'first'; change subject from "net: ns83820: drop pointless static qualifier in ns83820_probe_phy()" to "net: ns83820:

Re: [PATCH 3/8] dt-bindings: net: bluetooth: Add rtl8723bs-bluetooth

2019-02-19 Thread Stefan Wahren
Hi Vasily, Am 18.02.19 um 22:24 schrieb Vasily Khoruzhick: > On Mon, Feb 18, 2019 at 1:10 PM Rob Herring wrote: >> On Fri, Jan 18, 2019 at 09:02:27AM -0800, Vasily Khoruzhick wrote: >>> Add binding document for bluetooth part of RTL8723BS/RTL8723CS >>> >>> Signed-off-by: Vasily Khoruzhick >>> --

Re: [RFC v2] net: bridge: don't flood known multicast traffic when snooping is enabled

2019-02-19 Thread Linus Lüssing
On Mon, Feb 18, 2019 at 02:21:07PM +0200, Nikolay Aleksandrov wrote: > This is v2 of the RFC patch which aims to forward packets to known > mdsts' ports only (the no querier case). After v1 I've kept > the previous behaviour when it comes to unregistered traffic or when > a querier is present. All

Re: [PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread Julia Lawall
On Tue, 19 Feb 2019, Mao Wenan wrote: > This patch is to do code cleanup for ns83820_probe_phy(). > It deletes unused variable 'first' and commented out code. > > Signed-off-by: Mao Wenan Acked-by: Julia Lawall > --- > v2->v3: delte unused variable 'first'; change subject from > "net: ns

Re: general protection fault in tc_ctl_chain

2019-02-19 Thread Vlad Buslov
This is fixed by Dan Carpenter's patch "net: sched: potential NULL dereference in tcf_block_find()" that was submitted yesterday. On Mon 18 Feb 2019 at 20:02, Cong Wang wrote: > (Cc'ing Vlad, please fix it) > > On Wed, Feb 13, 2019 at 9:56 AM syzbot > wrote: >> >> Hello, >> >> syzbot found the f

Handling an Extra Signal at PHY Reset

2019-02-19 Thread Paul Kocialkowski
Hi, We are dealing with an Ethernet PHY (Marvell 88E1512) that comes with a CONFIG pin that must be connected to one of the other pins of the PHY to configure the LSB of the PHY address as well as I/O voltages (see section 2.18.1 Hardware Configuration of the datasheet). It must be connected "soon

Re: [PATCH v2] bonding: fix PACKET_ORIGDEV regression

2019-02-19 Thread Maciej Żenczykowski
Signed-off-by: Maciej Żenczykowski It certainly seems like this should do the trick, but I do agree some sort of tests would be nice.

Re: general protection fault in tc_ctl_chain

2019-02-19 Thread Dmitry Vyukov
On Tue, Feb 19, 2019 at 10:10 AM Vlad Buslov wrote: > > This is fixed by Dan Carpenter's patch "net: sched: potential NULL > dereference in tcf_block_find()" that was submitted yesterday. +Dan Let's tell syzbot that this is fixed: #syz fix: net: sched: potential NULL dereference in tcf_block_fi

Re: [PATCH AUTOSEL 3.18 12/16] sfc: suppress duplicate nvmem partition types in efx_ef10_mtd_probe

2019-02-19 Thread Bert Kenward
On 15/02/2019 02:15, Sasha Levin wrote: > From: Edward Cree > > [ Upstream commit 3366463513f544c12c6b88c13da4462ee9e7a1a1 ] > > Use a bitmap to keep track of which partition types we've already seen; > for duplicates, return -EEXIST from efx_ef10_mtd_probe_partition() and > thus skip adding t

Re: [PATCH] brcmfmac: remove set but not used variable 'old_state'

2019-02-19 Thread Arend Van Spriel
On 2/18/2019 9:08 AM, YueHaibing wrote: Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c: In function 'brcmf_usb_state_change': drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c:578:6: warning: variable 'old_state' set but not used [-W

Re: [Bridge] [RFC v2] net: bridge: don't flood known multicast traffic when snooping is enabled

2019-02-19 Thread Linus Lüssing
On Tue, Feb 19, 2019 at 09:57:16AM +0100, Linus Lüssing wrote: > On Mon, Feb 18, 2019 at 02:21:07PM +0200, Nikolay Aleksandrov wrote: > > This is v2 of the RFC patch which aims to forward packets to known > > mdsts' ports only (the no querier case). After v1 I've kept > > the previous behaviour whe

Re: [PATCH net-next 3/3] nfp: devlink: allow flashing the device via devlink

2019-02-19 Thread Jiri Pirko
Fri, Feb 15, 2019 at 04:44:29PM CET, jakub.kicin...@netronome.com wrote: >On Fri, 15 Feb 2019 11:15:14 +0100, Jiri Pirko wrote: >> > static const struct ethtool_ops nfp_net_ethtool_ops = { >> >> Why don't you use the compat fallback? I think you should. > >You and Michal both asked the same so l

Re: [PATCH net] mlxsw: __mlxsw_sp_port_headroom_set(): Fix a use of local variable

2019-02-19 Thread Ido Schimmel
On Sun, Feb 17, 2019 at 10:14:40AM -0800, David Miller wrote: > From: Ido Schimmel > Date: Sun, 17 Feb 2019 07:18:41 + > > > From: Petr Machata > > > > The function-local variable "delay" enters the loop interpreted as delay > > in bits. However, inside the loop it gets overwritten by the r

Re: Handling an Extra Signal at PHY Reset

2019-02-19 Thread Thomas Petazzoni
Hello Paul, On Tue, 19 Feb 2019 10:14:20 +0100 Paul Kocialkowski wrote: > We are dealing with an Ethernet PHY (Marvell 88E1512) that comes with a > CONFIG pin that must be connected to one of the other pins of the PHY > to configure the LSB of the PHY address as well as I/O voltages (see > secti

[PATCH net-next v3 3/3] net: stmmac: dwxgmac2: Also use TBU interrupt to clean TX path

2019-02-19 Thread Jose Abreu
TBU interrupt is a normal interrupt and can be used to trigger the cleaning of TX path. Lets check if it's active in DMA interrupt handler. While at it, refactor a little bit the function: - Don't check if RI is enabled because at function exit we will only clear the interrupts t

[PATCH net-next v3 1/3] net: stmmac: Fix NAPI poll in TX path when in multi-queue

2019-02-19 Thread Jose Abreu
Commit 8fce33317023 introduced the concept of NAPI per-channel and independent cleaning of TX path. This is currently breaking performance in some cases. The scenario happens when all packets are being received in Queue 0 but the TX is performed in Queue != 0. Fix this by using different NAPI ins

[PATCH net-next v3 2/3] net: stmmac: dwmac4: Also use TBU interrupt to clean TX path

2019-02-19 Thread Jose Abreu
TBU interrupt is a normal interrupt and can be used to trigger the cleaning of TX path. Lets check if it's active in DMA interrupt handler. While at it, refactor a little bit the function: - Don't check if RI is enabled because at function exit we will only clear the interrupts t

[PATCH net-next v3 0/3] net: stmmac: Performance improvements in Multi-Queue

2019-02-19 Thread Jose Abreu
Tested in XGMAC2 and GMAC5. Cc: Florian Fainelli Cc: Joao Pinto Cc: David S. Miller Cc: Giuseppe Cavallaro Cc: Alexandre Torgue Jose Abreu (3): net: stmmac: Fix NAPI poll in TX path when in multi-queue net: stmmac: dwmac4: Also use TBU interrupt to clean TX path net: stmmac: dwxgmac2:

Re: [PATCH net-next 01/12] net: sched: flower: don't check for rtnl on head dereference

2019-02-19 Thread Vlad Buslov
On Mon 18 Feb 2019 at 19:08, Cong Wang wrote: > On Wed, Feb 13, 2019 at 11:47 PM Vlad Buslov wrote: >> >> Flower classifier only changes root pointer during init and destroy. Cls >> API implements reference counting for tcf_proto, so there is no danger of >> concurrent access to tp when it is b

Re: [PATCH RESEND net] net: phy: xgmiitorgmii: Support generic PHY status read

2019-02-19 Thread Paul Kocialkowski
Hi, On Fri, 2019-02-15 at 10:53 -0800, Florian Fainelli wrote: > On 2/15/19 10:34 AM, Paul Kocialkowski wrote: > > As I was mentionning to Andrew in the initial submission of this patch, > > this driver is a bit unusual since it represents a GMII to RGMII > > bridge, so it's not actually a PHY dri

Re: [PATCH net-next 00/12] Refactor flower classifier to remove dependency on rtnl lock

2019-02-19 Thread Vlad Buslov
On Mon 18 Feb 2019 at 19:15, Cong Wang wrote: > Hi, > >> net/sched/cls_flower.c | 424 >> + >> 1 file changed, 321 insertions(+), 103 deletions(-) >> > > Given you change cls_flower so much, please also add a test case for > verifying your changes,

Re: [PATCH bpf-next 0/9] bpf: Network Resource Manager (NRM)

2019-02-19 Thread Daniel Borkmann
On 02/19/2019 06:38 AM, brakmo wrote: > Network Resource Manager is a framework for limiting the bandwidth used > by v2 cgroups. It consists of 4 BPF helpers and a sample BPF program to > limit egress bandwdith as well as a sample user program and script to > simplify NRM testing. > > The sample N

Re: [PATCH net-next 12/12] net: sched: flower: set unlocked flag for flower proto ops

2019-02-19 Thread Vlad Buslov
On Mon 18 Feb 2019 at 19:27, Cong Wang wrote: > On Wed, Feb 13, 2019 at 11:47 PM Vlad Buslov wrote: >> >> Set TCF_PROTO_OPS_DOIT_UNLOCKED for flower classifier to indicate that its >> ops callbacks don't require caller to hold rtnl lock. > > So, if this means RTNL is gone for all cls_flower cha

RE: [PATCH net-next] net/tls: Move protocol constants from cipher context to tls context

2019-02-19 Thread Vakul Garg
> -Original Message- > From: David Miller > Sent: Sunday, February 17, 2019 7:45 AM > To: Vakul Garg > Cc: netdev@vger.kernel.org; bor...@mellanox.com; > avia...@mellanox.com; davejwat...@fb.com; doro...@fb.com > Subject: Re: [PATCH net-next] net/tls: Move protocol constants from cipher

[PATCH bpf-next] bpf: add skb->queue_mapping write access from tc clsact

2019-02-19 Thread Jesper Dangaard Brouer
The skb->queue_mapping already have read access, via __sk_buff->queue_mapping. This patch allow BPF tc qdisc clsact write access to the queue_mapping via tc_cls_act_is_valid_access. It is already possible to change this via TC filter action skbedit tc-skbedit(8). Due to the lack of TC examples,

Re: [PATCH bpf-next 1/9] bpf: Add bpf helper bpf_tcp_enter_cwr

2019-02-19 Thread Daniel Borkmann
On 02/19/2019 06:38 AM, brakmo wrote: > This patch adds a new bpf helper BPF_FUNC_tcp_enter_cwr > "int bpf_tcp_enter_cwr(struct bpf_tcp_sock *tp)". > It is added to BPF_PROG_TYPE_CGROUP_SKB typed bpf_prog > which currently can be attached to the ingress and egress > path. > > This helper makes a t

Re: [PATCH net-next v4 10/17] net: sched: refactor tp insert/delete for concurrent execution

2019-02-19 Thread Vlad Buslov
On Mon 18 Feb 2019 at 19:55, Cong Wang wrote: > On Mon, Feb 18, 2019 at 3:19 AM Vlad Buslov wrote: >> >> >> On Fri 15 Feb 2019 at 23:17, Cong Wang wrote: >> > On Mon, Feb 11, 2019 at 12:56 AM Vlad Buslov wrote: >> >> +static bool tcf_proto_is_empty(struct tcf_proto *tp) >> >> +{ >> >> +

[PATCH net-next 4/5] bnxt_en: Add support for mdio read/write to external PHY

2019-02-19 Thread Michael Chan
From: Vasundhara Volam Add support for SIOCGMIIREG and SIOCSMIIREG ioctls to mdio read/write to external PHY. Signed-off-by: Vasundhara Volam Signed-off-by: Michael Chan --- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 69 ++- 1 file changed, 67 insertions(+), 2 del

[PATCH net-next 1/5] bnxt_en: Update firmware interface spec. to 1.10.0.47.

2019-02-19 Thread Michael Chan
Firmware error recover is the major change in this spec. Signed-off-by: Michael Chan --- drivers/net/ethernet/broadcom/bnxt/bnxt_hsi.h | 196 +++--- 1 file changed, 177 insertions(+), 19 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_hsi.h b/drivers/net/e

[PATCH net-next 5/5] bnxt_en: Return relevant error code when offload fails

2019-02-19 Thread Michael Chan
From: Sriharsha Basavapatna The driver returns -ENOSPC when tc_can_offload() check fails. Since that routine checks for flow parameters that are not supported by the driver, we should return the more appropriate -EOPNOTSUPP. Signed-off-by: Sriharsha Basavapatna Signed-off-by: Michael Chan ---

[PATCH net-next 3/5] bnxt_en: Propagate trusted VF attribute to firmware.

2019-02-19 Thread Michael Chan
Newer firmware understands the concept of a trusted VF, so propagate the trusted VF attribute set by the PF admin. to the firmware. Also, check the firmware trusted setting when considering the VF MAC address change and reporting the trusted setting to the user. Signed-off-by: Michael Chan ---

[PATCH net-next 2/5] bnxt_en: Add support for BCM957504

2019-02-19 Thread Michael Chan
From: Erik Burrows Add support for BCM957504 with device ID 1751 Signed-off-by: Erik Burrows Signed-off-by: Michael Chan --- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/n

[PATCH net-next 0/5] bnxt_en: Update for net-next.

2019-02-19 Thread Michael Chan
This series includes the usual firmware spec. update, a PCI ID addition, enhancements for VF trust, MDIO read/write for external PHY, and fixing the return code when TC flow offload fails. Erik Burrows (1): bnxt_en: Add support for BCM957504 Michael Chan (2): bnxt_en: Update firmware interfac

Re: [RFC PATCH net-next v3 12/21] ethtool: provide permanent hardware address in GET_INFO request

2019-02-19 Thread Jiri Pirko
Mon, Feb 18, 2019 at 07:22:24PM CET, mkube...@suse.cz wrote: >Add information about permanent hadware address of a device (as provided by >ETHTOOL_GPERMADDR ioctl command) in GET_INFO reply if ETH_INFO_IM_PERMADDR >flag is set in the request. > >There is no separate attribute for hardware address l

Re: [RFC PATCH net-next v3 00/21] ethtool netlink interface, part 1

2019-02-19 Thread Jiri Pirko
Mon, Feb 18, 2019 at 07:21:24PM CET, mkube...@suse.cz wrote: >Note: this is marked as RFC because it's rather late in the cycle; the plan >is to make a regular submission (with changes based on review) once >net-next reopens after the 5.1 merge window. The full (work in progress) >series, together

Re: [PATCH bpf-next 3/9] bpf: add bpf helper bpf_skb_set_ecn

2019-02-19 Thread Daniel Borkmann
On 02/19/2019 06:38 AM, brakmo wrote: > This patch adds a new bpf helper BPF_FUNC_skb_set_ecn > "int bpf_skb_set_Ecn(struct sk_buff *skb)". It is added to > BPF_PROG_TYPE_CGROUP_SKB typed bpf_prog which currently can > be attached to the ingress and egress path. This type of > bpf_prog cannot modif

Re: [PATCH bpf-next 4/9] bpf: Add bpf helper bpf_tcp_check_probe_timer

2019-02-19 Thread Daniel Borkmann
On 02/19/2019 06:38 AM, brakmo wrote: > This patch adds a new bpf helper BPF_FUNC_tcp_check_probe_timer > "int bpf_check_tcp_probe_timer(struct tcp_bpf_sock *tp, u32 when_us)". > It is added to BPF_PROG_TYPE_CGROUP_SKB typed bpf_prog which currently > can be attached to the ingress and egress path.

Re: [EXT] Re: [PATCH v2 3/8] net: thunderx: make CFG_DONE message to run through generic send-ack sequence

2019-02-19 Thread Vadim Lomovtsev
Hi David, On Mon, Feb 18, 2019 at 03:33:10PM -0800, David Miller wrote: > From: Vadim Lomovtsev > Date: Mon, 18 Feb 2019 09:52:14 + > > > @@ -169,6 +169,20 @@ static int nicvf_check_pf_ready(struct nicvf *nic) > > return 1; > > } > > > > +static int nicvf_send_cfg_done(struct nicvf *n

[net-next 2/3] tipc: introduce new capability flag for cluster

2019-02-19 Thread Hoang Le
As a preparation for introducing a moothly switching between replicast and broadcast method for multicast message. We have to introduce a new capability flag TIPC_MCAST_RBCTL to handle this new feature because of compatibility reasons. When a cluster upgrade a node can come back with this new capab

[net-next 3/3] tipc: smooth change between replicast and broadcast

2019-02-19 Thread Hoang Le
Currently, a multicast stream may start out using replicast, because there are few destinations, and then it should ideally switch to L2/broadcast IGMP/multicast when the number of destinations grows beyond a certain limit. The opposite should happen when the number decreases below the limit. To e

[net-next 1/3] tipc: support broadcast/replicast configurable for bc-link

2019-02-19 Thread Hoang Le
Currently, a multicast stream uses either broadcast or replicast as transmission method, based on the ratio between number of actual destinations nodes and cluster size. However, when an L2 interface (e.g., VXLAN) provides pseudo broadcast support, this becomes very inefficient, as it blindly repl

Re: [RFC PATCH net-next v3 12/21] ethtool: provide permanent hardware address in GET_INFO request

2019-02-19 Thread Michal Kubecek
On Tue, Feb 19, 2019 at 11:24:00AM +0100, Jiri Pirko wrote: > Mon, Feb 18, 2019 at 07:22:24PM CET, mkube...@suse.cz wrote: > >Add information about permanent hadware address of a device (as provided by > >ETHTOOL_GPERMADDR ioctl command) in GET_INFO reply if ETH_INFO_IM_PERMADDR > >flag is set in t

[iproute2-next 1/2] tipc: add link broadcast set method and ratio

2019-02-19 Thread Hoang Le
The command added here makes it possible to forcibly configure the broadcast link to use either broadcast or replicast, in addition to the already existing auto selection algorithm. A sample usage is shown below: $tipc link set broadcast BROADCAST $tipc link set broadcast AUTOSELECT ratio 25 $tip

[iproute2-next 2/2] tipc: add link broadcast get

2019-02-19 Thread Hoang Le
The command prints the actually method that multicast is running in the system. Also 'ratio' value for AUTOSELECT method. A sample usage is shown below: $tipc link get broadcast BROADCAST $tipc link get broadcast AUTOSELECT ratio:30% $tipc link get broadcast -j -p [ { "method": "AUTOSELE

Re: [PATCH net-next v3 1/2] net: phy: at803x: don't inline helpers

2019-02-19 Thread Marc Gonzalez
On 19/02/2019 07:17, Vinod Koul wrote: > Some helpers were inlined, but makes more sense to allow compiler > to do the right optimizations instead, so remove inline for > at803x_disable_rx_delay() and at803x_disable_tx_delay() I would word it slightly differently: Some helpers were declared with

Re: [PATCH bpf-next] bpf: add skb->queue_mapping write access from tc clsact

2019-02-19 Thread Daniel Borkmann
On 02/19/2019 11:24 AM, Jesper Dangaard Brouer wrote: > The skb->queue_mapping already have read access, via __sk_buff->queue_mapping. > > This patch allow BPF tc qdisc clsact write access to the queue_mapping via > tc_cls_act_is_valid_access. > > It is already possible to change this via TC filt

Re: [RFC PATCH net-next v3 00/21] ethtool netlink interface, part 1

2019-02-19 Thread Michal Kubecek
On Tue, Feb 19, 2019 at 11:35:08AM +0100, Jiri Pirko wrote: > >- some features provided by ethtool would rather belong to devlink (and > > some are already superseded by devlink); however, only few drivers > > provide devlink interface at the moment and as recent discussion on > > flashing revea

Re: [PATCH net-next v3 2/2] net: phy: at803x: disable delay only for RGMII mode

2019-02-19 Thread Marc Gonzalez
On 19/02/2019 07:18, Vinod Koul wrote: > Per "Documentation/devicetree/bindings/net/ethernet.txt" RGMII mode > should not have delay in PHY whereas RGMII_ID and RGMII_RXID/RGMII_TXID > can have delay in phy. PHY or phy? :-) > So disable the delay only for RGMII mode and enable for other modes. >

Re: [PATCH net-next v4 07/17] net: sched: protect filter_chain list with filter_chain_lock mutex

2019-02-19 Thread Vlad Buslov
On Tue 19 Feb 2019 at 05:26, Cong Wang wrote: > On Fri, Feb 15, 2019 at 7:35 AM Vlad Buslov wrote: >> >> Another problem that I found in cls_fw and cls_route is that they set >> arg->stop when empty. Both of them have code unchanged since it was >> committed initially in 2005 so I assume this c

Re: [PATCH net-next v3 2/2] net: phy: at803x: disable delay only for RGMII mode

2019-02-19 Thread Niklas Cassel
On Tue, Feb 19, 2019 at 12:57:18PM +0100, Marc Gonzalez wrote: > On 19/02/2019 07:18, Vinod Koul wrote: > > > Per "Documentation/devicetree/bindings/net/ethernet.txt" RGMII mode > > should not have delay in PHY whereas RGMII_ID and RGMII_RXID/RGMII_TXID > > can have delay in phy. > > PHY or phy?

Re: [RFC PATCH net-next v3 00/21] ethtool netlink interface, part 1

2019-02-19 Thread Jiri Pirko
Tue, Feb 19, 2019 at 12:57:27PM CET, mkube...@suse.cz wrote: >On Tue, Feb 19, 2019 at 11:35:08AM +0100, Jiri Pirko wrote: >> >- some features provided by ethtool would rather belong to devlink (and >> > some are already superseded by devlink); however, only few drivers >> > provide devlink interf

Re: [PATCH net-next v3 2/2] net: phy: at803x: disable delay only for RGMII mode

2019-02-19 Thread Marc Gonzalez
On 19/02/2019 13:33, Niklas Cassel wrote: > However, since the bootloader might have enabled delay on > TX, I still think that this patch does the right thing by > starting out with disabling delays for both RX and TX. > > But we should probably make the comment more elaborate: > > The value aft

Re: [PATCH mlx5-next v1] net/mlx5: Factor out HCA capabilities functions

2019-02-19 Thread Leon Romanovsky
On Sun, Feb 17, 2019 at 01:11:02PM +0200, Leon Romanovsky wrote: > From: Leon Romanovsky > > Combine all HCA capabilities setters under one function > and compile out the ODP related function in case kernel > was compiled without ODP support. > > Signed-off-by: Leon Romanovsky > --- > Changelog v

Re: L2TPv3 offset

2019-02-19 Thread James Chapman
On 19/02/2019 09:17, t.mart...@avm.de wrote: > > Hello, > > I saw that you removed the offset option from l2tp sessions in Linux > 4.16 (commit 900631ee6a2651dc4fbaecb8ef9fa5f1e3378853 l2tp: remove > configurable payload offset). Since we need something like that I'm > reaching out to you. > Adding

Re: [PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread Walter Harms
Am 19.02.2019 10:06, schrieb Mao Wenan: > This patch is to do code cleanup for ns83820_probe_phy(). > It deletes unused variable 'first' and commented out code. > > Signed-off-by: Mao Wenan > --- > v2->v3: delte unused variable 'first'; change subject from > "net: ns83820: drop pointless sta

Re: [PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread walter harms
Am 19.02.2019 10:06, schrieb Mao Wenan: > This patch is to do code cleanup for ns83820_probe_phy(). > It deletes unused variable 'first' and commented out code. > > Signed-off-by: Mao Wenan > --- > v2->v3: delte unused variable 'first'; change subject from > "net: ns83820: drop pointless st

Re: [PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread maowenan
On 2019/2/19 20:42, Walter Harms wrote: > > Am 19.02.2019 10:06, schrieb Mao Wenan: >> This patch is to do code cleanup for ns83820_probe_phy(). >> It deletes unused variable 'first' and commented out code. >> >> Signed-off-by: Mao Wenan >> --- >> v2->v3: delte unused variable 'first'; change

Re: [PATCH net-next v2] net: dsa: qca8k: Enable delay for RGMII_ID mode

2019-02-19 Thread Andrew Lunn
On Tue, Feb 19, 2019 at 12:29:43PM +0530, Vinod Koul wrote: > RGMII_ID specifies that we should have internal delay, so resurrect the > delay addition routine but under the RGMII_ID mode. > > Fixes: 40269aa9f40a ("net: dsa: qca8k: disable delay for RGMII mode") > Tested-by: Michal Vokáč > Signed-

Re: Handling an Extra Signal at PHY Reset

2019-02-19 Thread Paul Kocialkowski
Hi, On Tue, 2019-02-19 at 10:36 +0100, Thomas Petazzoni wrote: > Hello Paul, > > On Tue, 19 Feb 2019 10:14:20 +0100 > Paul Kocialkowski wrote: > > > We are dealing with an Ethernet PHY (Marvell 88E1512) that comes with a > > CONFIG pin that must be connected to one of the other pins of the PHY

[PATCH mlx5-next] net/mlx5: Separate ODP capabilities

2019-02-19 Thread Leon Romanovsky
From: Moni Shoua ODP support for XRC transport is not enabled by default in FW, so we need separate ODP checks to enable/disable it. While that, rewrite the set of ODP SRQ support capabilities in way that tests each field separately for clearness, which is not needed for current FW, but better t

Re: [RFC PATCH net-next v3 04/21] ethtool: helper functions for netlink interface

2019-02-19 Thread Michal Kubecek
On Mon, Feb 18, 2019 at 12:15:17PM -0800, Jakub Kicinski wrote: > On Mon, 18 Feb 2019 19:21:44 +0100 (CET), Michal Kubecek wrote: > > +/* create skb for a reply and fill device identification > > + * payload: payload length (without netlink and genetlink header) > > + * dev: device the reply is

Re: [PATCH] phy/marvell: prevent unneeded page switching

2019-02-19 Thread Andrew Lunn
On Tue, Feb 19, 2019 at 08:21:33AM +0100, Frank de Brabander wrote: > marvell_read_status() no longer first switches to the fiber page, instead > looks at the already active register page first. In case the link is down, > switches to the other (fiber/copper) register page. > > Unneeded register p

Re: [Bridge] [RFC v2] net: bridge: don't flood known multicast traffic when snooping is enabled

2019-02-19 Thread Nikolay Aleksandrov
On 19/02/2019 11:21, Linus Lüssing wrote: > On Tue, Feb 19, 2019 at 09:57:16AM +0100, Linus Lüssing wrote: >> On Mon, Feb 18, 2019 at 02:21:07PM +0200, Nikolay Aleksandrov wrote: >>> This is v2 of the RFC patch which aims to forward packets to known >>> mdsts' ports only (the no querier case). Afte

[PATCH v4] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread Mao Wenan
This patch is to do code cleanup for ns83820_probe_phy(). It deletes unused variable 'first', commented out code, and the pointless 'for' loop. Signed-off-by: Mao Wenan Acked-by: Julia Lawall --- v3->v4: delete pointless 'for' loop, variable 'i', and so on. drivers/net/ethernet/natsemi/ns83820

Re: Handling an Extra Signal at PHY Reset

2019-02-19 Thread Andrew Lunn
On Tue, Feb 19, 2019 at 10:14:20AM +0100, Paul Kocialkowski wrote: > Hi, > > We are dealing with an Ethernet PHY (Marvell 88E1512) that comes with a > CONFIG pin that must be connected to one of the other pins of the PHY > to configure the LSB of the PHY address as well as I/O voltages (see > sect

Re: [PATCH] bpf: bpftool, fix documentation for attach types

2019-02-19 Thread Alban Crequy
On Mon, Feb 11, 2019 at 2:26 PM Quentin Monnet wrote: > > 2019-02-11 13:54 UTC+0100 ~ Alban Crequy > > From: Alban Crequy > > > > bpftool has support for attach types "stream_verdict" and > > "stream_parser" but the documentation was referring to them with > > "skb_verdict" and "skb_parse". The

[PATCH][next] vhost: only return early if ret indicates an error or no iovecs have been processed

2019-02-19 Thread Colin King
From: Colin Ian King Currently the loop that calls log_write_hva on each iovec is never executed because of an incorrect error check on the return from the call to translate_desc. The check should be checking for a -ve error return and because it makes no sense to iterate over zero items, the ch

[PATCH iproute2] ip-rule/trivial: add comment about json key "to_tbl" for unspecific rule action

2019-02-19 Thread Thomas Haller
The key should not be called "to_tbl" because it is exactly not a FR_ACT_TO_TBL action. # ip rule add blackhole # ip -j rule | python -m json.tool ... { "priority": 0, "src": "all", "to_tbl": "blackhole" }, Still, as this is already released API from v4

Re: [PATCH net-next 09/12] net: sched: flower: handle concurrent tcf proto deletion

2019-02-19 Thread Vlad Buslov
On Mon 18 Feb 2019 at 20:47, Cong Wang wrote: > On Wed, Feb 13, 2019 at 11:47 PM Vlad Buslov wrote: >> >> Without rtnl lock protection tcf proto can be deleted concurrently. Check >> tcf proto 'deleting' flag after taking tcf spinlock to verify that no >> concurrent deletion is in progress. Ret

[PATCH bpf-next v2] bpf: bpftool, fix documentation for attach types

2019-02-19 Thread Alban Crequy
From: Alban Crequy bpftool has support for attach types "stream_verdict" and "stream_parser" but the documentation was referring to them as "skb_verdict" and "skb_parse". The inconsistency comes from commit b7d3826c2ed6 ("bpf: bpftool, add support for attaching programs to maps"). This patch cha

Re: [PATCH 3/8] dt-bindings: net: bluetooth: Add rtl8723bs-bluetooth

2019-02-19 Thread Rob Herring
On Mon, Feb 18, 2019 at 3:24 PM Vasily Khoruzhick wrote: > > On Mon, Feb 18, 2019 at 1:10 PM Rob Herring wrote: > > > > On Fri, Jan 18, 2019 at 09:02:27AM -0800, Vasily Khoruzhick wrote: > > > Add binding document for bluetooth part of RTL8723BS/RTL8723CS > > > > > > Signed-off-by: Vasily Khoruzh

Re: [PATCH 3/8] dt-bindings: net: bluetooth: Add rtl8723bs-bluetooth

2019-02-19 Thread Rob Herring
On Mon, Feb 18, 2019 at 4:28 PM Vasily Khoruzhick wrote: > > On Mon, Feb 18, 2019 at 2:08 PM Stefan Wahren wrote: > > > > Hi Vasily, > > Hi Stefan, > > > > Vasily Khoruzhick hat am 18. Februar 2019 um 22:24 > > > geschrieben: > > > > > > > > > On Mon, Feb 18, 2019 at 1:10 PM Rob Herring wrote:

[PATCH bpf-next v5 0/3] libbpf: adding AF_XDP support

2019-02-19 Thread Magnus Karlsson
This patch proposes to add AF_XDP support to libbpf. The main reason for this is to facilitate writing applications that use AF_XDP by offering higher-level APIs that hide many of the details of the AF_XDP uapi. This is in the same vein as libbpf facilitates XDP adoption by offering easy-to-use hig

[PATCH bpf-next v5 1/3] libbpf: add support for using AF_XDP sockets

2019-02-19 Thread Magnus Karlsson
This commit adds AF_XDP support to libbpf. The main reason for this is to facilitate writing applications that use AF_XDP by offering higher-level APIs that hide many of the details of the AF_XDP uapi. This is in the same vein as libbpf facilitates XDP adoption by offering easy-to-use higher level

[PATCH bpf-next v5 2/3] samples/bpf: convert xdpsock to use libbpf for AF_XDP access

2019-02-19 Thread Magnus Karlsson
This commit converts the xdpsock sample application to use the AF_XDP functions present in libbpf. This cuts down the size of it by nearly 300 lines of code. The default ring sizes plus the batch size has been increased and the size of the umem area has decreased. This so that the sample applicati

[PATCH bpf-next v5 3/3] xsk: add FAQ to facilitate for first time users

2019-02-19 Thread Magnus Karlsson
Added an FAQ section in Documentation/networking/af_xdp.rst to help first time users with common problems. As problems are getting identified, entries will be added to the FAQ. Signed-off-by: Magnus Karlsson --- Documentation/networking/af_xdp.rst | 36 +++- 1 fil

Re: [PATCH bpf-next v2] bpf: bpftool, fix documentation for attach types

2019-02-19 Thread Quentin Monnet
2019-02-19 15:13 UTC+0100 ~ Alban Crequy > From: Alban Crequy > > bpftool has support for attach types "stream_verdict" and > "stream_parser" but the documentation was referring to them as > "skb_verdict" and "skb_parse". The inconsistency comes from commit > b7d3826c2ed6 ("bpf: bpftool, add sup

[PATCH][next] ptp_qoriq: don't pass a large struct by value but instead pass it by reference

2019-02-19 Thread Colin King
From: Colin Ian King Passing the struct ptp_clock_info caps by parameter is passing over 130 bytes of data by value on the stack. Optimize this by passing it by reference instead. Also shinks the object code size: Before: textdata bss dec hex filename 125962160 64

Re: [PATCH] net: ns83820: drop pointless static qualifier in ns83820_probe_phy()

2019-02-19 Thread Dan Carpenter
Just delete it instead. regards, dan carpenter

Re: [PATCH v3 perf,bpf 05/11] perf, bpf: save bpf_prog_info in a rbtree in perf_env

2019-02-19 Thread Song Liu
> On Feb 19, 2019, at 12:51 AM, Jiri Olsa wrote: > > On Tue, Feb 19, 2019 at 05:52:20AM +, Song Liu wrote: >> >> >>> On Feb 17, 2019, at 3:05 PM, Jiri Olsa wrote: >>> >>> On Fri, Feb 15, 2019 at 01:53:48PM -0800, Song Liu wrote: >>> >>> SNIP >>> info_linear = bpf_program__get

Re: [PATCH v3] net: ns83820: code cleanup for ns83820_probe_phy()

2019-02-19 Thread Dan Carpenter
On Tue, Feb 19, 2019 at 01:42:03PM +0100, Walter Harms wrote: > > Am 19.02.2019 10:06, schrieb Mao Wenan: > > - > > for (i=1; i<2; i++) { > > > the loop here seems also pointless, so you can eliminate i. > (or did i muss something ?) > True. But please do it in a separate patch. regards,

Re: [PATCH bpf-next] bpf: add skb->queue_mapping write access from tc clsact

2019-02-19 Thread Jesper Dangaard Brouer
On Tue, 19 Feb 2019 12:46:57 +0100 Daniel Borkmann wrote: > On 02/19/2019 11:24 AM, Jesper Dangaard Brouer wrote: > > The skb->queue_mapping already have read access, via > > __sk_buff->queue_mapping. > > > > This patch allow BPF tc qdisc clsact write access to the queue_mapping via > > tc_cls_

Re: [PATCH] brcmfmac: remove set but not used variable 'old_state'

2019-02-19 Thread Kalle Valo
YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c: In function > 'brcmf_usb_state_change': > drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c:578:6: warning: > variable 'old_state' set but not used [-Wunused-but-

Re: [PATCH] arm64: do_csum: implement accelerated scalar version

2019-02-19 Thread Ilias Apalodimas
On Tue, Feb 19, 2019 at 12:08:42AM +0100, Ard Biesheuvel wrote: > It turns out that the IP checksumming code is still exercised often, > even though one might expect that modern NICs with checksum offload > have no use for it. However, as Lingyan points out, there are > combinations of features whe

Re: [PATCH] rsi: remove set but not used variables 'info, vif'

2019-02-19 Thread Kalle Valo
YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/net/wireless/rsi/rsi_91x_main.c: In function 'rsi_prepare_skb': > drivers/net/wireless/rsi/rsi_91x_main.c:127:24: warning: > variable 'vif' set but not used [-Wunused-but-set-variable] > > drivers/net/wireless/rsi/r

Re: Handling an Extra Signal at PHY Reset

2019-02-19 Thread Paul Kocialkowski
Hi Andrew, On Tue, 2019-02-19 at 14:36 +0100, Andrew Lunn wrote: > On Tue, Feb 19, 2019 at 10:14:20AM +0100, Paul Kocialkowski wrote: > > Hi, > > > > We are dealing with an Ethernet PHY (Marvell 88E1512) that comes with a > > CONFIG pin that must be connected to one of the other pins of the PHY >

  1   2   3   >