linux-kernel - Re: [PATCH] selftests/ftrace: Correctly enable event in instance-event.tc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2667182e-691e-2ab4-4c2d-a47d76f38107@huawei.com>
Date:   Wed, 6 Sep 2023 19:57:24 +0800
From:   Zheng Yejian <zhengyejian1@...wei.com>
To:     Steven Rostedt <rostedt@...dmis.org>
CC:     "Masami Hiramatsu (Google)" <mhiramat@...nel.org>,
        Ajay Kaher <akaher@...are.com>, <shuah@...nel.org>,
        <linux-kernel@...r.kernel.org>,
        <linux-trace-kernel@...r.kernel.org>,
        <linux-kselftest@...r.kernel.org>,
        Shuah Khan <skhan@...uxfoundation.org>,
        Ye Weihua <yeweihua4@...wei.com>
Subject: Re: [PATCH] selftests/ftrace: Correctly enable event in
 instance-event.tc

On 2023/9/6 19:27, Steven Rostedt wrote:
> On Wed, 6 Sep 2023 09:42:44 +0800
> Zheng Yejian <zhengyejian1@...wei.com> wrote:
> 
>> Hi, Steve, your new patch seems not fixes the KASAN issue.
>>
>> I run the instance-event.tc with your patch again and get same KASAN
>> report.
>>
>> My config is in:
>>   
>> https://lore.kernel.org/all/da5495b5-cdb5-cb82-dd9b-37e7a87ed97d@huawei.com/
> 
> I'm not able to reproduce it.
> 
> Can you send me the full dmesg.

Hi, Steve,

can you receive the attachments? dmesg log is too long, so
I put them in attachment file:

dmesg before the patch see attachment "dmesg_before_patch"
https://lore.kernel.org/all/20230905183332.628d7cc0@gandalf.local.home/
and dmesg after above patch see attachment "dmesg_after_patch".

BTW, I do reproduce with following testcase fix, then just run: 
./ftracetest test.d/instances/instance-event.tc
diff --git 
a/tools/testing/selftests/ftrace/test.d/instances/instance-event.tc 
b/tools/testing/selftests/ftrace/test.d/instances/instance-event.tc
index 0eb47fbb3f44..42422e425107 100644
--- a/tools/testing/selftests/ftrace/test.d/instances/instance-event.tc
+++ b/tools/testing/selftests/ftrace/test.d/instances/instance-event.tc
@@ -39,7 +39,7 @@ instance_read() {

  instance_set() {
          while :; do
-                echo 1 > foo/events/sched/sched_switch
+                echo 1 > foo/events/sched/sched_switch/enable
          done 2> /dev/null
  }


--

Thanks,
Zheng Yejian

> 
> Thanks,
> 
> -- Steve
> 

View attachment "dmesg_before_patch" of type "text/plain" (32548 bytes)

View attachment "dmesg_after_patch" of type "text/plain" (26286 bytes)