From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 71FE0C2BA19 for ; Fri, 24 Apr 2020 01:12:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5A0ED20736 for ; Fri, 24 Apr 2020 01:12:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728111AbgDXBMy (ORCPT ); Thu, 23 Apr 2020 21:12:54 -0400 Received: from mail.cn.fujitsu.com ([183.91.158.132]:59470 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725888AbgDXBMx (ORCPT ); Thu, 23 Apr 2020 21:12:53 -0400 X-IronPort-AV: E=Sophos;i="5.73,309,1583164800"; d="scan'208";a="90005343" Received: from unknown (HELO cn.fujitsu.com) ([10.167.33.5]) by heian.cn.fujitsu.com with ESMTP; 24 Apr 2020 09:12:50 +0800 Received: from G08CNEXMBPEKD06.g08.fujitsu.local (unknown [10.167.33.206]) by cn.fujitsu.com (Postfix) with ESMTP id 54E134BCC897; Fri, 24 Apr 2020 09:02:13 +0800 (CST) Received: from [10.167.220.69] (10.167.220.69) by G08CNEXMBPEKD06.g08.fujitsu.local (10.167.33.206) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Fri, 24 Apr 2020 09:12:49 +0800 Message-ID: <5EA23D0F.5020304@cn.fujitsu.com> Date: Fri, 24 Apr 2020 09:12:47 +0800 From: Xiao Yang User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.2; zh-CN; rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11 MIME-Version: 1.0 To: CC: Masami Hiramatsu , Steven Rostedt , , , Subject: Re: [PATCH] selftests/ftrace: Check the first record for kprobe_args_type.tc References: <20200407063419.292821-1-yangx.jy@cn.fujitsu.com> <20200407095400.2a13b821@gandalf.local.home> <20200408003026.24627a881c8f1eacb01cea48@kernel.org> <5E96611B.10100@cn.fujitsu.com> In-Reply-To: <5E96611B.10100@cn.fujitsu.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.167.220.69] X-ClientProxiedBy: G08CNEXCHPEKD05.g08.fujitsu.local (10.167.33.203) To G08CNEXMBPEKD06.g08.fujitsu.local (10.167.33.206) X-yoursite-MailScanner-ID: 54E134BCC897.AA628 X-yoursite-MailScanner: Found to be clean X-yoursite-MailScanner-From: yangx.jy@cn.fujitsu.com Sender: linux-kselftest-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kselftest@vger.kernel.org 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 wrote: >> >>> On Tue, 7 Apr 2020 14:34:19 +0800 >>> Xiao Yang 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 >> >> Thank you! >> >>> -- Steve >>> >>>> >>>> Signed-off-by: Xiao Yang >>>> --- >>>> .../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" >>> >> >> >