Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-06 Thread David Long
On 03/04/14 12:31, Oleg Nesterov wrote: On 03/04, Russell King - ARM Linux wrote: On Mon, Mar 03, 2014 at 09:50:39PM +0100, Oleg Nesterov wrote: And why CONFIG_UPROBES should depend on PERF_EVENTS? uprobes can be used by (say) systemtap without UPROBE_EVENT/PERF_EVENTS. But as Russell pointe

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-04 Thread Oleg Nesterov
On 03/04, Russell King - ARM Linux wrote: > > On Mon, Mar 03, 2014 at 09:50:39PM +0100, Oleg Nesterov wrote: > > > > And why CONFIG_UPROBES should depend on PERF_EVENTS? uprobes can be > > used by (say) systemtap without UPROBE_EVENT/PERF_EVENTS. > > > > But as Russell pointed out the events direct

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-03 Thread Russell King - ARM Linux
On Mon, Mar 03, 2014 at 09:50:39PM +0100, Oleg Nesterov wrote: > On 03/02, David Long wrote: > > > > Oleg, > > > > I've been looking at arch/Kconfig and kernel/trace/Kconfig where they > > deal with uprobes. The relevant items are CONFIG_UPROBES and > > CONFIG_UPROBE_EVENT. It just doesn't look r

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-03 Thread Oleg Nesterov
On 03/02, David Long wrote: > > Oleg, > > I've been looking at arch/Kconfig and kernel/trace/Kconfig where they > deal with uprobes. The relevant items are CONFIG_UPROBES and > CONFIG_UPROBE_EVENT. It just doesn't look right to me. Yes, this looks strange. I never understood these dependencies.

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-03 Thread Russell King - ARM Linux
On Mon, Mar 03, 2014 at 11:53:24AM +0530, Srikar Dronamraju wrote: > I know there were more discussions on this, but I cant dig them out at > this time. Finally it was decided that > 1. Users shouldnt have to select more than one config to select Uprobes. > 2. There was no point in selecting Uprob

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-03 Thread David Long
On 03/03/14 01:23, Srikar Dronamraju wrote: It should be me who should take the blame for this and not Oleg. This was discussed more than couple of times. I can recollect couple of discussions here. http://article.gmane.org/gmane.linux.kernel/1017186 http://article.gmane.org/gmane.linux.kernel

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-02 Thread Srikar Dronamraju
> Oleg, > > I've been looking at arch/Kconfig and kernel/trace/Kconfig where > they deal with uprobes. The relevant items are CONFIG_UPROBES and > CONFIG_UPROBE_EVENT. It just doesn't look right to me. It looks It should be me who should take the blame for this and not Oleg. This was discusse

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-02 Thread David Long
Oleg, I've been looking at arch/Kconfig and kernel/trace/Kconfig where they deal with uprobes. The relevant items are CONFIG_UPROBES and CONFIG_UPROBE_EVENT. It just doesn't look right to me. It looks like "select" is used in part maybe just to avoid the recursive dependency error that wou

Re: [PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-03-01 Thread Russell King - ARM Linux
On Mon, Feb 10, 2014 at 02:38:51AM -0500, David Long wrote: > This patch series adds basic uprobes support to ARM. It is based on patches > developed earlier by Rabin Vincent. That approach of adding hooks into > the kprobes instruction parsing code was not well received. This approach > separates

[PATCH v6 00/14] uprobes: Add uprobes support for ARM

2014-02-09 Thread David Long
From: "David A. Long" This patch series adds basic uprobes support to ARM. It is based on patches developed earlier by Rabin Vincent. That approach of adding hooks into the kprobes instruction parsing code was not well received. This approach separates the ARM instruction parsing code in kprobes