[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f7d8502e-21b8-43f9-ad81-dbbe07df5300@collabora.com>
Date: Mon, 1 Jul 2024 13:33:08 +0500
From: Muhammad Usama Anjum <usama.anjum@...labora.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Kees Cook <keescook@...omium.org>
Cc: Muhammad Usama Anjum <usama.anjum@...labora.com>, kernel@...labora.com,
linux-kselftest@...r.kernel.org, linux-kernel@...r.kernel.org,
Shuah Khan <shuah@...nel.org>
Subject: Re: [PATCH] selftests: firmware: conform test to TAP
Adding missing maintainer and reviser.
On 5/20/24 9:37 PM, Muhammad Usama Anjum wrote:
> Conform the layout, informational and status messages to TAP. No
> functional change is intended other than the layout of output messages.
> Without using TAP messages, the passed/failed/skip test names cannot be
> found. This is a prepartory patch, more patches will be sent to comform
> the test completely.
>
> Signed-off-by: Muhammad Usama Anjum <usama.anjum@...labora.com>
> ---
> .../selftests/firmware/fw_run_tests.sh | 63 ++++++++++++-------
> 1 file changed, 40 insertions(+), 23 deletions(-)
>
> diff --git a/tools/testing/selftests/firmware/fw_run_tests.sh b/tools/testing/selftests/firmware/fw_run_tests.sh
> index f6d95a2d51249..5063c75345680 100755
> --- a/tools/testing/selftests/firmware/fw_run_tests.sh
> +++ b/tools/testing/selftests/firmware/fw_run_tests.sh
> @@ -6,6 +6,9 @@
>
> set -e
>
> +DIR="$(dirname $(readlink -f "$0"))"
> +source "${DIR}"/../kselftest/ktap_helpers.sh
> +
> TEST_DIR=$(dirname $0)
> source $TEST_DIR/fw_lib.sh
>
> @@ -26,54 +29,68 @@ run_tests()
> proc_set_force_sysfs_fallback $1
> proc_set_ignore_sysfs_fallback $2
> $TEST_DIR/fw_upload.sh
> +
> + ktap_test_pass "Completed"
> }
>
> run_test_config_0001()
> {
> - echo "-----------------------------------------------------"
> - echo "Running kernel configuration test 1 -- rare"
> - echo "Emulates:"
> - echo "CONFIG_FW_LOADER=y"
> - echo "CONFIG_FW_LOADER_USER_HELPER=n"
> - echo "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=n"
> + ktap_print_msg "-----------------------------------------------------"
> + ktap_print_msg "Running kernel configuration test 1 -- rare"
> + ktap_print_msg "Emulates:"
> + ktap_print_msg "CONFIG_FW_LOADER=y"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER=n"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=n"
> run_tests 0 1
> }
>
> run_test_config_0002()
> {
> - echo "-----------------------------------------------------"
> - echo "Running kernel configuration test 2 -- distro"
> - echo "Emulates:"
> - echo "CONFIG_FW_LOADER=y"
> - echo "CONFIG_FW_LOADER_USER_HELPER=y"
> - echo "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=n"
> + ktap_print_msg "-----------------------------------------------------"
> + ktap_print_msg "Running kernel configuration test 2 -- distro"
> + ktap_print_msg "Emulates:"
> + ktap_print_msg "CONFIG_FW_LOADER=y"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER=y"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=n"
> proc_set_ignore_sysfs_fallback 0
> run_tests 0 0
> }
>
> run_test_config_0003()
> {
> - echo "-----------------------------------------------------"
> - echo "Running kernel configuration test 3 -- android"
> - echo "Emulates:"
> - echo "CONFIG_FW_LOADER=y"
> - echo "CONFIG_FW_LOADER_USER_HELPER=y"
> - echo "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=y"
> + ktap_print_msg "-----------------------------------------------------"
> + ktap_print_msg "Running kernel configuration test 3 -- android"
> + ktap_print_msg "Emulates:"
> + ktap_print_msg "CONFIG_FW_LOADER=y"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER=y"
> + ktap_print_msg "CONFIG_FW_LOADER_USER_HELPER_FALLBACK=y"
> run_tests 1 0
> }
>
> +ktap_print_header
> +
> check_mods
> check_setup
>
> -echo "Running namespace test: "
> -$TEST_DIR/fw_namespace $DIR/trigger_request
> -echo "OK"
> -
> if [ -f $FW_FORCE_SYSFS_FALLBACK ]; then
> + ktap_set_plan "4"
> +
> run_test_config_0001
> run_test_config_0002
> run_test_config_0003
> else
> - echo "Running basic kernel configuration, working with your config"
> + ktap_set_plan "2"
> +
> + ktap_print_msg "Running basic kernel configuration, working with your config"
> run_tests
> fi
> +
> +ktap_print_msg "Running namespace test: "
> +$TEST_DIR/fw_namespace $DIR/trigger_request
> +if [ $? -eq 0 ]; then
> + ktap_test_pass "fw_namespace completed successfully"
> +else
> + ktap_test_fail "fw_namespace failed"
> +fi
> +
> +ktap_finished
--
BR,
Muhammad Usama Anjum
Powered by blists - more mailing lists