Re: [PATCH v2] net: usb: rtl8150: prevent set_ethernet_addr from setting uninit address

2020-10-02 Thread Anant Thazhemadam
On 03-10-2020 04:08, David Miller wrote: > From: Anant Thazhemadam > Date: Fri, 2 Oct 2020 17:04:13 +0530 > >> But this patch is about ensuring that an uninitialized variable's >> value (whatever that may be) is not set as the ethernet address >> blindly (without any form of checking if get_regi

Re: [PATCH v4] staging: qlge: fix build breakage with dumping enabled

2020-10-02 Thread Benjamin Poirier
On 2020-10-03 07:59 +0800, Coiby Xu wrote: > This fixes commit 0107635e15ac > ("staging: qlge: replace pr_err with netdev_err") which introduced an > build breakage of missing `struct ql_adapter *qdev` for some functions > and a warning of type mismatch with dumping enabled, i.e., > > $ make CFLAG

Re: [PATCH v2] net: usb: rtl8150: prevent set_ethernet_addr from setting uninit address

2020-10-02 Thread Anant Thazhemadam
On 02-10-2020 19:59, Petko Manolov wrote: > On 20-10-02 17:35:25, Anant Thazhemadam wrote: >> Yes, this clears things up for me. I'll see to it that this gets done in a >> v3. > If set_ethernet_addr() fail, don't return error, but use eth_hw_addr_random() > instead to set random MAC address and

[PATCH v3 2/2] vhost-vdpa: fix page pinning leakage in error path

2020-10-02 Thread Si-Wei Liu
Pinned pages are not properly accounted particularly when mapping error occurs on IOTLB update. Clean up dangling pinned pages for the error path. As the inflight pinned pages, specifically for memory region that strides across multiple chunks, would need more than one free page for book keeping an

[PATCH v3 0/2] vhost-vdpa mapping error path fixes

2020-10-02 Thread Si-Wei Liu
Commit 4c8cf31885f6 ("vhost: introduce vDPA-based backend") has following issues in the failure path of IOTLB update: 1) vhost_vdpa_map() does not clean up dangling iotlb entry upon mapping failure 2) vhost_vdpa_process_iotlb_update() has leakage of pinned pages in case of vhost_vdpa_map()

[PATCH v3 1/2] vhost-vdpa: fix vhost_vdpa_map() on error condition

2020-10-02 Thread Si-Wei Liu
vhost_vdpa_map() should remove the iotlb entry just added if the corresponding mapping fails to set up properly. Fixes: 4c8cf31885f6 ("vhost: introduce vDPA-based backend") Signed-off-by: Si-Wei Liu --- drivers/vhost/vdpa.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/vhost/vdp

Re: [PATCH bpf-next 2/3] samples: bpf: count syscalls in xdpsock

2020-10-02 Thread Yonghong Song
On 10/2/20 6:36 AM, Ciara Loftus wrote: Categorise and record syscalls issued in the xdpsock sample app. The categories recorded are: rx_empty_polls:polls when the rx ring is empty fill_fail_polls: polls when failed to get addr from fill ring copy_tx_sendtos: sendtos issued f

Re: [PATCH bpf-next 1/3] samples: bpf: split xdpsock stats into new struct

2020-10-02 Thread Yonghong Song
On 10/2/20 6:36 AM, Ciara Loftus wrote: New statistics will be added in future commits. In preparation for this, let's split out the existing statistics into their own struct. Signed-off-by: Ciara Loftus Acked-by: Yonghong Song

Re: [PATCH bpf-next 1/2] samples/bpf: change Makefile to cope with latest llvm

2020-10-02 Thread Yonghong Song
On 10/2/20 9:22 PM, Andrii Nakryiko wrote: On Fri, Oct 2, 2020 at 7:19 PM Yonghong Song wrote: With latest llvm trunk, bpf programs under samples/bpf directory, if using CORE, may experience the following errors: LLVM ERROR: Cannot select: intrinsic %llvm.preserve.struct.access.index PLEAS

Re: [PATCH net-next v2 00/10] arm64: dts: layerscape: update MAC nodes with PHY information

2020-10-02 Thread Ioana Ciornei
On Fri, Oct 02, 2020 at 04:13:18PM -0700, David Miller wrote: > From: Ioana Ciornei > Date: Sat, 3 Oct 2020 00:07:27 +0300 > > > This patch set aims to add the necessary DTS nodes to complete the > > MAC/PCS/PHY representation on DPAA2 devices. The external MDIO bus nodes > > and the PHYs found

Re: [PATCH bpf-next 1/2] samples/bpf: change Makefile to cope with latest llvm

2020-10-02 Thread Andrii Nakryiko
On Fri, Oct 2, 2020 at 7:19 PM Yonghong Song wrote: > > With latest llvm trunk, bpf programs under samples/bpf > directory, if using CORE, may experience the following > errors: > > LLVM ERROR: Cannot select: intrinsic %llvm.preserve.struct.access.index > PLEASE submit a bug report to https://bugs

Re: [PATCH net-next 0/5] implement kthread based napi poll

2020-10-02 Thread Alexei Starovoitov
On Sat, Oct 03, 2020 at 05:54:38AM +0200, Eric Dumazet wrote: > > Sure, a WQ is already giving nice results on appliances, because there > you do not need strong isolation. > Would a kthread approach also work well on appliances ? Probably... Right. I think we're on the same page. The only reason

Re: [PATCH net-next 15/15] sctp: enable udp tunneling socks

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:49:07PM +0800, Xin Long wrote: > This patch is to enable udp tunneling socks by calling > sctp_udp_sock_start() in sctp_ctrlsock_init(), and > sctp_udp_sock_stop() in sctp_ctrlsock_exit(). > > Also add sysctl udp_port to allow changing the listening > sock's port by user

Re: [PATCH net-next 12/15] sctp: call sk_setup_caps in sctp_packet_transmit instead

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:49:04PM +0800, Xin Long wrote: > sk_setup_caps() was originally called in Commit 90017accff61 ("sctp: > Add GSO support"), as: > > "We have to refresh this in case we are xmiting to more than one >transport at a time" > > This actually happens in the loop of sctp_

Re: [PATCH net-next 11/15] sctp: add udphdr to overhead when udp_port is set

2020-10-02 Thread Marcelo Ricardo Leitner
On Wed, Sep 30, 2020 at 03:00:42AM +0800, kernel test robot wrote: > Hi Xin, > > Thank you for the patch! Yet something to improve: I wonder how are you planning to fix this. It is quite entangled. This is not performance critical. Maybe the cleanest way out is to move it to a .c file. Adding a

Re: [PATCH net-next 11/15] sctp: add udphdr to overhead when udp_port is set

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:49:03PM +0800, Xin Long wrote: > sctp_mtu_payload() is for calculating the frag size before making > chunks from a msg. So we should only add udphdr size to overhead > when udp socks are listening, as only then sctp can handling the

Re: [PATCH net-next 10/15] sctp: allow changing transport encap_port by peer packets

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:49:02PM +0800, Xin Long wrote: > static int sctp_udp_rcv(struct sock *sk, struct sk_buff *skb) > { > + memset(skb->cb, 0, sizeof(skb->cb)); > + SCTP_INPUT_CB(skb)->encap_port = ntohs(udp_hdr(skb)->source); Here it's in host order already. The fact that is does

Re: [PATCH net-next 09/15] sctp: add SCTP_REMOTE_UDP_ENCAPS_PORT sockopt

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:49:01PM +0800, Xin Long wrote: ... > +struct sctp_udpencaps { > + sctp_assoc_t sue_assoc_id; > + struct sockaddr_storage sue_address; > + uint16_t sue_port; > +}; ... > +static int sctp_setsockopt_encap_port(struct sock *sk, > +

Re: [PATCH net-next 03/15] udp: do checksum properly in skb_udp_tunnel_segment

2020-10-02 Thread Marcelo Ricardo Leitner
On Tue, Sep 29, 2020 at 09:48:55PM +0800, Xin Long wrote: > This patch fixes two things: > > When skb->ip_summed == CHECKSUM_PARTIAL, skb_checksum_help() should be > called do the checksum, instead of gso_make_checksum(), which is used > to do the checksum for current proto after calling skb

Re: [PATCH][next] bpf: verifier: Use fallthrough pseudo-keyword

2020-10-02 Thread Yonghong Song
On 10/2/20 4:42 PM, Gustavo A. R. Silva wrote: Replace /* fallthrough */ comments with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva

Re: WARNING in ieee80211_bss_info_change_notify

2020-10-02 Thread syzbot
syzbot has bisected this issue to: commit 135f971181d779c96ff3725c1a350a721785cc66 Author: Alex Deucher Date: Mon Nov 20 22:49:53 2017 + drm/amdgpu: don't skip attributes when powerplay is enabled bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=120f55bd90 start commit

Re: [PATCH bpf-next] bpf: deref map in BPF_PROG_BIND_MAP when it's already used

2020-10-02 Thread Alexei Starovoitov
On Fri, Oct 2, 2020 at 5:25 PM Stanislav Fomichev wrote: > > We are missing a deref for the case when we are doing BPF_PROG_BIND_MAP > on a map that's being already held by the program. > There is 'if (ret) bpf_map_put(map)' below which doesn't trigger > because we don't consider this an error. >

[PATCH bpf-next 1/2] samples/bpf: change Makefile to cope with latest llvm

2020-10-02 Thread Yonghong Song
With latest llvm trunk, bpf programs under samples/bpf directory, if using CORE, may experience the following errors: LLVM ERROR: Cannot select: intrinsic %llvm.preserve.struct.access.index PLEASE submit a bug report to https://bugs.llvm.org/ and include the crash backtrace. Stack dump: 0. P

[PATCH bpf-next 2/2] samples/bpf: fix a compilation error with fallthrough marking

2020-10-02 Thread Yonghong Song
Compiling samples/bpf hits an error related to fallthrough marking. ... CC samples/bpf/hbm.o samples/bpf/hbm.c: In function ‘main’: samples/bpf/hbm.c:486:4: error: ‘fallthrough’ undeclared (first use in this function) fallthrough; ^~~ The "fallthrough" is not defi

Re: [PATCH net-next v3 0/9] genetlink: support per-command policy dump

2020-10-02 Thread David Miller
From: Jakub Kicinski Date: Fri, 2 Oct 2020 14:49:51 -0700 > The objective of this series is to dump ethtool policies > to be able to tell which flags are supported by the kernel. > Current release adds ETHTOOL_FLAG_STATS for dumping extra > stats, but because of strict checking we need to make s

Re: [PATCH net-next 7/8] net: ethernet: ti: am65-cpsw: prepare xmit/rx path for multi-port devices in mac-only mode

2020-10-02 Thread David Miller
From: Grygorii Strashko Date: Thu, 1 Oct 2020 13:52:57 +0300 > This patch adds multi-port support to TI AM65x CPSW driver xmit/rx path in > preparation for adding support for multi-port devices, like Main CPSW0 on > K3 J721E SoC or future CPSW3g on K3 AM64x SoC. > Hence DMA channels are common/sh

Re: [Linux-kernel-mentees][PATCH 0/2] reorder members of structures in virtio_net for optimization

2020-10-02 Thread David Miller
From: Anant Thazhemadam Date: Wed, 30 Sep 2020 10:47:20 +0530 > The structures virtnet_info and receive_queue have byte holes in > middle, and their members could do with some rearranging > (order-of-declaration wise) in order to overcome this. > > Rearranging the members helps in: > * elimi

Re: [PATCH] vhost-vdpa: fix page pinning leakage in error path

2020-10-02 Thread Jason Wang
On 2020/10/2 上午4:23, Si-Wei Liu wrote: Pinned pages are not properly accounted particularly when mapping error occurs on IOTLB update. Clean up dangling pinned pages for the error path. As the inflight pinned pages, specifically for memory region that strides across multiple chunks, would need

Re: [PATCH v2 2/2] vhost: Don't call log_access_ok() when using IOTLB

2020-10-02 Thread Jason Wang
On 2020/9/30 上午12:30, Greg Kurz wrote: When the IOTLB device is enabled, the log_guest_addr that is passed by userspace to the VHOST_SET_VRING_ADDR ioctl, and which is then written to vq->log_addr, is a GIOVA. All writes to this address are translated by log_user() to writes to an HVA, and then

Re: [PATCH v2 1/2] vhost: Don't call access_ok() when using IOTLB

2020-10-02 Thread Jason Wang
On 2020/9/30 上午12:30, Greg Kurz wrote: When the IOTLB device is enabled, the vring addresses we get from userspace are GIOVAs. It is thus wrong to pass them down to access_ok() which only takes HVAs. Access validation is done at prefetch time with IOTLB. Teach vq_access_ok() about that by movi

hello

2020-10-02 Thread Mrs. Aisha Gaddafi.
Assalamu alaikum, I have a business Proposal for you and I need mutual respect, trust, honesty, transparency, adequate support and assistance, Hope to hear from you for more details. Warmest regards Mrs Aisha Gaddafi السلام عليكم، لدي اقتراح عمل لك وأنا بحاجة إلى الاحترام المتبادل والثقة والأ

Re: [PATCH bpf-next] bpf: deref map in BPF_PROG_BIND_MAP when it's already used

2020-10-02 Thread Andrii Nakryiko
On Fri, Oct 2, 2020 at 5:26 PM Stanislav Fomichev wrote: > > We are missing a deref for the case when we are doing BPF_PROG_BIND_MAP > on a map that's being already held by the program. > There is 'if (ret) bpf_map_put(map)' below which doesn't trigger > because we don't consider this an error. >

Re: [PATCH 0/7] TC-ETF support PTP clocks series

2020-10-02 Thread Thomas Gleixner
Vinicius, On Fri, Oct 02 2020 at 12:01, Vinicius Costa Gomes wrote: > I think that there's an underlying problem/limitation that is the cause > of the issue (or at least a step in the right direction) you are trying > to solve: the issue is that PTP clocks can't be used as hrtimers. That's only a

Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-02 Thread Guenter Roeck
On 10/2/20 4:41 PM, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > C

Re: [PATCH bpf-next] bpf: use raw_spin_trylock() for pcpu_freelist_push/pop in NMI

2020-10-02 Thread Song Liu
> On Oct 2, 2020, at 4:09 PM, Daniel Borkmann wrote: > > On 9/26/20 2:07 AM, Song Liu wrote: >> Recent improvements in LOCKDEP highlighted a potential A-A deadlock with >> pcpu_freelist in NMI: >> ./tools/testing/selftests/bpf/test_progs -t stacktrace_build_id_nmi >> [ 18.984807] ===

[PATCH v4] staging: qlge: fix build breakage with dumping enabled

2020-10-02 Thread Coiby Xu
This fixes commit 0107635e15ac ("staging: qlge: replace pr_err with netdev_err") which introduced an build breakage of missing `struct ql_adapter *qdev` for some functions and a warning of type mismatch with dumping enabled, i.e., $ make CFLAGS_MODULE="-DQL_ALL_DUMP -DQL_OB_DUMP -DQL_CB_DUMP \

[PATCH] dt-bindings: Another round of adding missing 'additionalProperties'

2020-10-02 Thread Rob Herring
Another round of wack-a-mole. The json-schema default is additional unknown properties are allowed, but for DT all properties should be defined. Cc: Thierry Reding Cc: Linus Walleij Cc: Stephen Boyd Cc: Shawn Guo Cc: Bjorn Andersson Cc: Baolin Wang Cc: Guenter Roeck Cc: Jonathan Cameron Cc

[RFC net-next] ptp: add clock_index as a device attribute

2020-10-02 Thread Jacob Keller
The PTP clock associated with a networking device is reported by the ETHTOOL_GET_TS_INFO ioctl, by using the ptp->index clock index variable. In order to associate this clock index with the proper device, userspace applications make the assumption that /dev/ptpX has a clock index of X. If this ass

[PATCH][next] bpf: verifier: Use fallthrough pseudo-keyword

2020-10-02 Thread Gustavo A. R. Silva
Replace /* fallthrough */ comments with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva --- kernel/bpf/verifier.c | 4 ++-- 1 file changed, 2

Re: [PATCH net-next v3 0/4] dpaa2-eth: add devlink parser error drop trap support

2020-10-02 Thread David Miller
From: Ioana Ciornei Date: Thu, 1 Oct 2020 18:11:44 +0300 > This patch set adds support in the dpaa2-eth driver for a new group of > devlink drop traps - PARSER_ERROR_DROPS. > > The first patch adds a new generic trap group and associated traps, > their definitions in devlink and their correspon

Re: [PATCH net-next 0/8] ionic error recovery

2020-10-02 Thread David Miller
From: Shannon Nelson Date: Thu, 1 Oct 2020 09:22:38 -0700 > This set of patches comes mostly from error recovery path testing, > as well as a couple of upstream review comments. Series applied, thank you.

Re: [PATCH net-next 0/7] s390/net: updates 2020-10-01

2020-10-02 Thread David Miller
From: Julian Wiedmann Date: Thu, 1 Oct 2020 19:11:29 +0200 > please apply the following patch series to netdev's net-next tree. > > Patches 1-3 enable qeth to also support the .set_channels() ethtool > callback for OSA devices. This completes support for the full range > of device types. > > T

Re: [pull request][net V3 00/14] mlx5 fixes 2020-09-30

2020-10-02 Thread David Miller
From: sa...@kernel.org Date: Fri, 2 Oct 2020 11:06:40 -0700 > This series introduces some fixes to mlx5 driver. > > v1->v2: > - Patch #1 Don't return while mutex is held. (Dave) > > v2->v3: > - Drop patch #1, will consider a better approach (Jakub) > - use cpu_relax() instead of cond_resched

[PATCH][next] bnx2x: Use fallthrough pseudo-keyword

2020-10-02 Thread Gustavo A. R. Silva
Replace /* no break */ comments with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva --- drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 4

Re: vsock fix backport to 5.4 stable

2020-10-02 Thread Sasha Levin
On Wed, Sep 30, 2020 at 10:55:37AM -0700, Vishnu Rangayyan wrote: Hi, Can we have this backport applied to 5.4 stable, its a useful fix. commit df12eb6d6cd920ab2f0e0a43cd6e1c23a05cea91 upstream The call has a minor api change in 5.4 vs higher, only the pkt arg is required. Queued up. I took

Re: [PATCH net-next 0/5] implement kthread based napi poll

2020-10-02 Thread Alexei Starovoitov
On Fri, Oct 2, 2020 at 4:02 PM David Miller wrote: > > From: Wei Wang > Date: Wed, 30 Sep 2020 12:21:35 -0700 > > ... > > And the reason we prefer 1 kthread per napi, instead of 1 workqueue > > entity per host, is that kthread is more configurable than workqueue, > > and we could leverage existi

Re: [PATCH net-next v2 00/10] arm64: dts: layerscape: update MAC nodes with PHY information

2020-10-02 Thread David Miller
From: Ioana Ciornei Date: Sat, 3 Oct 2020 00:07:27 +0300 > This patch set aims to add the necessary DTS nodes to complete the > MAC/PCS/PHY representation on DPAA2 devices. The external MDIO bus nodes > and the PHYs found on them are added, along with the PCS MDIO internal > buses and their PCS

Re: [PATCH bpf-next] bpf: use raw_spin_trylock() for pcpu_freelist_push/pop in NMI

2020-10-02 Thread Daniel Borkmann
On 9/26/20 2:07 AM, Song Liu wrote: Recent improvements in LOCKDEP highlighted a potential A-A deadlock with pcpu_freelist in NMI: ./tools/testing/selftests/bpf/test_progs -t stacktrace_build_id_nmi [ 18.984807] [ 18.984807] WARNING: inconsistent lock state

Re: [PATCH net-next 0/8] net: ethernet: ti: am65-cpsw: add multi port support in mac-only mode

2020-10-02 Thread Jakub Kicinski
On Fri, 2 Oct 2020 16:04:21 -0700 Jakub Kicinski wrote: > On Fri, 2 Oct 2020 12:56:43 +0300 Grygorii Strashko wrote: > > On 02/10/2020 02:08, Jakub Kicinski wrote: > > > On Thu, 1 Oct 2020 13:52:50 +0300 Grygorii Strashko wrote: > > >> This series adds multi-port support in mac-only mode (mul

Re: [PATCH net-next 0/8] net: ethernet: ti: am65-cpsw: add multi port support in mac-only mode

2020-10-02 Thread Jakub Kicinski
On Fri, 2 Oct 2020 12:56:43 +0300 Grygorii Strashko wrote: > On 02/10/2020 02:08, Jakub Kicinski wrote: > > On Thu, 1 Oct 2020 13:52:50 +0300 Grygorii Strashko wrote: > >> This series adds multi-port support in mac-only mode (multi MAC mode) to TI > >> AM65x CPSW driver in preparation for enablin

Re: [PATCH net-next 00/23] rxrpc: Fixes and preparation for RxGK

2020-10-02 Thread David Miller
From: David Howells Date: Thu, 01 Oct 2020 15:56:43 +0100 > The patches are tagged here: > > git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git > rxrpc-next-20201010 No, they aren't. git pull --no-ff git://git.kernel.org/pub/scm/linux/kernel/g

Re: [PATCH net-next 0/5] implement kthread based napi poll

2020-10-02 Thread David Miller
From: Wei Wang Date: Wed, 30 Sep 2020 12:21:35 -0700 ... > And the reason we prefer 1 kthread per napi, instead of 1 workqueue > entity per host, is that kthread is more configurable than workqueue, > and we could leverage existing tuning tools for threads, like taskset, > chrt, etc to tune sche

[PATCH][next] net: ksz884x: Use fallthrough pseudo-keyword

2020-10-02 Thread Gustavo A. R. Silva
Replace /* Fallthrough... */ comment with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva --- drivers/net/ethernet/micrel/ksz884x.c | 3 +--

Re: [PATCH net-next v2 6/6] bonding: make Kconfig toggle to disable legacy interfaces

2020-10-02 Thread David Miller
From: Jarod Wilson Date: Fri, 2 Oct 2020 16:23:46 -0400 > I'd had a bit of feedback that people would rather see both, and be > able to toggle off the old ones, rather than only having one or the > other, depending on the toggle, so I thought I'd give this a try. I > kind of liked the one or the

Re: [PATCH net-next v2 5/6] bonding: update Documentation for port/bond terminology

2020-10-02 Thread David Miller
From: Jarod Wilson Date: Fri, 2 Oct 2020 16:12:49 -0400 > The documentation was updated to point to the new names, but the old > ones still exist across the board, there should be no userspace > breakage here. (My lnst bonding tests actually fall flat currently > if the old names are gone). The

[PATCH][next] net: bna: Use fallthrough pseudo-keyword

2020-10-02 Thread Gustavo A. R. Silva
Replace /* !!! fall through !!! */ comments with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva --- drivers/net/ethernet/brocade/bna/bfa_ioc

Re: [PATCH net-next 0/5] implement kthread based napi poll

2020-10-02 Thread Jakub Kicinski
On Thu, 1 Oct 2020 18:44:40 -0700 Wei Wang wrote: > > Can you share relative performance delta of this banchmark? > > > > Could you explain why threads are slower than ksoftirqd if you pin the > > application away? From your cover letter it sounded like you want the > > scheduler to see the NAPI lo

Re: [PATCH net-next v2 6/6] bonding: make Kconfig toggle to disable legacy interfaces

2020-10-02 Thread David Miller
From: Stephen Hemminger Date: Fri, 2 Oct 2020 12:13:17 -0700 > On Fri, 2 Oct 2020 13:40:01 -0400 > Jarod Wilson wrote: > >> By default, enable retaining all user-facing API that includes the use of >> master and slave, but add a Kconfig knob that allows those that wish to >> remove it entirely

Re: [PATCH net-next 0/5] implement kthread based napi poll

2020-10-02 Thread Jakub Kicinski
On Fri, 2 Oct 2020 09:56:31 +0200 Eric Dumazet wrote: > On Thu, Oct 1, 2020 at 10:26 PM Jakub Kicinski wrote: > > > > On Thu, 1 Oct 2020 09:52:45 +0200 Eric Dumazet wrote: > > > > The unique work queue is a problem on server class platforms, with > > > NUMA placement. > > > We now have servers

[PATCH][next] usbnet: Use fallthrough pseudo-keyword

2020-10-02 Thread Gustavo A. R. Silva
Replace // FALLTHROUGH comment with the new pseudo-keyword macro fallthrough[1]. [1] https://www.kernel.org/doc/html/v5.7/process/deprecated.html?highlight=fallthrough#implicit-switch-case-fall-through Signed-off-by: Gustavo A. R. Silva --- drivers/net/usb/usbnet.c | 2 +- 1 file changed, 1 in

[PATCH net-next 1/2] net/sched: act_vlan: Add {POP,PUSH}_ETH actions

2020-10-02 Thread Guillaume Nault
Implement TCA_VLAN_ACT_POP_ETH and TCA_VLAN_ACT_PUSH_ETH, to respectively pop and push a base Ethernet header at the beginning of a frame. POP_ETH is just a matter of pulling ETH_HLEN bytes. VLAN tags, if any, must be stripped before calling POP_ETH. PUSH_ETH is restricted to skbs with no mac_hea

[PATCH net-next 2/2] net/sched: act_mpls: Add action to push MPLS LSE before Ethernet header

2020-10-02 Thread Guillaume Nault
Define the MAC_PUSH action which pushes an MPLS LSE before the mac header (instead of between the mac and the network headers as the plain PUSH action does). The only special case is when the skb has an offloaded VLAN. In that case, it has to be inlined before pushing the MPLS header. Signed-off-

[PATCH net-next 0/2] net/sched: Add actions for MPLS L2 VPNs

2020-10-02 Thread Guillaume Nault
This patch series adds the necessary TC actions for supporting layer 2 MPLS VPNs (VPLS). The objective is to give the possibility to add an MPLS header right before an skb's mac header, then to prepend this MPLS packet with a new Ethernet header with the MAC address of the next hop. Patch 1 imple

Re: pull-request: wireless-drivers-next-2020-10-02

2020-10-02 Thread David Miller
From: Kalle Valo Date: Fri, 2 Oct 2020 14:08:26 + (UTC) > here's a pull request to net-next tree, more info below. Please let me know if > there are any problems. Pulled, thank you.

Re: [PATCH net-next v2 6/6] bonding: make Kconfig toggle to disable legacy interfaces

2020-10-02 Thread Stephen Hemminger
On Fri, 2 Oct 2020 16:23:46 -0400 Jarod Wilson wrote: > On Fri, Oct 2, 2020 at 3:13 PM Stephen Hemminger > wrote: > > > > On Fri, 2 Oct 2020 13:40:01 -0400 > > Jarod Wilson wrote: > > > > > By default, enable retaining all user-facing API that includes the use of > > > master and slave, but

Re: [PATCH bpf-next] bpf: use raw_spin_trylock() for pcpu_freelist_push/pop in NMI

2020-10-02 Thread Martin KaFai Lau
On Fri, Sep 25, 2020 at 05:07:56PM -0700, Song Liu wrote: > Recent improvements in LOCKDEP highlighted a potential A-A deadlock with > pcpu_freelist in NMI: > > ./tools/testing/selftests/bpf/test_progs -t stacktrace_build_id_nmi > > [ 18.984807] > [ 18.984807]

Re: [PATCH net-next 0/9] Offload tc-flower to mscc_ocelot switch using VCAP chains

2020-10-02 Thread David Miller
From: Vladimir Oltean Date: Fri, 2 Oct 2020 15:02:19 +0300 > The purpose of this patch is to add more comprehensive support for flow > offloading in the mscc_ocelot library and switch drivers. > > The design (with chains) is the result of this discussion: > https://lkml.org/lkml/2020/6/2/203 >

Re: [PATCH v2] net: usb: rtl8150: prevent set_ethernet_addr from setting uninit address

2020-10-02 Thread David Miller
From: Anant Thazhemadam Date: Fri, 2 Oct 2020 17:04:13 +0530 > But this patch is about ensuring that an uninitialized variable's > value (whatever that may be) is not set as the ethernet address > blindly (without any form of checking if get_registers() worked > as expected, or not). Right, and

Re: [PATCH net] tcp: fix syn cookied MPTCP request socket leak

2020-10-02 Thread David Miller
From: Paolo Abeni Date: Fri, 2 Oct 2020 12:39:44 +0200 > If a syn-cookies request socket don't pass MPTCP-level > validation done in syn_recv_sock(), we need to release > it immediately, or it will be leaked. > > Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/89 > Fixes: 9466a1c

Re: pull-request: mac80211-next 2020-10-02

2020-10-02 Thread David Miller
From: Johannes Berg Date: Fri, 2 Oct 2020 12:11:25 +0200 > Here's a - probably final - set of patches for net-next. > Really the big thing is more complete S1G support, along > with a small list of other things, see the tag message. > > Please pull and let me know if there's any problem. Pulle

Re: [PATCH v9 0/7] Introduce sendpage_ok() to detect misused sendpage in network related drivers

2020-10-02 Thread David Miller
From: Coly Li Date: Fri, 2 Oct 2020 16:30:12 +0800 > Obviously my fault and no excuse for leaking this uncompleted version to > you. I just re-post a v10 version which I make sure all patches are the > latest version. > > Sorry for the inconvenience and thank you in advance for taking this set.

Re: [PATCH v10 0/7] Introduce sendpage_ok() to detect misused sendpage in network related drivers

2020-10-02 Thread David Miller
From: Coly Li Date: Fri, 2 Oct 2020 16:27:27 +0800 > As Sagi Grimberg suggested, the original fix is refind to a more common > inline routine: > static inline bool sendpage_ok(struct page *page) > { > return (!PageSlab(page) && page_count(page) >= 1); > } > If sendpage_ok()

Re: Why ping latency is smaller with shorter send interval?

2020-10-02 Thread David Miller
Can you please not send the same posting to the mailing list three times, from three different email addresses? Once is enough, thank you.

Re: [PATCH] netlink: fix policy dump leak

2020-10-02 Thread David Miller
From: Johannes Berg Date: Fri, 2 Oct 2020 09:46:04 +0200 > From: Johannes Berg > > If userspace doesn't complete the policy dump, we leak the > allocated state. Fix this. > > Fixes: d07dcf9aadd6 ("netlink: add infrastructure to expose policies to > userspace") > Signed-off-by: Johannes Berg

Re: [PATCH] net/smscx5xx: change to of_get_mac_address() eth_platform_get_mac_address()

2020-10-02 Thread David Miller
From: Lukasz Stelmach Date: Fri, 02 Oct 2020 09:39:40 +0200 > In both drivers the second to last line of the *_init_mac_address()[1][2] > functions is > > eth_hw_addr_random(dev->net); My bad, indeed it does take care of this already. Applied to net-next, thanks.

Re: [PATCH net-next] dt-bindings: net: dsa: b53: Add missing reg property to example

2020-10-02 Thread David Miller
From: Kurt Kanzenbach Date: Fri, 2 Oct 2020 08:20:51 +0200 > The switch has a certain MDIO address and this needs to be specified using the > reg property. Add it to the example. > > Signed-off-by: Kurt Kanzenbach Applied, thank you.

Re: [PATCH bpf-next v4 0/6] bpf: BTF support for ksyms

2020-10-02 Thread Alexei Starovoitov
On Tue, Sep 29, 2020 at 11:48 PM Hao Luo wrote: > > Ah, this is the bug in pahole described in > https://lkml.org/lkml/2020/8/20/1862. I proposed a fix [1] but it > hasn't reached pahole's master branch. Let me ask Arnaldo to see if he > is OK merging it. > > [1] https://www.spinics.net/lists/dwar

Re: [PATCH 1/6] net: core: document two new elements of struct net_device

2020-10-02 Thread David Miller
From: Mauro Carvalho Chehab Date: Fri, 2 Oct 2020 07:49:45 +0200 > As warned by "make htmldocs", there are two new struct elements > that aren't documented: > > ../include/linux/netdevice.h:2159: warning: Function parameter or > member 'unlink_list' not described in 'net_device' >

[PATCH net-next] net: dsa: propagate switchdev vlan_filtering prepare phase to drivers

2020-10-02 Thread Vladimir Oltean
A driver may refuse to enable VLAN filtering for any reason beyond what the DSA framework cares about, such as: - having tc-flower rules that rely on the switch being VLAN-aware - the particular switch does not support VLAN, even if the driver does (the DSA framework just checks for the presence

Re: [PATCH net-next v2 00/10] genetlink: support per-command policy dump

2020-10-02 Thread Jakub Kicinski
On Fri, 2 Oct 2020 14:22:05 -0700 Jakub Kicinski wrote: > > > Forgot the link ... > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git/log/?h=genetlink-op-policy-export > > > > > > > If it's not too late for you - do you want to merge the two series and > > pos

[PATCH net-next v3 4/9] genetlink: add a structure for dump state

2020-10-02 Thread Jakub Kicinski
Whenever netlink dump uses more than 2 cb->args[] entries code gets hard to read. We're about to add more state to ctrl_dumppolicy() so create a structure. Since the structure is typed and clearly named we can remove the local fam_id variable and use ctx->fam_id directly. v3: - rebase onto expli

[PATCH net-next v3 1/9] genetlink: reorg struct genl_family

2020-10-02 Thread Jakub Kicinski
There are holes and oversized members in struct genl_family. Before: /* size: 104, cachelines: 2, members: 16 */ After: /* size: 88, cachelines: 2, members: 16 */ The command field in struct genlmsghdr is a u8, so no point in the operation count being 32 bit. Also operation 0 is usually undefin

[PATCH net-next v3 3/9] genetlink: move to smaller ops wherever possible

2020-10-02 Thread Jakub Kicinski
Bulk of the genetlink users can use smaller ops, move them. Signed-off-by: Jakub Kicinski Reviewed-by: Johannes Berg --- drivers/block/nbd.c | 6 +++--- drivers/net/gtp.c| 6 +++--- drivers/net/ieee802154/mac802154_hwsim.c | 6 +++--- drivers/net/

[PATCH net-next v3 9/9] genetlink: switch control commands to per-op policies

2020-10-02 Thread Jakub Kicinski
In preparation for adding a new attribute to CTRL_CMD_GETPOLICY split the policies for getpolicy and getfamily apart. This will cause a slight user-visible change in that dumping the policies will switch from per family to per op, but supposedly sniffer-type applications (which are the main use ca

[PATCH net-next v3 5/9] genetlink: use .start callback for dumppolicy

2020-10-02 Thread Jakub Kicinski
The structure of ctrl_dumppolicy() is clearly split into init and dumping. Move the init to a .start callback for clarity, it's a more idiomatic netlink dump code structure. Signed-off-by: Jakub Kicinski Reviewed-by: Johannes Berg --- net/netlink/genetlink.c | 48 ---

[PATCH net-next v3 2/9] genetlink: add small version of ops

2020-10-02 Thread Jakub Kicinski
We want to add maxattr and policy back to genl_ops, to enable dumping per command policy to user space. This, however, would cause bloat for all the families with global policies. Introduce smaller version of ops (half the size of genl_ops). Translate these smaller ops into a full blown struct befo

[PATCH net-next v3 7/9] taskstats: move specifying netlink policy back to ops

2020-10-02 Thread Jakub Kicinski
commit 3b0f31f2b8c9 ("genetlink: make policy common to family") had to work around removal of policy from ops by parsing in the pre_doit callback. Now that policy is back in full ops we can switch again. Set maxattr to actual size of the policies - both commands set GENL_DONT_VALIDATE_STRICT so out

[PATCH net-next v3 8/9] genetlink: use parsed attrs in dumppolicy

2020-10-02 Thread Jakub Kicinski
Attributes are already parsed based on the policy specified in the family and ready-to-use in info->attrs. No need to call genlmsg_parse() again. Signed-off-by: Jakub Kicinski Reviewed-by: Johannes Berg --- net/netlink/genetlink.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-)

[PATCH net-next v3 6/9] genetlink: bring back per op policy

2020-10-02 Thread Jakub Kicinski
Add policy to the struct genl_ops structure, this time with maxattr, so it can be used properly. Propagate .policy and .maxattr from the family in genl_get_cmd() if needed, this way the rest of the code does not have to worry if the policy is per op or global. Signed-off-by: Jakub Kicinski Revie

[PATCH net-next v3 0/9] genetlink: support per-command policy dump

2020-10-02 Thread Jakub Kicinski
Hi! The objective of this series is to dump ethtool policies to be able to tell which flags are supported by the kernel. Current release adds ETHTOOL_FLAG_STATS for dumping extra stats, but because of strict checking we need to make sure that the flag is actually supported before setting it in a r

Re: [bpf-next PATCH v2 0/2] Add skb_adjust_room() for SK_SKB

2020-10-02 Thread Martin KaFai Lau
On Thu, Oct 01, 2020 at 06:09:34PM -0700, John Fastabend wrote: > This implements the helper skb_adjust_room() for BPF_SKS_SK_STREAM_VERDICT > programs so we can push/pop headers from the data on recieve. One use > case is to pop TLS headers off kTLS packets. > > The first patch implements the hel

Re: [PATCH net-next v2 00/10] genetlink: support per-command policy dump

2020-10-02 Thread Jakub Kicinski
On Fri, 2 Oct 2020 14:17:01 -0700 Jakub Kicinski wrote: > On Fri, 02 Oct 2020 23:00:15 +0200 Johannes Berg wrote: > > On Fri, 2020-10-02 at 22:59 +0200, Johannes Berg wrote: > > > On Fri, 2020-10-02 at 13:50 -0700, Jakub Kicinski wrote: > > > > My thinking was that until kernel actually start

Re: [PATCH net-next v2 00/10] genetlink: support per-command policy dump

2020-10-02 Thread Jakub Kicinski
On Fri, 02 Oct 2020 23:00:15 +0200 Johannes Berg wrote: > On Fri, 2020-10-02 at 22:59 +0200, Johannes Berg wrote: > > On Fri, 2020-10-02 at 13:50 -0700, Jakub Kicinski wrote: > > > My thinking was that until kernel actually start using separate dump > > > policies user space can assume policy 0 i

[PATCH net-next v2 09/10] arm64: dts: lx2160a: add PCS MDIO and PCS PHY nodes

2020-10-02 Thread Ioana Ciornei
Add PCS MDIO nodes for the internal MDIO buses on the LX2160A, along with their internal PCS PHYs, which will be used when the DPMAC is in TYPE_PHY mode. Signed-off-by: Ioana Ciornei --- Changes in v2: - none .../arm64/boot/dts/freescale/fsl-lx2160a.dtsi | 252 ++ 1 file change

[PATCH net-next v2 04/10] arm64: dts: ls1088ardb: add necessary DTS nodes for DPMAC2

2020-10-02 Thread Ioana Ciornei
Annotate the external MDIO2 node and describe the 10GBASER PHY found on the LS1088ARDB board and add a phy-handle for DPMAC2 to link it. Also, add the internal PCS MDIO node for the internal MDIO buses found on the LS1088A SoC along with its internal PCS PHY and link the corresponding DPMAC to the

[PATCH net-next v2 10/10] arm64: dts: lx2160ardb: add nodes for the AQR107 PHYs

2020-10-02 Thread Ioana Ciornei
Annotate the EMDIO1 node and describe the 2 AQR107 PHYs found on the LX2160ARDB board. Also, add the necessary phy-handles for DPMACs 3 and 4 to their associated PHY. Signed-off-by: Ioana Ciornei --- Changes in v2: - none .../boot/dts/freescale/fsl-lx2160a-rdb.dts| 32 +++

[PATCH net-next v2 03/10] arm64: dts: ls1088ardb: add QSGMII PHY nodes

2020-10-02 Thread Ioana Ciornei
Annotate the external MDIO1 node and describe the 8 QSGMII PHYs found on the LS1088ARDB board and add phy-handles for DPMACs 3-10 to its associated PHY. Also, add the internal PCS MDIO nodes for the internal MDIO buses found on the LS1088A SoC along with their internal PCS PHY and link the corresp

[PATCH net-next v2 05/10] arm64: dts: ls208xa: add the external MDIO nodes

2020-10-02 Thread Ioana Ciornei
Add the external MDIO device nodes found in the WRIOP global memory region. This is needed for management of external PHYs. Signed-off-by: Ioana Ciornei --- Changes in v2: - removed the 0x from the unit addresses arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi | 18 ++ 1 file ch

[PATCH net-next v2 01/10] dt-bindings: net: add the dpaa2-mac DTS definition

2020-10-02 Thread Ioana Ciornei
Add a documentation entry for the DTS bindings needed and supported by the dpaa2-mac driver. Signed-off-by: Ioana Ciornei --- Changes in v2: - new patch .../devicetree/bindings/net/dpaa2-mac.yaml| 55 +++ 1 file changed, 55 insertions(+) create mode 100644 Documentation/de

[PATCH net-next v2 02/10] arm64: dts: ls1088a: add external MDIO device nodes

2020-10-02 Thread Ioana Ciornei
Add the external MDIO device nodes found in the WRIOP global memory region. This is needed for management of external PHYs. Signed-off-by: Ioana Ciornei --- Changes in v2: - removed the 0x from the unit addresses arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi | 18 ++ 1 file ch

  1   2   3   4   >