Add a new test to ensure that when the transport changes a null pointer
dereference does not occur[1].
Note that this test does not fail, but it may hang on the client side if
it triggers a kernel oops.
This works by creating a socket, trying to connect to a server, and then
executing a second co
Reviewed-by: Xuan Zhuo
On Fri, 7 Mar 2025 01:12:08 +, Joe Damato wrote:
> Greetings:
>
> Welcome to v6. Only patch updated is patch 3. See changelog below.
>
> Jakub recently commented [1] that I should not hold this series on
> virtio-net linking queues to NAPIs behind other important work
On Wed, Mar 05, 2025 at 04:12:18PM +, Cosmin Ratiu wrote:
> +++ b/drivers/net/bonding/bond_main.c
> @@ -613,8 +613,11 @@ static void bond_ipsec_del_sa_all(struct bonding
> *bond)
>
> mutex_lock(&bond->ipsec_lock);
> list_for_each_entry(ipsec, &bond->ipsec_list, list) {
> -
Greetings:
Welcome to v6. Only patch updated is patch 3. See changelog below.
Jakub recently commented [1] that I should not hold this series on
virtio-net linking queues to NAPIs behind other important work that is
on-going and suggested I re-spin, so here we are :)
As per the discussion on th
Refactor virtnet_napi_enable and virtnet_napi_tx_enable to take a struct
receive_queue. Create a helper, virtnet_napi_do_enable, which contains
the logic to enable a NAPI.
Signed-off-by: Joe Damato
Acked-by: Michael S. Tsirkin
Acked-by: Jason Wang
Tested-by: Lei Yang
---
drivers/net/virtio_ne
Create virtnet_napi_disable helper and refactor virtnet_napi_tx_disable
to take a struct send_queue.
Signed-off-by: Joe Damato
Acked-by: Michael S. Tsirkin
Acked-by: Jason Wang
Tested-by: Lei Yang
---
drivers/net/virtio_net.c | 25 +
1 file changed, 17 insertions(+), 8
On Thu, Mar 06, 2025 at 06:28:58PM +0100, Christophe Leroy wrote:
>
>
> Le 06/03/2025 à 14:13, Petr Pavlu a écrit :
> > Section .static_call_sites holds data structures that need to be sorted and
> > processed only at module load time. This initial processing happens in
> > static_call_add_module
On Thu, Mar 06, 2025 at 05:09:33PM +0100, Luigi Leonardi wrote:
Add a new test to ensure that when the transport changes a null pointer
dereference does not occur[1].
Note that this test does not fail, but it may hang on the client side if
it triggers a kernel oops.
This works by creating a soc
On Thu, Mar 06, 2025 at 01:37:15PM +, Cosmin Ratiu wrote:
> On Thu, 2025-03-06 at 10:02 +, Hangbin Liu wrote:
> > > For bond_ipsec_add_sa_all(), I will move the xso.real_dev =
> > > real_dev
> > > after .xdo_dev_state_add() in case the following situation.
>
> xso.real_dev needs to be init
On 3/7/25 05:19, Hangbin Liu wrote:
> The fixed commit placed mutex_lock() inside spin_lock_bh(), which triggers
> a warning:
>
> BUG: sleeping function called from invalid context at...
>
> Fix this by moving the IPsec deletion operation to bond_ipsec_free_sa,
> which is not held by spin_lock_
This introduces a test for IPSec offload over bonding, utilizing netdevsim
for the testing process, as veth interfaces do not support IPSec offload.
The test will ensure that the IPSec offload functionality remains operational
even after a failover event occurs in the bonding configuration.
Here i
The active-backup bonding mode supports XFRM ESP offload. However, when
a bond is added using command like `ip link add bond0 type bond mode 1
miimon 100`, the `ethtool -k` command shows that the XFRM ESP offload is
disabled. This occurs because, in bond_newlink(), we change bond link
first and reg
Use persistent NAPI config so that NAPI IDs are not renumbered as queue
counts change.
$ sudo ethtool -l ens4 | tail -5 | egrep -i '(current|combined)'
Current hardware settings:
Combined: 4
$ ./tools/net/ynl/pyynl/cli.py \
--spec Documentation/netlink/specs/netdev.yaml \
--dump qu
On 1/15/25 03:24, Petr Mladek wrote:
> This commit updates the livepatch documentation to reflect recent changes
> in the behavior of states, callbacks, and shadow variables.
>
> Key changes include:
>
> - Per-state callbacks replace per-object callbacks, invoked only when a
> livepatch introdu
> Subject: [PATCH v4 7/8] reset: imx8mp-audiomix: Add support for DSP
> run/stall
>
> We can Run/Stall the DSP via audio block control bits found in
> audiomix.
> Implement this functionality using the reset controller and use assert
> for Stall and deassert for Run.
>
> Reviewed-by: Frank Li
>
On 28.02.25 17:54, Brendan Jackman wrote:
Some filesystems don't support funtract()ing unlinked files. They return
ENOENT. In that case, skip the test.
That's not documented in the man page, so is this a bug of these
filesystems?
What are examples for these weird filesystems?
As we have th
On Wed 2025-03-05 10:57:47, Tamir Duberstein wrote:
> On Wed, Mar 5, 2025 at 10:55 AM Andy Shevchenko
> wrote:
> >
> > On Wed, Mar 05, 2025 at 10:25:51AM -0500, Tamir Duberstein wrote:
> > > On Wed, Mar 5, 2025 at 10:01 AM Petr Mladek wrote:
> > > > On Fri 2025-02-14 11:20:01, Tamir Duberstein wr
On Wed, Mar 05, 2025 at 04:12:18PM +, Cosmin Ratiu wrote:
> On Wed, 2025-03-05 at 14:13 +, Hangbin Liu wrote:
> > On Wed, Mar 05, 2025 at 10:38:36AM +0200, Nikolay Aleksandrov wrote:
> > > > @@ -617,8 +614,18 @@ static void bond_ipsec_del_sa_all(struct
> > > > bonding *bond)
> > > >
> > >
On Thu, Mar 06, 2025 at 07:31:22AM +0100, Ignacio Encinas Rubio wrote:
>
>
> On 5/3/25 22:49, Charlie Jenkins wrote:
> > On Wed, Mar 05, 2025 at 05:39:28PM +0100, Ignacio Encinas wrote:
> >> Vector registers are zero initialized by the kernel. Stop accepting
> >> "all ones" as a clean value.
> >>
On Thu, Mar 06, 2025 at 09:37:53AM +, Hangbin Liu wrote:
> >
> > The reason the mutex was added (instead of the spinlock used before)
> > was exactly because the add and free offload operations could sleep.
> >
> > > With your reply, I also checked the xdo_dev_state_add() in
> > > bond_ipsec_
On 05/03/2025 17:56, Sabrina Dubroca wrote:
2025-03-05, 14:14:36 +0100, Antonio Quartulli wrote:
On 05/03/2025 12:20, Sabrina Dubroca wrote:
2025-03-05, 00:19:32 +0100, Antonio Quartulli wrote:
On 04/03/2025 19:37, Sabrina Dubroca wrote:
2025-03-04, 01:33:48 +0100, Antonio Quartulli wrote:
A
> On 27. Feb 2025, at 04:17, Alexei Starovoitov
> wrote:
>
> On Wed, Feb 26, 2025 at 6:43 PM Hou Tao wrote:
>>
lookup procedure A
A: find the old element (instead of the new old)
update procedure B
B: delete the old element
In the unlikely case that setting ro_after_init data to read-only fails, it
is too late to cancel loading of the module. The loader then issues only
a warning about the situation. Given that this reduces the kernel's
protection, it was suggested to make the failure more visible by tainting
the kern
On Thu, 6 Mar 2025 at 10:00, David Gow wrote:
>
> On Thu, 6 Mar 2025 at 08:29, Rae Moar wrote:
> >
> > A bug was identified where the KTAP below caused an infinite loop:
> >
> > TAP version 13
> > ok 4 test_case
> > 1..4
> >
> > The infinite loop was caused by the parser not parsing a test pla
From: Steven Rostedt
The tracing system needs a way to save all the currently loaded modules
and their addresses into persistent memory so that it can evaluate the
addresses on a reboot from a crash. When the persistent memory trace
starts, it will load the module addresses and names into the per
+To: Christophe Leroy
On 3/6/25 11:36, Petr Pavlu wrote:
> In the unlikely case that setting ro_after_init data to read-only fails, it
> is too late to cancel loading of the module. The loader then issues only
> a warning about the situation. Given that this reduces the kernel's
> protection, it
On 06.03.25 13:42, Brendan Jackman wrote:
On Thu, Mar 06, 2025 at 10:28:09AM +0100, David Hildenbrand wrote:
On 28.02.25 17:54, Brendan Jackman wrote:
Some filesystems don't support funtract()ing unlinked files. They return
ENOENT. In that case, skip the test.
That's not documented in the ma
Section .static_call_sites holds data structures that need to be sorted and
processed only at module load time. This initial processing happens in
static_call_add_module(), which is invoked as a callback to the
MODULE_STATE_COMING notification from prepare_coming_module().
The section is never mod
On Thu, Mar 06, 2025 at 10:02:34AM +, Hangbin Liu wrote:
> > Set xs->xso.real_dev = NULL is a good idea. As we will break
> > in bond_ipsec_del_sa()/bond_ipsec_free_sa() when there is no
> > xs->xso.real_dev.
> >
> > For bond_ipsec_add_sa_all(), I will move the xso.real_dev = real_dev
> > afte
On Thu, 2025-03-06 at 10:02 +, Hangbin Liu wrote:
> > For bond_ipsec_add_sa_all(), I will move the xso.real_dev =
> > real_dev
> > after .xdo_dev_state_add() in case the following situation.
xso.real_dev needs to be initialized before the call to
xdo_dev_state_add, since many of the implementa
On 5/3/25 22:49, Charlie Jenkins wrote:
> On Wed, Mar 05, 2025 at 05:39:28PM +0100, Ignacio Encinas wrote:
>> Vector registers are zero initialized by the kernel. Stop accepting
>> "all ones" as a clean value.
>>
>> Note that this was not working as expected given that
>> value == 0xff
>> c
On Thu, 6 Mar 2025 at 08:29, Rae Moar wrote:
>
> A bug was identified where the KTAP below caused an infinite loop:
>
> TAP version 13
> ok 4 test_case
> 1..4
>
> The infinite loop was caused by the parser not parsing a test plan
> if following a test result line.
>
> Fix bug to correctly parse
On Thu, Mar 6, 2025 at 7:26 AM Brendan Jackman wrote:
>
> On Thu, 6 Mar 2025 at 10:00, David Gow wrote:
> >
> > On Thu, 6 Mar 2025 at 08:29, Rae Moar wrote:
> > >
> > > A bug was identified where the KTAP below caused an infinite loop:
> > >
> > > TAP version 13
> > > ok 4 test_case
> > > 1..
Hello:
This series was applied to bpf/bpf-next.git (net)
by Martin KaFai Lau :
On Wed, 5 Mar 2025 21:34:32 + you wrote:
> Hi all,
>
> this v5 of the patch series is very similar to v4, but rebased onto the
> bpf-next/net branch instead of bpf-next/master.
> Because the commit c047e0e0e435 (
On Thu, Mar 06, 2025 at 05:20:59PM +0100, Petr Pavlu wrote:
> Change my role for MODULE SUPPORT from a reviewer to a maintainer. We
> started to rotate its maintainership and I currently look after the modules
> tree. This not being reflected in MAINTAINERS proved to confuse folks.
>
> Add lib/tes
On Thu, 06 Mar 2025 18:41:18 +, Louis Taylor wrote:
> openat is useful to avoid needing to construct relative paths, so expose
> a wrapper for using it directly.
>
>
Applied, thanks!
[1/5] tools/nolibc: add support for openat(2)
commit: b2edaad7f5192a79384b7d0bec62312b668da463
[2/5]
Add new ftrace helpers functions cleanup_tracing, trace_function and
check_traced_function.
Signed-off-by: Filipe Xavier
---
tools/testing/selftests/livepatch/functions.sh | 45 ++
1 file changed, 45 insertions(+)
diff --git a/tools/testing/selftests/livepatch/functions.
This new test makes sure that ftrace can trace a
function that was introduced by a livepatch.
Signed-off-by: Filipe Xavier
---
tools/testing/selftests/livepatch/test-ftrace.sh | 35
1 file changed, 35 insertions(+)
diff --git a/tools/testing/selftests/livepatch/test-ftr
: 20250306-ftrace-sftest-livepatch-60d9dc472235
Best regards,
--
Filipe Xavier
On 1/11/25 3:42 PM, Filipe Xavier wrote:
This new test makes sure that ftrace can trace a
function that was introduced by a livepatch.
Signed-off-by: Filipe Xavier
---
Changes in v2:
- functions.sh: added reset tracing on push and pop_config.
- test-ftrace.sh: enabled tracing_on before test in
On Thu, Mar 06, 2025 at 10:52:39PM +0100, Thomas Weißschuh wrote:
> The execution order of constructors in undefined and depends on the
> toolchain. While recent toolchains seems to have a stable order, it
> doesn't work for older ones and may also change at any time.
>
> Stop validating the orde
ust be last */
/* note: do not set any defaults so as to permit holes above */
---
base-commit: 6e406202a44a1a37176da0333cec10d5320c4b33
change-id: 20250306-nolibc-constructor-order-6921e8c93591
Best regards,
--
Thomas Weißschuh
vsock/vsock_test.c | 80
4 files changed, 90 insertions(+), 1 deletion(-)
---
base-commit: 848e076317446f9c663771ddec142d7c2eb4cb43
change-id: 20250306-test_vsock-3e77a9c7a245
Best regards,
--
Luigi Leonardi
Add `timeout_check_expired` function that returns true if the timeout
counter has expired.
This is useful in situations where a timeout does not necessarily mean a
failure.
Signed-off-by: Luigi Leonardi
---
tools/testing/vsock/timeout.c | 7 ++-
tools/testing/vsock/timeout.h | 3 +++
2 file
On Thu, Mar 06, 2025 at 02:13:51PM +0100, Petr Pavlu wrote:
> Section .static_call_sites holds data structures that need to be sorted and
> processed only at module load time. The section is never modified
> afterwards. Make it therefore read-only after module initialization to
> avoid any (non-)ac
Hi Thomas,
On Thu, Mar 06, 2025 at 06:22:45PM +0100, Thomas Weißschuh wrote:
> Hi Willy,
>
> On 2025-03-04 09:11:16+0100, Willy Tarreau wrote:
> > On Tue, Mar 04, 2025 at 07:58:15AM +, Louis Taylor wrote:
> > > openat is useful to avoid needing to construct relative paths, so expose
> > > a w
On 2025-03-04 07:58:15+, Louis Taylor wrote:
> openat is useful to avoid needing to construct relative paths, so expose
> a wrapper for using it directly.
>
> Signed-off-by: Louis Taylor
Looks good. I have some tiny nitpicks inline,
but if you prefer I can also pick it up as-is.
Acked-by: T
On Thu, 6 Mar 2025 09:00:02 -0800 Joe Damato wrote:
> +* - wrap all of the work in a lock (perhaps
> vi->refill_lock?)
> +* - check netif_running() and return early to avoid a race
> +*/
probably netdev instance lock is better here, as it will a
On 5 Mar 2025, at 17:38, Hugh Dickins wrote:
> On Wed, 5 Mar 2025, Zi Yan wrote:
>> On 5 Mar 2025, at 16:03, Hugh Dickins wrote:
>>>
>>> Beyond checking that, I didn't have time yesterday to investigate
>>> further, but I'll try again today (still using last weekend's mm.git).
>>
>> I am trying to
Change my role for MODULE SUPPORT from a reviewer to a maintainer. We
started to rotate its maintainership and I currently look after the modules
tree. This not being reflected in MAINTAINERS proved to confuse folks.
Add lib/tests/module/ and tools/testing/selftests/module/ to maintained
files. Th
On Thu, Mar 06, 2025 at 06:00:17PM +0100, Thomas Weißschuh wrote:
> On 2025-03-04 07:58:19+, Louis Taylor wrote:
> > It was mentioned that a new test_ function should be static, so go back
> > over existing functions and mark those static as well.
>
> Actually Willy wants these non-static for
Hello Andy Shevchenko.
On Wed, 05 Mar 2025 12:55:33 +0200, Andy Shevchenko wrote:
> The main part is the patch 3 that converts the driver to GPIO descriptor APIs,
> the first one is just an ad-hoc fix WRT sparse complains on the bitwise
> types misuse. The second one is a small cleanup that helps
Good morning,
On Wed, Mar 05, 2025 at 02:39:23PM +0200, Iuliana Prodan (OSS) wrote:
> From: Iuliana Prodan
>
> Some DSP firmware requires a FW_READY signal before proceeding,
> while others do not.
> Introduce imx_dsp_rproc_wait_fw_ready() to check the resource table
> and determine if waiting i
On 6 Mar 2025, at 4:19, David Hildenbrand wrote:
> On 05.03.25 22:08, Zi Yan wrote:
>> On 5 Mar 2025, at 15:50, Hugh Dickins wrote:
>>
>>> On Wed, 5 Mar 2025, Zi Yan wrote:
On 4 Mar 2025, at 6:49, Hugh Dickins wrote:
>
> I think (might be wrong, I'm in a rush) my mods are all to this
On Thu, 6 Mar 2025 17:20:59 +0100
Petr Pavlu wrote:
> Change my role for MODULE SUPPORT from a reviewer to a maintainer. We
> started to rotate its maintainership and I currently look after the modules
> tree. This not being reflected in MAINTAINERS proved to confuse folks.
>
> Add lib/tests/mo
A bug was identified where the KTAP below caused an infinite loop:
TAP version 13
ok 4 test_case
1..4
The infinite loop was caused by the parser not parsing a test plan
if following a test result line.
Fix this bug to correctly parse test plan line.
Signed-off-by: Rae Moar
---
Changes since
Hi Willy,
On 2025-03-04 09:11:16+0100, Willy Tarreau wrote:
> On Tue, Mar 04, 2025 at 07:58:15AM +, Louis Taylor wrote:
> > openat is useful to avoid needing to construct relative paths, so expose
> > a wrapper for using it directly.
>
> Reviewed the whole series, no comments from me. Let's w
On 3/4/2025 9:55 AM, Paul E. McKenney wrote:
> On Mon, Mar 03, 2025 at 11:08:24AM -0500, Joel Fernandes wrote:
>> On Fri, Feb 28, 2025 at 01:13:56PM +0100, Uladzislau Rezki (Sony) wrote:
>>> Currently kvfree_rcu() APIs use a system workqueue which is
>>> "system_unbound_wq" to driver RCU machine
Vector registers are zero initialized by the kernel. Stop accepting
"all ones" as a clean value.
Note that this was not working as expected given that
value == 0xff
can be assumed to be always false by the compiler as value's range is
[-128, 127]. Both GCC (-Wtype-limits) and clang
(-Wtaut
On Thu, Mar 06, 2025 at 10:28:09AM +0100, David Hildenbrand wrote:
> On 28.02.25 17:54, Brendan Jackman wrote:
> > Some filesystems don't support funtract()ing unlinked files. They return
> > ENOENT. In that case, skip the test.
> >
>
> That's not documented in the man page, so is this a bug of t
On Wed, Mar 05, 2025 at 06:21:18PM -0800, Jakub Kicinski wrote:
> On Wed, 5 Mar 2025 17:42:35 -0800 Joe Damato wrote:
> > Two spots that come to mind are:
> > - in virtnet_probe where all the other netdev ops are plumbed
> >through, or
> > - above virtnet_disable_queue_pair which I assume a f
openat is useful to avoid needing to construct relative paths, so expose
a wrapper for using it directly.
Signed-off-by: Louis Taylor
---
tools/include/nolibc/sys.h | 25
tools/testing/selftests/nolibc/nolibc-test.c | 17 +
2 files changed, 42 i
62 matches
Mail list logo
