Re: Re: [PATCH] phy: nxp-c45: add driver for tja1103

2021-04-12 Thread Christian Herber
Hi Andrew, On 4/12/2021 6:52 PM, Andrew Lunn wrote: So what you are say is, you don't care if the IP is completely different, it all goes in one driver. So lets put this driver into nxp-tja11xx.c. And then we avoid all the naming issues. Andrew As this seems to be a key question, let

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Leon Romanovsky
On Mon, Apr 12, 2021 at 07:58:47PM -0300, Jason Gunthorpe wrote: > On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > > ib_modify_qp() is an expensive operation on some HCAs running > > virtualized. This series removes two ib_modify_qp() calls from RDS. > > > > I am sending this as a v

Re: [PATCH net-next v4 01/10] netdevice: priv_flags extend to 64bit

2021-04-12 Thread kernel test robot
Hi Xuan, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Xuan-Zhuo/virtio-net-support-xdp-socket-zero-copy-xmit/20210413-111831 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-

[PATCH net-next 2/2] net: hns3: VF not request link status when PF support push link status feature

2021-04-12 Thread Huazhong Tan
From: Guangbin Huang To reduce the processing of unnecessary mailbox command when PF supports actively push its link status to VFs, VFs stop sending request link status command in periodic service task in this case. Signed-off-by: Guangbin Huang Signed-off-by: Huazhong Tan --- drivers/net/eth

[PATCH net-next 0/2] net: hns3: updates for -next

2021-04-12 Thread Huazhong Tan
This series adds support for pushing link status to VFs for the HNS3 ethernet driver. Guangbin Huang (2): net: hns3: PF add support for pushing link status to VFs net: hns3: VF not request link status when PF support push link status feature drivers/net/ethernet/hisilicon/hns3/hclge_mbx.

[PATCH net-next 1/2] net: hns3: PF add support for pushing link status to VFs

2021-04-12 Thread Huazhong Tan
From: Guangbin Huang Previously, VF updates its link status every second by send query command to PF in periodic service task. If link stats of PF is changed, VF may need at most one second to update its link status. To reduce delay of link status between PF and VFs, PF actively push its link st

[syzbot] KASAN: null-ptr-deref Write in rhashtable_free_and_destroy (2)

2021-04-12 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:d93a0d43 Merge tag 'block-5.12-2021-04-02' of git://git.ke.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=12d81cfcd0 kernel config: https://syzkaller.appspot.com/x/.config?x=71a75beb62b62a34 das

[syzbot] KASAN: slab-out-of-bounds Read in __xfrm_decode_session (2)

2021-04-12 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:1678e493 Merge tag 'lto-v5.12-rc6' of git://git.kernel.org.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=1565bf7cd0 kernel config: https://syzkaller.appspot.com/x/.config?x=71a75beb62b62a34 das

[PATCH RFC v2 1/4] virtio: fix up virtio_disable_cb

2021-04-12 Thread Michael S. Tsirkin
virtio_disable_cb is currently a nop for split ring with event index. This is because it used to be always called from a callback when we know device won't trigger more events until we update the index. However, now that we run with interrupts enabled a lot we also poll without a callback so that

[PATCH RFC v2 4/4] virtio_net: move txq wakeups under tx q lock

2021-04-12 Thread Michael S. Tsirkin
We currently check num_free outside tx q lock which is unsafe: new packets can arrive meanwhile and there won't be space in the queue. Thus a spurious queue wakeup causing overhead and even packet drops. Move the check under the lock to fix that. Signed-off-by: Michael S. Tsirkin --- drivers/ne

[PATCH RFC v2 2/4] virtio_net: disable cb aggressively

2021-04-12 Thread Michael S. Tsirkin
There are currently two cases where we poll TX vq not in response to a callback: start xmit and rx napi. We currently do this with callbacks enabled which can cause extra interrupts from the card. Used not to be a big issue as we run with interrupts disabled but that is no longer the case, and in

[PATCH RFC v2 3/4] virtio_net: move tx vq operation under tx queue lock

2021-04-12 Thread Michael S. Tsirkin
It's unsafe to operate a vq from multiple threads. Unfortunately this is exactly what we do when invoking clean tx poll from rx napi. As a fix move everything that deals with the vq to under tx lock. Signed-off-by: Michael S. Tsirkin --- drivers/net/virtio_net.c | 22 +- 1 fi

[PATCH RFC v2 0/4] virtio net: spurious interrupt related fixes

2021-04-12 Thread Michael S. Tsirkin
With the implementation of napi-tx in virtio driver, we clean tx descriptors from rx napi handler, for the purpose of reducing tx complete interrupts. But this introduces a race where tx complete interrupt has been raised, but the handler finds there is no work to do because we have done the work i

pull-request: wireless-drivers-next-2021-04-13

2021-04-12 Thread Kalle Valo
Hi, here's a pull request to net-next tree, more info below. Please let me know if there are any problems. Kalle The following changes since commit 2117fce81f6b862aac0673abe8df0c60dca64bfa: Merge branch 'psample-Add-additional-metadata-attributes' (2021-03-14 15:00:44 -0700) are available i

Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Leon Romanovsky
On Mon, Apr 12, 2021 at 10:51:25AM -0700, Michael Chan wrote: > On Mon, Apr 12, 2021 at 10:33 AM Leon Romanovsky wrote: > > > > On Mon, Apr 12, 2021 at 09:31:33AM -0700, Michael Chan wrote: > > > On Mon, Apr 12, 2021 at 12:37 AM Leon Romanovsky wrote: > > > > > > > > On Sun, Apr 11, 2021 at 08:18

Re: [PATCH net-next v4 01/10] netdevice: priv_flags extend to 64bit

2021-04-12 Thread kernel test robot
Hi Xuan, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Xuan-Zhuo/virtio-net-support-xdp-socket-zero-copy-xmit/20210413-111831 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-

Re: [PATCH net] virtio-net: suppress bad irq warning for tx napi

2021-04-12 Thread Michael S. Tsirkin
On Fri, Feb 05, 2021 at 02:28:33PM -0800, Wei Wang wrote: > On Thu, Feb 4, 2021 at 12:48 PM Willem de Bruijn > wrote: > > > > On Wed, Feb 3, 2021 at 6:53 PM Wei Wang wrote: > > > > > > On Wed, Feb 3, 2021 at 3:10 PM Michael S. Tsirkin wrote: > > > > > > > > On Wed, Feb 03, 2021 at 01:24:08PM -05

Re: [PATCH net-next v3 5/5] virtio-net: keep tx interrupts disabled unless kick

2021-04-12 Thread Michael S. Tsirkin
On Mon, Apr 24, 2017 at 01:49:30PM -0400, Willem de Bruijn wrote: > From: Willem de Bruijn > > Tx napi mode increases the rate of transmit interrupts. Suppress some > by masking interrupts while more packets are expected. The interrupts > will be reenabled before the last packet is sent. > > Thi

Re: [PATCH net-next v3 1/1] net: stmmac: Add support for external trigger timestamping

2021-04-12 Thread Wong Vee Khee
On Sun, Apr 11, 2021 at 08:10:55AM -0700, Richard Cochran wrote: > On Sun, Apr 11, 2021 at 10:40:28AM +0800, Wong Vee Khee wrote: > > diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c > > b/drivers/net/ethernet/stmicro/stmmac/dwmac-intel.c > > index 60566598d644..60e17fd24aba 100644 >

Re: [PATCH bpf-next v2] libbpf: clarify flags in ringbuf helpers

2021-04-12 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to bpf/bpf-next.git (refs/heads/master): On Mon, 12 Apr 2021 16:24:32 -0300 you wrote: > In 'bpf_ringbuf_reserve()' we require the flag to '0' at the moment. > > For 'bpf_ringbuf_{discard,submit,output}' a flag of '0' might send a > notification to the process if ne

Re: [PATCH bpf-next v2] libbpf: clarify flags in ringbuf helpers

2021-04-12 Thread Andrii Nakryiko
On Mon, Apr 12, 2021 at 12:25 PM Pedro Tammela wrote: > > In 'bpf_ringbuf_reserve()' we require the flag to '0' at the moment. > > For 'bpf_ringbuf_{discard,submit,output}' a flag of '0' might send a > notification to the process if needed. > > Signed-off-by: Pedro Tammela > --- Great, thanks! A

Re: Re: [PATCH v6 09/10] vduse: Introduce VDUSE - vDPA Device in Userspace

2021-04-12 Thread Yongji Xie
On Tue, Apr 13, 2021 at 11:35 AM Jason Wang wrote: > > > 在 2021/4/12 下午5:59, Yongji Xie 写道: > > On Mon, Apr 12, 2021 at 5:37 PM Jason Wang wrote: > >> > >> 在 2021/4/12 下午4:02, Yongji Xie 写道: > >>> On Mon, Apr 12, 2021 at 3:16 PM Jason Wang wrote: > 在 2021/4/9 下午4:02, Yongji Xie 写道: > >>

Re: [PATCH net] virtio-net: suppress bad irq warning for tx napi

2021-04-12 Thread Michael S. Tsirkin
On Mon, Apr 12, 2021 at 04:14:58PM -0700, David Miller wrote: > From: "Michael S. Tsirkin" > Date: Mon, 12 Apr 2021 18:33:45 -0400 > > > On Mon, Apr 12, 2021 at 06:08:21PM -0400, Michael S. Tsirkin wrote: > >> OK I started looking at this again. My idea is simple. > >> A. disable callbacks before

RE: [PATCH v4 05/23] ice: Add devlink params support

2021-04-12 Thread Parav Pandit
> From: Parav Pandit > Sent: Tuesday, April 13, 2021 12:38 AM > > > From: Saleem, Shiraz > > Sent: Monday, April 12, 2021 8:21 PM > > > > > Subject: Re: [PATCH v4 05/23] ice: Add devlink params support > > > > > > On Wed, Apr 07, 2021 at 08:58:25PM +, Saleem, Shiraz wrote: > > > > > Subje

Re: [RFC v4 net-next 1/4] net: phy: add MediaTek PHY driver

2021-04-12 Thread DENG Qingfang
On Mon, Apr 12, 2021 at 11:08:36PM +0800, DENG Qingfang wrote: > On Mon, Apr 12, 2021 at 07:04:49AM +, René van Dorst wrote: > > Hi Qingfang, > > > +static void mtk_phy_config_init(struct phy_device *phydev) > > > +{ > > > + /* Disable EEE */ > > > + phy_write_mmd(phydev, MDIO_MMD_AN, MDIO_AN_E

RE: [net-next, v3, 2/2] enetc: support PTP Sync packet one-step timestamping

2021-04-12 Thread Y.b. Lu
> -Original Message- > From: Yangbo Lu > Sent: 2021年4月12日 17:03 > To: netdev@vger.kernel.org > Cc: Y.b. Lu ; David S . Miller ; > Richard Cochran ; Claudiu Manoil > ; Jakub Kicinski ; Vladimir > Oltean ; Russell King > Subject: [net-next, v3, 2/2] enetc: support PTP Sync packet one-step >

[net-next] enetc: fix locking for one-step timestamping packet transfer

2021-04-12 Thread Yangbo Lu
The previous patch to support PTP Sync packet one-step timestamping described one-step timestamping packet handling logic as below in commit message: - Trasmit packet immediately if no other one in transfer, or queue to skb queue if there is already one in transfer. The test_and_set_bit_lock()

Re: [PATCH v6 09/10] vduse: Introduce VDUSE - vDPA Device in Userspace

2021-04-12 Thread Jason Wang
在 2021/4/12 下午5:59, Yongji Xie 写道: On Mon, Apr 12, 2021 at 5:37 PM Jason Wang wrote: 在 2021/4/12 下午4:02, Yongji Xie 写道: On Mon, Apr 12, 2021 at 3:16 PM Jason Wang wrote: 在 2021/4/9 下午4:02, Yongji Xie 写道: +}; + +struct vduse_dev_config_data { + __u32 offset; /* offset from the beginni

Re: [PATCH net v3] net: sched: fix packet stuck problem for lockless qdisc

2021-04-12 Thread Yunsheng Lin
On 2021/4/13 11:26, Hillf Danton wrote: > On Tue, 13 Apr 2021 10:56:42 Yunsheng Lin wrote: >> On 2021/4/13 10:21, Hillf Danton wrote: >>> On Mon, 12 Apr 2021 20:00:43 Yunsheng Lin wrote: Yes, the below patch seems to fix the data race described in the commit log. Then what is t

[PATCH net-next v4 05/10] xsk: XDP_SETUP_XSK_POOL support option IFF_NOT_USE_DMA_ADDR

2021-04-12 Thread Xuan Zhuo
Some devices, such as virtio-net, do not directly use dma addr. These devices do not initialize dma after completing the xsk setup, so the dma check is skipped here. Signed-off-by: Xuan Zhuo Reviewed-by: Dust Li --- net/xdp/xsk_buff_pool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH net-next v4 09/10] virtio-net: xsk zero copy xmit implement wakeup and xmit

2021-04-12 Thread Xuan Zhuo
This patch implements the core part of xsk zerocopy xmit. When the user calls sendto to consume the data in the xsk tx queue, virtnet_xsk_wakeup() will be called. In wakeup, it will try to send a part of the data directly. There are two purposes for this realization: 1. Send part of the data qui

[PATCH net-next v4 07/10] virtio-net: virtnet_poll_tx support budget check

2021-04-12 Thread Xuan Zhuo
virtnet_poll_tx() check the work done like other network card drivers. When work < budget, napi_poll() in dev.c will exit directly. And virtqueue_napi_complete() will be called to close napi. If closing napi fails or there is still data to be processed, virtqueue_napi_complete() will make napi sch

[PATCH net-next v4 04/10] xsk: support get page by addr

2021-04-12 Thread Xuan Zhuo
xsk adds an interface and returns the page corresponding to data. virtio-net does not initialize dma, so it needs page to construct scatterlist to pass to vring. Signed-off-by: Xuan Zhuo Reviewed-by: Dust Li --- include/net/xdp_sock_drv.h | 11 +++ 1 file changed, 11 insertions(+) diff

[PATCH net-next v4 10/10] virtio-net: xsk zero copy xmit kick by threshold

2021-04-12 Thread Xuan Zhuo
After testing, the performance of calling kick every time is not stable. And if all the packets are sent and kicked again, the performance is not good. So add a module parameter to specify how many packets are sent to call a kick. 8 is a relatively stable value with the best performance. Here is

[PATCH net-next v4 08/10] virtio-net: xsk zero copy xmit setup

2021-04-12 Thread Xuan Zhuo
xsk is a high-performance packet receiving and sending technology. This patch implements the binding and unbinding operations of xsk and the virtio-net queue for xsk zero copy xmit. The xsk zero copy xmit depends on tx napi. So if tx napi is not true, an error will be reported. And the entire ope

[PATCH net-next v4 03/10] virtio-net: add priv_flags IFF_NOT_USE_DMA_ADDR

2021-04-12 Thread Xuan Zhuo
virtio-net not use dma addr directly. So add this priv_flags IFF_NOT_USE_DMA_ADDR. Signed-off-by: Xuan Zhuo --- drivers/net/virtio_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index bb4ea9dbc16b..52653e234a20 100644

[PATCH net-next v4 06/10] virtio-net: unify the code for recycling the xmit ptr

2021-04-12 Thread Xuan Zhuo
Now there are two types of "skb" and "xdp frame" during recycling old xmit. There are two completely similar and independent implementations. This is inconvenient for the subsequent addition of new types. So extract a function from this piece of code and call this function uniformly to recover old

[PATCH net-next v4 01/10] netdevice: priv_flags extend to 64bit

2021-04-12 Thread Xuan Zhuo
The size of priv_flags is 32 bits, and the number of flags currently available has reached 32. It is time to expand the size of priv_flags to 64 bits. Here the priv_flags is modified to 8 bytes, but the size of struct net_device has not changed, it is still 2176 bytes. It is because _tx is aligned

[PATCH net-next v4 00/10] virtio-net support xdp socket zero copy xmit

2021-04-12 Thread Xuan Zhuo
XDP socket is an excellent by pass kernel network transmission framework. The zero copy feature of xsk (XDP socket) needs to be supported by the driver. The performance of zero copy is very good. mlx5 and intel ixgbe already support this feature, This patch set allows virtio-net to support xsk's ze

[PATCH net-next v4 02/10] netdevice: add priv_flags IFF_NOT_USE_DMA_ADDR

2021-04-12 Thread Xuan Zhuo
Some driver devices, such as virtio-net, do not directly use dma addr. For upper-level frameworks such as xdp socket, that need to be aware of this. So add a new priv_flag IFF_NOT_USE_DMA_ADDR. Signed-off-by: Xuan Zhuo --- include/linux/netdevice.h | 3 +++ 1 file changed, 3 insertions(+) diff

Re: [PATCH net v3] net: sched: fix packet stuck problem for lockless qdisc

2021-04-12 Thread Yunsheng Lin
On 2021/4/13 10:21, Hillf Danton wrote: > On Mon, 12 Apr 2021 20:00:43 Yunsheng Lin wrote: >> >> Yes, the below patch seems to fix the data race described in >> the commit log. >> Then what is the difference between my patch and your patch below:) > > Hehe, this is one of the tough questions over

[PATCH net v2] i40e: fix the panic when running bpf in xdpdrv mode

2021-04-12 Thread kerneljasonxing
From: Jason Xing Fix this panic by adding more rules to calculate the value of @rss_size_max which could be used in allocating the queues when bpf is loaded, which, however, could cause the failure and then trigger the NULL pointer of vsi->rx_rings. Prio to this fix, the machine doesn't care abou

Re: [PATCH net-next 2/2] ibmvnic: add sysfs entry for timeout and fatal reset

2021-04-12 Thread Jakub Kicinski
On Mon, 12 Apr 2021 15:26:00 -0500 Lijun Pan wrote: > On Mon, Apr 12, 2021 at 1:23 PM Jakub Kicinski wrote: > > On Mon, 12 Apr 2021 02:43:30 -0500 Lijun Pan wrote: > > > Add timeout and fatal reset sysfs entries so that both functions > > > can be triggered manually the tested. Otherwise, you ha

Re: [PATCH] i40e: fix the panic when running bpf in xdpdrv mode

2021-04-12 Thread Jason Xing
On Tue, Apr 13, 2021 at 5:52 AM Jesse Brandeburg wrote: > > kerneljasonx...@gmail.com wrote: > > > From: Jason Xing > > > > Re: [PATCH] i40e: fix the panic when running bpf in xdpdrv mode > > Please use netdev style subject lines when patching net kernel to > indicate which kernel tree this is ta

Re: [PATCH net-next 0/8] ionic: hwstamp tweaks

2021-04-12 Thread Richard Cochran
On Mon, Apr 12, 2021 at 09:33:29AM -0700, Shannon Nelson wrote: > If the original patches hadn't already been pulled into net-next, this is > what I would have done.  My understanding is that once the patches have been > pulled into the repo that we need to do delta patches, not new versions of > t

[PATCH] mac80211_hwsim: indicate support for 60GHz channels

2021-04-12 Thread Ramon Fontes
Advertise 60GHz channels to mac80211. --- drivers/net/wireless/mac80211_hwsim.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/mac80211_hwsim.c b/drivers/net/wireless/mac80211_hwsim.c index fa7d4c20d..70d0af475 100644 --- a/drivers/n

Re: [PATCH net-next v4 2/2] of: net: fix of_get_mac_addr_nvmem() for non-platform devices

2021-04-12 Thread Andrew Lunn
On Mon, Apr 12, 2021 at 07:47:18PM +0200, Michael Walle wrote: > of_get_mac_address() already supports fetching the MAC address by an > nvmem provider. But until now, it was just working for platform devices. > Esp. it was not working for DSA ports and PCI devices. It gets more > common that PCI de

Re: [PATCH net-next v4 1/2] of: net: pass the dst buffer to of_get_mac_address()

2021-04-12 Thread Andrew Lunn
On Mon, Apr 12, 2021 at 07:47:17PM +0200, Michael Walle wrote: > of_get_mac_address() returns a "const void*" pointer to a MAC address. > Lately, support to fetch the MAC address by an NVMEM provider was added. > But this will only work with platform devices. It will not work with > PCI devices (e.

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 02:27:30 +0200 Marek Behun wrote: > On Tue, 13 Apr 2021 01:54:50 +0200 > Marek Behun wrote: > > > I will look into this, maybe ask some follow-up questions. > > Tobias, > > it seems that currently the LAGs in mv88e6xxx driver do not use the > HashTrunk feature (which can b

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 01:54:50 +0200 Marek Behun wrote: > I will look into this, maybe ask some follow-up questions. Tobias, it seems that currently the LAGs in mv88e6xxx driver do not use the HashTrunk feature (which can be enabled via bit 11 of the MV88E6XXX_G2_TRUNK_MAPPING register). If we u

Re: [RFC v4 net-next 2/4] net: dsa: mt7530: add interrupt support

2021-04-12 Thread Andrew Lunn
> > > +static void > > > +mt7530_setup_mdio_irq(struct mt7530_priv *priv) > > > +{ > > > + struct dsa_switch *ds = priv->ds; > > > + int p; > > > + > > > + for (p = 0; p < MT7530_NUM_PHYS; p++) { > > > + if (BIT(p) & ds->phys_mii_mask) { > > > + unsigned int irq; > > > + > >

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 01:13:53 +0200 Tobias Waldekranz wrote: > > ...you could get the isolation in place. But you will still lookup the > > DA in the ATU, and there you will find a destination of either cpu0 or > > cpu1. So for one of the ports, the destination will be outside of its > > port base

Re: [PATCH] net: phy: marvell: fix detection of PHY on Topaz switches

2021-04-12 Thread Marek Behún
On Mon, 12 Apr 2021 18:38:29 +0200 Pali Rohár wrote: > On Monday 12 April 2021 18:12:35 Andrew Lunn wrote: > > On Mon, Apr 12, 2021 at 05:52:39PM +0200, Pali Rohár wrote: > > > On Monday 12 April 2021 17:32:33 Andrew Lunn wrote: > > > > > Anyway, now I'm looking at phy/marvell.c driver again

Re: [PATCH net-next] net: dsa: lantiq_gswip: Add support for dumping the registers

2021-04-12 Thread Andrew Lunn
On Tue, Apr 13, 2021 at 12:24:49AM +0200, Martin Blumenstingl wrote: > Hi Andrew, > > On Mon, Apr 12, 2021 at 1:16 AM Andrew Lunn wrote: > > > > On Sun, Apr 11, 2021 at 10:55:11PM +0200, Martin Blumenstingl wrote: > > > Add support for .get_regs_len and .get_regs so it is easier to find out > > >

Re: [PATCH net-next 1/2] net: phy: marvell-88x2222: check that link is operational

2021-04-12 Thread Andrew Lunn
On Mon, Apr 12, 2021 at 03:16:59PM +0300, Ivan Bornyakov wrote: > Some SFP modules uses RX_LOS for link indication. In such cases link > will be always up, even without cable connected. RX_LOS changes will > trigger link_up()/link_down() upstream operations. Thus, check that SFP > link is operation

Re: [PATCH net-next 1/2] net: phy: marvell-88x2222: check that link is operational

2021-04-12 Thread Marek Behún
On Mon, 12 Apr 2021 15:16:59 +0300 Ivan Bornyakov wrote: > Some SFP modules uses RX_LOS for link indication. In such cases link > will be always up, even without cable connected. RX_LOS changes will > trigger link_up()/link_down() upstream operations. Thus, check that SFP > link is operational be

[PATCH stable 4.14] net: phy: broadcom: Only advertise EEE for supported modes

2021-04-12 Thread Florian Fainelli
commit c056d480b40a68f2520ccc156c7fae672d69d57d upstream We should not be advertising EEE for modes that we do not support, correct that oversight by looking at the PHY device supported linkmodes. Fixes: 99cec8a4dda2 ("net: phy: broadcom: Allow enabling or disabling of EEE") Signed-off-by: Floria

[PATCh stable 4.19] net: phy: broadcom: Only advertise EEE for supported modes

2021-04-12 Thread Florian Fainelli
commit c056d480b40a68f2520ccc156c7fae672d69d57d upstream We should not be advertising EEE for modes that we do not support, correct that oversight by looking at the PHY device supported linkmodes. Fixes: 99cec8a4dda2 ("net: phy: broadcom: Allow enabling or disabling of EEE") Signed-off-by: Floria

Re: [PATCH v2] net: phy: marvell: fix detection of PHY on Topaz switches

2021-04-12 Thread Marek Behún
> + /* Some internal PHYs don't have a model number. */ > + if (reg == MII_PHYSID2 && !(val & 0x3f0) && > + chip->info->family < ARRAY_SIZE(family_prod_id_table)) { > + prod_id = family_prod_id_table[chip->info->family]; > + if (prod_id) > +

Re: [PATCH net 1/7] netfilter: flowtable: fix NAT IPv6 offload mangling

2021-04-12 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net.git (refs/heads/master): On Tue, 13 Apr 2021 00:30:53 +0200 you wrote: > Fix out-of-bound access in the address array. > > Fixes: 5c27d8d76ce8 ("netfilter: nf_flow_table_offload: add IPv6 support") > Signed-off-by: Pablo Neira Ayuso > --- > net/netf

Re: [PATCH net] virtio-net: suppress bad irq warning for tx napi

2021-04-12 Thread David Miller
From: "Michael S. Tsirkin" Date: Mon, 12 Apr 2021 18:33:45 -0400 > On Mon, Apr 12, 2021 at 06:08:21PM -0400, Michael S. Tsirkin wrote: >> OK I started looking at this again. My idea is simple. >> A. disable callbacks before we try to drain skbs >> B. actually do disable callbacks even with event

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Tue, Apr 13, 2021 at 01:09, Tobias Waldekranz wrote: > On Tue, Apr 13, 2021 at 00:55, Marek Behun wrote: >> On Tue, 13 Apr 2021 00:05:51 +0200 >> Tobias Waldekranz wrote: >> >>> On Mon, Apr 12, 2021 at 23:50, Marek Behun wrote: >>> > On Mon, 12 Apr 2021 23:22:45 +0200 >>> > Tobias Waldekranz

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Tue, Apr 13, 2021 at 00:55, Marek Behun wrote: > On Tue, 13 Apr 2021 00:05:51 +0200 > Tobias Waldekranz wrote: > >> On Mon, Apr 12, 2021 at 23:50, Marek Behun wrote: >> > On Mon, 12 Apr 2021 23:22:45 +0200 >> > Tobias Waldekranz wrote: >> > >> >> On Mon, Apr 12, 2021 at 21:30, Marek Behun

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 01:48:05 +0300 Vladimir Oltean wrote: > On Tue, Apr 13, 2021 at 12:26:52AM +0200, Tobias Waldekranz wrote: > > On Tue, Apr 13, 2021 at 01:06, Vladimir Oltean wrote: > > > On Mon, Apr 12, 2021 at 11:49:22PM +0200, Tobias Waldekranz wrote: > > >> On Tue, Apr 13, 2021 at 00:

Re: [RFC Patch bpf-next] bpf: introduce bpf timer

2021-04-12 Thread Alexei Starovoitov
On Mon, Apr 05, 2021 at 05:36:27PM -0700, Cong Wang wrote: > On Fri, Apr 2, 2021 at 4:45 PM Alexei Starovoitov > wrote: > > > > On Fri, Apr 02, 2021 at 02:24:51PM -0700, Cong Wang wrote: > > > > > where the key is the timer ID and the value is the timer expire > > > > > timer. > > > > > > > > The

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Jason Gunthorpe
On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > ib_modify_qp() is an expensive operation on some HCAs running > virtualized. This series removes two ib_modify_qp() calls from RDS. > > I am sending this as a v3, even though it is the first sent to > net. This because the IB Core comm

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 00:05:51 +0200 Tobias Waldekranz wrote: > On Mon, Apr 12, 2021 at 23:50, Marek Behun wrote: > > On Mon, 12 Apr 2021 23:22:45 +0200 > > Tobias Waldekranz wrote: > > > >> On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > >> > On Mon, 12 Apr 2021 14:46:11 +0200 > >> > To

Re: [PATCH rdma-next 00/10] Enable relaxed ordering for ULPs

2021-04-12 Thread Jason Gunthorpe
On Mon, Apr 12, 2021 at 04:20:47PM -0400, Tom Talpey wrote: > So the issue is only in testing all the providers and platforms, > to be sure this new behavior isn't tickling anything that went > unnoticed all along, because no RDMA provider ever issued RO. The mlx5 ethernet driver has run in RO mo

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Vladimir Oltean
On Tue, Apr 13, 2021 at 12:26:52AM +0200, Tobias Waldekranz wrote: > On Tue, Apr 13, 2021 at 01:06, Vladimir Oltean wrote: > > On Mon, Apr 12, 2021 at 11:49:22PM +0200, Tobias Waldekranz wrote: > >> On Tue, Apr 13, 2021 at 00:34, Vladimir Oltean wrote: > >> > On Mon, Apr 12, 2021 at 11:22:45PM +0

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Tue, 13 Apr 2021 01:17:21 +0300 Vladimir Oltean wrote: > On Tue, Apr 13, 2021 at 12:04:57AM +0200, Marek Behun wrote: > > On Mon, 12 Apr 2021 19:32:11 +0300 > > Vladimir Oltean wrote: > > > > > On Mon, Apr 12, 2021 at 11:00:45PM +0800, DENG Qingfang wrote: > > > > On Sun, Apr 11, 2021 at

Re: [PATCH net] ice: Re-organizes reqstd/avail {R,T}XQ check/code for efficiency+readability

2021-04-12 Thread Nguyen, Anthony L
On Sun, 2021-04-11 at 02:45 +0100, Salil Mehta wrote: > If user has explicitly requested the number of {R,T}XQs, then it is > unnecessary > to get the count of already available {R,T}XQs from the PF > avail_{r,t}xqs > bitmap. This value will get overriden by user specified value in any s/override

Re: [PATCH net] virtio-net: suppress bad irq warning for tx napi

2021-04-12 Thread Michael S. Tsirkin
On Mon, Apr 12, 2021 at 06:08:21PM -0400, Michael S. Tsirkin wrote: > OK I started looking at this again. My idea is simple. > A. disable callbacks before we try to drain skbs > B. actually do disable callbacks even with event idx > > To make B not regress, we need to > C. detect the common case o

[PATCH net 1/7] netfilter: flowtable: fix NAT IPv6 offload mangling

2021-04-12 Thread Pablo Neira Ayuso
Fix out-of-bound access in the address array. Fixes: 5c27d8d76ce8 ("netfilter: nf_flow_table_offload: add IPv6 support") Signed-off-by: Pablo Neira Ayuso --- net/netfilter/nf_flow_table_offload.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/netfilter/nf_flow_tabl

[PATCH net 0/7] Netfilter fixes for net

2021-04-12 Thread Pablo Neira Ayuso
Hi, The following patchset contains Netfilter fixes for net: 1) Fix NAT IPv6 offload in the flowtable. 2) icmpv6 is printed as unknown in /proc/net/nf_conntrack. 3) Use div64_u64() in nft_limit, from Eric Dumazet. 4) Use pre_exit to unregister ebtables and arptables hooks, from Florian West

[PATCH net 6/7] netfilter: x_tables: fix compat match/target pad out-of-bound write

2021-04-12 Thread Pablo Neira Ayuso
From: Florian Westphal xt_compat_match/target_from_user doesn't check that zeroing the area to start of next rule won't write past end of allocated ruleset blob. Remove this code and zero the entire blob beforehand. Reported-by: syzbot+cfc0247ac173f597a...@syzkaller.appspotmail.com Reported-by:

[PATCH net 7/7] netfilter: nftables: clone set element expression template

2021-04-12 Thread Pablo Neira Ayuso
memcpy() breaks when using connlimit in set elements. Use nft_expr_clone() to initialize the connlimit expression list, otherwise connlimit garbage collector crashes when walking on the list head copy. [ 493.064656] Workqueue: events_power_efficient nft_rhash_gc [nf_tables] [ 493.064685] RIP: 00

[PATCH net 5/7] netfilter: arp_tables: add pre_exit hook for table unregister

2021-04-12 Thread Pablo Neira Ayuso
From: Florian Westphal Same problem that also existed in iptables/ip(6)tables, when arptable_filter is removed there is no longer a wait period before the table/ruleset is free'd. Unregister the hook in pre_exit, then remove the table in the exit function. This used to work correctly because the

[PATCH net 2/7] netfilter: conntrack: do not print icmpv6 as unknown via /proc

2021-04-12 Thread Pablo Neira Ayuso
/proc/net/nf_conntrack shows icmpv6 as unknown. Fixes: 09ec82f5af99 ("netfilter: conntrack: remove protocol name from l4proto struct") Signed-off-by: Pablo Neira Ayuso --- net/netfilter/nf_conntrack_standalone.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/netfilter/nf_conntrack_stan

[PATCH net 3/7] netfilter: nft_limit: avoid possible divide error in nft_limit_init

2021-04-12 Thread Pablo Neira Ayuso
From: Eric Dumazet div_u64() divides u64 by u32. nft_limit_init() wants to divide u64 by u64, use the appropriate math function (div64_u64) divide error: [#1] PREEMPT SMP KASAN CPU: 1 PID: 8390 Comm: syz-executor188 Not tainted 5.12.0-rc4-syzkaller #0 Hardware name: Google Google Compute E

[PATCH net 4/7] netfilter: bridge: add pre_exit hooks for ebtable unregistration

2021-04-12 Thread Pablo Neira Ayuso
From: Florian Westphal Just like ip/ip6/arptables, the hooks have to be removed, then synchronize_rcu() has to be called to make sure no more packets are being processed before the ruleset data is released. Place the hook unregistration in the pre_exit hook, then call the new ebtables pre_exit f

Re: [PATCH net-next] net: dsa: lantiq_gswip: Add support for dumping the registers

2021-04-12 Thread Martin Blumenstingl
Hi Andrew, On Mon, Apr 12, 2021 at 1:16 AM Andrew Lunn wrote: > > On Sun, Apr 11, 2021 at 10:55:11PM +0200, Martin Blumenstingl wrote: > > Add support for .get_regs_len and .get_regs so it is easier to find out > > about the state of the ports on the GSWIP hardware. For this we > > specifically a

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Tue, Apr 13, 2021 at 01:06, Vladimir Oltean wrote: > On Mon, Apr 12, 2021 at 11:49:22PM +0200, Tobias Waldekranz wrote: >> On Tue, Apr 13, 2021 at 00:34, Vladimir Oltean wrote: >> > On Mon, Apr 12, 2021 at 11:22:45PM +0200, Tobias Waldekranz wrote: >> >> On Mon, Apr 12, 2021 at 21:30, Marek Be

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Vladimir Oltean
On Tue, Apr 13, 2021 at 12:04:57AM +0200, Marek Behun wrote: > On Mon, 12 Apr 2021 19:32:11 +0300 > Vladimir Oltean wrote: > > > On Mon, Apr 12, 2021 at 11:00:45PM +0800, DENG Qingfang wrote: > > > On Sun, Apr 11, 2021 at 09:50:17PM +0300, Vladimir Oltean wrote: > > > > > > > > So I'd be tempted t

Re: [PATCH net] virtio-net: suppress bad irq warning for tx napi

2021-04-12 Thread Michael S. Tsirkin
OK I started looking at this again. My idea is simple. A. disable callbacks before we try to drain skbs B. actually do disable callbacks even with event idx To make B not regress, we need to C. detect the common case of disable after event triggering and skip the write then. I added a new event_

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Vladimir Oltean
On Mon, Apr 12, 2021 at 11:49:22PM +0200, Tobias Waldekranz wrote: > On Tue, Apr 13, 2021 at 00:34, Vladimir Oltean wrote: > > On Mon, Apr 12, 2021 at 11:22:45PM +0200, Tobias Waldekranz wrote: > >> On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > >> > On Mon, 12 Apr 2021 14:46:11 +0200 > >> >

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Mon, Apr 12, 2021 at 23:50, Marek Behun wrote: > On Mon, 12 Apr 2021 23:22:45 +0200 > Tobias Waldekranz wrote: > >> On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: >> > On Mon, 12 Apr 2021 14:46:11 +0200 >> > Tobias Waldekranz wrote: >> > >> >> I agree. Unless you only have a few really

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Mon, 12 Apr 2021 19:32:11 +0300 Vladimir Oltean wrote: > On Mon, Apr 12, 2021 at 11:00:45PM +0800, DENG Qingfang wrote: > > On Sun, Apr 11, 2021 at 09:50:17PM +0300, Vladimir Oltean wrote: > > > > > > So I'd be tempted to say 'tough luck' if all your ports are not up, and > > > the ones that

[PATCH 1/2] net: sit: Unregister catch-all devices

2021-04-12 Thread Hristo Venev
A sit interface created without a local or a remote address is linked into the `sit_net::tunnels_wc` list of its original namespace. When deleting a network namespace, delete the devices that have been moved. The following script triggers a null pointer dereference if devices linked in a deleted `

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Mon, 12 Apr 2021 23:49:22 +0200 Tobias Waldekranz wrote: > On Tue, Apr 13, 2021 at 00:34, Vladimir Oltean wrote: > > On Mon, Apr 12, 2021 at 11:22:45PM +0200, Tobias Waldekranz wrote: > >> On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > >> > On Mon, 12 Apr 2021 14:46:11 +0200 > >> >

[PATCH 2/2] net: ip6_tunnel: Unregister catch-all devices

2021-04-12 Thread Hristo Venev
Similarly to the sit case, we need to remove the tunnels with no addresses that have been moved to another network namespace. Fixes: 0bd8762824e73 ("ip6tnl: add x-netns support") Signed-off-by: Hristo Venev --- net/ipv6/ip6_tunnel.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a

Re: [PATCH] i40e: fix the panic when running bpf in xdpdrv mode

2021-04-12 Thread Jesse Brandeburg
kerneljasonx...@gmail.com wrote: > From: Jason Xing > > Re: [PATCH] i40e: fix the panic when running bpf in xdpdrv mode Please use netdev style subject lines when patching net kernel to indicate which kernel tree this is targeted at, "net" or "net-next" [PATCH net v2] i40e: ... > Fix this by a

Re: [PATCH bpf-next v3 3/3] bpf: selftests: update array map tests for per-cpu batched ops

2021-04-12 Thread Martin KaFai Lau
On Mon, Apr 12, 2021 at 04:40:01PM -0300, Pedro Tammela wrote: > Follows the same logic as the hashtable tests. > > Signed-off-by: Pedro Tammela > --- > .../bpf/map_tests/array_map_batch_ops.c | 110 +- > 1 file changed, 80 insertions(+), 30 deletions(-) > > diff --git a/t

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Tue, Apr 13, 2021 at 00:34, Vladimir Oltean wrote: > On Mon, Apr 12, 2021 at 11:22:45PM +0200, Tobias Waldekranz wrote: >> On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: >> > On Mon, 12 Apr 2021 14:46:11 +0200 >> > Tobias Waldekranz wrote: >> > >> >> I agree. Unless you only have a few rea

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Marek Behun
On Mon, 12 Apr 2021 23:22:45 +0200 Tobias Waldekranz wrote: > On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > > On Mon, 12 Apr 2021 14:46:11 +0200 > > Tobias Waldekranz wrote: > > > >> I agree. Unless you only have a few really wideband flows, a LAG will > >> typically do a great job with

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Vladimir Oltean
On Mon, Apr 12, 2021 at 11:22:45PM +0200, Tobias Waldekranz wrote: > On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > > On Mon, 12 Apr 2021 14:46:11 +0200 > > Tobias Waldekranz wrote: > > > >> I agree. Unless you only have a few really wideband flows, a LAG will > >> typically do a great job w

Re: [PATCH net resend] ethtool: fix kdoc attr name

2021-04-12 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Mon, 12 Apr 2021 11:47:07 -0700 you wrote: > Add missing 't' in attrtype. > > Signed-off-by: Jakub Kicinski > --- > net/ethtool/netlink.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Here is the summary w

Re: [PATCH v2] net: phy: marvell: fix detection of PHY on Topaz switches

2021-04-12 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Mon, 12 Apr 2021 18:57:39 +0200 you wrote: > Since commit fee2d546414d ("net: phy: marvell: mv88e6390 temperature > sensor reading"), Linux reports the temperature of Topaz hwmon as > constant -75°C. > > This is because swit

[PATCH net-next V2] icmp: ICMPV6: pass RFC 8335 reply messages to ping_rcv

2021-04-12 Thread Andreas Roeseler
The current icmp_rcv function drops all unknown ICMP types, including ICMP_EXT_ECHOREPLY (type 43). In order to parse Extended Echo Reply messages, we have to pass these packets to the ping_rcv function, which does not do any other filtering and passes the packet to the designated socket. Pass in

Re: [PATCH RFC net-next 0/3] Multi-CPU DSA support

2021-04-12 Thread Tobias Waldekranz
On Mon, Apr 12, 2021 at 21:30, Marek Behun wrote: > On Mon, 12 Apr 2021 14:46:11 +0200 > Tobias Waldekranz wrote: > >> I agree. Unless you only have a few really wideband flows, a LAG will >> typically do a great job with balancing. This will happen without the >> user having to do any configurat

Re: [PATCH V4 1/2] dt-bindings: net: renesas,etheravb: Add additional clocks

2021-04-12 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Mon, 12 Apr 2021 08:26:18 -0500 you wrote: > The AVB driver assumes there is an external crystal, but it could > be clocked by other means. In order to enable a programmable > clock, it needs to be added to the clocks

  1   2   3   4   5   >