linux-kernel - Re: [PATCH v9 3.2 0/9] Uprobes patchset with perf probe support

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: <20120126111041.GG3853@elte.hu>
Date:	Thu, 26 Jan 2012 12:10:41 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Peter Zijlstra <peterz@...radead.org>
Cc:	Srikar Dronamraju <srikar@...ux.vnet.ibm.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Arnaldo Carvalho de Melo <acme@...hat.com>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Oleg Nesterov <oleg@...hat.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Linux-mm <linux-mm@...ck.org>, Andi Kleen <andi@...stfloor.org>,
	Christoph Hellwig <hch@...radead.org>,
	Steven Rostedt <rostedt@...dmis.org>,
	Roland McGrath <roland@...k.frob.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>,
	Arnaldo Carvalho de Melo <acme@...radead.org>,
	Anton Arapov <anton@...hat.com>,
	Ananth N Mavinakayanahalli <ananth@...ibm.com>,
	Jim Keniston <jkenisto@...ux.vnet.ibm.com>,
	Stephen Rothwell <sfr@...b.auug.org.au>
Subject: Re: [PATCH v9 3.2 0/9] Uprobes patchset with perf probe support


* Peter Zijlstra <peterz@...radead.org> wrote:

> On Tue, 2012-01-17 at 10:39 +0100, Ingo Molnar wrote:
> 
> > I did not suggest anything complex or intrusive: just basically 
> > unify the namespace, have a single set of callbacks, and call 
> > into the uprobes and perf code from those callbacks - out of the 
> > sight of MM code.
> > 
> > That unified namespace could be called:
> > 
> >     event_mmap(...);
> >     event_fork(...);
> > 
> > etc. - and from event_mmap() you could do a simple:
> > 
> > 	perf_event_mmap(...)
> > 	uprobes_event_mmap(...)
> > 
> > [ Once all this is updated to use tracepoints it would turn into 
> >   a notification callback chain kind of thing. ]
> 
> We keep disagreeing on this. I utterly loathe hiding stuff in 
> notifier lists. It makes it completely non-obvious who all 
> does what.

My immediate suggestion was not a notifier list but an 
open-coded list of function calls done in helper inline 
functions - to minimize the impact of the callbacks on mm/.

> Another very good reason to not do what you suggest is that 
> perf_event_mmap() is a pure consumer, it doesn't have a return 
> value, whereas uprobes_mmap() can actually fail the mmap.

You know that i disagree with that, there is no fundamental 
reason why event callbacks couldnt participate in program logic, 
as long as the call site explicitly wants such side effects. It 
avoids senseless duplication of callbacks.

Anyway, if Andrew is fine with the current callbacks as-is then 
it's fine to me as well.

Thanks,

	Ingo
--
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/