Re: [PATCH mlx5-next v6 1/4] PCI: Add sysfs callback to allow MSI-X table size change of SR-IOV VFs

2021-02-18 Thread Leon Romanovsky
On Thu, Feb 18, 2021 at 04:39:50PM -0600, Bjorn Helgaas wrote: > On Thu, Feb 18, 2021 at 12:15:51PM +0200, Leon Romanovsky wrote: > > On Wed, Feb 17, 2021 at 12:02:39PM -0600, Bjorn Helgaas wrote: > > > [+cc Greg in case he wants to chime in on the sysfs discussion. > > > TL;DR: we're trying to add

Re: [PATCHv3 3/3] CDC-NCM: record speed in status method

2021-02-18 Thread Grant Grundler
Oliver, Can you include a 4th patch in the series to bring cdc_ether driver in line with the cdc_ncm behavior? I apologize for not including the patch inline - but it's late and I don't want to fight with gmail at this point. Patch is attached. Not tested. cheers, grant On Thu, Feb 18, 2021 at 1

Re: [PATCHv3 3/3] CDC-NCM: record speed in status method

2021-02-18 Thread Grant Grundler
On Thu, Feb 18, 2021 at 10:21 AM Oliver Neukum wrote: > > The driver has a status method for receiving speed updates. > The framework, however, had support functions only for devices > that reported their speed upon an explicit query over a MDIO > interface. > CDC_NCM however gets direct notificat

Re: [PATCHv3 1/3] usbnet: specify naming of usbnet_set/get_link_ksettings

2021-02-18 Thread Grant Grundler
On Thu, Feb 18, 2021 at 7:50 PM Andrew Lunn wrote: > > On Thu, Feb 18, 2021 at 07:31:41PM +, Grant Grundler wrote: > > Oliver, Jakub, > > Can I post v4 and deal with the issues below? > > You should probably wait for two weeks. We are far enough into the > merge window that i doubt it will get

Re: [PATCH bpf-next V2 0/2] bpf: Updates for BPF-helper bpf_check_mtu

2021-02-18 Thread Jesper Dangaard Brouer
On Thu, 18 Feb 2021 12:49:53 +0100 Jesper Dangaard Brouer wrote: > The FIB lookup example[1] show how the IP-header field tot_len > (iph->tot_len) is used as input to perform the MTU check. The recently > added MTU check helper bpf_check_mtu() should also support this type > of MTU check. > > Le

Re: [PATCH] perf machine: Use true and false for bool variable

2021-02-18 Thread kajoljain
On 2/18/21 2:54 PM, Jiapeng Chong wrote: > Fix the following coccicheck warnings: > > ./tools/perf/util/machine.c:2000:9-10: WARNING: return of 0/1 in > function 'symbol__match_regex' with return type bool. > > Reported-by: Abaci Robot > Signed-off-by: Jiapeng Chong > --- > tools/perf/util/

Re: [PATCH] [v13] wireless: Initial driver submission for pureLiFi STA devices

2021-02-18 Thread Srinivasan Raju
Hi, Please find a few responses to the comments , We will fix rest of the comments and submit v14 of the patch. > Also, you *really* need some validation here, rather than blindly > trusting that the file is well-formed, otherwise you immediately have a > security issue here. The firmware is si

Re: [PATCH v1] vdpa/mlx5: Restore the hardware used index after change map

2021-02-18 Thread Jason Wang
On 2021/2/18 8:43 下午, Si-Wei Liu wrote: On 2/17/2021 8:44 PM, Jason Wang wrote: On 2021/2/10 下午4:59, Si-Wei Liu wrote: On 2/9/2021 7:53 PM, Jason Wang wrote: On 2021/2/10 上午10:30, Si-Wei Liu wrote: On 2/8/2021 10:37 PM, Jason Wang wrote: On 2021/2/9 下午2:12, Eli Cohen wrote: On Tu

[PATCH net v4] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
The icmp{,v6}_send functions make all sorts of use of skb->cb, casting it with IPCB or IP6CB, assuming the skb to have come directly from the inet layer. But when the packet comes from the ndo layer, especially when forwarded, there's no telling what might be in skb->cb at that point. As a result,

Re: [PATCH net v3] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
On Thu, Feb 18, 2021 at 11:06 PM Willem de Bruijn wrote: > > On Thu, Feb 18, 2021 at 3:39 PM Jason A. Donenfeld wrote: > > > > The icmp{,v6}_send functions make all sorts of use of skb->cb, casting > > Again, if respinning, please briefly describe the specific buggy code > path. I think it's info

Re: [PATCH net-next] net: mscc: Fix MRP switchdev driver

2021-02-18 Thread Horatiu Vultur
The 02/18/2021 13:17, Vladimir Oltean wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > Hi Vladimir, > Hi Horatiu, > > On Thu, Feb 18, 2021 at 12:47:26PM +0100, Horatiu Vultur wrote: > > This patch fixes the ocelot MRP switchdev driver such

[PATCH net 6/8] i40e: Fix VFs not created

2021-02-18 Thread Tony Nguyen
From: Sylwester Dziedziuch When creating VFs they were sometimes not getting resources. It was caused by not executing i40e_reset_all_vfs due to flag __I40E_VF_DISABLE being set on PF. Because of this IAVF was never able to finish setup sequence never getting reset indication from PF. Changed tes

[PATCH net 2/8] i40e: Fix memory leak in i40e_probe

2021-02-18 Thread Tony Nguyen
From: Keita Suzuki Struct i40e_veb is allocated in function i40e_setup_pf_switch, and stored to an array field veb inside struct i40e_pf. However when i40e_setup_misc_vector fails, this memory leaks. Fix this by calling exit and teardown functions. Signed-off-by: Keita Suzuki Tested-by: Tony B

[PATCH net 7/8] i40e: Fix add TC filter for IPv6

2021-02-18 Thread Tony Nguyen
From: Mateusz Palczewski Fix insufficient distinction between IPv4 and IPv6 addresses when creating a filter. IPv4 and IPv6 are kept in the same memory area. If IPv6 is added, then it's caught by IPv4 check, which leads to err -95. Fixes: 2f4b411a3d67("i40e: Enable cloud filters via tc-flower")

[PATCH net 8/8] i40e: Fix endianness conversions

2021-02-18 Thread Tony Nguyen
From: Norbert Ciosek Fixes the following sparse warnings: i40e_main.c:5953:32: warning: cast from restricted __le16 i40e_main.c:8008:29: warning: incorrect type in assignment (different base types) i40e_main.c:8008:29:expected unsigned int [assigned] [usertype] ipa i40e_main.c:8008:29:go

[PATCH net 3/8] i40e: Add zero-initialization of AQ command structures

2021-02-18 Thread Tony Nguyen
From: Mateusz Palczewski Zero-initialize AQ command data structures to comply with API specifications. Fixes: 2f4b411a3d67 ("i40e: Enable cloud filters via tc-flower") Fixes: f4492db16df8 ("i40e: Add NPAR BW get and set functions") Signed-off-by: Andrzej Sawuła Signed-off-by: Mateusz Palczewski

[PATCH net 4/8] i40e: Fix overwriting flow control settings during driver loading

2021-02-18 Thread Tony Nguyen
From: Mateusz Palczewski During driver loading flow control settings were written to FW using a variable which was always zero, since it was being set only by ethtool. This behavior has been corrected and driver no longer overwrites the default FW/NVM settings. Fixes: 373149fc99a0 ("i40e: Decrea

[PATCH net 0/8][pull request] Intel Wired LAN Driver Updates 2021-02-18

2021-02-18 Thread Tony Nguyen
This series contains updates to i40e driver only. Slawomir resolves an issue with the IPv6 extension headers being processed incorrectly. Keita Suzuki fixes a memory leak on probe failure. Mateusz initializes AQ command structures to zero to comply with spec, fixes FW flow control settings being

[PATCH net 1/8] i40e: Fix flow for IPv6 next header (extension header)

2021-02-18 Thread Tony Nguyen
From: Slawomir Laba When a packet contains an IPv6 header with next header which is an extension header and not a protocol one, the kernel function skb_transport_header called with such sk_buff will return a pointer to the extension header and not to the TCP one. The above explained call caused

[PATCH net 5/8] i40e: Fix addition of RX filters after enabling FW LLDP agent

2021-02-18 Thread Tony Nguyen
From: Mateusz Palczewski Fix addition of VLAN filter for PF after enabling FW LLDP agent. Changing LLDP Agent causes FW to re-initialize per NVM settings. Remove default PF filter and move "Enable/Disable" to currently used reset flag. Without this patch PF would try to add MAC VLAN filter with d

FW: [PATCH net-next] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-02-18 Thread Min Li
-Original Message- From: Min Li Sent: February 18, 2021 11:14 AM To: 'Arnd Bergmann' Cc: Derek Kiernan ; Dragan Cvetic ; Arnd Bergmann ; gregkh ; linux-ker...@vger.kernel.org; Networking ; Richard Cochran Subject: RE: [PATCH net-next] misc: Add Renesas Synchronization Management Uni

Re: [PATCH mlx5-next v6 1/4] PCI: Add sysfs callback to allow MSI-X table size change of SR-IOV VFs

2021-02-18 Thread Bjorn Helgaas
On Thu, Feb 18, 2021 at 12:15:51PM +0200, Leon Romanovsky wrote: > On Wed, Feb 17, 2021 at 12:02:39PM -0600, Bjorn Helgaas wrote: > > [+cc Greg in case he wants to chime in on the sysfs discussion. > > TL;DR: we're trying to add/remove sysfs files when a PCI driver that > > supports certain callbac

Re: [PATCH net v3] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 3:39 PM Jason A. Donenfeld wrote: > > The icmp{,v6}_send functions make all sorts of use of skb->cb, casting Again, if respinning, please briefly describe the specific buggy code path. I think it's informative and cannot be gleaned from the fix. > it with IPCB or IP6CB, a

Re: [PATCH bpf-next 00/17] Improve BPF syscall command documentation

2021-02-18 Thread Joe Stringer
On Thu, Feb 18, 2021 at 11:49 AM Jonathan Corbet wrote: > > Joe Stringer writes: > > * The changes in patch 16 here extended Documentation/bpf/index.rst, > > but to assist in improving the overall kernel documentation > > organisation / hierarchy, you would prefer to instead introduce a > > dedic

[PATCH v15 0/4] Adding the Sparx5 Serdes driver

2021-02-18 Thread Steen Hegelund
Adding the Sparx5 Serdes driver This series of patches provides the serdes driver for the Microchip Sparx5 ethernet switch. The serdes driver supports the 10G and 25G serdes instances available in the Sparx5. The Sparx5 serdes support several interface modes with several speeds and also allows t

[PATCH v15 1/4] dt-bindings: phy: Add sparx5-serdes bindings

2021-02-18 Thread Steen Hegelund
Document the Sparx5 ethernet serdes phy driver bindings. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Rob Herring Reviewed-by: Andrew Lunn Reviewed-by: Alexandre Belloni --- .../bindings/phy/microchip,sparx5-serdes.yaml | 100 ++ 1 file changed, 100

[PATCH v15 4/4] arm64: dts: sparx5: Add Sparx5 serdes driver node

2021-02-18 Thread Steen Hegelund
Add Sparx5 serdes driver node, and enable it generally for all reference boards. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Andrew Lunn Reviewed-by: Alexandre Belloni --- arch/arm64/boot/dts/microchip/sparx5.dtsi | 8 1 file changed, 8 insertions(+) diff

[PATCH v15 2/4] phy: Add media type and speed serdes configuration interfaces

2021-02-18 Thread Steen Hegelund
Provide new phy configuration interfaces for media type and speed that allows e.g. PHYs used for ethernet to be configured with this information. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Andrew Lunn Reviewed-by: Alexandre Belloni --- drivers/phy/phy-core.c | 30

Re: [PATCH v14 2/4] phy: Add media type and speed serdes configuration interfaces

2021-02-18 Thread Steen Hegelund
Hi Kishon, On Tue, 2021-02-16 at 15:54 +0530, Kishon Vijay Abraham I wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > Hi, > > On 16/02/21 2:07 pm, Steen Hegelund wrote: > > Hi Andrew and Kishon, > > > > On Mon, 2021-02-15 at 15:07 +0100,

Re: [PATCH v15 0/4] Adding the Sparx5 Serdes driver

2021-02-18 Thread David Miller
From: Steen Hegelund Date: Thu, 18 Feb 2021 17:14:47 +0100 > Adding the Sparx5 Serdes driver > The net-next tree is closed so this will have to wait until the next merge window. Thanks.

[PATCH net] net: phy: micrel: set soft_reset callback to genphy_soft_reset for KSZ8081

2021-02-18 Thread Christian Melki
Following a similar reinstate for the KSZ9031. Older kernels would use the genphy_soft_reset if the PHY did not implement a .soft_reset. Bluntly removing that default may expose a lot of situations where various PHYs/board implementations won't recover on various changes. Like with implement

Re: [PATCH net-next v2 0/2] net: phy: at803x: paging support

2021-02-18 Thread David Miller
From: Michael Walle Date: Thu, 18 Feb 2021 20:46:10 +0100 > Am 2021-02-18 20:26, schrieb Vladimir Oltean: >> On Thu, Feb 18, 2021 at 07:52:38PM +0100, Michael Walle wrote: >>> Add paging support to the QCA AR8031/33 PHY. This will be needed if we >>> add support for the .config_inband_aneg callba

linux-next: build warnings after merge of the net-next tree

2021-02-18 Thread Stephen Rothwell
Hi all, After merging the net-next tree, today's linux-next build (htmldocs) produced these warnings: Documentation/networking/filter.rst:1053: WARNING: Inline emphasis start-string without end-string. Documentation/networking/filter.rst:1053: WARNING: Inline emphasis start-string without end-s

[PATCH v8 bpf-next 5/5] xsk: build skb by page (aka generic zerocopy xmit)

2021-02-18 Thread Alexander Lobakin
From: Xuan Zhuo This patch is used to construct skb based on page to save memory copy overhead. This function is implemented based on IFF_TX_SKB_NO_LINEAR. Only the network card priv_flags supports IFF_TX_SKB_NO_LINEAR will use page to directly construct skb. If this feature is not supported, it

[PATCH v8 bpf-next 4/5] xsk: respect device's headroom and tailroom on generic xmit path

2021-02-18 Thread Alexander Lobakin
xsk_generic_xmit() allocates a new skb and then queues it for xmitting. The size of new skb's headroom is desc->len, so it comes to the driver/device with no reserved headroom and/or tailroom. Lots of drivers need some headroom (and sometimes tailroom) to prepend (and/or append) some headers or dat

[PATCH v8 bpf-next 3/5] virtio-net: support IFF_TX_SKB_NO_LINEAR

2021-02-18 Thread Alexander Lobakin
From: Xuan Zhuo Virtio net supports the case where the skb linear space is empty, so add priv_flags. Signed-off-by: Xuan Zhuo Acked-by: Michael S. Tsirkin Signed-off-by: Alexander Lobakin Acked-by: John Fastabend --- drivers/net/virtio_net.c | 3 ++- 1 file changed, 2 insertions(+), 1 delet

[PATCH v8 bpf-next 2/5] net: add priv_flags for allow tx skb without linear

2021-02-18 Thread Alexander Lobakin
From: Xuan Zhuo In some cases, we hope to construct skb directly based on the existing memory without copying data. In this case, the page will be placed directly in the skb, and the linear space of skb is empty. But unfortunately, many the network card does not support this operation. For exampl

[PATCH v8 bpf-next 1/5] netdevice: add missing IFF_PHONY_HEADROOM self-definition

2021-02-18 Thread Alexander Lobakin
This is harmless for now, but can be fatal for future refactors. Fixes: 871b642adebe3 ("netdev: introduce ndo_set_rx_headroom") Signed-off-by: Alexander Lobakin Acked-by: John Fastabend --- include/linux/netdevice.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/netdevice.h b

[PATCH v8 bpf-next 0/5] xsk: build skb by page (aka generic zerocopy xmit)

2021-02-18 Thread Alexander Lobakin
This series introduces XSK generic zerocopy xmit by adding XSK umem pages as skb frags instead of copying data to linear space. The only requirement for this for drivers is to be able to xmit skbs with skb_headlen(skb) == 0, i.e. all data including hard headers starts from frag 0. To indicate wheth

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
On Thu, Feb 18, 2021 at 9:37 PM Willem de Bruijn wrote: > > On Thu, Feb 18, 2021 at 3:25 PM Jason A. Donenfeld wrote: > > > > On Thu, Feb 18, 2021 at 9:16 PM Willem de Bruijn > > wrote: > > > > > > On Thu, Feb 18, 2021 at 12:58 PM Jason A. Donenfeld > > > wrote: > > > > > > > > On Thu, Feb 18,

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 3:25 PM Jason A. Donenfeld wrote: > > On Thu, Feb 18, 2021 at 9:16 PM Willem de Bruijn > wrote: > > > > On Thu, Feb 18, 2021 at 12:58 PM Jason A. Donenfeld wrote: > > > > > > On Thu, Feb 18, 2021 at 5:34 PM Willem de Bruijn > > > wrote: > > > > Thanks for respinning. > >

[PATCH net v3] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
The icmp{,v6}_send functions make all sorts of use of skb->cb, casting it with IPCB or IP6CB, assuming the skb to have come directly from the inet layer. But when the packet comes from the ndo layer, especially when forwarded, there's no telling what might be in skb->cb at that point. As a result,

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
On Thu, Feb 18, 2021 at 9:16 PM Willem de Bruijn wrote: > > On Thu, Feb 18, 2021 at 12:58 PM Jason A. Donenfeld wrote: > > > > On Thu, Feb 18, 2021 at 5:34 PM Willem de Bruijn > > wrote: > > > Thanks for respinning. > > > > > > Making ipv4 and ipv6 more aligned is a good goal, but more for > > >

Re: [PATCH net-next RFC v4] net: hdlc_x25: Queue outgoing LAPB frames

2021-02-18 Thread Xie He
On Thu, Feb 18, 2021 at 12:06 PM Xie He wrote: > > On Thu, Feb 18, 2021 at 11:55 AM Leon Romanovsky wrote: > > > > This is how we write code, we use defines instead of constant numbers, > > comments to describe tricky parts and assign already preprocessed result. > > > > There is nothing I can do

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 12:58 PM Jason A. Donenfeld wrote: > > On Thu, Feb 18, 2021 at 5:34 PM Willem de Bruijn > wrote: > > Thanks for respinning. > > > > Making ipv4 and ipv6 more aligned is a good goal, but more for > > net-next than bug fixes that need to be backported to many stable > > bran

Re: [PATCH net-next RFC v4] net: hdlc_x25: Queue outgoing LAPB frames

2021-02-18 Thread Xie He
On Thu, Feb 18, 2021 at 11:55 AM Leon Romanovsky wrote: > > This is how we write code, we use defines instead of constant numbers, > comments to describe tricky parts and assign already preprocessed result. > > There is nothing I can do If you don't like or don't want to use Linux kernel > style.

Re: BUG: unable to handle kernel paging request in nl802154_del_llsec_key

2021-02-18 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:3af409ca net: enetc: fix destroyed phylink dereference dur.. git tree: net console output: https://syzkaller.appspot.com/x/log.txt?x=12ee9f72d0 kernel config: https://syzkaller.appspot.com/x/.config?x=8cb23303ddb9

Re: [PATCH net-next RFC v4] net: hdlc_x25: Queue outgoing LAPB frames

2021-02-18 Thread Leon Romanovsky
On Thu, Feb 18, 2021 at 09:36:54AM -0800, Xie He wrote: > On Thu, Feb 18, 2021 at 2:37 AM Leon Romanovsky wrote: > > > > It is not me who didn't explain, it is you who didn't want to write clear > > comment that describes the headroom size without need of "3 - 1". > > Why do I need to write unnece

Re: [PATCHv3 1/3] usbnet: specify naming of usbnet_set/get_link_ksettings

2021-02-18 Thread Andrew Lunn
On Thu, Feb 18, 2021 at 07:31:41PM +, Grant Grundler wrote: > Oliver, Jakub, > Can I post v4 and deal with the issues below? You should probably wait for two weeks. We are far enough into the merge window that i doubt it will get picked up. So please wait, rebase, and then post. > Nit: The v2

Re: [PATCH v7 bpf-next 0/6] xsk: build skb by page (aka generic zerocopy xmit)

2021-02-18 Thread Jakub Kicinski
On Wed, 17 Feb 2021 22:08:55 -0800 John Fastabend wrote: > > Performance Testing > > > > The test environment is Aliyun ECS server. > > Test cmd: > > ``` > > xdpsock -i eth0 -t -S -s > > ``` > > > > Test result data: > > > > size64 512 10241500 >

Re: [PATCH net-next v2 0/2] net: phy: at803x: paging support

2021-02-18 Thread Michael Walle
Am 2021-02-18 20:26, schrieb Vladimir Oltean: On Thu, Feb 18, 2021 at 07:52:38PM +0100, Michael Walle wrote: Add paging support to the QCA AR8031/33 PHY. This will be needed if we add support for the .config_inband_aneg callback, see series [1]. The driver itself already accesses the fiber page

Re: [PATCHv3 1/3] usbnet: specify naming of usbnet_set/get_link_ksettings

2021-02-18 Thread Grant Grundler
Oliver, Jakub, Can I post v4 and deal with the issues below? I can compile and test against ToT for now and I'll like to get this mess behind me. I still think the first patch in this series should revert my previous change (de658a195ee23ca6aaffe197d1d2ea040beea0a2). On Thu, Feb 18, 2021 at 10:2

Re: [PATCH net-next v2 0/2] net: phy: at803x: paging support

2021-02-18 Thread Vladimir Oltean
On Thu, Feb 18, 2021 at 07:52:38PM +0100, Michael Walle wrote: > Add paging support to the QCA AR8031/33 PHY. This will be needed if we > add support for the .config_inband_aneg callback, see series [1]. > > The driver itself already accesses the fiber page (without proper locking). > The former ve

Re: [PATCH bpf-next 00/17] Improve BPF syscall command documentation

2021-02-18 Thread Jonathan Corbet
Joe Stringer writes: > Hey Jon, thanks for the feedback. Absolutely, what you say makes > sense. The intent here wasn't to come up with something new. Based on > your prompt from this email (and a quick look at your KR '19 > presentation), I'm hearing a few observations: > * Storing the documenta

Re: [PATCH net] net: icmp: zero-out cb in icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
On Thu, Feb 18, 2021 at 8:08 PM Jakub Kicinski wrote: > > On Thu, 18 Feb 2021 13:30:53 +0100 Jason A. Donenfeld wrote: > > The icmp{,v6}_send functions make all sorts of use of skb->cb, assuming > > the skb to have come directly from the inet layer. But when the packet > > comes from the ndo layer

BUG: unable to handle kernel paging request in nl802154_del_llsec_key

2021-02-18 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:f40ddce8 Linux 5.11 git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=11261a4cd0 kernel config: https://syzkaller.appspot.com/x/.config?x=67894355b1dbeb07 dashboard link: https://syzkaller.appspot.co

Re: [PATCH net] net: icmp: zero-out cb in icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jakub Kicinski
On Thu, 18 Feb 2021 13:30:53 +0100 Jason A. Donenfeld wrote: > The icmp{,v6}_send functions make all sorts of use of skb->cb, assuming > the skb to have come directly from the inet layer. But when the packet > comes from the ndo layer, especially when forwarded, there's no telling > what might be i

Re: [PATCHv3 2/3] usbnet: add method for reporting speed without MDIO

2021-02-18 Thread Jakub Kicinski
On Thu, 18 Feb 2021 11:20:37 +0100 Oliver Neukum wrote: > The old method for reporting network speed upwards > assumed that a device uses MDIO and uses the generic phy > functions based on that. > Add a a primitive internal version not making the assumption > reporting back directly what the status

Re: [PATCHv3 1/3] usbnet: specify naming of usbnet_set/get_link_ksettings

2021-02-18 Thread Jakub Kicinski
On Thu, 18 Feb 2021 11:20:36 +0100 Oliver Neukum wrote: > The old generic functions assume that the devices includes > an MDIO interface. This is true only for genuine ethernet. > Devices with a higher level of abstraction or based on different > technologies do not have it. So in preparation for >

RE: [RFC PATCH v3 00/11] NVMeTCP Offload ULP and QEDN Device Driver

2021-02-18 Thread Shai Malin
> > With the goal of enabling a generic infrastructure that allows NVMe/TCP > offload devices like NICs to seamlessly plug into the NVMe-oF stack, this > patch series introduces the nvme-tcp-offload ULP host layer, which will be a > new transport type called "tcp-offload" and will serve as an abst

[PATCH net-next v2 2/2] net: phy: at803x: remove at803x_aneg_done()

2021-02-18 Thread Michael Walle
at803x_aneg_done() is pretty much dead code since the patch series "net: phy: improve and simplify phylib state machine" [1]. Just remove it. [1] https://lore.kernel.org/netdev/922c223b-7bc0-e0ec-345d-2034b796a...@gmail.com/ Suggested-by: Vladimir Oltean Signed-off-by: Michael Walle --- drive

Re: [net-next PATCH v6 02/15] net: phy: Introduce fwnode_mdio_find_device()

2021-02-18 Thread Jakub Kicinski
On Thu, 18 Feb 2021 10:56:41 +0530 Calvin Johnson wrote: > +/** > + * fwnode_mdio_find_device - Given a fwnode, find the mdio_device > + * @np: pointer to the mdio_device's fwnode > + * > + * If successful, returns a pointer to the mdio_device with the embedded > + * struct device refcount incremen

[PATCH net-next v2 1/2] net: phy: at803x: add pages support to AR8031/33

2021-02-18 Thread Michael Walle
The AR8031 has two register sets: Copper and Fiber. The fiber page is used in case of 100Base-FX and 1000Base-X. But more importantly it is also used for the SGMII link. Add support to switch between these two. Signed-off-by: Michael Walle --- drivers/net/phy/at803x.c | 35 ++

[PATCH net-next v2 0/2] net: phy: at803x: paging support

2021-02-18 Thread Michael Walle
Add paging support to the QCA AR8031/33 PHY. This will be needed if we add support for the .config_inband_aneg callback, see series [1]. The driver itself already accesses the fiber page (without proper locking). The former version of this patchset converted the access to phy_read_paged(), but Vla

Re: linux-next: manual merge of the net-next tree with the net tree

2021-02-18 Thread Sukadev Bhattiprolu
Stephen Rothwell [s...@canb.auug.org.au] wrote: > Hi all, > > Today's linux-next merge of the net-next tree got conflicts in: > > drivers/net/ethernet/ibm/ibmvnic.c > drivers/net/ethernet/ibm/ibmvnic.h > > between commit: > > 4a41c421f367 ("ibmvnic: serialize access to work queue on remov

[PATCH net] dpaa_eth: fix the access method for the dpaa_napi_portal

2021-02-18 Thread Camelia Groza
The current use of container_of is flawed and unnecessary. Obtain the dpaa_napi_portal reference from the private percpu data instead. Fixes: a1e031ffb422 ("dpaa_eth: add XDP_REDIRECT support") Reported-by: Sascha Hauer Signed-off-by: Camelia Groza --- drivers/net/ethernet/freescale/dpaa/dpaa_e

Re: possible deadlock in inet_stream_connect

2021-02-18 Thread syzbot
syzbot has bisected this issue to: commit 40947e13997a1cba4e875893ca6e5d5e61a0689d Author: Florian Westphal Date: Fri Feb 12 23:59:56 2021 + mptcp: schedule worker when subflow is closed bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=109b88f2d0 start commit: 9ec5ee

Re: [PATCH] kcov: Remove kcov include from sched.h and move it to its users.

2021-02-18 Thread Johannes Berg
On Thu, 2021-02-18 at 18:31 +0100, Sebastian Andrzej Siewior wrote: > The recent addition of in_serving_softirq() to kconv.h results in You typo'ed "kconv.h" pretty consistently ;-) But yes, that makes sense. Acked-by: Johannes Berg johannes

Re: [PATCH net-next RFC v4] net: hdlc_x25: Queue outgoing LAPB frames

2021-02-18 Thread Xie He
On Thu, Feb 18, 2021 at 2:37 AM Leon Romanovsky wrote: > > It is not me who didn't explain, it is you who didn't want to write clear > comment that describes the headroom size without need of "3 - 1". Why do I need to write unnecessary comments when "3 - 1" and the current comment already explain

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
On Thu, Feb 18, 2021 at 5:34 PM Willem de Bruijn wrote: > Thanks for respinning. > > Making ipv4 and ipv6 more aligned is a good goal, but more for > net-next than bug fixes that need to be backported to many stable > branches. > > Beyond that, I'm not sure this fixes additional cases vs the previ

[PATCH] kcov: Remove kcov include from sched.h and move it to its users.

2021-02-18 Thread Sebastian Andrzej Siewior
The recent addition of in_serving_softirq() to kconv.h results in compile failure on PREEMPT_RT because it requires task_struct::softirq_disable_cnt. This is not available if kconv.h is included from sched.h. It is not needed to include kconv.h from sched.h. All but the net/ user already include t

Re: [PATCH net] net: icmp: zero-out cb in icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 10:40 AM Jason A. Donenfeld wrote: > > Hi Willem, > > On Thu, Feb 18, 2021 at 3:57 PM Willem de Bruijn > wrote: > > > > On Thu, Feb 18, 2021 at 7:31 AM Jason A. Donenfeld wrote: > > > > > > The icmp{,v6}_send functions make all sorts of use of skb->cb, assuming > > > > In

UBSAN: shift-out-of-bounds in netlink_recvmsg

2021-02-18 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:b646acd5 net: re-solve some conflicts after net -> net-nex.. git tree: net-next console output: https://syzkaller.appspot.com/x/log.txt?x=129fbe04d0 kernel config: https://syzkaller.appspot.com/x/.config?x=dbc1ca9e55dc1f9f das

[PATCH net-next] net: mscc: Fix MRP switchdev driver

2021-02-18 Thread Horatiu Vultur
This patch fixes the ocelot MRP switchdev driver such that also DSA driver can use these functions. Before the driver presumed that the net_device uses a 'struct ocelot_port_private' as priv which was wrong. The only reason for using ocelot_port_private was to access the net_device, but this can b

general protection fault in nl802154_del_llsec_dev

2021-02-18 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:57baf8cc net: axienet: Handle deferred probe on clock prop.. git tree: net console output: https://syzkaller.appspot.com/x/log.txt?x=16d83be2d0 kernel config: https://syzkaller.appspot.com/x/.config?x=8cb23303ddb9411f dashboar

Re: bug report: WARNING in bonding

2021-02-18 Thread Tariq Toukan
On 11/12/2020 6:33 PM, Ido Schimmel wrote: On Thu, Nov 12, 2020 at 05:54:30PM +0200, Tariq Toukan wrote: On 11/12/2020 5:46 PM, Ido Schimmel wrote: On Thu, Nov 12, 2020 at 05:38:44PM +0200, Tariq Toukan wrote: Hi all, In the past ~2-3 weeks, we started seeing the following WARNING and tr

Re: [PATCH] net: hsr: add support for EntryForgetTime

2021-02-18 Thread George McCollister
On Thu, Feb 18, 2021 at 9:01 AM Marco Wenzel wrote: > > In IEC 62439-3 EntryForgetTime is defined with a value of 400 ms. When a > node does not send any frame within this time, the sequence number check > for can be ignored. This solves communication issues with Cisco IE 2000 > in Redbox mode. >

[PATCH/v2] bpf: add bpf_skb_adjust_room flag BPF_F_ADJ_ROOM_ENCAP_L2_ETH

2021-02-18 Thread Xuesen Huang
From: Xuesen Huang bpf_skb_adjust_room sets the inner_protocol as skb->protocol for packets encapsulation. But that is not appropriate when pushing Ethernet header. Add an option to further specify encap L2 type and set the inner_protocol as ETH_P_TEB. Suggested-by: Willem de Bruijn Signed-off

Re: [PATCH/v2] bpf: add bpf_skb_adjust_room flag BPF_F_ADJ_ROOM_ENCAP_L2_ETH

2021-02-18 Thread 黄学森
Thanks Daniel and Willem! So sorry to reply to you late for I just took the Chinese Spring Festival vacation. I will resubmit this patch. Thanks again! > 2021年2月11日 下午11:26,Daniel Borkmann 写道: > > On 2/10/21 3:50 PM, Willem de Bruijn wrote: >> On Wed, Feb 10, 2021 at 1:59 AM huangxuesen wro

Re: [PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 11:08 AM Jason A. Donenfeld wrote: > > The icmp{,v6}_send functions make all sorts of use of skb->cb, casting > it with IPCB or IP6CB, assuming the skb to have come directly from the > inet layer. But when the packet comes from the ndo layer, especially > when forwarded, th

RE: [PATCH net-next] misc: Add Renesas Synchronization Management Unit (SMU) support

2021-02-18 Thread Min Li
> -Original Message- > From: Arnd Bergmann > Sent: February 18, 2021 5:51 AM > To: Min Li > Cc: Derek Kiernan ; Dragan Cvetic > ; Arnd Bergmann ; gregkh > ; linux-ker...@vger.kernel.org; Networking > ; Richard Cochran > Subject: Re: [PATCH net-next] misc: Add Renesas Synchronization >

[PATCH net v2] net: icmp: pass zeroed opts from icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
The icmp{,v6}_send functions make all sorts of use of skb->cb, casting it with IPCB or IP6CB, assuming the skb to have come directly from the inet layer. But when the packet comes from the ndo layer, especially when forwarded, there's no telling what might be in skb->cb at that point. As a result,

Re: [PATCH v2 04/24] dt-bindings: net: dwmac: Refactor snps,*-config properties

2021-02-18 Thread Serge Semin
On Thu, Feb 11, 2021 at 12:58:00AM +0300, Serge Semin wrote: > On Tue, Feb 09, 2021 at 04:26:08PM -0600, Rob Herring wrote: > > On Mon, Feb 08, 2021 at 04:55:48PM +0300, Serge Semin wrote: > > > Currently the "snps,axi-config", "snps,mtl-rx-config" and > > > "snps,mtl-tx-config" properties are decl

Re: [PATCH 01/16] dt-bindings: net: dwmac: Add DW GMAC GPIOs properties

2021-02-18 Thread Serge Semin
On Thu, Feb 11, 2021 at 01:28:06AM +0300, Serge Semin wrote: > On Tue, Feb 09, 2021 at 05:13:52PM -0600, Rob Herring wrote: > > On Mon, Feb 08, 2021 at 05:08:05PM +0300, Serge Semin wrote: > > > Synopsys DesignWare Ethernet controllers can be synthesized with > > > General-Purpose IOs support. GPIO

Re: [PATCH] net: check if protocol extracted by virtio_net_hdr_set_proto is correct

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 10:01 AM Balazs Nemeth wrote: > > For gso packets, virtio_net_hdr_set_proto sets the protocol (if it isn't > set) based on the type in the virtio net hdr, but the skb could contain > anything since it could come from packet_snd through a raw socket. If > there is a mismatch

Re: [PATCH net] net: icmp: zero-out cb in icmp{,v6}_ndo_send before sending

2021-02-18 Thread Jason A. Donenfeld
Hi Willem, On Thu, Feb 18, 2021 at 3:57 PM Willem de Bruijn wrote: > > On Thu, Feb 18, 2021 at 7:31 AM Jason A. Donenfeld wrote: > > > > The icmp{,v6}_send functions make all sorts of use of skb->cb, assuming > > Indeed that also casts skb->cb, to read IP6CB(skb)->iif, good catch. > > Still, mig

RE: [PATCH v10 01/20] dlb: add skeleton for DLB driver

2021-02-18 Thread Chen, Mike Ximing
> -Original Message- > From: gre...@linuxfoundation.org > Sent: Thursday, February 18, 2021 2:53 AM > To: Chen, Mike Ximing > Cc: netdev@vger.kernel.org; Linux Kernel Mailing List ker...@vger.kernel.org>; da...@davemloft.net; k...@kernel.org; a...@arndb.de; > Williams, Dan J ; > pier

Re: [net-next PATCH v6 10/15] net: mdio: Add ACPI support code for mdio

2021-02-18 Thread Andy Shevchenko
On Thu, Feb 18, 2021 at 7:28 AM Calvin Johnson wrote: > > Define acpi_mdiobus_register() to Register mii_bus and create PHYs for > each ACPI child node. > +#include > +#include Perhaps it's better to provide the headers that this file is direct user of, i.e. bits.h dev_printk.h module.h ty

Re: [net-next] net: mvpp2: skip RSS configurations on loopback port

2021-02-18 Thread Marcin Wojtas
Hi, czw., 18 lut 2021 o 13:42 napisał(a): > > From: Stefan Chulski > > PPv2 loopback port doesn't support RSS, so we should > skip RSS configurations for this port. > > Signed-off-by: Stefan Chulski > --- > drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 25 +++- > 1 file ch

Re: [net-next PATCH v6 15/15] net: dpaa2-mac: Add ACPI support for DPAA2 MAC driver

2021-02-18 Thread Andy Shevchenko
On Thu, Feb 18, 2021 at 7:29 AM Calvin Johnson wrote: > > Modify dpaa2_mac_get_node() to get the dpmac fwnode from either > DT or ACPI. > > Modify dpaa2_mac_get_if_mode() to get interface mode from dpmac_node > which is a fwnode. > > Modify dpaa2_pcs_create() to create pcs from dpmac_node fwnode.

[PATCH] net: check if protocol extracted by virtio_net_hdr_set_proto is correct

2021-02-18 Thread Balazs Nemeth
For gso packets, virtio_net_hdr_set_proto sets the protocol (if it isn't set) based on the type in the virtio net hdr, but the skb could contain anything since it could come from packet_snd through a raw socket. If there is a mismatch between what virtio_net_hdr_set_proto sets and the actual protoc

[PATCH] net: hsr: add support for EntryForgetTime

2021-02-18 Thread Marco Wenzel
In IEC 62439-3 EntryForgetTime is defined with a value of 400 ms. When a node does not send any frame within this time, the sequence number check for can be ignored. This solves communication issues with Cisco IE 2000 in Redbox mode. Fixes: f421436a591d ("net/hsr: Add support for the High-availabi

Re: [PATCH net] net: icmp: zero-out cb in icmp{,v6}_ndo_send before sending

2021-02-18 Thread Willem de Bruijn
On Thu, Feb 18, 2021 at 7:31 AM Jason A. Donenfeld wrote: > > The icmp{,v6}_send functions make all sorts of use of skb->cb, assuming Indeed that also casts skb->cb, to read IP6CB(skb)->iif, good catch. Still, might be good to more precisely detail the relevant bug: icmp_send casts the cb to an

Re: [net-next PATCH v6 07/15] net: mdiobus: Introduce fwnode_mdiobus_register_phy()

2021-02-18 Thread Andy Shevchenko
On Thu, Feb 18, 2021 at 7:28 AM Calvin Johnson wrote: > > Introduce fwnode_mdiobus_register_phy() to register PHYs on the > mdiobus. From the compatible string, identify whether the PHY is > c45 and based on this create a PHY device instance which is > registered on the mdiobus. Thanks for an upd

Re: [PATCH v1] vdpa/mlx5: Restore the hardware used index after change map

2021-02-18 Thread Si-Wei Liu
On 2/17/2021 8:44 PM, Jason Wang wrote: On 2021/2/10 下午4:59, Si-Wei Liu wrote: On 2/9/2021 7:53 PM, Jason Wang wrote: On 2021/2/10 上午10:30, Si-Wei Liu wrote: On 2/8/2021 10:37 PM, Jason Wang wrote: On 2021/2/9 下午2:12, Eli Cohen wrote: On Tue, Feb 09, 2021 at 11:20:14AM +0800, Jason

Re: WARNING in dst_release

2021-02-18 Thread Florian Westphal
syzbot wrote: > Hello, > > syzbot has tested the proposed patch and the reproducer did not trigger any > issue: > > Reported-and-tested-by: syzbot+b53bbea2ad64f9cf8...@syzkaller.appspotmail.com #syz-fix: mptcp: reset last_snd on subflow close [ This patch is currently in mptcp-next ]

Re: [PATCH] Revert "ath9k: fix ath_tx_process_buffer() potential null ptr dereference"

2021-02-18 Thread Shuah Khan
On 2/17/21 11:23 PM, Kalle Valo wrote: Shuah Khan wrote: This reverts commit a56c14bb21b296fb6d395164ab62ef2e419e5069. ath_tx_process_buffer() doesn't dereference or check sta and passes it to ath_tx_complete_aggr() and ath_tx_complete_buf(). ath_tx_complete_aggr() checks the pointer before

RE: [PATCH] Revert "dpaa_eth: add XDP_REDIRECT support"

2021-02-18 Thread Camelia Alexandra Groza
> -Original Message- > From: Jesper Dangaard Brouer > Sent: Thursday, February 18, 2021 13:43 > To: Sascha Hauer > Cc: bro...@redhat.com; netdev@vger.kernel.org; Camelia Alexandra Groza > ; Madalin Bucur ; > Jakub Kicinski ; ker...@pengutronix.de; Ioana Ciornei > ; Ioana Ciocoi Radulescu

Re: [PATCH net-next] net: mscc: Fix MRP switchdev driver

2021-02-18 Thread Vladimir Oltean
Hi Horatiu, On Thu, Feb 18, 2021 at 12:47:26PM +0100, Horatiu Vultur wrote: > This patch fixes the ocelot MRP switchdev driver such that also DSA > driver can use these functions. Before the driver presumed that the > net_device uses a 'struct ocelot_port_private' as priv which was wrong. > > The

Re: [PATCH] perf tools: Simplify the calculation of variables

2021-02-18 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 05, 2021 at 10:49:15AM +0100, Jiri Olsa escreveu: > On Fri, Feb 05, 2021 at 11:54:15AM +0800, Jiapeng Chong wrote: > > Fix the following coccicheck warnings: > > > > ./tools/perf/util/header.c:3809:18-20: WARNING !A || A && B is > > equivalent to !A || B. > > > > Reported-by: Abaci Ro

  1   2   >