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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <fcc5d688-9d58-3b4c-58de-472330160142@huawei.com>
Date:   Wed, 6 Sep 2023 21:31:32 +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 21:02, Steven Rostedt wrote:
> On Wed, 6 Sep 2023 08:56:21 -0400
> Steven Rostedt <rostedt@...dmis.org> wrote:
> 
>> On Wed, 6 Sep 2023 19:57:24 +0800
>> Zheng Yejian <zhengyejian1@...wei.com> wrote:
>>
>>> 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
>>>    }
>>
>> Ah thanks. I didn't have that on my test box :-p
>>
>> Although I'm still having issues reproducing it. Do you have anything
>> special on your kernel command line?
>>

I can reproduce it on qemu, boot command seems nothing special (replace
${kernel} with bzImage path and replace ${rootfs} with a ubuntu image):

qemu-system-x86_64 -M pc -m 8G -enable-kvm -smp cpus=12 -kernel 
${kernel} -drive file=${rootfs},if=virtio,format=raw,id=hd0 -fsdev 
local,security_model=passthrough,id=fsdev0,path=/home/test -nographic 
-append "rootwait root=/dev/vda rw console=ttyS0 rodata=off"

> 
> Also, which branch are you testing? What's the sha of the code you are testing?
>

I think it can be reproduced after any commit after commit
27152bceea1d ("eventfs: Move tracing/events to eventfs")

also can be reproduced with the newest:
65d6e954e378 ("Merge tag 'gfs2-v6.5-rc5-fixes' of 
git://git.kernel.org/pub/scm/linux/kernel/git/gfs2/linux-gfs2")

And revert commit 27152bceea1d ("eventfs: Move tracing/events to
eventfs"), the issue just seems gone.

--

Thanks,
Zheng Yejian

> -- Steve
>