ACK: [SRU F][PATCH] UBUNTU: SAUCE: Fix ftrace/Meta-selftests bashism check
Roxana Nicolescu
roxana.nicolescu at canonical.com
Tue Feb 7 10:20:27 UTC 2023
On 07-02-2023 10:48, Stefan Bader wrote:
> BugLink: https://bugs.launchpad.net/bugs/2006453
>
> Upstream commit a1d6cd88c897 "selftests/ftrace: event_triggers: wait longer for
> test_event_enable" introduces a test which is considered a bashism and
> as a result causes the Meta-selftests subtest to fail. Fix up the "=="
> to be "=" in order to avoid this.
>
> Fixes: a1d6cd88c897 "selftests/ftrace: event_triggers: wait longer for test_event_enable"
> Signed-off-by: Stefan Bader <stefan.bader at canonical.com>
> ---
> .../selftests/ftrace/test.d/ftrace/func_event_triggers.tc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc
> index f261eeccfaf6..020fbdaa3fba 100644
> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc
> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_event_triggers.tc
> @@ -46,7 +46,7 @@ test_event_enabled() {
>
> while [ $check_times -ne 0 ]; do
> e=`cat $EVENT_ENABLE`
> - if [ "$e" == $val ]; then
> + if [ "$e" = $val ]; then
> return 0
> fi
> sleep $SLEEP_TIME
Nice catch.
Acked-by: Roxana Nicolescu <roxana.nicolescu at canonical.com
More information about the kernel-team
mailing list