* [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc
@ 2020-04-07 6:34 Xiao Yang
2020-04-07 13:54 ` Steven Rostedt
0 siblings, 1 reply; 8+ messages in thread
From: Xiao Yang @ 2020-04-07 6:34 UTC (permalink / raw)
To: rostedt, mingo; +Cc: shuah, ice_yangxiao, linux-kselftest, Xiao Yang
It is possible to get multiple records from trace during test and then more
than 4 arguments are assigned to ARGS. This situation results in the failure
of kprobe_args_type.tc. For example:
-----------------------------------------------------------
grep testprobe trace
ftracetest-5902 [001] d... 111195.682227: testprobe: (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 arg4=7
pmlogger-5949 [000] d... 111195.709898: testprobe: (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 arg4=7
grep testprobe trace
sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'
ARGS='334823024 334823024 0x13f4fe70 7
345308784 345308784 0x1494fe70 7'
-----------------------------------------------------------
We don't care which process calls do_fork so just check the first record to
fix the issue.
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
.../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc
index 1bcb67dcae26..81490ecaaa92 100644
--- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc
+++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc
@@ -38,7 +38,7 @@ for width in 64 32 16 8; do
echo 0 > events/kprobes/testprobe/enable
: "Confirm the arguments is recorded in given types correctly"
- ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'`
+ ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'`
check_types $ARGS $width
: "Clear event for next loop"
--
2.25.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-07 6:34 [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc Xiao Yang @ 2020-04-07 13:54 ` Steven Rostedt 2020-04-07 15:30 ` Masami Hiramatsu 0 siblings, 1 reply; 8+ messages in thread From: Steven Rostedt @ 2020-04-07 13:54 UTC (permalink / raw) To: Xiao Yang; +Cc: mingo, shuah, ice_yangxiao, linux-kselftest, Masami Hiramatsu On Tue, 7 Apr 2020 14:34:19 +0800 Xiao Yang <yangx.jy@cn.fujitsu.com> wrote: > It is possible to get multiple records from trace during test and then more > than 4 arguments are assigned to ARGS. This situation results in the failure > of kprobe_args_type.tc. For example: > ----------------------------------------------------------- > grep testprobe trace > ftracetest-5902 [001] d... 111195.682227: testprobe: (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 arg4=7 > pmlogger-5949 [000] d... 111195.709898: testprobe: (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 arg4=7 > grep testprobe trace > sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/' > ARGS='334823024 334823024 0x13f4fe70 7 > 345308784 345308784 0x1494fe70 7' > ----------------------------------------------------------- > > We don't care which process calls do_fork so just check the first record to > fix the issue. Looks good to me, but Masami will need to ack it. -- Steve > > Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> > --- > .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > index 1bcb67dcae26..81490ecaaa92 100644 > --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > @@ -38,7 +38,7 @@ for width in 64 32 16 8; do > echo 0 > events/kprobes/testprobe/enable > > : "Confirm the arguments is recorded in given types correctly" > - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > check_types $ARGS $width > > : "Clear event for next loop" ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-07 13:54 ` Steven Rostedt @ 2020-04-07 15:30 ` Masami Hiramatsu 2020-04-15 1:19 ` Xiao Yang 0 siblings, 1 reply; 8+ messages in thread From: Masami Hiramatsu @ 2020-04-07 15:30 UTC (permalink / raw) To: Steven Rostedt Cc: Xiao Yang, mingo, shuah, ice_yangxiao, linux-kselftest, Masami Hiramatsu On Tue, 7 Apr 2020 09:54:00 -0400 Steven Rostedt <rostedt@goodmis.org> wrote: > On Tue, 7 Apr 2020 14:34:19 +0800 > Xiao Yang <yangx.jy@cn.fujitsu.com> wrote: > > > It is possible to get multiple records from trace during test and then more > > than 4 arguments are assigned to ARGS. This situation results in the failure > > of kprobe_args_type.tc. For example: > > ----------------------------------------------------------- > > grep testprobe trace > > ftracetest-5902 [001] d... 111195.682227: testprobe: (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 arg4=7 > > pmlogger-5949 [000] d... 111195.709898: testprobe: (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 arg4=7 > > grep testprobe trace > > sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/' > > ARGS='334823024 334823024 0x13f4fe70 7 > > 345308784 345308784 0x1494fe70 7' > > ----------------------------------------------------------- > > > > We don't care which process calls do_fork so just check the first record to > > fix the issue. > > Looks good to me, but Masami will need to ack it. > Ah, good catch! I had fixed similar issue but missed this one. Acked-by: Masami Hiramatsu <mhiramat@kernel.org> Thank you! > -- Steve > > > > > Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> > > --- > > .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > > index 1bcb67dcae26..81490ecaaa92 100644 > > --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > > +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > > @@ -38,7 +38,7 @@ for width in 64 32 16 8; do > > echo 0 > events/kprobes/testprobe/enable > > > > : "Confirm the arguments is recorded in given types correctly" > > - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > > + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > > check_types $ARGS $width > > > > : "Clear event for next loop" > -- Masami Hiramatsu <mhiramat@kernel.org> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-07 15:30 ` Masami Hiramatsu @ 2020-04-15 1:19 ` Xiao Yang 2020-04-24 1:12 ` Xiao Yang 0 siblings, 1 reply; 8+ messages in thread From: Xiao Yang @ 2020-04-15 1:19 UTC (permalink / raw) To: shuah Cc: Masami Hiramatsu, Steven Rostedt, mingo, ice_yangxiao, linux-kselftest Hi Shuah, Sorry to bother you. Steven and Masami have reviewed this patch so can you apply it to your tree? Thanks, Xiao Yang On 2020/4/7 23:30, Masami Hiramatsu wrote: > On Tue, 7 Apr 2020 09:54:00 -0400 > Steven Rostedt<rostedt@goodmis.org> wrote: > >> On Tue, 7 Apr 2020 14:34:19 +0800 >> Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: >> >>> It is possible to get multiple records from trace during test and then more >>> than 4 arguments are assigned to ARGS. This situation results in the failure >>> of kprobe_args_type.tc. For example: >>> ----------------------------------------------------------- >>> grep testprobe trace >>> ftracetest-5902 [001] d... 111195.682227: testprobe: (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 arg4=7 >>> pmlogger-5949 [000] d... 111195.709898: testprobe: (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 arg4=7 >>> grep testprobe trace >>> sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/' >>> ARGS='334823024 334823024 0x13f4fe70 7 >>> 345308784 345308784 0x1494fe70 7' >>> ----------------------------------------------------------- >>> >>> We don't care which process calls do_fork so just check the first record to >>> fix the issue. >> >> Looks good to me, but Masami will need to ack it. >> > > Ah, good catch! I had fixed similar issue but missed this one. > > Acked-by: Masami Hiramatsu<mhiramat@kernel.org> > > Thank you! > >> -- Steve >> >>> >>> Signed-off-by: Xiao Yang<yangx.jy@cn.fujitsu.com> >>> --- >>> .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>> index 1bcb67dcae26..81490ecaaa92 100644 >>> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>> @@ -38,7 +38,7 @@ for width in 64 32 16 8; do >>> echo 0> events/kprobes/testprobe/enable >>> >>> : "Confirm the arguments is recorded in given types correctly" >>> - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>> + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>> check_types $ARGS $width >>> >>> : "Clear event for next loop" >> > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-15 1:19 ` Xiao Yang @ 2020-04-24 1:12 ` Xiao Yang 2020-04-24 1:18 ` Steven Rostedt 0 siblings, 1 reply; 8+ messages in thread From: Xiao Yang @ 2020-04-24 1:12 UTC (permalink / raw) To: shuah Cc: Masami Hiramatsu, Steven Rostedt, mingo, ice_yangxiao, linux-kselftest Hi Shuah, Ping. Best Regards, Xiao Yang On 2020/4/15 9:19, Xiao Yang wrote: > Hi Shuah, > > Sorry to bother you. > Steven and Masami have reviewed this patch so can you apply it to your > tree? > > Thanks, > Xiao Yang > On 2020/4/7 23:30, Masami Hiramatsu wrote: >> On Tue, 7 Apr 2020 09:54:00 -0400 >> Steven Rostedt<rostedt@goodmis.org> wrote: >> >>> On Tue, 7 Apr 2020 14:34:19 +0800 >>> Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: >>> >>>> It is possible to get multiple records from trace during test and >>>> then more >>>> than 4 arguments are assigned to ARGS. This situation results in the >>>> failure >>>> of kprobe_args_type.tc. For example: >>>> ----------------------------------------------------------- >>>> grep testprobe trace >>>> ftracetest-5902 [001] d... 111195.682227: testprobe: >>>> (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 >>>> arg4=7 >>>> pmlogger-5949 [000] d... 111195.709898: testprobe: >>>> (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 >>>> arg4=7 >>>> grep testprobe trace >>>> sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 >>>> \3 \4/' >>>> ARGS='334823024 334823024 0x13f4fe70 7 >>>> 345308784 345308784 0x1494fe70 7' >>>> ----------------------------------------------------------- >>>> >>>> We don't care which process calls do_fork so just check the first >>>> record to >>>> fix the issue. >>> >>> Looks good to me, but Masami will need to ack it. >>> >> >> Ah, good catch! I had fixed similar issue but missed this one. >> >> Acked-by: Masami Hiramatsu<mhiramat@kernel.org> >> >> Thank you! >> >>> -- Steve >>> >>>> >>>> Signed-off-by: Xiao Yang<yangx.jy@cn.fujitsu.com> >>>> --- >>>> .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git >>>> a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>> b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>> index 1bcb67dcae26..81490ecaaa92 100644 >>>> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>> @@ -38,7 +38,7 @@ for width in 64 32 16 8; do >>>> echo 0> events/kprobes/testprobe/enable >>>> >>>> : "Confirm the arguments is recorded in given types correctly" >>>> - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) >>>> arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>>> + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* >>>> arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>>> check_types $ARGS $width >>>> >>>> : "Clear event for next loop" >>> >> >> > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-24 1:12 ` Xiao Yang @ 2020-04-24 1:18 ` Steven Rostedt 2020-04-24 1:36 ` Xiao Yang 0 siblings, 1 reply; 8+ messages in thread From: Steven Rostedt @ 2020-04-24 1:18 UTC (permalink / raw) To: Xiao Yang Cc: shuah, Masami Hiramatsu, mingo, ice_yangxiao, linux-kselftest, Shuah Khan On Fri, 24 Apr 2020 09:12:47 +0800 Xiao Yang <yangx.jy@cn.fujitsu.com> wrote: > Hi Shuah, > > Ping. Shuah usually responds to her linuxfoundation.org email a bit better [ added ]. -- Steve > > Best Regards, > Xiao Yang > On 2020/4/15 9:19, Xiao Yang wrote: > > Hi Shuah, > > > > Sorry to bother you. > > Steven and Masami have reviewed this patch so can you apply it to your > > tree? > > > > Thanks, > > Xiao Yang > > On 2020/4/7 23:30, Masami Hiramatsu wrote: > >> On Tue, 7 Apr 2020 09:54:00 -0400 > >> Steven Rostedt<rostedt@goodmis.org> wrote: > >> > >>> On Tue, 7 Apr 2020 14:34:19 +0800 > >>> Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: > >>> > >>>> It is possible to get multiple records from trace during test and > >>>> then more > >>>> than 4 arguments are assigned to ARGS. This situation results in the > >>>> failure > >>>> of kprobe_args_type.tc. For example: > >>>> ----------------------------------------------------------- > >>>> grep testprobe trace > >>>> ftracetest-5902 [001] d... 111195.682227: testprobe: > >>>> (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 > >>>> arg4=7 > >>>> pmlogger-5949 [000] d... 111195.709898: testprobe: > >>>> (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 > >>>> arg4=7 > >>>> grep testprobe trace > >>>> sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 > >>>> \3 \4/' > >>>> ARGS='334823024 334823024 0x13f4fe70 7 > >>>> 345308784 345308784 0x1494fe70 7' > >>>> ----------------------------------------------------------- > >>>> > >>>> We don't care which process calls do_fork so just check the first > >>>> record to > >>>> fix the issue. > >>> > >>> Looks good to me, but Masami will need to ack it. > >>> > >> > >> Ah, good catch! I had fixed similar issue but missed this one. > >> > >> Acked-by: Masami Hiramatsu<mhiramat@kernel.org> > >> > >> Thank you! > >> > >>> -- Steve > >>> > >>>> > >>>> Signed-off-by: Xiao Yang<yangx.jy@cn.fujitsu.com> > >>>> --- > >>>> .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- > >>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>> > >>>> diff --git > >>>> a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > >>>> b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > >>>> index 1bcb67dcae26..81490ecaaa92 100644 > >>>> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > >>>> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc > >>>> @@ -38,7 +38,7 @@ for width in 64 32 16 8; do > >>>> echo 0> events/kprobes/testprobe/enable > >>>> > >>>> : "Confirm the arguments is recorded in given types correctly" > >>>> - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) > >>>> arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > >>>> + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* > >>>> arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` > >>>> check_types $ARGS $width > >>>> > >>>> : "Clear event for next loop" > >>> > >> > >> > > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-24 1:18 ` Steven Rostedt @ 2020-04-24 1:36 ` Xiao Yang 2020-04-24 12:22 ` Shuah Khan 0 siblings, 1 reply; 8+ messages in thread From: Xiao Yang @ 2020-04-24 1:36 UTC (permalink / raw) To: Steven Rostedt Cc: shuah, Masami Hiramatsu, mingo, ice_yangxiao, linux-kselftest, Shuah Khan On 2020/4/24 9:18, Steven Rostedt wrote: > On Fri, 24 Apr 2020 09:12:47 +0800 > Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: > >> Hi Shuah, >> >> Ping. > > Shuah usually responds to her linuxfoundation.org email a bit better [ added ]. Hi Steve, Thanks for your reminder. Sorry for the wrong email(shuah@kernel.org). Thanks, Xiao Yang > > -- Steve > >> >> Best Regards, >> Xiao Yang >> On 2020/4/15 9:19, Xiao Yang wrote: >>> Hi Shuah, >>> >>> Sorry to bother you. >>> Steven and Masami have reviewed this patch so can you apply it to your >>> tree? >>> >>> Thanks, >>> Xiao Yang >>> On 2020/4/7 23:30, Masami Hiramatsu wrote: >>>> On Tue, 7 Apr 2020 09:54:00 -0400 >>>> Steven Rostedt<rostedt@goodmis.org> wrote: >>>> >>>>> On Tue, 7 Apr 2020 14:34:19 +0800 >>>>> Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: >>>>> >>>>>> It is possible to get multiple records from trace during test and >>>>>> then more >>>>>> than 4 arguments are assigned to ARGS. This situation results in the >>>>>> failure >>>>>> of kprobe_args_type.tc. For example: >>>>>> ----------------------------------------------------------- >>>>>> grep testprobe trace >>>>>> ftracetest-5902 [001] d... 111195.682227: testprobe: >>>>>> (_do_fork+0x0/0x460) arg1=334823024 arg2=334823024 arg3=0x13f4fe70 >>>>>> arg4=7 >>>>>> pmlogger-5949 [000] d... 111195.709898: testprobe: >>>>>> (_do_fork+0x0/0x460) arg1=345308784 arg2=345308784 arg3=0x1494fe70 >>>>>> arg4=7 >>>>>> grep testprobe trace >>>>>> sed -e 's/.* arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 >>>>>> \3 \4/' >>>>>> ARGS='334823024 334823024 0x13f4fe70 7 >>>>>> 345308784 345308784 0x1494fe70 7' >>>>>> ----------------------------------------------------------- >>>>>> >>>>>> We don't care which process calls do_fork so just check the first >>>>>> record to >>>>>> fix the issue. >>>>> >>>>> Looks good to me, but Masami will need to ack it. >>>>> >>>> >>>> Ah, good catch! I had fixed similar issue but missed this one. >>>> >>>> Acked-by: Masami Hiramatsu<mhiramat@kernel.org> >>>> >>>> Thank you! >>>> >>>>> -- Steve >>>>> >>>>>> >>>>>> Signed-off-by: Xiao Yang<yangx.jy@cn.fujitsu.com> >>>>>> --- >>>>>> .../testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc | 2 +- >>>>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>>>> >>>>>> diff --git >>>>>> a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>>>> b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>>>> index 1bcb67dcae26..81490ecaaa92 100644 >>>>>> --- a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>>>> +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_type.tc >>>>>> @@ -38,7 +38,7 @@ for width in 64 32 16 8; do >>>>>> echo 0> events/kprobes/testprobe/enable >>>>>> >>>>>> : "Confirm the arguments is recorded in given types correctly" >>>>>> - ARGS=`grep "testprobe" trace | sed -e 's/.* arg1=\(.*\) >>>>>> arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>>>>> + ARGS=`grep "testprobe" trace | head -n 1 | sed -e 's/.* >>>>>> arg1=\(.*\) arg2=\(.*\) arg3=\(.*\) arg4=\(.*\)/\1 \2 \3 \4/'` >>>>>> check_types $ARGS $width >>>>>> >>>>>> : "Clear event for next loop" >>>>> >>>> >>>> >>> >> >> > > > > . > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc 2020-04-24 1:36 ` Xiao Yang @ 2020-04-24 12:22 ` Shuah Khan 0 siblings, 0 replies; 8+ messages in thread From: Shuah Khan @ 2020-04-24 12:22 UTC (permalink / raw) To: Xiao Yang, Steven Rostedt Cc: shuah, Masami Hiramatsu, mingo, ice_yangxiao, linux-kselftest, Shuah Khan On 4/23/20 7:36 PM, Xiao Yang wrote: > On 2020/4/24 9:18, Steven Rostedt wrote: >> On Fri, 24 Apr 2020 09:12:47 +0800 >> Xiao Yang<yangx.jy@cn.fujitsu.com> wrote: >> >>> Hi Shuah, >>> >>> Ping. >> >> Shuah usually responds to her linuxfoundation.org email a bit better [ >> added ]. > Hi Steve, > > Thanks for your reminder. > Sorry for the wrong email(shuah@kernel.org). > Thanks for the ping. Will be applied shortly. thanks, -- Shuah ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-04-24 12:22 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-04-07 6:34 [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc Xiao Yang 2020-04-07 13:54 ` Steven Rostedt 2020-04-07 15:30 ` Masami Hiramatsu 2020-04-15 1:19 ` Xiao Yang 2020-04-24 1:12 ` Xiao Yang 2020-04-24 1:18 ` Steven Rostedt 2020-04-24 1:36 ` Xiao Yang 2020-04-24 12:22 ` Shuah Khan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox