* [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT
@ 2022-02-09 16:24 Krzysztof Kozlowski
2022-02-09 19:26 ` Shuah Khan
2022-02-10 5:00 ` Steven Rostedt
0 siblings, 2 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-09 16:24 UTC (permalink / raw)
To: Steven Rostedt, Ingo Molnar, Shuah Khan, Krzysztof Kozlowski,
linux-kselftest, linux-kernel, linux-rt-users, joseph.salisbury
The PREEMPT_RT patchset does not use soft IRQs thus trying to filter for
do_softirq fails for such kernel:
echo do_softirq
ftracetest: 81: echo: echo: I/O error
Choose some other externally visible function for the test.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
I understand that the failure does not exist on mainline kernel (only
with PREEMPT_RT patchset) but the change does not harm it.
If it is not suitable alone, please consider it for RT patchset.
---
.../selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
index e96e279e0533..1d0c7601865f 100644
--- a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
+++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
@@ -19,7 +19,7 @@ fail() { # mesg
FILTER=set_ftrace_filter
FUNC1="schedule"
-FUNC2="do_softirq"
+FUNC2="_printk"
ALL_FUNCS="#### all functions enabled ####"
--
2.32.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT
2022-02-09 16:24 [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT Krzysztof Kozlowski
@ 2022-02-09 19:26 ` Shuah Khan
2022-02-10 5:00 ` Steven Rostedt
1 sibling, 0 replies; 4+ messages in thread
From: Shuah Khan @ 2022-02-09 19:26 UTC (permalink / raw)
To: Krzysztof Kozlowski, Steven Rostedt, Ingo Molnar, Shuah Khan,
linux-kselftest, linux-kernel, linux-rt-users, joseph.salisbury,
Shuah Khan
On 2/9/22 9:24 AM, Krzysztof Kozlowski wrote:
> The PREEMPT_RT patchset does not use soft IRQs thus trying to filter for
> do_softirq fails for such kernel:
>
> echo do_softirq
> ftracetest: 81: echo: echo: I/O error
>
> Choose some other externally visible function for the test.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>
> ---
>
> I understand that the failure does not exist on mainline kernel (only
> with PREEMPT_RT patchset) but the change does not harm it.
>
> If it is not suitable alone, please consider it for RT patchset.
> ---
> .../selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> index e96e279e0533..1d0c7601865f 100644
> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> @@ -19,7 +19,7 @@ fail() { # mesg
>
> FILTER=set_ftrace_filter
> FUNC1="schedule"
> -FUNC2="do_softirq"
> +FUNC2="_printk"
>
> ALL_FUNCS="#### all functions enabled ####"
>
>
Change looks good to me.
Reviewed-by: Shuah Khan <skhan@linuxfoundation.org>
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT
2022-02-09 16:24 [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT Krzysztof Kozlowski
2022-02-09 19:26 ` Shuah Khan
@ 2022-02-10 5:00 ` Steven Rostedt
2022-02-10 8:10 ` Krzysztof Kozlowski
1 sibling, 1 reply; 4+ messages in thread
From: Steven Rostedt @ 2022-02-10 5:00 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Ingo Molnar, Shuah Khan, linux-kselftest, linux-kernel,
linux-rt-users, joseph.salisbury
On Wed, 9 Feb 2022 17:24:30 +0100
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> wrote:
> The PREEMPT_RT patchset does not use soft IRQs thus trying to filter for
> do_softirq fails for such kernel:
>
> echo do_softirq
> ftracetest: 81: echo: echo: I/O error
>
> Choose some other externally visible function for the test.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>
> ---
>
> I understand that the failure does not exist on mainline kernel (only
> with PREEMPT_RT patchset) but the change does not harm it.
>
> If it is not suitable alone, please consider it for RT patchset.
> ---
> .../selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> index e96e279e0533..1d0c7601865f 100644
> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
> @@ -19,7 +19,7 @@ fail() { # mesg
>
> FILTER=set_ftrace_filter
> FUNC1="schedule"
> -FUNC2="do_softirq"
> +FUNC2="_printk"
The problem with the above, is that it is not guaranteed to trigger (and
probably will not), where as the do_softirq is. The filtering is suppose to
trace something that actually happens.
How about using: scheduler_tick ?
-- Steve
>
> ALL_FUNCS="#### all functions enabled ####"
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT
2022-02-10 5:00 ` Steven Rostedt
@ 2022-02-10 8:10 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2022-02-10 8:10 UTC (permalink / raw)
To: Steven Rostedt
Cc: Ingo Molnar, Shuah Khan, linux-kselftest, linux-kernel,
linux-rt-users, joseph.salisbury
On 10/02/2022 06:00, Steven Rostedt wrote:
> On Wed, 9 Feb 2022 17:24:30 +0100
> Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> wrote:
>
>> The PREEMPT_RT patchset does not use soft IRQs thus trying to filter for
>> do_softirq fails for such kernel:
>>
>> echo do_softirq
>> ftracetest: 81: echo: echo: I/O error
>>
>> Choose some other externally visible function for the test.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>>
>> ---
>>
>> I understand that the failure does not exist on mainline kernel (only
>> with PREEMPT_RT patchset) but the change does not harm it.
>>
>> If it is not suitable alone, please consider it for RT patchset.
>> ---
>> .../selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
>> index e96e279e0533..1d0c7601865f 100644
>> --- a/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
>> +++ b/tools/testing/selftests/ftrace/test.d/ftrace/func_set_ftrace_file.tc
>> @@ -19,7 +19,7 @@ fail() { # mesg
>>
>> FILTER=set_ftrace_filter
>> FUNC1="schedule"
>> -FUNC2="do_softirq"
>> +FUNC2="_printk"
>
> The problem with the above, is that it is not guaranteed to trigger (and
> probably will not), where as the do_softirq is. The filtering is suppose to
> trace something that actually happens.
>
> How about using: scheduler_tick ?
>
This works as well. I had impression that the test is only about
filtering interface and actual function does not have to hit/trigger.
I'll send v2 with scheduler_tick.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-02-10 8:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-09 16:24 [PATCH] selftests/ftrace: Do not trace do_softirq because of PREEMPT_RT Krzysztof Kozlowski
2022-02-09 19:26 ` Shuah Khan
2022-02-10 5:00 ` Steven Rostedt
2022-02-10 8:10 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).