All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jakub Kicinski <kuba@kernel.org>
To: Breno Leitao <leitao@debian.org>
Cc: Andrew Lunn <andrew+netdev@lunn.ch>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Paolo Abeni <pabeni@redhat.com>, Shuah Khan <shuah@kernel.org>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-kselftest@vger.kernel.org, kernel-team@meta.com,
	Simon Horman <horms@kernel.org>
Subject: Re: [PATCH net-next v2 2/4] netconsole: selftest: Split the helpers from the selftest
Date: Mon, 6 Jan 2025 16:06:53 -0800	[thread overview]
Message-ID: <20250106160653.2fcdb192@kernel.org> (raw)
In-Reply-To: <20250103-netcons_overflow_test-v2-2-a49f9be64c21@debian.org>

On Fri, 03 Jan 2025 04:57:50 -0800 Breno Leitao wrote:
> Split helper functions from the netconsole basic test into a separate
> library file to enable reuse across different netconsole tests. This
> change only moves the existing helper functions to lib/sh/lib_netcons.sh
> while preserving the same test functionality.
> 
> The helpers provide common functions for:
> - Setting up network namespaces and interfaces
> - Managing netconsole dynamic targets
> - Setting user data
> - Handling test dependencies
> - Cleanup operations
> 
> Do not make any change in the code, other than the mechanical
> separation.

You have to update the Makefile, we don't wildcard all files under
drivers/net/lib, the lib support in ksft is not great :(

$ make -C tools/testing/selftests/ \
	TARGETS="drivers/net drivers/net/hw" \
	install INSTALL_PATH=/tmp/ksft-net-drv
[... noise ...]
$ find /tmp/ksft-net-drv/ -iname '*netcons*'
/tmp/ksft-net-drv/drivers/net/netcons_basic.sh
/tmp/ksft-net-drv/drivers/net/netcons_overflow.sh

.. your new lib file won't get installed.
-- 
pw-bot: cr

  reply	other threads:[~2025-01-07  0:06 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-03 12:57 [PATCH net-next v2 0/4] netconsole: selftest for userdata overflow Breno Leitao
2025-01-03 12:57 ` [PATCH net-next v2 1/4] netconsole: Warn if MAX_USERDATA_ITEMS limit is exceeded Breno Leitao
2025-01-03 12:57 ` [PATCH net-next v2 2/4] netconsole: selftest: Split the helpers from the selftest Breno Leitao
2025-01-07  0:06   ` Jakub Kicinski [this message]
2025-01-08 11:19     ` Breno Leitao
2025-01-03 12:57 ` [PATCH net-next v2 3/4] netconsole: selftest: Delete all userdata keys Breno Leitao
2025-01-03 12:57 ` [PATCH net-next v2 4/4] netconsole: selftest: verify userdata entry limit Breno Leitao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250106160653.2fcdb192@kernel.org \
    --to=kuba@kernel.org \
    --cc=andrew+netdev@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=kernel-team@meta.com \
    --cc=leitao@debian.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=shuah@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.