[PATCH] x86/uaccess: use unrolled string copy for short strings

2017-06-21 Thread Paolo Abeni
2(4%) 64 51 38 13(25%) Above 64 bytes the gain fades away. Very similar values are collectd for __copy_to_user(). UDP receive performances under flood with small packets using recvfrom() increase by ~5%. Signed-off-by: Paolo Abeni --- arch/x86/include/as

Re: [RFC PATCH 0/4] softirq: Per vector threading v3

2018-01-23 Thread Paolo Abeni
Hi, On Fri, 2018-01-19 at 16:46 +0100, Frederic Weisbecker wrote: > As per Linus suggestion, this take doesn't limit the number of occurences > per jiffy anymore but instead defers a vector to workqueues as soon as > it gets re-enqueued on IRQ tail. > > No tunable here, so testing should be easie

Re: [RFC PATCH 0/4] softirq: Per vector threading v3

2018-01-23 Thread Paolo Abeni
On Tue, 2018-01-23 at 11:22 -0500, David Miller wrote: > From: Paolo Abeni > Date: Tue, 23 Jan 2018 11:13:52 +0100 > > > Hi, > > > > On Fri, 2018-01-19 at 16:46 +0100, Frederic Weisbecker wrote: > >> As per Linus suggestion, this take doesn't limi

Re: [RFC PATCH 0/4] softirq: Per vector threading v3

2018-01-24 Thread Paolo Abeni
On Tue, 2018-01-23 at 09:42 -0800, Linus Torvalds wrote: > On Tue, Jan 23, 2018 at 8:57 AM, Paolo Abeni wrote: > > > > > Or is it that the workqueue execution is simply not yielding for some > > > reason? > > > > It's like that. > > > >

Re: [RFC PATCH 0/4] softirq: Per vector threading v3

2018-01-24 Thread Paolo Abeni
On Wed, 2018-01-24 at 10:05 -0500, David Miller wrote: > From: Paolo Abeni > Date: Wed, 24 Jan 2018 15:54:05 +0100 > > > Niklas suggested a possible relation with CONFIG_IRQ_TIME_ACCOUNTING=y > > and indeed he was right. > > > > The patched kernel under test h

Re: [RFC PATCH 2/2] softirq: Per vector thread deferment

2018-01-12 Thread Paolo Abeni
afonov > Cc: Eric Dumazet > Cc: Linus Torvalds > Cc: Peter Zijlstra > Cc: Andrew Morton > Cc: David Miller > Cc: Hannes Frederic Sowa > Cc: Ingo Molnar > Cc: Levin Alexander > Cc: Paolo Abeni > Cc: Paul E. McKenney > Cc: Radu Rendec > Cc: Rik v

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

2018-03-05 Thread Paolo Abeni
On Mon, 2018-03-05 at 00:21 -0800, syzbot wrote: > Hello, > > syzbot hit the following crash on upstream commit > 5fbdefcf685defd8bc5a8f37b17538d25c58d77a (Fri Mar 2 21:05:20 2018 +) > Merge branch 'parisc-4.16-1' of > git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux > > So

Re: [PATCH] x86: only use ERMS for user copies for larger sizes

2018-11-21 Thread Paolo Abeni
On Wed, 2018-11-21 at 06:32 -0700, Jens Axboe wrote: > I did some more investigation yesterday, and found this: > > commit 236222d39347e0e486010f10c1493e83dbbdfba8 > Author: Paolo Abeni > Date: Thu Jun 29 15:55:58 2017 +0200 > > x86/uaccess: Optimize copy_user_enhan

Re: WARNING in static_key_disable_cpuslocked

2018-11-17 Thread Paolo Abeni
cffdef9288529077...@syzkaller.appspotmail.com This should be addressed in current net-next, by commit: commit 9c48060141bd937497774546e4bb89b8992be383 Author: Paolo Abeni Date: Thu Nov 15 02:34:50 2018 +0100 udp: fix jump label misuse Cheers, Paolo

Re: [PATCH net-next v2 0/4] net: mitigate retpoline overhead

2018-12-07 Thread Paolo Abeni
Hi, On Thu, 2018-12-06 at 22:28 -0800, David Miller wrote: > From: David Miller > Date: Thu, 06 Dec 2018 22:24:09 -0800 (PST) > > > Series applied, thanks! > > Erm... actually reverted. Please fix these build failures: oops ... I'm sorry for the late reply. I'm travelling and I will not able

Re: [PATCH net-next v2 1/4] indirect call wrappers: helpers to speed-up indirect calls of builtin

2018-12-07 Thread Paolo Abeni
On Fri, 2018-12-07 at 09:46 +, David Woodhouse wrote: > On Wed, 2018-12-05 at 19:13 +0100, Paolo Abeni wrote: > > +/* > > + * We can use INDIRECT_CALL_$NR for ipv6 related functions only if ipv6 is > > + * builtin, this macro simplify dealing with indirect calls wit

Re: [PATCH] add param that allows bootline control of hardened usercopy

2018-06-26 Thread Paolo Abeni
Hi, On Mon, 25 Jun 2018 11:21:38 -0700 Kees Cook wrote: > On Mon, Jun 25, 2018 at 8:08 AM, Chris von Recklinghausen > wrote: > > Enabling HARDENED_USER_COPY causes measurable regressions in the > > networking performances, up to 8% under UDP flood. > > Which function is "hot"? i.e. which copy*u

Re: [PATCH] add param that allows bootline control of hardened usercopy

2018-06-26 Thread Paolo Abeni
[hopefully fixed the 'mm' recipient] On Tue, 2018-06-26 at 09:54 -0700, Kees Cook wrote: > On Tue, Jun 26, 2018 at 2:48 AM, Paolo Abeni wrote: > > With CONFIG_HARDENED_USERCOPY=y, perf shows ~6% of CPU time spent > > cumulatively in __check_object_size (~4%) and __virt_a

Re: [PATCH] ipvlan: flush arp table when mac address changed

2018-05-14 Thread Paolo Abeni
Hi, On Sat, 2018-05-12 at 19:00 +0800, liuq...@huawei.com wrote: > From: Keefe Liu > > When master device's mac has been changed, the > commit <32c10bbfe914> "ipvlan: always use the current L2 > addr of the master" makes the IPVlan devices's mac changed > also, but it doesn't flush the IPVlan's

[PATCH net-next v3 1/4] indirect call wrappers: helpers to speed-up indirect calls of builtin

2018-12-14 Thread Paolo Abeni
implement the boilerplate for the above schema and will be used by the next patches. rfc -> v1: - use branch prediction hint, as suggested by Eric v1 -> v2: - list explicitly the builtin function names in INDIRECT_CALL_*(), as suggested by Ed Cree Suggested-by: Eric Dumazet Signed-off-by:

[PATCH net-next v3 2/4] net: use indirect call wrappers at GRO network layer

2018-12-14 Thread Paolo Abeni
LL_ changes Signed-off-by: Paolo Abeni --- include/net/inet_common.h | 2 ++ net/core/dev.c| 15 +-- net/ipv6/ip6_offload.c| 6 +++--- 3 files changed, 18 insertions(+), 5 deletions(-) diff --git a/include/net/inet_common.h b/include/net/inet_common.h index 3ca969cbd

[PATCH net-next v3 4/4] udp: use indirect call wrappers for GRO socket lookup

2018-12-14 Thread Paolo Abeni
This avoids another indirect call for UDP GRO. Again, the test for the IPv6 variant is performed first. v1 -> v2: - adapted to INDIRECT_CALL_ changes Signed-off-by: Paolo Abeni --- net/ipv4/udp_offload.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/net/i

[PATCH net-next v3 0/4] net: mitigate retpoline overhead

2018-12-14 Thread Paolo Abeni
f [2] https://linuxplumbersconf.org/event/2/contributions/99/attachments/98/117/lpc18_paper_af_xdp_perf-v2.pdf Paolo Abeni (4): indirect call wrappers: helpers to speed-up indirect calls of builtin net: use indirect call wrappers at GRO network layer net: use indirect call wrappers at GRO tra

[PATCH net-next v3 3/4] net: use indirect call wrappers at GRO transport layer

2018-12-14 Thread Paolo Abeni
some special care. v1 -> v2: - adapted to INDIRECT_CALL_ changes v2 -> v3: - fix build issue with CONFIG_IPV6=m Signed-off-by: Paolo Abeni --- include/net/inet_common.h | 7 +++ net/ipv4/af_inet.c| 13 +++-- net/ipv4/tcp_offload.c| 6 -- net/ipv4/udp_off

Re: [PATCH net-next v3 0/4] net: mitigate retpoline overhead

2018-12-16 Thread Paolo Abeni
On Sat, 2018-12-15 at 13:23 -0800, David Miller wrote: > From: Paolo Abeni > Date: Fri, 14 Dec 2018 11:51:56 +0100 > > > The spectre v2 counter-measures, aka retpolines, are a source of measurable > > overhead[1]. We can partially address that when the function pointer

Re: linux-next: build failure after merge of the net-next tree

2018-12-17 Thread Paolo Abeni
Hi, On Mon, 2018-12-17 at 12:36 +1100, Stephen Rothwell wrote: > After merging the net-next tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > In file included from net/core/dev.c:148: > net/core/dev.c: In function 'napi_gro_complete': > net/core/dev.c:5364:26: error:

Re: [PATCH net-next v2 1/4] indirect call wrappers: helpers to speed-up indirect calls of builtin

2018-12-11 Thread Paolo Abeni
Hi, I'm sorry for the long delay, I was (and still I am) diverted by some other duty. On Fri, 2018-12-07 at 21:46 +, David Woodhouse wrote: > On Fri, 2018-12-07 at 21:46 +0100, Paolo Abeni wrote: > > > I wonder if we can declare the common case functions as 'weak&#

Re: [PATCH net-next v4 2/5] page_frag: unify gfp bits for order 3 page allocation

2024-02-01 Thread Paolo Abeni
On Tue, 2024-01-30 at 19:37 +0800, Yunsheng Lin wrote: > Currently there seems to be three page frag implementions > which all try to allocate order 3 page, if that fails, it > then fail back to allocate order 0 page, and each of them > all allow order 3 page allocation to fail under certain > cond

Re: [PATCH net-next v4 2/5] page_frag: unify gfp bits for order 3 page allocation

2024-02-02 Thread Paolo Abeni
On Fri, 2024-02-02 at 10:10 +0800, Yunsheng Lin wrote: > On 2024/2/1 21:16, Paolo Abeni wrote: > > > from the __page_frag_cache_refill() allocator - which never accesses > > the memory reserves. > > I am not really sure I understand the above commemt. >

Re: [PATCH net-next v5] virtio_net: Support RX hash XDP hint

2024-02-06 Thread Paolo Abeni
On Sat, 2024-02-03 at 10:56 +0800, Liang Chen wrote: > On Sat, Feb 3, 2024 at 12:20 AM Jesper Dangaard Brouer > wrote: > > On 02/02/2024 13.11, Liang Chen wrote: [...] > > > @@ -1033,6 +1039,16 @@ static void put_xdp_frags(struct xdp_buff *xdp) > > > } > > > } > > > > > > +static void vi

Re: [PATCH net-next v3] net: dqs: add NIC stall detector based on BQL

2024-02-06 Thread Paolo Abeni
On Fri, 2024-02-02 at 08:53 -0800, Breno Leitao wrote: > From: Jakub Kicinski > > softnet_data->time_squeeze is sometimes used as a proxy for > host overload or indication of scheduling problems. In practice > this statistic is very noisy and has hard to grasp units - > e.g. is 10 squeezes a seco

Re: [PATCH net-next v5] virtio_net: Support RX hash XDP hint

2024-02-07 Thread Paolo Abeni
On Wed, 2024-02-07 at 10:54 +0800, Liang Chen wrote: > On Tue, Feb 6, 2024 at 6:44 PM Paolo Abeni wrote: > > > > On Sat, 2024-02-03 at 10:56 +0800, Liang Chen wrote: > > > On Sat, Feb 3, 2024 at 12:20 AM Jesper Dangaard Brouer > > > wrote: > > &g

Re: [PATCH net-next v5] virtio_net: Support RX hash XDP hint

2024-02-09 Thread Paolo Abeni
On Fri, 2024-02-09 at 18:39 +0800, Liang Chen wrote: > On Wed, Feb 7, 2024 at 10:27 PM Paolo Abeni wrote: > > > > On Wed, 2024-02-07 at 10:54 +0800, Liang Chen wrote: > > > On Tue, Feb 6, 2024 at 6:44 PM Paolo Abeni wrote: > > > > > > > > On

Re: [PATCH net-next v2 1/3] xsk: Remove non-zero 'dma_page' check in xp_assign_dev

2024-02-29 Thread Paolo Abeni
On Wed, 2024-02-28 at 19:05 +0800, Yunjian Wang wrote: > Now dma mappings are used by the physical NICs. However the vNIC > maybe do not need them. So remove non-zero 'dma_page' check in > xp_assign_dev. > > Signed-off-by: Yunjian Wang > --- > net/xdp/xsk_buff_pool.c | 7 --- > 1 file change

Re: [PATCH net-next v2 2/3] vhost_net: Call peek_len when using xdp

2024-02-29 Thread Paolo Abeni
On Wed, 2024-02-28 at 19:05 +0800, Yunjian Wang wrote: > If TUN supports AF_XDP TX zero-copy, the XDP program will enqueue > packets to the XDP ring and wake up the vhost worker. This requires > the vhost worker to call peek_len(), which can be used to consume > XDP descriptors. > > Signed-off-by:

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

2024-02-29 Thread Paolo Abeni
On Wed, 2024-02-28 at 19:05 +0800, Yunjian Wang wrote: > @@ -2661,6 +2776,54 @@ static int tun_ptr_peek_len(void *ptr) > } > } > > +static void tun_peek_xsk(struct tun_file *tfile) > +{ > + struct xsk_buff_pool *pool; > + u32 i, batch, budget; > + void *frame; > + > + if (!

Re: [PATCH net-next v6 5/5] tools: virtio: introduce vhost_net_test

2024-03-05 Thread Paolo Abeni
On Wed, 2024-02-28 at 17:30 +0800, Yunsheng Lin wrote: > introduce vhost_net_test for both vhost_net tx and rx basing > on virtio_test to test vhost_net changing in the kernel. > > Steps for vhost_net tx testing: > 1. Prepare a out buf. > 2. Kick the vhost_net to do tx processing. > 3. Do the rece

Re: [PATCH] net: hns3: tracing: fix hclgevf trace event strings

2024-03-14 Thread Paolo Abeni
'&' of a char array is not compatible with strcmp(). > > The '&' is not necessary, remove it. > > Fixes: d8355240cf8fb ("net: hns3: add trace event support for PF/VF mailbox") checkpactch in strict mode complains the hash is not 12 char long. > Signed-off-by: Steven Rostedt (Google) FWIW Acked-by: Paolo Abeni

Re: [PATCH] net: make config lines follow common pattern

2023-11-27 Thread Paolo Abeni
On Thu, 2023-11-23 at 12:12 +0100, Lukas Bulwahn wrote: > The Kconfig parser is quite relaxed when parsing config definition lines. > However, there are just a few config definition lines that do not follow > the common regular expression 'config [0-9A-Z]', i.e., there are only a few > cases where

Re: [PATCH v4 net-next 1/7] net: add rx_sk to trace_kfree_skb

2024-06-14 Thread Paolo Abeni
On Wed, 2024-06-12 at 09:59 +0200, Jesper Dangaard Brouer wrote: > > On 11/06/2024 22.11, Yan Zhai wrote: > > skb does not include enough information to find out receiving > > sockets/services and netns/containers on packet drops. In theory > > skb->dev tells about netns, but it can get cleared/re

Re: [PATCH v5 3/4] net: dropreason: use new __print_sym() in tracing

2024-06-18 Thread Paolo Abeni
On Fri, 2024-06-14 at 10:19 +0200, Johannes Berg wrote: > From: Johannes Berg > > The __print_symbolic() could only ever print the core > drop reasons, since that's the way the infrastructure > works. Now that we have __print_sym() with all the > advantages mentioned in that commit, convert to th

Re: [PATCH v4] tracing/net_sched: NULL pointer dereference in perf_trace_qdisc_reset()

2024-06-27 Thread Paolo Abeni
On Tue, 2024-06-25 at 02:33 +0900, ysk...@gmail.com wrote: > From: Yunseong Kim > > In the TRACE_EVENT(qdisc_reset) NULL dereference occurred from > > qdisc->dev_queue->dev ->name > > This situation simulated from bunch of veths and Bluetooth disconnection > and reconnection. > > During qdis

Re: [PATCH] ptp: Add vDSO-style vmclock support

2024-07-25 Thread Paolo Abeni
Hi, Just a bunch of 'nits below On 7/24/24 19:16, David Woodhouse wrote: diff --git a/drivers/ptp/ptp_vmclock.c b/drivers/ptp/ptp_vmclock.c new file mode 100644 index ..9c508c21c062 --- /dev/null +++ b/drivers/ptp/ptp_vmclock.c [...] +/* + * Multiply a 64-bit count by a 64-bit t

Re: [PATCH v5] ptp: Add support for the AMZNC10C 'vmclock' device

2024-09-03 Thread Paolo Abeni
On 8/23/24 12:09, David Woodhouse wrote: diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig index 604541dcb320..e98c9767e0ef 100644 --- a/drivers/ptp/Kconfig +++ b/drivers/ptp/Kconfig @@ -131,6 +131,19 @@ config PTP_1588_CLOCK_KVM To compile this driver as a module, choose M here: t

Re: WARNING in ip_recv_error

2018-05-24 Thread Paolo Abeni
On Wed, 2018-05-23 at 11:40 -0400, Willem de Bruijn wrote: > On Sun, May 20, 2018 at 7:13 PM, Willem de Bruijn > wrote: > > On Fri, May 18, 2018 at 2:59 PM, Willem de Bruijn > > wrote: > > > On Fri, May 18, 2018 at 2:46 PM, Willem de Bruijn > > > wrote: > > > > On Fri, May 18, 2018 at 2:44 PM, W

Re: [PATCH] net: improve ipv4 performances

2018-04-04 Thread Paolo Abeni
On Sun, 2018-04-01 at 20:31 +0200, Anton Gary Ceph wrote: > After a few profiling and analysis, turned out that the ethertype field > of the packets has the following distribution: [...] > 0.6% don't know/no opinion Am I the only one finding the submission date and the above info suspicious ?

Re: [v3 net-next 08/10] skbuff: reuse NAPI skb cache on allocation path (__build_skb())

2021-02-10 Thread Paolo Abeni
Hello, I'm sorry for the late feedback, I could not step-in before. Also adding Jesper for awareness, as he introduced the bulk free infrastructure. On Tue, 2021-02-09 at 20:48 +, Alexander Lobakin wrote: > @@ -231,7 +256,7 @@ struct sk_buff *__build_skb(void *data, unsigned int > frag_size

Re: [v3 net-next 08/10] skbuff: reuse NAPI skb cache on allocation path (__build_skb())

2021-02-10 Thread Paolo Abeni
On Wed, 2021-02-10 at 12:25 +, Alexander Lobakin wrote: > Paolo Abeni on Wed, 10 Feb 2021 11:21:06 +0100 wrote: > > Perhaps giving the device drivers the ability to opt-in on this infra > > via a new helper - as done back then with napi_consume_skb() - would > > m

Re: [PATCH v4 net-next 09/11] skbuff: allow to optionally use NAPI cache from __alloc_skb()

2021-02-11 Thread Paolo Abeni
On Wed, 2021-02-10 at 16:30 +, Alexander Lobakin wrote: > Reuse the old and forgotten SKB_ALLOC_NAPI to add an option to get > an skbuff_head from the NAPI cache instead of inplace allocation > inside __alloc_skb(). > This implies that the function is called from softirq or BH-off > context, no

Re: [PATCH v4 net-next 09/11] skbuff: allow to optionally use NAPI cache from __alloc_skb()

2021-02-11 Thread Paolo Abeni
On Thu, 2021-02-11 at 14:28 +, Alexander Lobakin wrote: > From: Paolo Abeni on Thu, 11 Feb 2021 11:16:40 +0100 > wrote: > > What about changing __napi_alloc_skb() to always use > > the __napi_build_skb(), for both kmalloc and page backed skbs? That is, > > always doi

Re: [MPTCP] KASAN: use-after-free Read in mptcp_established_options

2021-02-17 Thread Paolo Abeni
On Wed, 2021-02-17 at 09:30 -0800, syzbot wrote: > syzbot found the following issue on: > > HEAD commit:966df6de lan743x: sync only the received area of an rx rin.. > git tree: net-next > console output: https://syzkaller.appspot.com/x/log.txt?x=11afe082d0 > kernel config: https://s

Re: KASAN: use-after-free Read in tcp_current_mss

2021-02-17 Thread Paolo Abeni
On Wed, 2021-02-17 at 07:36 -0800, syzbot wrote: > Hello, > > syzbot found the following issue on: > > HEAD commit:773dc50d Merge branch 'Xilinx-axienet-updates' > git tree: net-next > console output: https://syzkaller.appspot.com/x/log.txt?x=13460822d0 > kernel config: https://syz

Re: possible deadlock in mptcp_push_pending

2021-02-18 Thread Paolo Abeni
On Wed, 2021-02-17 at 09:31 -0800, syzbot wrote: > syzbot found the following issue on: > > HEAD commit:c48f8607 Merge branch 'PTP-for-DSA-tag_ocelot_8021q' > git tree: net-next > console output: https://syzkaller.appspot.com/x/log.txt?x=16525cb0d0 > kernel config: https://syzkaller

Re: [PATCH net-next] mptcp: use mptcp_for_each_subflow in mptcp_stream_accept

2020-08-03 Thread Paolo Abeni
tcp_subflow_tcp_sock(subflow); > > if (!ssk->sk_socket) Acked-by: Paolo Abeni

Re: [PATCH] net: openvswitch: silence suspicious RCU usage warning

2020-08-05 Thread Paolo Abeni
.@syzkaller.appspotmail.com > Reported-by: syzbot+f612c02823acb02ff...@syzkaller.appspotmail.com > Signed-off-by: Tonghao Zhang Acked-by: Paolo Abeni

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

2021-01-13 Thread Paolo Abeni
On Wed, 2021-01-13 at 10:32 +, 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 2e4e

Re: [PATCH net-next 2/4] gro: add combined call_gro_receive() + INDIRECT_CALL_INET() helper

2021-03-19 Thread Paolo Abeni
Hello, On Thu, 2021-03-18 at 18:42 +, Alexander Lobakin wrote: > call_gro_receive() is used to limit GRO recursion, but it works only > with callback pointers. > There's a combined version of call_gro_receive() + INDIRECT_CALL_2() > in , but it doesn't check for IPv6 modularity. AFAICS, ip6_o

Re: [PATCH net-next 2/4] gro: add combined call_gro_receive() + INDIRECT_CALL_INET() helper

2021-03-19 Thread Paolo Abeni
On Fri, 2021-03-19 at 11:43 +, Alexander Lobakin wrote: > I'm not sure if you did it on purpose in commit aaa5d90b395a7 > ("net: use indirect call wrappers at GRO network layer"). > Was that intentional I must admit that 2y+ later my own intentions are not so clear to me too;) > for the sake

Re: [PATCH] udp: Add support for getsockopt(..., ..., UDP_GRO, ..., ...)

2021-03-31 Thread Paolo Abeni
On Wed, 2021-03-31 at 15:10 +0200, Norman Maurer wrote: > As this missing change was most likely an oversight in the original > commit I do think it should go into 5.12 and subsequently stable as > well. That’s also the reason why I didn’t send a v2 and changed the > commit message / subject for th

Re: [PATCH] udp: Add support for getsockopt(..., ..., UDP_GRO, ..., ...)

2021-03-26 Thread Paolo Abeni
Hello, On Thu, 2021-03-25 at 20:56 +0100, Norman Maurer wrote: > From: Norman Maurer > > Support for UDP_GRO was added in the past but the implementation for > getsockopt was missed which did lead to an error when we tried to > retrieve the setting for UDP_GRO. This patch adds the missing switch

Re: [PATCH] udp: Add support for getsockopt(..., ..., UDP_GRO, ..., ...)

2021-03-26 Thread Paolo Abeni
On Fri, 2021-03-26 at 11:22 +0100, Norman Maurer wrote: > On 26. Mar 2021, at 10:36, Paolo Abeni wrote: > > One thing you can do to simplifies the maintainer's life, would be post > > a v2 with the correct tag (and ev. obsolete this patch in patchwork). > > I am quite

Re: [PATCH] net/core: use xx_zalloc instead xx_alloc and memset

2020-11-18 Thread Paolo Abeni
On Wed, 2020-11-18 at 16:15 +0800, Tian Tao wrote: > use kmem_cache_zalloc instead kmem_cache_alloc and memset. > > Signed-off-by: Tian Tao > --- > net/core/skbuff.c | 10 +++--- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/net/core/skbuff.c b/net/core/skbuff.c > index

Re: possible deadlock in ipv6_sock_mc_close

2021-03-01 Thread Paolo Abeni
Hello, On Mon, 2021-03-01 at 14:52 +, Chuck Lever wrote: > > On Mar 1, 2021, at 8:49 AM, syzbot > > wrote: > > > > Hello, > > > > syzbot found the following issue on: > > > > HEAD commit:eee7ede6 Merge branch 'bnxt_en-error-recovery-bug-fixes' > > git tree: net > > console outpu

Re: [PATCH v4 11/13] task_isolation: net: don't flush backlog on CPUs running isolated tasks

2021-01-22 Thread Paolo Abeni
On Fri, 2021-01-22 at 11:13 -0300, Marcelo Tosatti wrote: > On Thu, Oct 01, 2020 at 04:47:31PM +0200, Frederic Weisbecker wrote: > > On Wed, Jul 22, 2020 at 02:58:24PM +, Alex Belits wrote: > > > From: Yuri Norov > > > > > > so we don't need to flush it. > > > > What guarantees that we have

[RFC PATCH 2/2] net: add sysfs attribute to control napi threaded mode

2016-05-10 Thread Paolo Abeni
this patch addis a new sysfs attribute to the network device class. Said attribute is a bitmask that allows controlling the threaded mode for all the napi instances of the given network device. The threaded mode can be switched only if related network device is down. Signed-off-by: Paolo Abeni

[RFC PATCH 1/2] net: implement threaded-able napi poll loop support

2016-05-10 Thread Paolo Abeni
is started, napi_schedule() will wake-up such thread instead of scheduling the softirq. The threaded poll loop behaves quite likely the net_rx_action, but it does not have to manipulate local irqs and uses an explicit scheduling point based on netdev_budget. Signed-off-by: Paolo Abeni Signed-off

[RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Paolo Abeni
pervisor uses the L4 header to compute the rx hash. Paolo Abeni (2): net: implement threaded-able napi poll loop support net: add sysfs attribute to control napi threaded mode include/linux/netdevice.h | 4 ++ net/core/dev.c| 113 ++

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Paolo Abeni
Hi, On Tue, 2016-05-10 at 07:29 -0700, Eric Dumazet wrote: > On Tue, 2016-05-10 at 16:11 +0200, Paolo Abeni wrote: > > Currently, the softirq loop can be scheduled both inside the ksofirqd kernel > > thread and inside any running process. This makes nearly impossible for the > &

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Paolo Abeni
On Tue, 2016-05-10 at 09:08 -0700, Eric Dumazet wrote: > On Tue, 2016-05-10 at 18:03 +0200, Paolo Abeni wrote: > > > If a single core host is under network flood, i.e. ksoftirqd is > > scheduled and it eventually (after processing ~640 packets) will let the > > user space

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Paolo Abeni
On Tue, 2016-05-10 at 17:57 +0200, Thomas Gleixner wrote: > On Tue, 10 May 2016, Paolo Abeni wrote: > > Nice patch set and very promising results! > > > At this point we are not really sure if we should go with this simpler > > approach by putting NAPI itself into

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-11 Thread Paolo Abeni
Hi Eric, On Tue, 2016-05-10 at 15:51 -0700, Eric Dumazet wrote: > On Wed, 2016-05-11 at 00:32 +0200, Hannes Frederic Sowa wrote: > > > Not only did we want to present this solely as a bugfix but also as as > > performance enhancements in case of virtio (as you can see in the cover > > letter). Giv

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-11 Thread Paolo Abeni
On Wed, 2016-05-11 at 06:08 -0700, Eric Dumazet wrote: > On Wed, 2016-05-11 at 11:48 +0200, Paolo Abeni wrote: > > Hi Eric, > > On Tue, 2016-05-10 at 15:51 -0700, Eric Dumazet wrote: > > > On Wed, 2016-05-11 at 00:32 +0200, Hannes Frederic Sowa wrote: > > > > &

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-12 Thread Paolo Abeni
On Wed, 2016-05-11 at 14:56 -0700, Eric Dumazet wrote: > On Wed, 2016-05-11 at 08:55 +0200, Peter Zijlstra wrote: > > On Tue, May 10, 2016 at 03:51:37PM -0700, Eric Dumazet wrote: > > > diff --git a/kernel/softirq.c b/kernel/softirq.c > > > index 17caf4b63342..22463217e3cf 100644 > > > --- a/kernel

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-12 Thread Paolo Abeni
On Thu, 2016-05-12 at 13:49 -0700, Eric Dumazet wrote: > On Thu, 2016-05-12 at 22:07 +0200, Paolo Abeni wrote: > > > > > static inline bool ksoftirqd_running(void) > > > > { > > > > return __this_cpu_read(ksoftirqd)->state == TASK_RUNNING; >

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-13 Thread Paolo Abeni
On Wed, 2016-05-11 at 14:56 -0700, Eric Dumazet wrote: > On Wed, 2016-05-11 at 08:55 +0200, Peter Zijlstra wrote: > > On Tue, May 10, 2016 at 03:51:37PM -0700, Eric Dumazet wrote: > > > diff --git a/kernel/softirq.c b/kernel/softirq.c > > > index 17caf4b63342..22463217e3cf 100644 > > > --- a/kernel

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-13 Thread Paolo Abeni
On Fri, 2016-05-13 at 10:03 -0700, Eric Dumazet wrote: > On Fri, May 13, 2016 at 9:50 AM, Paolo Abeni wrote: > > >> Indeed, and the patch looks quite simple now ;) > >> > >> diff --git a/kernel/softirq.c b/kernel/softirq.c > >> index >

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-16 Thread Paolo Abeni
On Fri, 2016-05-13 at 10:36 -0700, Eric Dumazet wrote: > On Fri, May 13, 2016 at 10:19 AM, Paolo Abeni wrote: > > > The difference is small, in the noise range: > > > > [with this patch applied] > > super_netperf 100 -H 192.168.122.1 -t UDP_STREAM -l 60 -- -m 1 >

[PATCH net] ipv4/fib: don't warn when primary address is missing if in_dev is dead

2016-04-21 Thread Paolo Abeni
t; warnings. This commit avoids warning if the in_dev is dead Signed-off-by: Paolo Abeni --- net/ipv4/fib_frontend.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c index 8a9246d..63566ec 100644 --- a/net/ipv4/fib_

Re: [RFC -next 2/2] virtio_net: Read and use the advised MTU

2016-03-10 Thread Paolo Abeni
On Thu, 2016-03-10 at 09:28 -0500, Aaron Conole wrote: > This patch checks the feature bit for the VIRTIO_NET_F_MTU feature. If it > exists, read the advised MTU and use it. > > No proper error handling is provided for the case where a user changes the > negotiated MTU. A future commit will add pr

[PATCH net-next] security: drop the unused hook skb_owned_by

2016-04-08 Thread Paolo Abeni
ithout any users nor caller. This commit removes the said left-over field. Fixes: b1d9e6b0646d ("LSM: Switch to lists of hooks") Signed-off-by: Paolo Abeni --- include/linux/lsm_hooks.h | 1 - security/security.c | 1 - 2 files changed, 2 deletions(-) diff --git a/include/lin

Re: Packet gets stuck in NOLOCK pfifo_fast qdisc

2019-10-09 Thread Paolo Abeni
On Wed, 2019-10-09 at 08:46 +0200, Jonas Bonn wrote: > Hi, > > The lockless pfifo_fast qdisc has an issue with packets getting stuck in > the queue. What appears to happen is: > > i) Thread 1 holds the 'seqlock' on the qdisc and dequeues packets. > ii) Thread 1 dequeues the last packet in the

Re: [PATCH net-next] net: ipvlan: forward ingress packet to slave's l2 in l3s mode

2019-06-26 Thread Paolo Abeni
Hi, On Tue, 2019-06-25 at 14:42 +0800, Zhiyuan Hou wrote: > In ipvlan l3s mode, ingress packet is switched to slave interface and > delivers to l4 stack. This may cause two problems: > > 1. When slave is in an ns different from master, the behavior of stack > in slave ns may cause confusion

Re: [BUG] v4.20 - bridge not getting DHCP responses? (works in 4.19.13)

2019-01-10 Thread Paolo Abeni
On Thu, 2019-01-10 at 01:38 +0100, Ian Kumlien wrote: > Confirmed, sending a new mail with summary etc > > On Thu, Jan 10, 2019 at 1:16 AM Ian Kumlien wrote: > > On Wed, Jan 9, 2019 at 12:17 AM Ian Kumlien wrote: > > > On Wed, Jan 9, 2019, 00:09 Florian Fainelli > > > [--8<---] > > > > > > >

Re: [BUG] moving fq back to clock monotonic breaks my setup

2019-01-10 Thread Paolo Abeni
On Thu, 2019-01-10 at 09:25 +0100, Ian Kumlien wrote: > On Thu, Jan 10, 2019 at 6:53 AM Eric Dumazet wrote: > > On Wed, Jan 9, 2019 at 4:48 PM Ian Kumlien wrote: > > > Hi, > > > > > > Just been trough ~5+ hours of bisecting and eventually actually found > > > the culprit =) > > > > > > commit f

Re: [bug, bisected] pfifo_fast causes packet reordering

2018-03-19 Thread Paolo Abeni
Hi, On Fri, 2018-03-16 at 11:26 +0100, Jakob Unterwurzacher wrote: > On 15.03.18 23:30, John Fastabend wrote: > > > I have reproduced it using two USB network cards connected to each other. > > > The test tool sends UDP packets containing a counter and listens on the > > > other interface, it is

[PATCH 1/5] genirq: implement support for runtime switch to threaded irqs

2016-06-15 Thread Paolo Abeni
t irq mode change, can register a notifier with irq_set_mode_notifier(). Such notifier will be invoked in atomic context just after each irq reconfiguration. Signed-off-by: Paolo Abeni Signed-off-by: Hannes Frederic Sowa --- include/linux/interrupt.h | 15 kernel/irq/internals.h| 3 +

[PATCH 3/5] sched/preempt: cond_resched_softirq() must check for softirq

2016-06-15 Thread Paolo Abeni
Currently cond_resched_softirq() fails to reschedule if there are pending softirq but no other running process. This happens i.e. when receiving an interrupt with local bh disabled. Reported-by: Eric Dumazet Signed-off-by: Paolo Abeni Signed-off-by: Hannes Frederic Sowa --- kernel/sched

[PATCH 0/5] genirq: threadable IRQ support

2016-06-15 Thread Paolo Abeni
to adjust the weight of the networking kthreads, preferring certain networking cards or setting cpus affinity on packet processing threads. Maybe also using deadline scheduling or other scheduler features might be worthwhile. * scheduler statistics can be used to observe network packet processi

[PATCH 5/5] ixgbe: add support for threadable rx irq

2016-06-15 Thread Paolo Abeni
Plug-in the threadable irq infrastructure to allow run-time configuration of rx irqs, when msix irqs are used. Signed-off-by: Paolo Abeni Signed-off-by: Hannes Frederic Sowa --- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion

[PATCH 2/5] genirq: add flags for controlling the default threaded irq behavior

2016-06-15 Thread Paolo Abeni
scheduled under SCHED_FIFO. Setting the IRQF_TH_SCHED_NORMAL flag on irq will cause the irq thread to run by default under the SCHED_NORMAL scheduler. Signed-off-by: Paolo Abeni Signed-off-by: Hannes Frederic Sowa --- include/linux/interrupt.h | 6 ++ kernel/irq/manage.c | 17

[PATCH 4/5] netdev: implement infrastructure for threadable napi irq

2016-06-15 Thread Paolo Abeni
mode change handler which actually set napi->thread and register it after requesting the irq. Signed-off-by: Paolo Abeni Signed-off-by: Hannes Frederic Sowa --- include/linux/netdevice.h | 4 net/core/dev.c| 59 +++ 2 files changed,

Re: [PATCH 3/5] sched/preempt: cond_resched_softirq() must check for softirq

2016-06-15 Thread Paolo Abeni
On Wed, 2016-06-15 at 15:48 +0200, Peter Zijlstra wrote: > On Wed, Jun 15, 2016 at 03:42:04PM +0200, Paolo Abeni wrote: > > Currently cond_resched_softirq() fails to reschedule if there > > are pending softirq but no other running process. This happens > > i.e. when receiv

Re: [PATCH 4/5] netdev: implement infrastructure for threadable napi irq

2016-06-15 Thread Paolo Abeni
On Wed, 2016-06-15 at 07:17 -0700, Eric Dumazet wrote: > On Wed, Jun 15, 2016 at 6:42 AM, Paolo Abeni wrote: > > This commit adds the infrastructure needed for threadable > > rx interrupt. A reference to the irq thread is used to > > mark the threaded irq mode. > > In

Re: [PATCH 4/5] netdev: implement infrastructure for threadable napi irq

2016-06-16 Thread Paolo Abeni
On Wed, 2016-06-15 at 10:04 -0700, Eric Dumazet wrote: > On Wed, Jun 15, 2016 at 9:42 AM, Paolo Abeni wrote: > > On Wed, 2016-06-15 at 07:17 -0700, Eric Dumazet wrote: > > >> > >> I really appreciate the effort, but as I already said this is not going to > &

Re: [PATCH 4/5] netdev: implement infrastructure for threadable napi irq

2016-06-16 Thread Paolo Abeni
On Thu, 2016-06-16 at 04:19 -0700, Eric Dumazet wrote: > On Thu, Jun 16, 2016 at 3:39 AM, Paolo Abeni wrote: > > We used a different setup to explicitly avoid the (guest) userspace > > starvation issue. Using a guest with 2vCPUs (or more) and a single queue > > avoids

Re: [PATCH] mm: page_alloc: High-order per-cpu page allocator v3

2016-12-01 Thread Paolo Abeni
On Thu, 2016-12-01 at 18:34 +0100, Jesper Dangaard Brouer wrote: > (Cc. netdev, we might have an issue with Paolo's UDP accounting and > small socket queues) > > On Wed, 30 Nov 2016 16:35:20 + > Mel Gorman wrote: > > > > I don't quite get why you are setting the socket recv size > > > (with

Re: [PATCH] mm: page_alloc: High-order per-cpu page allocator v3

2016-12-02 Thread Paolo Abeni
On Fri, 2016-12-02 at 16:37 +0100, Jesper Dangaard Brouer wrote: > On Thu, 01 Dec 2016 23:17:48 +0100 > Paolo Abeni wrote: > > > On Thu, 2016-12-01 at 18:34 +0100, Jesper Dangaard Brouer wrote: > > > (Cc. netdev, we might have an issue with Paolo's UDP accounting

Re: net: deadlock between ip_expire/sch_direct_xmit

2017-03-14 Thread Paolo Abeni
On Tue, 2017-03-14 at 07:41 -0700, Eric Dumazet wrote: > On Tue, 2017-03-14 at 14:31 +0100, Dmitry Vyukov wrote: > > Hello, > > > > I've got the following deadlock report while running syzkaller fuzzer > > on net-next/92cd12c5ed432c5eebd2462d666772a8d8442c3b: > > > > > > [ INFO: possible circula

Re: net: deadlock between ip_expire/sch_direct_xmit

2017-03-14 Thread Paolo Abeni
On Tue, 2017-03-14 at 08:09 -0700, Eric Dumazet wrote: > On Tue, Mar 14, 2017 at 8:03 AM, Paolo Abeni wrote: > > > I'm wondering if we really need to keep the fragment queue lock held > > while sending the icmp packet ? we hold a reference to the struct, so > > it

Re: [PATCH] kbuild/mkspec: clean boot loader configuration on rpm removal

2016-03-03 Thread Paolo Abeni
On Wed, 2016-03-02 at 10:09 -0500, Josh Boyer wrote: > On Wed, Mar 2, 2016 at 9:38 AM, Hannes Frederic Sowa > wrote: > > On 02.03.2016 15:28, Paolo Abeni wrote: > >> > >> This patch add a rpm preuninstall scriptlet to cleanup the > >> boot loader co

[PATCH] kbuild/mkspec: clean boot loader configuration on rpm removal

2016-03-02 Thread Paolo Abeni
This patch add a rpm preuninstall scriptlet to cleanup the boot loader configuration on kernel package uninstall. The initrd for the to-be-removed kernel is deleted, too. Signed-off-by: Paolo Abeni --- scripts/package/mkspec | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts

Re: Packet gets stuck in NOLOCK pfifo_fast qdisc

2020-07-02 Thread Paolo Abeni
Hi all, On Thu, 2020-07-02 at 08:14 +0200, Jonas Bonn wrote: > Hi Cong, > > On 01/07/2020 21:58, Cong Wang wrote: > > On Wed, Jul 1, 2020 at 9:05 AM Cong Wang wrote: > > > On Tue, Jun 30, 2020 at 2:08 PM Josh Hunt wrote: > > > > Do either of you know if there's been any development on a fix for

Re: Packet gets stuck in NOLOCK pfifo_fast qdisc

2020-07-07 Thread Paolo Abeni
On Thu, 2020-07-02 at 11:08 -0700, Josh Hunt wrote: > On 7/2/20 2:45 AM, Paolo Abeni wrote: > > Hi all, > > > > On Thu, 2020-07-02 at 08:14 +0200, Jonas Bonn wrote: > > > Hi Cong, > > > > > > On 01/07/2020 21:58, Cong Wang wrote: > > > &

Re: [PATCH net-next 2/2] ipv6: fib6: avoid indirect calls from fib6_rule_lookup

2020-06-22 Thread Paolo Abeni
On Mon, 2020-06-22 at 09:25 -0700, Brian Vazquez wrote: > > Hi Paolo > On Mon, Jun 22, 2020 at 3:13 AM Paolo Abeni wrote: > > Hi, > > > > On Fri, 2020-06-19 at 20:14 -0700, Brian Vazquez wrote: > > > @@ -111,11 +111,13 @@ struct dst_entry *fib6_rule_lookup(s

Re: [PATCH net-next 2/2] ipv6: fib6: avoid indirect calls from fib6_rule_lookup

2020-06-23 Thread Paolo Abeni
On Mon, 2020-06-22 at 12:26 -0700, Brian Vazquez wrote: > > > On Mon, Jun 22, 2020 at 11:00 AM Paolo Abeni wrote: > > On Mon, 2020-06-22 at 09:25 -0700, Brian Vazquez wrote: > > > > > > Hi Paolo > > > On Mon, Jun 22, 2020 at 3:13 AM Paolo Abeni wrote:

  1   2   >