linux-kernel - Re: [PATCH 08/12] perf: Carve out mmap helpers for general use

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, 26 Jan 2011 02:00:02 +0100
From:	Borislav Petkov <bp@...en8.de>
To:	Arnaldo Carvalho de Melo <acme@...radead.org>
Cc:	Borislav Petkov <bp@...64.org>,
	"peterz@...radead.org" <peterz@...radead.org>,
	"mingo@...e.hu" <mingo@...e.hu>,
	"tony.luck@...el.com" <tony.luck@...el.com>,
	"rostedt@...dmis.org" <rostedt@...dmis.org>,
	"fweisbec@...il.com" <fweisbec@...il.com>,
	"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 08/12] perf: Carve out mmap helpers for general use

On Mon, Jan 24, 2011 at 10:39:36AM -0200, Arnaldo Carvalho de Melo wrote:
> Em Mon, Jan 24, 2011 at 10:04:10AM +0100, Borislav Petkov escreveu:
> > Ok, I see at least one problem with my patch - you've reworked the
> > mmaping functionality in evlist.c/evsel.c and I should use it too, I
> > guess. For that, I think you'd want me to apply my stuff ontop of your
> > perf/core branch, right?
> 
> Right, I hope to have that branch merged by Ingo soon.
>  
> > Am I missing something else?
> 
> Nope, you're not. Doing that we erode your patchset a bit, reducing its
> size.

.. which means less work for me, heey, nice! :)

> One related experience I'm doing now is to have a perf.so python
> binding, the setup.py file for it with the list of files needed for this
> specific binding is at:
> 
> http://userweb.kernel.org/~acme/setup.py
> 
> A simple tool using the resulting binding is a thread
> fork/comm/exit/sample watcher, available at:
> 
> http://userweb.kernel.org/~acme/twatch
> 
> In this process I'm moving functions around so as to reduce the number
> of tools/perf/util.c files to link into this perf.so python binding,
> untangling things as much as possible.
> 
> The binding proper is:
> 
> http://userweb.kernel.org/~acme/python.c
> 
> I'm digressing, but twatch is an example of a simple "daemon" consuming
> perf events where performance is not much of a problem.
> 
> And provides a prototyping ground when starting to design perf events
> consuming daemons :-)

Yaay, twatch looks almost like a child's play and even my grandma can
profile her system now :).

But yeah, I thought about pythonizing the ras thingy too but the reasons
against it are that we might run on systems which don't have python,
have some obscure/old version of it or we simply don't want to depend on
it or any other additional tool for that matter. Generally, we want to
run with as low overhead as possible when handling hw error info and be
as self-contained as possible.

-- 
Regards/Gruss,
    Boris.
--
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/