linux-kernel - Re: register_timer_hook use in arch/sh/oprofile

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]

Date:	Wed, 24 Jun 2009 21:25:33 +0900
From:	Paul Mundt <lethal@...ux-sh.org>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Martin Schwidefsky <schwidefsky@...ibm.com>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Paul Mackerras <paulus@...ba.org>
Subject: Re: register_timer_hook use in arch/sh/oprofile

On Wed, Jun 24, 2009 at 02:17:50PM +0200, Ingo Molnar wrote:
> 
> * Paul Mundt <lethal@...ux-sh.org> wrote:
> 
> > In practice oprofile has never been a good fit for these sorts of 
> > counters, so this has fairly limited use. If there's a way to 
> > wiggle these types of counters in to the new perf_counter API, 
> > then I'll convert that over and just kill the old oprofile driver 
> > off completely. Barring that, I'll just end up converting it over 
> > to hrtimers as well, so don't let that stop you from ripping out 
> > the timer hook bits.
> > 
> > Most of this code predates hrtimers anyways, and it also predates 
> > the timer hook, which is only something that we converted to some 
> > years back.
> 
> Note, the current initial upstream SH support for perfcounters:
> 
>  arch/sh/include/asm/perf_counter.h:#define set_perf_counter_pending()	do { } while (0)
>  arch/sh/include/asm/unistd_32.h:#define __NR_perf_counter_open	336
>  arch/sh/include/asm/unistd_64.h:#define __NR_perf_counter_open	364
>  arch/sh/kernel/syscalls_32.S:	.long sys_perf_counter_open
>  arch/sh/kernel/syscalls_64.S:	.long sys_perf_counter_open
> 
> Should already give you hrtimers straight away.
> 
> To test it, could you try to run 'perf top' after:
> 
> 	cd tools/perf/
> 	make install
> 
> It should display a hrtimer driven kernel profile already. You can 
> increase/decrease the frequency of sampling by using -F option - say 
> 'perf top -F 10000' should sample at 10 KHz.
> 
> Please let me know if any of this does not work as expected.
> 
Yes, that all works fine. My comment was more in reference to the
hardware performance counters that don't have IRQs of their own, which
still need to be tied in to the perf_counter API.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/