netdev - Re: [PATCH net-next 00/13] selftests: Mark auto-deferring functions clearly

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: 
 <175893604399.115459.16652616612941464675.git-patchwork-notify@kernel.org>
Date: Sat, 27 Sep 2025 01:20:43 +0000
From: patchwork-bot+netdevbpf@...nel.org
To: Petr Machata <petrm@...dia.com>
Cc: davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
 pabeni@...hat.com, netdev@...r.kernel.org, horms@...nel.org,
 idosch@...dia.com, razor@...ckwall.org, shuah@...nel.org,
 linux-kselftest@...r.kernel.org, mlxsw@...dia.com
Subject: Re: [PATCH net-next 00/13] selftests: Mark auto-deferring functions
 clearly

Hello:

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

On Thu, 25 Sep 2025 19:31:43 +0200 you wrote:
> selftests/net/lib.sh contains a suite of iproute2 wrappers that
> automatically schedule the corresponding cleanup through defer. The fact
> they do so is however not immediately obvious, one needs to know which
> functions are handling the deferral behind the scenes, and which expect the
> caller to handle cleanups themselves.
> 
> A convention for these auto-deferring functions would help both writing and
> patch review. This patchset does so by marking these functions with an adf_
> prefix. We already have a few such functions: forwarding/lib.sh has
> adf_mcd_start() and a few selftests add private helpers that conform to
> this convention.
> 
> [...]

Here is the summary with links:
  - [net-next,01/13] selftests: net: lib: Rename ip_link_add() to adf_*
    https://git.kernel.org/netdev/net-next/c/191c4912f9c3
  - [net-next,02/13] selftests: net: lib: Rename ip_link_set_master() to adf_*
    https://git.kernel.org/netdev/net-next/c/c3cbd21fe18e
  - [net-next,03/13] selftests: net: lib: Rename ip_link_set_addr() to adf_*
    https://git.kernel.org/netdev/net-next/c/beb98a347762
  - [net-next,04/13] selftests: net: lib: Rename ip_link_set_up() to adf_*
    https://git.kernel.org/netdev/net-next/c/34d3f8b75e2b
  - [net-next,05/13] selftests: net: lib: Rename ip_link_set_down() to adf_*
    https://git.kernel.org/netdev/net-next/c/a55f9fb3432e
  - [net-next,06/13] selftests: net: lib: Rename ip_addr_add() to adf_*
    https://git.kernel.org/netdev/net-next/c/773603d6db30
  - [net-next,07/13] selftests: net: lib: Rename ip_route_add() to adf_*
    https://git.kernel.org/netdev/net-next/c/d85bcf6505d2
  - [net-next,08/13] selftests: net: lib: Rename bridge_vlan_add() to adf_*
    https://git.kernel.org/netdev/net-next/c/b628dfcd54cb
  - [net-next,09/13] selftests: net: vlan_bridge_binding: Rename dfr_set_binding_*() to adf_*
    https://git.kernel.org/netdev/net-next/c/14b72996ae80
  - [net-next,10/13] selftests: forwarding: lib: Add an autodefer variant of vrf_prepare()
    https://git.kernel.org/netdev/net-next/c/02aabe00b2e1
  - [net-next,11/13] selftests: forwarding: lib: Add an autodefer variant of simple_if_init()
    https://git.kernel.org/netdev/net-next/c/f53748d56d10
  - [net-next,12/13] selftests: forwarding: lib: Add an autodefer variant of forwarding_enable()
    https://git.kernel.org/netdev/net-next/c/040a6cbead5d
  - [net-next,13/13] selftests: forwarding: README: Mention defer, adf_
    https://git.kernel.org/netdev/net-next/c/fca6ff9191bd

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html