[PATCH net-next v2 2/4] net: Introduce {netdev,napi}_alloc_frag_align()

2021-01-30 Thread Kevin Hao
In the current implementation of {netdev,napi}_alloc_frag(), it doesn't have any align guarantee for the returned buffer address, But for some hardwares they do require the DMA buffer to be aligned correctly, so we would have to use some workarounds like below if the buffers allocated by the {netde

[PATCH net-next v2 4/4] net: dpaa2: Use napi_alloc_frag_align() to avoid the memory waste

2021-01-30 Thread Kevin Hao
The napi_alloc_frag_align() will guarantee that a correctly align buffer address is returned. So use this function to simplify the buffer alloc and avoid the unnecessary memory waste. Signed-off-by: Kevin Hao --- v2: No change. drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c | 3 +-- 1 file ch

[PATCH net-next v2 1/4] mm: page_frag: Introduce page_frag_alloc_align()

2021-01-30 Thread Kevin Hao
In the current implementation of page_frag_alloc(), it doesn't have any align guarantee for the returned buffer address. But for some hardwares they do require the DMA buffer to be aligned correctly, so we would have to use some workarounds like below if the buffers allocated by the page_frag_alloc

[PATCH net-next v2 0/4] net: Avoid the memory waste in some Ethernet drivers

2021-01-30 Thread Kevin Hao
Hi, v2: - Inline page_frag_alloc() and {netdev,napi}_alloc_frag() - Adopt Vlastimil's suggestion and add his Acked-by In the current implementation of napi_alloc_frag(), it doesn't have any align guarantee for the returned buffer address. We would have to use some ugly workarounds to make sur

Re: [PATCH v2 net-next 06/21] nvme-tcp: Add DDP offload control path

2021-01-30 Thread Boris Pismenny
On 19/01/2021 5:47, David Ahern wrote: > On 1/14/21 8:10 AM, Boris Pismenny wrote: >> +static >> +int nvme_tcp_offload_socket(struct nvme_tcp_queue *queue) >> +{ >> +struct net_device *netdev = get_netdev_for_sock(queue->sock->sk, true); >> +struct nvme_tcp_ddp_config config = {}; >> +i

Re: [Patch v2 net-next 2/7] octeontx2-af: Add new CGX_CMD to get PHY FEC statistics

2021-01-30 Thread Hariprasad Kelam
Hi Willem, > -Original Message- > From: Willem de Bruijn > Sent: Saturday, January 30, 2021 7:57 PM > To: Hariprasad Kelam > Cc: Network Development ; LKML ker...@vger.kernel.org>; David Miller ; Jakub > Kicinski ; Sunil Kovvuri Goutham > ; Linu Cherian ; > Geethasowjanya Akula ; Jerin

Re: [PATCH] Add documentation of ss filter to man page

2021-01-30 Thread Thayne McCombs
Oh. I think I should have added "[PATCH iproute2-next] ss:" to my subject line. Sorry, this was my first post to this list.

Re: [PATCH net-next 9/9] net: ipa: don't disable NAPI in suspend

2021-01-30 Thread Alex Elder
On 1/30/21 1:22 PM, Jakub Kicinski wrote: On Sat, 30 Jan 2021 10:25:16 -0500 Willem de Bruijn wrote: @@ -894,12 +894,16 @@ int gsi_channel_start(struct gsi *gsi, u32 channel_id) struct gsi_channel *channel = &gsi->channel[channel_id]; int ret; - /* Enable the completion

Re: [PATCH net-next 9/9] net: ipa: don't disable NAPI in suspend

2021-01-30 Thread Alex Elder
On 1/30/21 9:25 AM, Willem de Bruijn wrote: On Fri, Jan 29, 2021 at 3:29 PM Alex Elder wrote: The channel stop and suspend paths both call __gsi_channel_stop(), which quiesces channel activity, disables NAPI, and (on other than SDM845) stops the channel. Similarly, the start and resume paths

Re: [PATCH net] net: hdlc_x25: Use qdisc to queue outgoing LAPB frames

2021-01-30 Thread Xie He
On Sat, Jan 30, 2021 at 11:16 AM Jakub Kicinski wrote: > > Sounds like too much afford for a sub-optimal workaround. > The qdisc semantics are borken in the proposed scheme (double > counting packets) - both in term of statistics and if user decides > to add a policer, filter etc. Hmm... Another

[Patch net-next v2] net: fix dev_ifsioc_locked() race condition

2021-01-30 Thread Cong Wang
From: Cong Wang dev_ifsioc_locked() is called with only RCU read lock, so when there is a parallel writer changing the mac address, it could get a partially updated mac address, as shown below: Thread 1Thread 2 // eth_commit_mac_addr_change() memcpy(dev->dev_addr, addr->s

RE: [PATCH net-next v1 1/6] lan743x: boost performance on cpu archs w/o dma cache snooping

2021-01-30 Thread Bryan.Whitehead
Sven, see below comments > @@ -2148,11 +2149,18 @@ static int lan743x_rx_process_packet(struct > lan743x_rx *rx) > descriptor = &rx->ring_cpu_ptr[first_index]; > > /* unmap from dma */ > + packet_length = RX_DESC_DATA0_FRAME_LE

Re: [PATCH net] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

2021-01-30 Thread DENG Qingfang
On Sun, Jan 31, 2021 at 8:39 AM Vladimir Oltean wrote: > > Tobias has a point in a way too, you should get used to adding the > 'master static' flags to your bridge fdb commands, otherwise weird > things like this could happen. The faulty code can only be triggered > when going through dsa_legacy_

Re: [PATCH net] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

2021-01-30 Thread Vladimir Oltean
On Sat, Jan 30, 2021 at 09:43:34PM +0800, DENG Qingfang wrote: > Having multiple destination ports for a unicast address does not make > sense. > Make port_db_load_purge override existent unicast portvec instead of > adding a new port bit. > > Fixes: 884729399260 ("net: dsa: mv88e6xxx: handle mult

Re: [PATCH net] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

2021-01-30 Thread Vladimir Oltean
On Sat, Jan 30, 2021 at 09:47:02PM +0100, Tobias Waldekranz wrote: > root@envoy:~# bridge fdb add 02:00:de:ad:00:01 dev eth1 static vlan 1 > Why does the second add operation succeed? Am I missing some magic flag? Yes, 'master'. We talked about this before. 'bridge fdb add' is implicitly 'self' wh

Re: [PATCH net-next v1 1/6] lan743x: boost performance on cpu archs w/o dma cache snooping

2021-01-30 Thread Sven Van Asbroeck
On Sat, Jan 30, 2021 at 5:11 PM wrote: > > It appears you moved this packet_length assignment from just below the > following if block, however you left out the le32_to_cpu.See next comment PS this merge snafu is removed completely by the next patch in the set. So this will not prevent you from

[PATCH 6/9] net: usb: lan78xx: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/usb/lan78xx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/lan78xx.

[PATCH 5/9] net: usb: hso: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/usb/hso.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/net/usb/hso.c b

[PATCH 1/9] arcnet: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/arcnet/arcnet.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/net/arcnet/ar

[PATCH 0/9] drivers: net: update tasklet_init callers

2021-01-30 Thread Emil Renner Berthing
This updates the remaining callers of tasklet_init() in drivers/net to the new API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") All changes are done by coccinelle using the following semantic patch. Coccinelle needs a little help parsing drivers/net/arcnet/arcne

[PATCH 9/9] net: usb: rtl8150: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/usb/rtl8150.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/usb/rtl8150.

[PATCH 8/9] net: usb: r8152: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/usb/r8152.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/net/usb/r8152.c

[PATCH 7/9] net: usb: pegasus: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/usb/pegasus.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/net/usb/pegasus

Re: [PATCH net-next v1 1/6] lan743x: boost performance on cpu archs w/o dma cache snooping

2021-01-30 Thread Sven Van Asbroeck
Hi Bryan, thank you so much for reviewing, I really appreciate it. On Sat, Jan 30, 2021 at 5:11 PM wrote: > > > /* unmap from dma */ > > + packet_length = RX_DESC_DATA0_FRAME_LENGTH_GET_ > > + (descriptor->data0);

[PATCH 4/9] ppp: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the async and synctty drivers to use the new tasklet API n commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/ppp/ppp_async.c | 8 drivers/net/ppp/ppp_synctty.c | 8 2 files changed, 8 insertions

[PATCH 3/9] ifb: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/ifb.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/net/ifb.c b/drivers/net

[PATCH 2/9] caif_virtio: use new tasklet API

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") Signed-off-by: Emil Renner Berthing --- drivers/net/caif/caif_virtio.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/net/caif/

[PATCH] net: usb: cdc_ncm: use new API for bh tasklet

2021-01-30 Thread Emil Renner Berthing
This converts the driver to use the new tasklet API introduced in commit 12cc923f1ccc ("tasklet: Introduce new initialization API") It is unfortunate that we need to add a pointer to the driver context to get back to the usbnet device, but the space will be reclaimed once there are no more users o

Re: [PATCH v2 net-next 3/4] net: introduce common dev_page_is_reserved()

2021-01-30 Thread John Hubbard
On 1/30/21 11:45 AM, Alexander Lobakin wrote: From: Jakub Kicinski Date: Sat, 30 Jan 2021 11:07:07 -0800 On Sat, 30 Jan 2021 15:42:29 + Alexander Lobakin wrote: On Wed, 27 Jan 2021 20:11:23 + Alexander Lobakin wrote: + * dev_page_is_reserved - check whether a page can be reused for

Re: [PATCH net] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

2021-01-30 Thread Tobias Waldekranz
On Sat, Jan 30, 2021 at 21:43, DENG Qingfang wrote: > Having multiple destination ports for a unicast address does not make > sense. > Make port_db_load_purge override existent unicast portvec instead of > adding a new port bit. Is this the layer we want to solve this problem at? What are the con

Re: [RFC PATCH 15/16] gtp: add ability to send GTP controls headers

2021-01-30 Thread Pravin Shelar
On Sat, Jan 30, 2021 at 10:44 AM Jakub Kicinski wrote: > > On Fri, 29 Jan 2021 22:59:06 -0800 Pravin Shelar wrote: > > On Fri, Jan 29, 2021 at 6:08 AM Jonas Bonn wrote: > > > On 28/01/2021 22:29, Pravin Shelar wrote: > > > > Receive path: LWT extracts tunnel metadata into tunnel-metadata > > > >

Re: [PATCH v2 net-next 3/4] net: introduce common dev_page_is_reserved()

2021-01-30 Thread Alexander Lobakin
From: Jakub Kicinski Date: Sat, 30 Jan 2021 11:07:07 -0800 > On Sat, 30 Jan 2021 15:42:29 + Alexander Lobakin wrote: > > > On Wed, 27 Jan 2021 20:11:23 + Alexander Lobakin wrote: > > > > + * dev_page_is_reserved - check whether a page can be reused for > > > > network Rx > > > > + * @pag

Re: [PATCH net-next 9/9] net: ipa: don't disable NAPI in suspend

2021-01-30 Thread Jakub Kicinski
On Sat, 30 Jan 2021 10:25:16 -0500 Willem de Bruijn wrote: > > @@ -894,12 +894,16 @@ int gsi_channel_start(struct gsi *gsi, u32 channel_id) > > struct gsi_channel *channel = &gsi->channel[channel_id]; > > int ret; > > > > - /* Enable the completion interrupt */ > > + /*

Re: [PATCH net] net: hdlc_x25: Use qdisc to queue outgoing LAPB frames

2021-01-30 Thread Jakub Kicinski
On Sat, 30 Jan 2021 06:29:20 -0800 Xie He wrote: > On Fri, Jan 29, 2021 at 5:36 PM Jakub Kicinski wrote: > > I'm still struggling to wrap my head around this. > > > > Did you test your code with lockdep enabled? Which Qdisc are you using? > > You're queuing the frames back to the interface they ca

Re: [PATCH v2 net-next 3/4] net: introduce common dev_page_is_reserved()

2021-01-30 Thread Jakub Kicinski
On Sat, 30 Jan 2021 15:42:29 + Alexander Lobakin wrote: > > On Wed, 27 Jan 2021 20:11:23 + Alexander Lobakin wrote: > > > + * dev_page_is_reserved - check whether a page can be reused for network > > > Rx > > > + * @page: the page to test > > > + * > > > + * A page shouldn't be considere

[PATCH net] docs: networking: swap words in icmp_errors_use_inbound_ifaddr doc

2021-01-30 Thread Vincent Bernat
Signed-off-by: Vincent Bernat --- Documentation/networking/ip-sysctl.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst index dd2b12a32b73..48d9db9151ac 100644 --- a/Documentation/networking/ip-s

Re: [RFC PATCH 15/16] gtp: add ability to send GTP controls headers

2021-01-30 Thread Jakub Kicinski
On Fri, 29 Jan 2021 22:59:06 -0800 Pravin Shelar wrote: > On Fri, Jan 29, 2021 at 6:08 AM Jonas Bonn wrote: > > On 28/01/2021 22:29, Pravin Shelar wrote: > > > Receive path: LWT extracts tunnel metadata into tunnel-metadata > > > struct. This object has 5-tuple info from outer header and tunnel

Re: [PATCH net 1/4] igc: Report speed and duplex as unknown when device is runtime suspended

2021-01-30 Thread Jakub Kicinski
On Sat, 30 Jan 2021 16:00:06 +0200 Neftin, Sasha wrote: > On 1/30/2021 08:22, Jakub Kicinski wrote: > > On Thu, 28 Jan 2021 13:38:48 -0800 Tony Nguyen wrote: > >> From: Kai-Heng Feng > >> > >> Similar to commit 165ae7a8feb5 ("igb: Report speed and duplex as unknown > >> when device is runtime su

[net-next 08/14] octeontx2-af: cn10k: Add RPM MAC support

2021-01-30 Thread Geetha sowjanya
From: Hariprasad Kelam OcteonTx2's next gen platform the CN10K has RPM MAC which has a different serdes when compared to CGX MAC. Though the underlying HW is different, the CSR interface has been designed largely inline with CGX MAC, with few exceptions though. So we are using the same CGX driver

[net-next 12/14] octeontx2-af: cn10k: Add RPM LMAC pause frame support

2021-01-30 Thread Geetha sowjanya
From: Rakesh Babu Flow control configuration is different for CGX(Octeontx2) and RPM(CN10K) functional blocks. This patch adds the necessary changes for RPM to support 802.3 pause frames configuration on cn10k platforms. Signed-off-by: Rakesh Babu Signed-off-by: Geetha sowjanya Signed-off-by:

[net-next 09/14] octeontx2-af: cn10k: Add support for programmable channels

2021-01-30 Thread Geetha sowjanya
From: Subbaraya Sundeep NIX uses unique channel numbers to identify the packet sources/sinks like CGX,LBK and SDP. The channel numbers assigned to each block are hardwired in CN9xxx silicon. The fixed channel numbers in CN9xxx are: 0x0 | a << 8 | b- LBK(0..3)_CH(0..63) 0x0 | a << 8

[net-next 11/14] octeontx2-pf: cn10k: Get max mtu supported from admin function

2021-01-30 Thread Geetha sowjanya
From: Hariprasad Kelam CN10K supports max mtu of 16K on lmac links and 64k on lbk links and Octeontx2 silicon supports 9K mtu on both links. Get the same from nix_get_hw_info mbox message in netdev probe. remove updating port field value in ethtool(get_link_ksettings) as firmware does not suppor

[net-next 13/14] octeontx2-af: cn10k: Add RPM Rx/Tx stats support

2021-01-30 Thread Geetha sowjanya
From: Hariprasad Kelam RPM supports below list of counters as an extension to existing counters * class based flow control pause frames * vlan/jabber/fragmented packets * fcs/alignment/oversized error packets This patch adds support to display supported RPM counters via debugfs and define

[net-next 10/14] octeontx2-af: cn10K: Add MTU configuration

2021-01-30 Thread Geetha sowjanya
From: Hariprasad Kelam OcteonTx3 CN10K silicon supports bigger MTU when compared to 9216 MTU supported by OcteonTx2 silicon variants. Lookback interface supports upto 64K and RPM LMAC interfaces support upto 16K. This patch does the necessary configuration and adds support for PF/VF drivers to r

[net-next 01/14] octeontx2-af: cn10k: Add mbox support for CN10K platform

2021-01-30 Thread Geetha sowjanya
Firmware allocates memory regions for PFs and VFs in DRAM. The PFs memory region is used for AF-PF and PF-VF mailbox. This mbox facilitates communication between AF-PF and PF-VF. On CN10K platform: The DRAM region allocated to PF is enumerated as PF BAR4 memory. PF BAR4 contains AF-PF mbox region

[net-next 02/14] octeontx2-pf: cn10k: Add mbox support for CN10K

2021-01-30 Thread Geetha sowjanya
From: Subbaraya Sundeep Firmware allocates memory regions for PFs and VFs in DRAM. The PFs memory region is used for AF-PF and PF-VF mailbox. This mbox facilitate communication between AF-PF and PF-VF. On CN10K platform: The DRAM region allocated to PF is enumerated as PF BAR4 memory. PF BAR4 co

[net-next 14/14] octeontx2-af: cn10k: MAC internal loopback support

2021-01-30 Thread Geetha sowjanya
From: Hariprasad Kelam MAC on CN10K silicon support loopback for selftest or debug purposes. This patch does necessary configuration to loopback packets upon receiving request from LMAC mapped RVU PF's netdev via mailbox. Also MAC (CGX) on OcteonTx2 silicon variants and MAC (RPM) on OcteonTx3 CN

[net-next 03/14] octeontx2-af: cn10k: Update NIX/NPA context structure

2021-01-30 Thread Geetha sowjanya
NIX hardware context structure got changed to accommodate new features like bandwidth steering, L3/L4 outer/inner checksum enable/disable etc., on CN10K platform. This patch defines new mbox message NIX_CN10K_AQ_INST for new NIX context initialization. This patch also updates the NPA context struc

[net-next 07/14] octeontx2-pf: cn10k: Use LMTST lines for NPA/NIX operations

2021-01-30 Thread Geetha sowjanya
This patch adds support to use new LMTST lines for NPA batch free and burst SQE flush. Adds new dev_hw_ops structure to hold platform specific functions and create new files cn10k.c and cn10k.h. Signed-off-by: Geetha sowjanya Signed-off-by: Sunil Goutham --- .../net/ethernet/marvell/octeontx2/n

[net-next 04/14] octeontx2-af: cn10k: Update NIX and NPA context in debugfs

2021-01-30 Thread Geetha sowjanya
On CN10K platform NPA and NIX context structure bit fields had changed to support new features like bandwidth steering etc. This patch dumps approprate context for CN10K platform. Signed-off-by: Geetha sowjanya Signed-off-by: Sunil Goutham --- .../ethernet/marvell/octeontx2/af/rvu_debugfs.c

[net-next 05/14] octeontx2-pf: cn10k: Initialise NIX context

2021-01-30 Thread Geetha sowjanya
On CN10K platform NIX RQ and SQ context structure got changed. This patch uses new mbox message "NIX_CN10K_AQ_ENQ" for NIX context initialization on CN10K platform. This patch also updates the nix_rx_parse_s and nix_sqe_sg_s structures to add packet steering bit feilds. Signed-off-by: Geetha sowj

[net-next 06/14] octeontx2-pf: cn10k: Map LMTST region

2021-01-30 Thread Geetha sowjanya
On CN10K platform transmit/receive buffer alloc and free from/to hardware had changed to support burst operation. Whereas pervious silicon's only support single buffer free at a time. To Support the same firmware allocates a DRAM region for each PF/VF for storing LMTLINES. These LMTLINES are used f

[net-next 00/14] Add Marvell CN10K support

2021-01-30 Thread Geetha sowjanya
The current admin function (AF) driver and the netdev driver supports OcteonTx2 silicon variants. The same OcteonTx2's Resource Virtualization Unit (RVU) is carried forward to the next-gen silicon ie OcteonTx3, with some changes and feature enhancements. This patch set adds support for OcteonTx3

Re: [PATCH nf-next v4 5/5] af_packet: Introduce egress hook

2021-01-30 Thread Willem de Bruijn
On Sat, Jan 30, 2021 at 11:26 AM Lukas Wunner wrote: > > On Sun, Jan 24, 2021 at 11:18:00AM -0500, Willem de Bruijn wrote: > > On Sun, Jan 24, 2021 at 6:14 AM Lukas Wunner wrote: > > > On Fri, Jan 22, 2021 at 11:13:19AM -0500, Willem de Bruijn wrote: > > > > On Fri, Jan 22, 2021 at 4:44 AM Lukas

[PATCH net-next 0/2] net: dsa: hellcreek: Report tables sizes

2021-01-30 Thread Kurt Kanzenbach
Hi, Florian, Andrew and Vladimir suggested at some point to use devlink for reporting tables, features and debugging counters instead of using debugfs and printk. So, start by reporting the VLAN and FDB table sizes. Thanks, Kurt Kurt Kanzenbach (2): net: dsa: hellcreek: Report VLAN table occu

[PATCH net-next 2/2] net: dsa: hellcreek: Report FDB table occupancy

2021-01-30 Thread Kurt Kanzenbach
Report the FDB table size and occupancy via devlink. The actual size depends on the used Hellcreek version: |root@tsn:~# devlink resource show platform/ff24.switch |platform/ff24.switch: | name VLAN size 4096 occ 2 unit entry dpipe_tables none | name FDB size 256 occ 6 unit entry dpipe_t

Re: [PATCH net] net: hdlc_x25: Use qdisc to queue outgoing LAPB frames

2021-01-30 Thread Xie He
On Fri, Jan 29, 2021 at 5:36 PM Jakub Kicinski wrote: > > I'm still struggling to wrap my head around this. > > Did you test your code with lockdep enabled? Which Qdisc are you using? > You're queuing the frames back to the interface they came from - won't > that cause locking issues? Hmm... Than

Re: [PATCH nf-next v4 5/5] af_packet: Introduce egress hook

2021-01-30 Thread Lukas Wunner
On Sun, Jan 24, 2021 at 11:18:00AM -0500, Willem de Bruijn wrote: > On Sun, Jan 24, 2021 at 6:14 AM Lukas Wunner wrote: > > On Fri, Jan 22, 2021 at 11:13:19AM -0500, Willem de Bruijn wrote: > > > On Fri, Jan 22, 2021 at 4:44 AM Lukas Wunner wrote: > > > > Add egress hook for AF_PACKET sockets tha

Re: [PATCH net-next 3/3] net: mhi: Add mbim proto

2021-01-30 Thread Bjørn Mork
Loic Poulain writes: > MBIM has initially been specified by USB-IF for transporting data (IP) > between a modem and a host over USB. However some modern modems also > support MBIM over PCIe (via MHI). In the same way as QMAP(rmnet), it > allows to aggregate IP packets and to perform context multi

Re: [PATCH net-next 05/10] net: ipv4: Emit notification when fib hardware flags are changed

2021-01-30 Thread Ido Schimmel
On Thu, Jan 28, 2021 at 08:33:22PM -0700, David Ahern wrote: > On 1/28/21 8:04 PM, Jakub Kicinski wrote: > > On Tue, 26 Jan 2021 15:23:06 +0200 Ido Schimmel wrote: > >> Emit RTM_NEWROUTE notifications whenever RTM_F_OFFLOAD/RTM_F_TRAP flags > >> are changed. The aim is to provide an indication to u

Re: [PATCH net 2/2] ibmvnic: fix race with multiple open/close

2021-01-30 Thread Willem de Bruijn
On Thu, Jan 28, 2021 at 10:51 PM Sukadev Bhattiprolu wrote: > > If two or more instances of 'ip link set' commands race and first one > already brings the interface up (or down), the subsequent instances > can simply return without redoing the up/down operation. > > Fixes: ed651a10875f ("ibmvnic:

Re: [PATCH] neighbour: Prevent a dead entry from updating gc_list

2021-01-30 Thread David Ahern
On 1/27/21 9:54 AM, Chinmay Agarwal wrote: > Following race condition was detected: > - neigh_flush_dev() is under execution and calls > neigh_mark_dead(n) marking the neighbour entry 'n' as dead. > > - Executing: __netif_receive_skb() -> > __netif_receive_skb_core() -> arp_rcv() -> arp_process(

Re: [PATCH nf-next v4 1/5] net: sched: Micro-optimize egress handling

2021-01-30 Thread Lukas Wunner
On Tue, Jan 26, 2021 at 11:58:17AM +0300, Dan Carpenter wrote: > On Mon, Jan 25, 2021 at 11:39:08AM -0800, Jakub Kicinski wrote: > > On Sun, 24 Jan 2021 11:33:01 +0100 Lukas Wunner wrote: > > > On Fri, Jan 22, 2021 at 10:40:05AM +0100, Eric Dumazet wrote: > > > > On Fri, Jan 22, 2021 at 9:55 AM Luk

Re: [RESEND PATCH net v4] udp: ipv4: manipulate network header of NATed UDP GRO fraglist

2021-01-30 Thread Alexander Lobakin
From: Dongseok Yi Date: Sat, 30 Jan 2021 08:13:27 +0900 > UDP/IP header of UDP GROed frag_skbs are not updated even after NAT > forwarding. Only the header of head_skb from ip_finish_output_gso -> > skb_gso_segment is updated but following frag_skbs are not updated. > > A call path skb_mac_gso_s

Re: [PATCH v2 net-next 3/4] net: introduce common dev_page_is_reserved()

2021-01-30 Thread Alexander Lobakin
From: Jakub Kicinski Date: Fri, 29 Jan 2021 18:39:07 -0800 > On Wed, 27 Jan 2021 20:11:23 + Alexander Lobakin wrote: > > + * dev_page_is_reserved - check whether a page can be reused for network Rx > > + * @page: the page to test > > + * > > + * A page shouldn't be considered for reusing/recy

Re: [PATCH net-next 05/10] net: ipv4: Emit notification when fib hardware flags are changed

2021-01-30 Thread Ido Schimmel
On Thu, Jan 28, 2021 at 08:15:45PM -0800, Jakub Kicinski wrote: > My impression from working on this problem in TC is that the definition > of "all" becomes problematic especially if one takes into account > drivers getting reloaded. But I think routing offload has stronger > semantics than TC, so

Re: [PATCH net-next 9/9] net: ipa: don't disable NAPI in suspend

2021-01-30 Thread Willem de Bruijn
On Fri, Jan 29, 2021 at 3:29 PM Alex Elder wrote: > > The channel stop and suspend paths both call __gsi_channel_stop(), > which quiesces channel activity, disables NAPI, and (on other than > SDM845) stops the channel. Similarly, the start and resume paths > share __gsi_channel_start(), which sta

Re: [PATCH net-next v4] net: psample: Introduce stubs to remove NIC driver dependency

2021-01-30 Thread Ido Schimmel
On Fri, Jan 29, 2021 at 12:30:09PM -0800, Jakub Kicinski wrote: > On Fri, 29 Jan 2021 14:08:39 +0800 Chris Mi wrote: > > Instead of discussing it several days, maybe it's better to review > > current patch, so that we can move forward :) > > It took you 4 revisions to post a patch which builds cl

Re: [PATCH net-next v5] net: psample: Introduce stubs to remove NIC driver dependency

2021-01-30 Thread Ido Schimmel
On Sat, Jan 30, 2021 at 10:33:19AM +0800, Chris Mi wrote: > In order to send sampled packets to userspace, NIC driver calls > psample api directly. But it creates a hard dependency on module > psample. Introduce psample_ops to remove the hard dependency. > It is initialized when psample module is l

Re: [PATCH net-next v2] net/sched: act_police: add support for packet-per-second policing

2021-01-30 Thread Ido Schimmel
On Fri, Jan 29, 2021 at 03:04:51PM -0800, Cong Wang wrote: > On Fri, Jan 29, 2021 at 2:29 AM Simon Horman > wrote: I didn't get v2 (didn't made it to the list), but I did leave feedback on v1 [1]. Not sure if you got it or not given the recent issues. [1] https://lore.kernel.org/netdev/20210128

Re: [PATCH stable v5.4 2/2] IPv6: reply ICMP error if the first fragment don't include all headers

2021-01-30 Thread Greg KH
On Sat, Jan 30, 2021 at 05:24:52PM +0530, Aviraj CJ wrote: > From: Hangbin Liu > > commit 2efdaaaf883a143061296467913c01aa1ff4b3ce upstream. > > Based on RFC 8200, Section 4.5 Fragment Header: > > - If the first fragment does not include all headers through an > Upper-Layer header, then

Re: [Patch v2 net-next 2/7] octeontx2-af: Add new CGX_CMD to get PHY FEC statistics

2021-01-30 Thread Willem de Bruijn
On Sat, Jan 30, 2021 at 4:53 AM Hariprasad Kelam wrote: > > Hi Willem, > > > -Original Message- > > From: Willem de Bruijn > > Sent: Thursday, January 28, 2021 1:50 AM > > To: Hariprasad Kelam > > Cc: Network Development ; LKML > ker...@vger.kernel.org>; David Miller ; Jakub > > Kicinsk

Re: [patch net-next RFC 00/10] introduce line card support for modular switch

2021-01-30 Thread Jiri Pirko
Fri, Jan 29, 2021 at 06:31:59PM CET, and...@lunn.ch wrote: >> Platform line card driver is aware of line card I2C topology, its >> responsibility is to detect line card basic hardware type, create I2C >> topology (mux), connect all the necessary I2C devices, like hotswap >> devices, voltage and pow

[PATCH net-next 1/2] net: dsa: hellcreek: Report VLAN table occupancy

2021-01-30 Thread Kurt Kanzenbach
The VLAN membership configuration is cached in software already. So, it can be reported via devlink. Add support for it: |root@tsn:~# devlink resource show platform/ff24.switch |platform/ff24.switch: | name VLAN size 4096 occ 4 unit entry dpipe_tables none Signed-off-by: Kurt Kanzenbach

Re: [PATCH net 1/4] igc: Report speed and duplex as unknown when device is runtime suspended

2021-01-30 Thread Neftin, Sasha
On 1/30/2021 08:22, Jakub Kicinski wrote: On Thu, 28 Jan 2021 13:38:48 -0800 Tony Nguyen wrote: From: Kai-Heng Feng Similar to commit 165ae7a8feb5 ("igb: Report speed and duplex as unknown when device is runtime suspended"), if we try to read speed and duplex sysfs while the device is runtime

Re: [PATCH bpf-next V13 4/7] bpf: add BPF-helper for MTU checking

2021-01-30 Thread Jesper Dangaard Brouer
On Sat, 30 Jan 2021 01:08:17 +0100 Daniel Borkmann wrote: > On 1/29/21 4:50 PM, John Fastabend wrote: > > Jesper Dangaard Brouer wrote: > >> On Thu, 28 Jan 2021 22:51:23 -0800 > >> John Fastabend wrote: > >>> Jesper Dangaard Brouer wrote: > This BPF-helper bpf_check_mtu() works for bo

[PATCH net] net: dsa: mv88e6xxx: override existent unicast portvec in port_fdb_add

2021-01-30 Thread DENG Qingfang
Having multiple destination ports for a unicast address does not make sense. Make port_db_load_purge override existent unicast portvec instead of adding a new port bit. Fixes: 884729399260 ("net: dsa: mv88e6xxx: handle multiple ports in ATU") Signed-off-by: DENG Qingfang --- drivers/net/dsa/mv88

[PATCH v2] rtl8xxxu: remove unused assignment value

2021-01-30 Thread samirweng1979
From: wengjianfeng at first, ret was assigned to zero, but later assigned to a funciton,so the assignment to zero is no use, which can simple be removed instead. Signed-off-by: wengjianfeng --- drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu_8192e.c | 2 -- 1 file changed, 2 deletions(-) diff

Re: [PATCH net 1/1] netfilter: conntrack: Check offload bit on table dump

2021-01-30 Thread Pablo Neira Ayuso
Hi Roi, On Thu, Jan 28, 2021 at 09:40:52AM +0200, Roi Dayan wrote: > Currently, offloaded flows might be deleted when executing conntrack -L > or cat /proc/net/nf_conntrack while rules being offloaded. > Ct timeout is not maintained for offloaded flows as aging > of offloaded flows are managed by

[PATCH stable v5.4 2/2] IPv6: reply ICMP error if the first fragment don't include all headers

2021-01-30 Thread Aviraj CJ
From: Hangbin Liu commit 2efdaaaf883a143061296467913c01aa1ff4b3ce upstream. Based on RFC 8200, Section 4.5 Fragment Header: - If the first fragment does not include all headers through an Upper-Layer header, then that fragment should be discarded and an ICMP Parameter Problem, Code

[PATCH stable v5.4 1/2] ICMPv6: Add ICMPv6 Parameter Problem, code 3 definition

2021-01-30 Thread Aviraj CJ
From: Hangbin Liu commit b59e286be280fa3c2e94a0716ddcee6ba02bc8ba upstream. Based on RFC7112, Section 6: IANA has added the following "Type 4 - Parameter Problem" message to the "Internet Control Message Protocol version 6 (ICMPv6) Parameters" registry: CODE NAME/DESCRIPTION

Re: [Internal review][PATCH stable v5.4 1/2] ICMPv6: Add ICMPv6 Parameter Problem, code 3 definition

2021-01-30 Thread Greg KH
On Sat, Jan 30, 2021 at 11:31:11AM +, Aviraj Cj (acj) wrote: > > > On 30/01/21, 2:06 PM, "Greg KH" wrote: > > On Sat, Jan 30, 2021 at 12:57:40AM +0530, Aviraj CJ wrote: > > From: Hangbin Liu > > > > commit b59e286be280fa3c2e94a0716ddcee6ba02bc8ba upstream. > > > > Based on RFC7112, Sect

[PATCH] wl1251: cmd: remove redundant assignment

2021-01-30 Thread samirweng1979
From: wengjianfeng -ENOMEM has been used as a return value,it is not necessary to assign it, and if kzalloc fail,not need free it,so just return -ENOMEM when kzalloc fail. Signed-off-by: wengjianfeng --- drivers/net/wireless/ti/wl1251/cmd.c | 36 1 file cha

Re: [Internal review][PATCH stable v5.4 1/2] ICMPv6: Add ICMPv6 Parameter Problem, code 3 definition

2021-01-30 Thread Aviraj Cj (acj)
On 30/01/21, 2:06 PM, "Greg KH" wrote: On Sat, Jan 30, 2021 at 12:57:40AM +0530, Aviraj CJ wrote: > From: Hangbin Liu > > commit b59e286be280fa3c2e94a0716ddcee6ba02bc8ba upstream. > > Based on RFC7112, Section 6: > >IANA has added the following "Type 4 - Parameter Problem" message to >

Re: Re: [RFC v3 03/11] vdpa: Remove the restriction that only supports virtio-net devices

2021-01-30 Thread Yongji Xie
On Fri, Jan 29, 2021 at 11:04 PM Stefano Garzarella wrote: > > On Thu, Jan 28, 2021 at 11:11:49AM +0800, Jason Wang wrote: > > > >On 2021/1/27 下午4:57, Stefano Garzarella wrote: > >>On Wed, Jan 27, 2021 at 11:33:03AM +0800, Jason Wang wrote: > >>> > >>>On 2021/1/20 下午7:08, Stefano Garzarella wrote:

Re: [PATCH v8 net-next 08/11] net: dsa: allow changing the tag protocol via the "tagging" device attribute

2021-01-30 Thread Jakub Kicinski
On Sat, 30 Jan 2021 01:43:04 +0200 Vladimir Oltean wrote: > Jakub, I was stupid and I pasted the ping command output into the commit > message, so git will trim anything past the dotted line as not part of > the commit message, which makes your netdev/verify_signedoff test fail. > If by some sort o

Re: [PATCH net-next v3 2/8] taprio: Add support for frame preemption offload

2021-01-30 Thread Jakub Kicinski
On Fri, 29 Jan 2021 15:12:58 -0800 Vinicius Costa Gomes wrote: > Jakub Kicinski writes: > >> Good catch :-) > >> > >> I wanted to have this (at least one express queue) handled in a > >> centralized way, but perhaps this should be handled best per driver. > > > > Centralized is good. Much easie

Re: [RESEND net v3] net: ip_tunnel: fix mtu calculation

2021-01-30 Thread Cong Wang
On Fri, Jan 29, 2021 at 2:30 PM Vadim Fedorenko wrote: > > dev->hard_header_len for tunnel interface is set only when header_ops > are set too and already contains full overhead of any tunnel encapsulation. > That's why there is not need to use this overhead twice in mtu calc. > > Fixes: fdafed459

Re: [PATCH v2 net-next] hv_netvsc: Copy packets sent by Hyper-V out of the receive buffer

2021-01-30 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Tue, 26 Jan 2021 17:29:07 +0100 you wrote: > Pointers to receive-buffer packets sent by Hyper-V are used within the > guest VM. Hyper-V can send packets with erroneous values or modify > packet fields after they are pro

[PATCH net-next v2 0/2] rework the memory barrier for SCRQ entry

2021-01-30 Thread Lijun Pan
This series rework the memory barrier for SCRQ (Sub-Command-Response Queue) entry. v2: send to net-next. Lijun Pan (2): ibmvnic: rework to ensure SCRQ entry reads are properly ordered ibmvnic: remove unnecessary rmb() inside ibmvnic_poll drivers/net/ethernet/ibm/ibmvnic.c | 31 +++--

[PATCH net-next v2 2/2] ibmvnic: remove unnecessary rmb() inside ibmvnic_poll

2021-01-30 Thread Lijun Pan
rmb() can be removed since: 1. pending_scrq() has dma_rmb() at the function end; 2. dma_rmb(), though weaker, is enough here. Signed-off-by: Lijun Pan Acked-by: Dwip Banerjee Acked-by: Thomas Falcon Reviewed-by: Brian King --- drivers/net/ethernet/ibm/ibmvnic.c | 1 - 1 file changed, 1 deleti

RE: [PATCH 07/22] RDMA/irdma: Register an auxiliary driver and implement private channel OPs

2021-01-30 Thread Saleem, Shiraz
> Subject: Re: [PATCH 07/22] RDMA/irdma: Register an auxiliary driver and > implement private channel OPs > > On Mon, Jan 25, 2021 at 02:42:48PM -0400, Jason Gunthorpe wrote: > > On Fri, Jan 22, 2021 at 05:48:12PM -0600, Shiraz Saleem wrote: > > > +/** > > > + * irdma_init_dev - GEN_2 device init

RE: [PATCH 09/22] RDMA/irdma: Implement HW Admin Queue OPs

2021-01-30 Thread Saleem, Shiraz
> Subject: Re: [PATCH 09/22] RDMA/irdma: Implement HW Admin Queue OPs > > On Wed, Jan 27, 2021 at 12:41:59AM +, Saleem, Shiraz wrote: > > > Subject: Re: [PATCH 09/22] RDMA/irdma: Implement HW Admin Queue OPs > > > > > > On Fri, Jan 22, 2021 at 05:48:14PM -0600, Shiraz Saleem wrote: > > > > +#d

Re: [PATCH net] net: hdlc_x25: Use qdisc to queue outgoing LAPB frames

2021-01-30 Thread Jakub Kicinski
On Fri, 29 Jan 2021 06:56:10 +0100 Martin Schiller wrote: > On 2021-01-28 23:06, Xie He wrote: > > On Thu, Jan 28, 2021 at 11:47 AM Jakub Kicinski > > wrote: > >> > >> Noob question - could you point at or provide a quick guide to > >> layering > >> here? I take there is only one netdev, and

Re: [PATCH net-next 2/8] net: dsa: tag_ksz: add tag handling for Microchip LAN937x

2021-01-30 Thread Vladimir Oltean
On Thu, Jan 28, 2021 at 12:11:06PM +0530, Prasanna Vengateshan wrote: > diff --git a/net/dsa/tag_ksz.c b/net/dsa/tag_ksz.c > index 4820dbcedfa2..6fac39c2b7d5 100644 > --- a/net/dsa/tag_ksz.c > +++ b/net/dsa/tag_ksz.c > @@ -190,10 +190,84 @@ static const struct dsa_device_ops ksz9893_netdev_ops = >

Re: [PATCH net-next 0/8] net: dsa: microchip: DSA driver support for LAN937x switch

2021-01-30 Thread Vladimir Oltean
On Thu, Jan 28, 2021 at 09:55:58AM -0800, Florian Fainelli wrote: > Could you also feed back to your hardware organization to settle on a > tag format that is not a snowflake? Almost *every* switch you have has a > different tagging format, this is absurd. All other vendors in tree have > been able

[PATCH] mwl8k: assign value when defining variables

2021-01-30 Thread samirweng1979
From: wengjianfeng define refilled and then assign value to it, which should do at the same time. Signed-off-by: wengjianfeng --- drivers/net/wireless/marvell/mwl8k.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/marvell/mwl8k.c b/drivers/net/wirel

Re: [Patch v2 net-next 3/7] octeontx2-pf: ethtool fec mode support

2021-01-30 Thread Hariprasad Kelam
Hi Willem, > -Original Message- > From: Willem de Bruijn > Sent: Thursday, January 28, 2021 2:01 AM > To: Hariprasad Kelam > Cc: Network Development ; LKML ker...@vger.kernel.org>; David Miller ; Jakub > Kicinski ; Sunil Kovvuri Goutham > ; Linu Cherian ; > Geethasowjanya Akula ; Jerin

Re: [PATCH net-next v4 0/2] Add nci suit and virtual nci device driver

2021-01-30 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (refs/heads/master): On Wed, 27 Jan 2021 22:08:27 +0900 you wrote: > From: Bongsu Jeon > > 1/2 is the Virtual NCI device driver. > 2/2 is the NCI selftest suite > > v4: > 1/2 > - flip the condition for the ioctl. > - refactor some code.

Re: [PATCH net-next] tcp: shrink inet_connection_sock icsk_mtup enabled and probe_size

2021-01-30 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to netdev/net-next.git (refs/heads/master): On Fri, 29 Jan 2021 13:54:38 -0500 you wrote: > From: Neal Cardwell > > This commit shrinks inet_connection_sock by 4 bytes, by shrinking > icsk_mtup.enabled from 32 bits to 1 bit, and shrinking > icsk_mtup.probe_size fro

  1   2   >