[LTP] [PATCH v3 5/5] busy_poll: Move TST_{SETUP, CLEANUP} to the tests
Petr Vorel
pvorel@suse.cz
Thu Apr 28 08:56:57 CEST 2022
Although having variables in both busy_poll_lib.sh and the tests which
are using it isn't optimal, hooking up callbacks on the reverse end of
include is even worse code.
Suggested-by: Martin Doucha <mdoucha@suse.cz>
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Follow-up of "v3 shell: Cleanup getopts usage" patchset [1],
replacing first commit.
Kind regards,
Petr
[1] https://patchwork.ozlabs.org/project/ltp/list/?series=297175
testcases/network/busy_poll/busy_poll01.sh | 3 +++
testcases/network/busy_poll/busy_poll02.sh | 3 +++
testcases/network/busy_poll/busy_poll03.sh | 2 ++
testcases/network/busy_poll/busy_poll_lib.sh | 3 +--
4 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/testcases/network/busy_poll/busy_poll01.sh b/testcases/network/busy_poll/busy_poll01.sh
index 65f4db3fe..1f7097771 100755
--- a/testcases/network/busy_poll/busy_poll01.sh
+++ b/testcases/network/busy_poll/busy_poll01.sh
@@ -4,6 +4,9 @@
#
# Author: Alexey Kodanev <alexey.kodanev@oracle.com>
+TST_SETUP="setup"
+TST_CLEANUP="cleanup"
+
cleanup()
{
[ -n "$busy_read_old" ] && \
diff --git a/testcases/network/busy_poll/busy_poll02.sh b/testcases/network/busy_poll/busy_poll02.sh
index ebae4d2f5..634bbd6bd 100755
--- a/testcases/network/busy_poll/busy_poll02.sh
+++ b/testcases/network/busy_poll/busy_poll02.sh
@@ -4,6 +4,9 @@
#
# Author: Alexey Kodanev <alexey.kodanev@oracle.com>
+TST_SETUP="setup"
+TST_CLEANUP="cleanup"
+
cleanup()
{
[ -n "$busy_poll_old" ] && \
diff --git a/testcases/network/busy_poll/busy_poll03.sh b/testcases/network/busy_poll/busy_poll03.sh
index 04d5978f7..b2e1c0a7a 100755
--- a/testcases/network/busy_poll/busy_poll03.sh
+++ b/testcases/network/busy_poll/busy_poll03.sh
@@ -4,6 +4,8 @@
#
# Author: Alexey Kodanev <alexey.kodanev@oracle.com>
+TST_SETUP="setup"
+TST_CLEANUP="cleanup"
TST_TEST_DATA="udp udp_lite"
cleanup()
diff --git a/testcases/network/busy_poll/busy_poll_lib.sh b/testcases/network/busy_poll/busy_poll_lib.sh
index de61d3fcd..446ae3d65 100755
--- a/testcases/network/busy_poll/busy_poll_lib.sh
+++ b/testcases/network/busy_poll/busy_poll_lib.sh
@@ -1,10 +1,9 @@
#!/bin/sh
# SPDX-License-Identifier: GPL-2.0-or-later
+# Copyright (c) Linux Test Project, 2016-2022
# Copyright (c) 2016-2018 Oracle and/or its affiliates. All Rights Reserved.
-TST_SETUP="setup"
TST_TESTFUNC="test"
-TST_CLEANUP="cleanup"
TST_MIN_KVER="3.11"
TST_NEEDS_TMPDIR=1
TST_NEEDS_ROOT=1
--
2.35.3
More information about the ltp
mailing list