linux-kernel - Re: [PATCH 0/3] Add support for perf user stack dump in powerpc

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: <20160428152958.GA2730@redhat.com>
Date:	Thu, 28 Apr 2016 12:29:58 -0300
From:	Arnaldo Carvalho de Melo <acme@...hat.com>
To:	"Naveen N. Rao" <naveen.n.rao@...ux.vnet.ibm.com>
Cc:	Chandan Kumar <chandan.kumar@...ux.vnet.ibm.com>,
	linux-kernel@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org,
	maddy@...ux.vnet.ibm.com, hemant@...ux.vnet.ibm.com,
	anju@...ux.vnet.ibm.com, jolsa@...hat.com
Subject: Re: [PATCH 0/3] Add support for perf user stack dump in powerpc

Em Thu, Apr 28, 2016 at 07:53:41PM +0530, Naveen N. Rao escreveu:
> On 2016/04/28 03:01PM, Chandan Kumar wrote:
> > This patch set enables perf user stack dump on powerpc
> 
> For this patch-set:
> Reviewed-by: Naveen N. Rao <naveen.n.rao@...ux.vnet.ibm.com>
> 
> I suppose that implies my testing, but nonetheless:
> Tested-by: Naveen N. Rao <naveen.n.rao@...ux.vnet.ibm.com>
> 
> > 
> > Example with "ls" command as user program
> > ./perf record --call-graph=dwarf,8192 ls
> > 
> > ./perf script
> > ---------8< -----
> > ..
> > ...
> > perf 12549 75938.934539:       1771 cycles:ppp: 
> >                   210d90 .perf_event_exec (/boot/vmlinux)
> >                   34b914 .load_elf_binary (/boot/vmlinux)
> >                   2dcabc .search_binary_handler (/boot/vmlinux)
> >                   2dd6f0 .do_execveat_common.isra.28 (/boot/vmlinux)
> >                   2ddcd8 .sys_execve (/boot/vmlinux)
> >                    19258 system_call (/boot/vmlinux)
> 
> To expand on this, with upstream libunwind 
> (git://git.sv.gnu.org/libunwind.git) and with debuginfo installed:
> 
> # ./perf record -e cycles -g --call-graph dwarf -- find . -iname
> '*linux*'
>  ./tests/vmlinux-kallsyms.c
> ./tests/vmlinux-kallsyms.o
> ./tests/.vmlinux-kallsyms.o.cmd
> ./util/include/linux
> [ perf record: Woken up 2 times to write data ]
> [ perf record: Captured and wrote 0.371 MB perf.data (45 samples) ]
> # ./perf script
> <snip>
> find 23858 167788.657182:     881967 cycles: 
>                   8e758c _cond_resched (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                   3f83bc selinux_inode_getattr (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                   3ee254 security_inode_getattr (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                   3063a4 vfs_getattr (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                   306530 vfs_fstatat (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                   306c60 sys_newfstatat (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>                    19210 system_call (/lib/modules/4.6.0-rc3-nnr+/build/vmlinux)
>             3fffa31829a8 __fxstatat64 (/usr/lib64/libc-2.17.so)
>                 10025ac8 fts_stat.isra.2 (/usr/bin/find)
>                 10027c04 fts_read (/usr/bin/find)
>                 10003e18 find (/usr/bin/find)
>                 100035ec main (/usr/bin/find)
>             3fffa30a4580 generic_start_main.isra.0 (/usr/lib64/libc-2.17.so)
>             3fffa30a4774 __libc_start_main (/usr/lib64/libc-2.17.so)
> 
> <snip>
> 
> 
> Arnaldo, Michael,
> Can we take this through powerpc/next as well since this pretty much 
> rides on the recent perf regs work?

I haven't tested with the bits that touch the common code, have you? But
I think it should be harmless and overall this patchset looks fine, so,
if you, just in case, have tested this on a x86_64 machine and all seems
well, i.e. perf test, perf record/report with a perf-probe generated
event, to exercise this stuff, then:

Acked-by: Arnaldo Carvalho de Melo <acme@...hat.com>