Re: [PATCH net-next v2 3/3] tun: AF_XDP Tx zero-copy support

2024-03-01 Thread Willem de Bruijn
Maciej Fijalkowski wrote: > On Wed, Feb 28, 2024 at 07:05:56PM +0800, Yunjian Wang wrote: > > This patch set allows TUN to support the AF_XDP Tx zero-copy feature, > > which can significantly reduce CPU utilization for XDP programs. > > Why no Rx ZC support though? What will happen if I try

Re: [PATCH net-next 2/2] tun: AF_XDP Rx zero-copy support

2024-01-24 Thread Willem de Bruijn
Yunjian Wang wrote: > Now the zero-copy feature of AF_XDP socket is supported by some > drivers, which can reduce CPU utilization on the xdp program. > This patch set allows tun to support AF_XDP Rx zero-copy feature. > > This patch tries to address this by: > - Use peek_len to consume a

Re: [PATCH net-next v3 3/3] net: add netmem_ref to skb_frag_t

2023-12-21 Thread Willem de Bruijn
Mina Almasry wrote: > Use netmem_ref instead of page in skb_frag_t. Currently netmem_ref > is always a struct page underneath, but the abstraction allows efforts > to add support for skb frags not backed by pages. > > There is unfortunately 1 instance where the skb_frag_t is assumed to be > a

Re: [PATCH net-next v3 1/3] vsock/virtio: use skb_frag_*() helpers

2023-12-21 Thread Willem de Bruijn
Mina Almasry wrote: > Minor fix for virtio: code wanting to access the fields inside an skb > frag should use the skb_frag_*() helpers, instead of accessing the > fields directly. This allows for extensions where the underlying > memory is not a page. > > Signed-off-by: Mina Almasry > > --- >

Re: [PATCH RFC 0/4] virtio-net: add tx-hash, rx-tstamp, tx-tstamp and tx-time

2023-12-19 Thread Willem de Bruijn
Jason Wang wrote: > On Tue, Dec 19, 2023 at 12:36 AM Willem de Bruijn > wrote: > > > > Steffen Trumtrar wrote: > > > This series tries to pick up the work on the virtio-net timestamping > > > feature from Willem de Bruijn. > > > > > > Origina

Re: [PATCH RFC 0/4] virtio-net: add tx-hash, rx-tstamp, tx-tstamp and tx-time

2023-12-18 Thread Willem de Bruijn
Steffen Trumtrar wrote: > This series tries to pick up the work on the virtio-net timestamping > feature from Willem de Bruijn. > > Original series > Message-Id: 20210208185558.995292-1-willemdebruijn.ker...@gmail.com > Subject: [PATCH RFC v2 0/4] virtio-net: add

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

2021-04-13 Thread Willem de Bruijn
> > > > > > > > > but even yours is also fixed I think. > > > > > > The common point is that a single spurious interrupt is not a problem. > > > The problem only exists if there are tons of spurious interrupts with no > > > real ones. For this to trigger, we keep polling the ring and while we do >

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

2021-04-13 Thread Willem de Bruijn
On Tue, Apr 13, 2021 at 3:54 PM Michael S. Tsirkin wrote: > > On Tue, Apr 13, 2021 at 10:01:11AM -0400, Willem de Bruijn wrote: > > On Tue, Apr 13, 2021 at 1:47 AM Michael S. Tsirkin wrote: > > > > > > virtio_disable_cb is currently a nop for split ring with even

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

2021-04-13 Thread Willem de Bruijn
On Tue, Apr 13, 2021 at 10:03 AM Michael S. Tsirkin wrote: > > On Tue, Apr 13, 2021 at 04:54:42PM +0800, Jason Wang wrote: > > > > 在 2021/4/13 下午1:47, Michael S. Tsirkin 写道: > > > It's unsafe to operate a vq from multiple threads. > > > Unfortunately this is exactly what we do when invoking > > >

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

2021-04-13 Thread Willem de Bruijn
On Tue, Apr 13, 2021 at 4:53 AM Jason Wang wrote: > > > 在 2021/4/13 下午1:47, 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

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

2021-04-13 Thread Willem de Bruijn
On Tue, Apr 13, 2021 at 1:47 AM Michael S. Tsirkin wrote: > > 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

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

2021-04-11 Thread Willem de Bruijn
On Sun, Apr 11, 2021 at 9:31 PM Hao Sun wrote: > > Hi > > When using Healer(https://github.com/SunHao-0/healer/tree/dev) to fuzz > the Linux kernel, I found the following bug report, but I'm not sure > about this. > Sorry, I do not have a reproducing program for this bug. > I hope that the stack

Re: [RFC net] net: skbuff: fix stack variable out of bounds access

2021-03-23 Thread Willem de Bruijn
On Tue, Mar 23, 2021 at 12:30 PM Arnd Bergmann wrote: > > On Tue, Mar 23, 2021 at 3:42 PM Willem de Bruijn > wrote: > > > > On Tue, Mar 23, 2021 at 8:52 AM Arnd Bergmann wrote: > > >> > > A similar fix already landed in 5.12-rc3: commit b228c9b05876 (&quo

Re: [RFC net] net: skbuff: fix stack variable out of bounds access

2021-03-23 Thread Willem de Bruijn
On Tue, Mar 23, 2021 at 8:52 AM Arnd Bergmann wrote: > > From: Arnd Bergmann > > gcc-11 warns that the TS_SKB_CB()) cast in skb_find_text() > leads to an out-of-bounds access in skb_prepare_seq_read() after > the addition of a new struct member made skb_seq_state longer > than ts_state: > >

Re: [PATCH net v3 1/2] net: check if protocol extracted by virtio_net_hdr_set_proto is correct

2021-03-09 Thread Willem de Bruijn
ol set by virtio_net_hdr_set_proto. Note that since the protocol > is only checked if skb->dev implements header_ops->parse_protocol, > packets from devices without the implementation are not checked at this > stage. > > Fixes: 9274124f023b ("net: stricter validation of untrus

Re: [PATCH net v3 2/2] net: avoid infinite loop in mpls_gso_segment when mpls_hlen == 0

2021-03-09 Thread Willem de Bruijn
> again call mpls_gso_segment with the same packet leading to an infinite > loop. In addition, ensure that the header length is a multiple of four, > which should hold irrespective of the number of stacked labels. > > Signed-off-by: Balazs Nemeth Acked-by: Willem de Bruijn The compil

Re: [PATCH v2 1/2] net: check if protocol extracted by virtio_net_hdr_set_proto is correct

2021-03-09 Thread Willem de Bruijn
On Tue, Mar 9, 2021 at 6:26 AM Michael S. Tsirkin wrote: > > On Mon, Mar 08, 2021 at 11:31:25AM +0100, 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

Re: [PATCH v2 2/2] net: avoid infinite loop in mpls_gso_segment when mpls_hlen == 0

2021-03-08 Thread Willem de Bruijn
On Mon, Mar 8, 2021 at 11:43 AM David Ahern wrote: > > On 3/8/21 9:26 AM, Balazs Nemeth wrote: > > On Mon, 2021-03-08 at 09:17 -0700, David Ahern wrote: > >> On 3/8/21 9:07 AM, Willem de Bruijn wrote: > >>>> diff --git a/net/mpls/mpls_gso.c b/net/mpls/

Re: [PATCH v2 2/2] net: avoid infinite loop in mpls_gso_segment when mpls_hlen == 0

2021-03-08 Thread Willem de Bruijn
On Mon, Mar 8, 2021 at 5:32 AM Balazs Nemeth wrote: > > A packet with skb_inner_network_header(skb) == skb_network_header(skb) > and ETH_P_MPLS_UC will prevent mpls_gso_segment from pulling any headers > from the packet. Subsequently, the call to skb_mac_gso_segment will > again call

Re: [PATCH v2 1/2] net: check if protocol extracted by virtio_net_hdr_set_proto is correct

2021-03-08 Thread Willem de Bruijn
On Mon, Mar 8, 2021 at 5:32 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 bpf-next] selftests_bpf: extend test_tc_tunnel test with vxlan

2021-03-05 Thread Willem de Bruijn
On Fri, Mar 5, 2021 at 11:10 AM Daniel Borkmann wrote: > > On 3/5/21 4:08 PM, Willem de Bruijn wrote: > > On Fri, Mar 5, 2021 at 7:34 AM Xuesen Huang wrote: > >> > >> From: Xuesen Huang > >> > >> Add BPF_F_ADJ_ROOM_ENCAP_L2_ETH flag to the exi

Re: [PATCH bpf-next] selftests_bpf: extend test_tc_tunnel test with vxlan

2021-03-05 Thread Willem de Bruijn
g > Signed-off-by: Li Wang > Signed-off-by: Willem de Bruijn Please don't add my signed off by without asking. That said, Acked-by: Willem de Bruijn

Re: [PATCH] selftests_bpf: extend test_tc_tunnel test with vxlan

2021-03-04 Thread Willem de Bruijn
g > Signed-off-by: Li Wang > Signed-off-by: Willem de Bruijn Please mark patch target: [PATCH bpf-next] > --- > tools/testing/selftests/bpf/progs/test_tc_tunnel.c | 113 > ++--- > tools/testing/selftests/bpf/test_tc_tunnel.sh | 15 ++- >

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

2021-03-04 Thread Willem de Bruijn
fy encap L2 type and set the inner_protocol > as ETH_P_TEB. > > Suggested-by: Willem de Bruijn > Signed-off-by: Xuesen Huang > Signed-off-by: Zhiyong Cheng > Signed-off-by: Li Wang Acked-by: Willem de Bruijn

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

2021-03-03 Thread Willem de Bruijn
> > Instead of untyped macros, I'd define encap_ipv4 as a function that > > calls __encap_ipv4. > > > > And no need for encap_ipv4_with_ext_proto equivalent to __encap_ipv4. > > > I defined these macros to try to keep the existing invocation for > encap_ipv4/6 > as the same, if we define this as

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

2021-03-03 Thread Willem de Bruijn
fy encap L2 type and set the inner_protocol > as ETH_P_TEB. > > Update test_tc_tunnel to verify adding vxlan encapsulation works with > this flag. > > Suggested-by: Willem de Bruijn > Signed-off-by: Xuesen Huang > Signed-off-by: Zhiyong Cheng > Signed-off-by: Li Wang

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

2021-02-26 Thread Willem de Bruijn
On Fri, Feb 26, 2021 at 3:15 PM Cong Wang wrote: > > On Thu, Feb 25, 2021 at 7:59 PM Xuesen Huang wrote: > > v3: > > - Fix the code format. > > > > v2: > > Suggested-by: Willem de Bruijn > > - Add a new flag to specify the type of the inne

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

2021-02-23 Thread Willem de Bruijn
On Tue, Feb 23, 2021 at 8:48 AM Balazs Nemeth wrote: > > On Mon, 2021-02-22 at 11:39 +0800, Jason Wang wrote: > > > > On 2021/2/19 10:55 下午, Willem de Bruijn wrote: > > > On Fri, Feb 19, 2021 at 3:53 AM Jason Wang > > > wrote: > > > > >

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

2021-02-19 Thread Willem de Bruijn
On Fri, Feb 19, 2021 at 3:53 AM Jason Wang wrote: > > > On 2021/2/18 11:50 下午, Willem de Bruijn wrote: > > 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

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

Re: possible stack corruption in icmp_send (__stack_chk_fail)

2021-02-17 Thread Willem de Bruijn
On Wed, Feb 17, 2021 at 6:18 PM Jason A. Donenfeld wrote: > > On 2/18/21, Willem de Bruijn wrote: > > On Wed, Feb 17, 2021 at 5:56 PM Jason A. Donenfeld wrote: > >> > >> Hi Willem, > >> > >> On Wed, Feb 17, 2021 at 11:27 PM Willem de Bruijn &

Re: possible stack corruption in icmp_send (__stack_chk_fail)

2021-02-17 Thread Willem de Bruijn
On Wed, Feb 17, 2021 at 5:56 PM Jason A. Donenfeld wrote: > > Hi Willem, > > On Wed, Feb 17, 2021 at 11:27 PM Willem de Bruijn > wrote: > > A vmlinux image might help. I couldn't find one for this kernel. > > https://data.zx2c4.com/icmp_send-crash-e03b4a42-706a-43

Re: possible stack corruption in icmp_send (__stack_chk_fail)

2021-02-17 Thread Willem de Bruijn
On Wed, Feb 17, 2021 at 1:12 PM Jason A. Donenfeld wrote: > > Hi Netdev & Willem, > > I've received a report of stack corruption -- via the stack protector > check -- in icmp_send. I was sent a vmcore, and was able to extract > the OOPS from there. However, I've been unable to produce the bug and

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

2021-02-10 Thread Willem de Bruijn
fy encap L2 type and set the inner_protocol > as ETH_P_TEB. > > Suggested-by: Willem de Bruijn > Signed-off-by: huangxuesen > Signed-off-by: chengzhiyong > Signed-off-by: wangli Thanks, this is exactly what I meant. Acked-by: Willem de Bruijn One small point regarding Signed-of

Re: [PATCH] bpf: in bpf_skb_adjust_room correct inner protocol for vxlan

2021-02-09 Thread Willem de Bruijn
mac header. I would only not infer it based on some imprecise measure, such as inner_maclen being 14. But add a new explicit flag BPF_F_ADJ_ROOM_ENCAP_L2_ETH. Update inner protocol if the flag is passed and inner_maclen >= ETH_HLEN. Fail the operation if the flag is passed and inner_maclen is too sh

Re: [PATCH] bpf: in bpf_skb_adjust_room correct inner protocol for vxlan

2021-02-08 Thread Willem de Bruijn
On Mon, Feb 8, 2021 at 7:16 AM huangxuesen wrote: > > From: huangxuesen > > When pushing vxlan tunnel header, set inner protocol as ETH_P_TEB in skb > to avoid HW device disabling udp tunnel segmentation offload, just like > vxlan_build_skb does. > > Drivers for NIC may invoke

Re: [PATCH net-next] net/packet: Improve the comment about LL header visibility criteria

2021-02-05 Thread Willem de Bruijn
der if > dev->header_ops->create is present. > > Signed-off-by: Xie He Acked-by: Willem de Bruijn Indeed, existence of dev->header_ops->create is the deciding factor. Thanks Xie.

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

2021-02-01 Thread Willem de Bruijn
n separate > steps (patches). Now both are moved around together in patch 5 and > 6, which eliminates the need for the final (NAPI-only) patch. > > I won't repeat the patch summaries provided in v1: > https://lore.kernel.org/netdev/20210129202019.2099259-1-el...@linaro.org/ > > Many

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

2021-02-01 Thread Willem de Bruijn
On Mon, Feb 1, 2021 at 9:35 AM Alex Elder wrote: > > On 1/31/21 7:36 PM, Willem de Bruijn wrote: > > On Sun, Jan 31, 2021 at 10:32 AM Alex Elder wrote: > >> > >> On 1/31/21 8:52 AM, Willem de Bruijn wrote: > >>> On Sat, Jan 30, 2021 at 11:29 PM Alex Elde

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

2021-01-31 Thread Willem de Bruijn
On Sun, Jan 31, 2021 at 10:32 AM Alex Elder wrote: > > On 1/31/21 8:52 AM, Willem de Bruijn wrote: > > On Sat, Jan 30, 2021 at 11:29 PM Alex Elder wrote: > >> > >> On 1/30/21 9:25 AM, Willem de Bruijn wrote: > >>> On Fri, Jan 29, 2021 at 3:29 PM Alex El

Re: [Patch v3 net-next 0/7] ethtool support for fec and link configuration

2021-01-31 Thread Willem de Bruijn
On Sun, Jan 31, 2021 at 8:11 AM Hariprasad Kelam wrote: > > This series of patches add support for forward error correction(fec) and > physical link configuration. Patches 1&2 adds necessary mbox handlers for fec > mode configuration request and to fetch stats. Patch 3 registers driver >

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

2021-01-31 Thread Willem de Bruijn
On Sat, Jan 30, 2021 at 12:04 PM Geetha sowjanya wrote: > > 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 >

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

2021-01-31 Thread Willem de Bruijn
On Sat, Jan 30, 2021 at 11:29 PM Alex Elder wrote: > > 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

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

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

Re: [PATCH net-next v1 2/6] lan743x: support rx multi-buffer packets

2021-01-29 Thread Willem de Bruijn
On Fri, Jan 29, 2021 at 6:03 PM Sven Van Asbroeck wrote: > > Hoi Willem, thanks a lot for reviewing this patch, much appreciated !! > > On Fri, Jan 29, 2021 at 5:11 PM Willem de Bruijn > wrote: > > > > > +static struct sk_buff * > > > +lan743x_rx_trim_sk

Re: [PATCH net-next v1 2/6] lan743x: support rx multi-buffer packets

2021-01-29 Thread Willem de Bruijn
On Fri, Jan 29, 2021 at 2:56 PM Sven Van Asbroeck wrote: > > From: Sven Van Asbroeck > > Multi-buffer packets enable us to use rx ring buffers smaller than > the mtu. This will allow us to change the mtu on-the-fly, without > having to stop the network interface in order to re-size the rx > ring

Re: [PATCH] rtlwifi: rtl8192se: remove redundant initialization of variable rtstatus

2021-01-28 Thread Willem de Bruijn
esses-Coverity: ("Unused value") > Signed-off-by: Colin Ian King (for netdrv) Acked-by: Willem de Bruijn

Re: [Patch v2 net-next 4/7] octeontx2-af: Physical link configuration support

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 4:02 AM Hariprasad Kelam wrote: > > From: Christina Jacob > > CGX LMAC, the physical interface support link configuration parameters > like speed, auto negotiation, duplex etc. Firmware saves these into > memory region shared between firmware and this driver. > > This

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

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 4:03 AM Hariprasad Kelam wrote: > > From: Christina Jacob > > Add ethtool support to configure fec modes baser/rs and > support to fecth FEC stats from CGX as well PHY. > > Configure fec mode > - ethtool --set-fec eth0 encoding rs/baser/off/auto > Query fec mode >

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

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 4:04 AM Hariprasad Kelam wrote: > > From: Felix Manlunas > > This patch adds support to fetch fec stats from PHY. The stats are > put in the shared data struct fwdata. A PHY driver indicates > that it has FEC stats by setting the flag fwdata.phy.misc.has_fec_stats > >

Re: [Patch v2 net-next 1/7] octeontx2-af: forward error correction configuration

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 4:05 AM Hariprasad Kelam wrote: > > From: Christina Jacob > > CGX block supports forward error correction modes baseR > and RS. This patch adds support to set encoding mode > and to read corrected/uncorrected block counters > > Adds new mailbox handlers set_fec to

Re: [PATCH v4 net-next 00/19] net: mvpp2: Add TX Flow Control support

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 6:50 AM wrote: > > From: Stefan Chulski > > Armada hardware has a pause generation mechanism in GOP (MAC). > The GOP generate flow control frames based on an indication programmed in > Ports Control 0 Register. There is a bit per port. > However assertion of the PortX

Re: [PATCH v4 net-next 11/19] net: mvpp2: add spinlock for FW FCA configuration path

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 7:19 AM wrote: > > From: Stefan Chulski > > Spinlock added to MSS shared memory configuration space. > > Signed-off-by: Stefan Chulski > --- > drivers/net/ethernet/marvell/mvpp2/mvpp2.h | 5 + > drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c | 3 +++ > 2 files

Re: [PATCH v4 net-next 10/19] net: mvpp2: add FCA RXQ non occupied descriptor threshold

2021-01-27 Thread Willem de Bruijn
On Wed, Jan 27, 2021 at 7:26 AM wrote: > > From: Stefan Chulski > > RXQ non occupied descriptor threshold would be used by > Flow Control Firmware feature to move to the XOFF mode. > RXQ non occupied threshold would change interrupt cause > that polled by CM3 Firmware. > Actual non occupied

Re: [PATCH] rtlwifi: use tasklet_setup to initialize rx_work_tasklet

2021-01-27 Thread Willem de Bruijn
); > > without all the casting. > > Fixes: 6b8c7574a5f8 ("rtlwifi: fix build warning") > Fixes: d3ccc14dfe95 ("rtlwifi/rtw88: convert tasklets to use new > tasklet_setup() API") > Signed-off-by: Emil Renner Berthing Since the current code works, this could target net-next without Fixes tags. Acked-by: Willem de Bruijn

Re: [PATCH net-next v2 0/6] net: ipa: hardware pipeline cleanup fixes

2021-01-27 Thread Willem de Bruijn
++-- > drivers/net/ipa/ipa_cmd.h | 24 ++- > drivers/net/ipa/ipa_endpoint.c | 79 ++ > drivers/net/ipa/ipa_main.c | 1 + > 5 files changed, 109 insertions(+), 42 deletions(-) For netdrv Acked-by: Willem de Bruijn

Re: [PATCH] rtlwifi: halbtc8723b2ant: Remove redundant code

2021-01-27 Thread Willem de Bruijn
gested-by: Jiapeng Zhong > Signed-off-by: Abaci Team Signed-off-by lines need to have a real name. See Documentation/process/submitting-patches.rst With that change Acked-by: Willem de Bruijn

Re: [PATCH net-next 3/4] bridge: mrp: Extend br_mrp_switchdev to detect better the errors

2021-01-25 Thread Willem de Bruijn
On Sat, Jan 23, 2021 at 11:23 AM Horatiu Vultur wrote: > > This patch extends the br_mrp_switchdev functions to be able to have a > better understanding what cause the issue and if the SW needs to be used > as a backup. > > There are the following cases: > - when the code is compiled without

Re: [PATCH v4 net-next 2/2] udp: allow forwarding of plain (non-fraglisted) UDP GRO packets

2021-01-22 Thread Willem de Bruijn
_F_GRO_UDP_FWD is > enabled on a receiving netdevice. > > If both NETIF_F_GRO_FRAGLIST and NETIF_F_GRO_UDP_FWD are set, > fraglisted GRO takes precedence. This keeps the current behaviour > and is generally more optimal for now, as the number of NICs with > hardware USO offload is relatively small. > > Signed-off-by: Alexander Lobakin Acked-by: Willem de Bruijn

Re: [PATCH net-next 2/2] udp: allow forwarding of plain (non-fraglisted) UDP GRO packets

2021-01-22 Thread Willem de Bruijn
On Fri, Jan 22, 2021 at 6:25 AM Alexander Lobakin wrote: > > From: Willem de Bruijn > Date: Thu, 21 Jan 2021 21:47:47 -0500 > > > On Mon, Jan 18, 2021 at 2:33 PM Alexander Lobakin wrote: > > > > > > Commit 9fd1ff5d2ac7 ("udp: Support UDP fraglist

Re: [PATCH net-next 2/2] udp: allow forwarding of plain (non-fraglisted) UDP GRO packets

2021-01-21 Thread Willem de Bruijn
On Mon, Jan 18, 2021 at 2:33 PM Alexander Lobakin wrote: > > Commit 9fd1ff5d2ac7 ("udp: Support UDP fraglist GRO/GSO.") actually > not only added a support for fraglisted UDP GRO, but also tweaked > some logics the way that non-fraglisted UDP GRO started to work for > forwarding too. > Commit

Re: [RFC PATCH 0/7] Support for virtio-net hash reporting

2021-01-18 Thread Willem de Bruijn
> > > What it does not give is a type indication, such as > > > VIRTIO_NET_HASH_TYPE_TCPv6. I don't understand how this would be used. > > > In datapaths where the NIC has already computed the four-tuple hash > > > and stored it in skb->hash --the common case for servers--, That

Re: arch/arm64/include/asm/syscall_wrapper.h:41:25: warning: no previous prototype for '__arm64_compat_sys_epoll_pwait2'

2021-01-14 Thread Willem de Bruijn
On Thu, Jan 14, 2021 at 3:05 AM kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > master > head: 65f0d2414b7079556fbbcc070b3d1c9f9587606d > commit: b0a0c2615f6f199a656ed8549d7dce625d77aa77 epoll: wire up syscall > epoll_pwait2 > date:

Re: [RFC PATCH 0/7] Support for virtio-net hash reporting

2021-01-13 Thread Willem de Bruijn
On Tue, Jan 12, 2021 at 11:11 PM Jason Wang wrote: > > > On 2021/1/13 上午7:47, Willem de Bruijn wrote: > > On Tue, Jan 12, 2021 at 3:29 PM Yuri Benditovich > > wrote: > >> On Tue, Jan 12, 2021 at 9:49 PM Yuri Benditovich > >> wrote: > >>&g

Re: [RFC PATCH 0/7] Support for virtio-net hash reporting

2021-01-12 Thread Willem de Bruijn
o 'hash_report_type'. > > > > > > When TUN module is called later to allocate and fill the virtio-net > > > header and push it to destination virtqueue it populates the hash > > > and the hash type into virtio-net header. > > > > > > VHOST driver

Re: [PATCH 0/6] fs: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
On Mon, Jan 11, 2021 at 7:58 PM Al Viro wrote: > > On Mon, Jan 11, 2021 at 07:30:11PM -0500, Willem de Bruijn wrote: > > From: Willem de Bruijn > > > > Use in_compat_syscall() to differentiate compat handling exactly > > where needed, including in nest

[PATCH 1/6] selftests/filesystems: add initial select and poll selftest

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Add initial code coverage for select, pselect, poll and ppoll. Open a socketpair and wait for a read event. 1. run with data waiting 2. run to timeout, if a (short) timeout is specified. Also optionally pass sigset to pselect and ppoll, to exercise all datapaths. Build

[PATCH 3/6] ppoll: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Apply the same compat deduplication strategy to ppoll that was previously applied to select and pselect. Like pselect, ppoll has timespec and sigmask arguments, which have compat variants. poll has neither, so is not modified. Convert the ppoll syscall to a do_ppoll

[PATCH 2/6] select: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Select and pselect have multiple syscall implementations to handle compat and 32-bit time variants. Deduplicate core logic, which can cause divergence over time as changes may not be applied consistently. See vmalloc support in select, for one example. Handle compat

[PATCH 5/6] compat: add set_maybe_compat_user_sigmask helper

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Deduplicate the open coded branch on sigmask compat handling. Signed-off-by: Willem de Bruijn Cc: Jens Axboe --- fs/eventpoll.c | 5 + fs/io_uring.c | 9 + fs/select.c| 10 ++ include/linux/compat.h | 10 ++ 4

[PATCH 6/6] io_pgetevents: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn io_pgetevents has four variants, including compat variants of both timespec and sigmask. With set_maybe_compat_user_sigmask helper, the latter can be deduplicated. Move the shared logic to new do_io_pgetevents, analogous to do_io_getevents. Signed-off-by: Willem de

[PATCH 0/6] fs: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Use in_compat_syscall() to differentiate compat handling exactly where needed, including in nested function calls. Then remove duplicated code in callers. Changes RFC[1]->v1 - remove kselftest dependency on variant support in teardown (patch is out for rev

[PATCH 4/6] epoll: deduplicate compat logic

2021-01-11 Thread Willem de Bruijn
From: Willem de Bruijn Apply the same compat deduplication strategy to epoll that was previously applied to (p)select and ppoll. Make do_epoll_wait handle both variants of sigmask. This removes the need for near duplicate do_compat_epoll_pwait. Signed-off-by: Willem de Bruijn --- fs

Re: [PATCH v3 1/2] epoll: add nsec timeout support with epoll_pwait2

2021-01-11 Thread Willem de Bruijn
On Thu, Dec 10, 2020 at 5:59 PM Willem de Bruijn wrote: > > On Thu, Dec 10, 2020 at 3:34 PM Arnd Bergmann wrote: > > > > On Thu, Dec 10, 2020 at 6:33 PM Willem de Bruijn > > wrote: > > > On Sat, Nov 21, 2020 at 4:27 AM Arnd Bergmann wrote: > > > &g

Re: [BUG] from x86: Support kmap_local() forced debugging

2021-01-07 Thread Willem de Bruijn
On Thu, Jan 7, 2021 at 3:53 PM Steven Rostedt wrote: > > On Thu, 7 Jan 2021 11:47:02 -0800 > Linus Torvalds wrote: > > > On Wed, Jan 6, 2021 at 8:45 PM Willem de Bruijn wrote: > > > > > > But there are three other kmap_atomic callers under net/ that do

Re: [PATCH net v2] net: fix use-after-free when UDP GRO with shared fraglist

2021-01-07 Thread Willem de Bruijn
On Thu, Jan 7, 2021 at 8:33 AM Daniel Borkmann wrote: > > On 1/7/21 2:05 PM, Willem de Bruijn wrote: > > On Thu, Jan 7, 2021 at 7:52 AM Daniel Borkmann wrote: > >> On 1/7/21 12:40 PM, Dongseok Yi wrote: > >>> On 2021-01-07 20:05, Daniel Borkmann wrote: > >&

Re: [PATCH net v2] net: fix use-after-free when UDP GRO with shared fraglist

2021-01-07 Thread Willem de Bruijn
3.426844] packet_set_ring+0x5f0/0x820 > >>> [ 4443.426849] packet_setsockopt+0x5a4/0xcd0 > >>> [ 4443.426853] __sys_setsockopt+0x188/0x278 > >>> [ 4443.426858] __arm64_sys_setsockopt+0x28/0x38 > >>> [ 4443.426869] el0_svc_common+0xf0/0x1d0 > >&g

Re: [BUG] from x86: Support kmap_local() forced debugging

2021-01-06 Thread Willem de Bruijn
On Wed, Jan 6, 2021 at 9:11 PM Willem de Bruijn wrote: > > On Wed, Jan 6, 2021 at 8:49 PM Jakub Kicinski wrote: > > > > On Wed, 6 Jan 2021 17:03:48 -0800 Linus Torvalds wrote: > > > I wonder whether there is other code that "knows" about kmap() only >

Re: [BUG] from x86: Support kmap_local() forced debugging

2021-01-06 Thread Willem de Bruijn
On Wed, Jan 6, 2021 at 8:49 PM Jakub Kicinski wrote: > > On Wed, 6 Jan 2021 17:03:48 -0800 Linus Torvalds wrote: > > I wonder whether there is other code that "knows" about kmap() only > > affecting PageHighmem() pages thing that is no longer true. > > > > Looking at some other code,

Re: [PATCH net] net: fix use-after-free when UDP GRO with shared fraglist

2021-01-06 Thread Willem de Bruijn
On Tue, Jan 5, 2021 at 10:32 PM Dongseok Yi wrote: > > On 2021-01-06 12:07, Willem de Bruijn wrote: > > > > On Tue, Jan 5, 2021 at 8:29 PM Dongseok Yi wrote: > > > > > > On 2021-01-05 06:03, Willem de Bruijn wrote: > > > > > >

Re: [PATCH net] net: fix use-after-free when UDP GRO with shared fraglist

2021-01-05 Thread Willem de Bruijn
On Tue, Jan 5, 2021 at 8:29 PM Dongseok Yi wrote: > > On 2021-01-05 06:03, Willem de Bruijn wrote: > > > > On Mon, Jan 4, 2021 at 4:00 AM Dongseok Yi wrote: > > > > > > skbs in frag_list could be shared by pskb_expand_head() from BPF. > > &

Re: [PATCH net] net: fix use-after-free when UDP GRO with shared fraglist

2021-01-04 Thread Willem de Bruijn
On Mon, Jan 4, 2021 at 4:00 AM Dongseok Yi wrote: > > skbs in frag_list could be shared by pskb_expand_head() from BPF. Can you elaborate on the BPF connection? > While tcpdump, sk_receive_queue of PF_PACKET has the original frag_list. > But the same frag_list is queued to PF_INET (or PF_INET6)

Re: [PATCH] epoll: fix compat syscall wire up of epoll_pwait2

2020-12-20 Thread Willem de Bruijn
On Sun, Dec 20, 2020 at 6:43 AM Arnd Bergmann wrote: > > On Sun, Dec 20, 2020 at 11:00 AM Heiko Carstens wrote: > > > > Commit b0a0c2615f6f ("epoll: wire up syscall epoll_pwait2") wired up > > the 64 bit syscall instead of the compat variant in a couple of pl

Re: [epoll] fb72873666: WARNING:at_kernel/tracepoint.c:#tracepoint_probe_register_prio

2020-12-14 Thread Willem de Bruijn
On Mon, Dec 14, 2020 at 9:59 AM kernel test robot wrote: > > Greeting, > > FYI, we noticed the following commit (built with gcc-9): > > commit: fb728736669f7805bcc0fa1c4d578faf991d62a8 ("epoll: wire up syscall > epoll_pwait2") > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git

Re: [PATCH 1/3] Add TX sending hardware timestamp.

2020-12-11 Thread Willem de Bruijn
> >> I did not use "Fair Queue traffic policing". > >> As for ETF, it is all about ordering packets from different > >> applications. > >> How can we achive it with skiping queuing? > >> Could you elaborate on this point? > >> >>> > >> >>> The qdisc can only defer pacing

Re: [PATCH 1/3] Add TX sending hardware timestamp.

2020-12-10 Thread Willem de Bruijn
> > If I understand correctly, you are trying to achieve a single delivery time. > > The need for two separate timestamps passed along is only because the > > kernel is unable to do the time base conversion. > > Yes, a correct point. > > > > > Else, ETF could program the qdisc watchdog in system

Re: [PATCH v3 1/2] epoll: add nsec timeout support with epoll_pwait2

2020-12-10 Thread Willem de Bruijn
On Thu, Dec 10, 2020 at 3:34 PM Arnd Bergmann wrote: > > On Thu, Dec 10, 2020 at 6:33 PM Willem de Bruijn > wrote: > > On Sat, Nov 21, 2020 at 4:27 AM Arnd Bergmann wrote: > > > On Fri, Nov 20, 2020 at 11:28 PM Willem de Bruijn > > > wrote: > > > I w

Re: [PATCH 1/3] Add TX sending hardware timestamp.

2020-12-10 Thread Willem de Bruijn
On Wed, Dec 9, 2020 at 3:18 PM Geva, Erez wrote: > > > On 09/12/2020 18:37, Willem de Bruijn wrote: > > On Wed, Dec 9, 2020 at 10:25 AM Geva, Erez > > wrote: > >> > >> > >> On 09/12/2020 15:48, Willem de Bruijn wrote: > >>

Re: [PATCH v3 1/2] epoll: add nsec timeout support with epoll_pwait2

2020-12-10 Thread Willem de Bruijn
On Sat, Nov 21, 2020 at 4:27 AM Arnd Bergmann wrote: > > On Fri, Nov 20, 2020 at 11:28 PM Willem de Bruijn > wrote: > > On Fri, Nov 20, 2020 at 2:23 PM Arnd Bergmann wrote: > > > On Fri, Nov 20, 2020 at 5:01 PM Willem de Bruijn > > > wrote: > &g

Re: [PATCH 1/3] Add TX sending hardware timestamp.

2020-12-09 Thread Willem de Bruijn
On Wed, Dec 9, 2020 at 10:25 AM Geva, Erez wrote: > > > On 09/12/2020 15:48, Willem de Bruijn wrote: > > On Wed, Dec 9, 2020 at 9:37 AM Erez Geva wrote: > >> > >> Configure and send TX sending hardware timestamp from > >> user space application to the

Re: [PATCH 1/3] Add TX sending hardware timestamp.

2020-12-09 Thread Willem de Bruijn
On Wed, Dec 9, 2020 at 9:37 AM Erez Geva wrote: > > Configure and send TX sending hardware timestamp from > user space application to the socket layer, > to provide to the TC ETC Qdisc, and pass it to > the interface network driver. > > - New flag for the SO_TXTIME socket option. > - New

Re: [PATCH net-next] net: switch to storing KCOV handle directly in sk_buff

2020-11-27 Thread Willem de Bruijn
On Fri, Nov 27, 2020 at 7:26 AM Marco Elver wrote: > > On Thu, 26 Nov 2020 at 17:35, Willem de Bruijn > wrote: > > On Thu, Nov 26, 2020 at 3:19 AM Marco Elver wrote: > [...] > > > Will send v2. > > > > Does it make more sense to revert the patch that

Re: [PATCH] media: gp8psk: initialize stats at power control logic

2020-11-27 Thread Willem de Bruijn
f the proposed solution, let's just set > status = 0. > > Reported-by: syzbot > Reported-by: Willem de Bruijn > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/usb/dvb-usb/gp8psk.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --gi

Re: [PATCH net-next] net: switch to storing KCOV handle directly in sk_buff

2020-11-26 Thread Willem de Bruijn
On Thu, Nov 26, 2020 at 3:19 AM Marco Elver wrote: > > On Wed, 25 Nov 2020 at 21:43, Jakub Kicinski wrote: > > > > On Wed, 25 Nov 2020 18:34:36 +0100 Marco Elver wrote: > > > diff --git a/net/core/skbuff.c b/net/core/skbuff.c > > > index ffe3dcc0ebea..070b1077d976 100644 > > > ---

[PATCH v4 4/4] selftests/filesystems: expand epoll with epoll_pwait2

2020-11-21 Thread Willem de Bruijn
From: Willem de Bruijn Code coverage for the epoll_pwait2 syscall. epoll62: Repeat basic test epoll1, but exercising the new syscall. epoll63: Pass a timespec and exercise the timeout wakeup path. Changes v4: - fix sys_epoll_pwait2 to take __kernel_timespec (Arnd). - fix sys_epoll_pwait2

[PATCH v4 3/4] epoll: wire up syscall epoll_pwait2

2020-11-21 Thread Willem de Bruijn
From: Willem de Bruijn Split off from prev patch in the series that implements the syscall. Signed-off-by: Willem de Bruijn --- arch/alpha/kernel/syscalls/syscall.tbl | 1 + arch/arm/tools/syscall.tbl | 1 + arch/arm64/include/asm/unistd.h | 2 +- arch/arm64

[PATCH v4 1/4] epoll: convert internal api to timespec64

2020-11-21 Thread Willem de Bruijn
From: Willem de Bruijn Make epoll more consistent with select/poll: pass along the timeout as timespec64 pointer. In anticipation of additional changes affecting all three polling mechanisms: - add epoll_pwait2 syscall with timespec semantics, and share poll_select_set_timeout implementation

[PATCH v4 2/4] epoll: add syscall epoll_pwait2

2020-11-21 Thread Willem de Bruijn
From: Willem de Bruijn Add syscall epoll_pwait2, an epoll_wait variant with nsec resolution that replaces int timeout with struct timespec. It is equivalent otherwise. int epoll_pwait2(int fd, struct epoll_event *events, int maxevents, const struct

[PATCH v4 0/4] add epoll_pwait2 syscall

2020-11-21 Thread Willem de Bruijn
From: Willem de Bruijn Enable nanosecond timeouts for epoll. Analogous to pselect and ppoll, introduce an epoll_wait syscall variant that takes a struct timespec instead of int timeout. See patch 2 for more details. patch 1: pre patch cleanup: convert internal epoll to timespec64 patch 2: add

  1   2   3   4   5   6   7   >