mbox series

[net,0/3] selftests: net: lib: small fixes

Message ID 20240605-upstream-net-20240605-selftests-net-lib-fixes-v1-0-b3afadd368c9@kernel.org (mailing list archive)
Headers show
Series selftests: net: lib: small fixes | expand

Message

Matthieu Baerts (NGI0) June 5, 2024, 9:21 a.m. UTC
While looking at using 'lib.sh' for the MPTCP selftests [1], we found
some small issues with 'lib.sh'. Here they are:

- Patch 1: fix 'errexit' (set -e) support with busywait. 'errexit' is
  supported in some functions, not all. A fix for v6.8+.

- Patch 2: avoid confusing error messages linked to the cleaning part
  when the netns setup fails. A fix for v6.8+.

- Patch 3: set a variable as local to avoid accidentally changing the
  value of a another one with the same name on the caller side. A fix
  for v6.10-rc1+.

Link: https://lore.kernel.org/mptcp/5f4615c3-0621-43c5-ad25-55747a4350ce@kernel.org/T/ [1]
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
Matthieu Baerts (NGI0) (3):
      selftests: net: lib: support errexit with busywait
      selftests: net: lib: avoid error removing empty netns name
      selftests: net: lib: set 'i' as local

 tools/testing/selftests/net/lib.sh | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)
---
base-commit: a535d59432370343058755100ee75ab03c0e3f91
change-id: 20240605-upstream-net-20240605-selftests-net-lib-fixes-7a90a1a8d9d2

Best regards,

Comments

patchwork-bot+netdevbpf@kernel.org June 6, 2024, 3:40 p.m. UTC | #1
Hello:

This series was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 05 Jun 2024 11:21:15 +0200 you wrote:
> While looking at using 'lib.sh' for the MPTCP selftests [1], we found
> some small issues with 'lib.sh'. Here they are:
> 
> - Patch 1: fix 'errexit' (set -e) support with busywait. 'errexit' is
>   supported in some functions, not all. A fix for v6.8+.
> 
> - Patch 2: avoid confusing error messages linked to the cleaning part
>   when the netns setup fails. A fix for v6.8+.
> 
> [...]

Here is the summary with links:
  - [net,1/3] selftests: net: lib: support errexit with busywait
    https://git.kernel.org/netdev/net/c/41b02ea4c0ad
  - [net,2/3] selftests: net: lib: avoid error removing empty netns name
    https://git.kernel.org/netdev/net/c/79322174bcc7
  - [net,3/3] selftests: net: lib: set 'i' as local
    https://git.kernel.org/netdev/net/c/84a8bc3ec225

You are awesome, thank you!