[for-next][PATCH 4/9] tracing: Move find_event_field() into trace_events.c

2013-03-15 Thread Steven Rostedt
l function names. Link: http://lkml.kernel.org/r/513d8426.9070...@huawei.com Signed-off-by: zhangwei(Jovi) [ rostedt: Modified trace_find_field() to trace_find_event_field() ] Signed-off-by: Steven Rostedt --- kernel/trace/trace.h |6 ++ kernel/trace/trace_events.c

[for-next][PATCH 5/9] tracing: Convert trace_destroy_fields() to static

2013-03-15 Thread Steven Rostedt
From: "zhangwei(Jovi)" trace_destroy_fields() is not used outside of the file. It can be a static function. Link: http://lkml.kernel.org/r/513d842a.2000...@huawei.com Signed-off-by: zhangwei(Jovi) Signed-off-by: Steven Rostedt --- kernel/trace/trace_events.c |2 +- 1 file

[for-next][PATCH 2/9] tracing: Use pr_warn_once instead of open coded implementation

2013-03-15 Thread Steven Rostedt
From: "zhangwei(Jovi)" Use pr_warn_once, instead of making an open coded implementation. Link: http://lkml.kernel.org/r/513d8419.20...@huawei.com Signed-off-by: zhangwei(Jovi) Signed-off-by: Steven Rostedt --- kernel/trace/trace.c |9 ++--- 1 file changed, 2 insert

[for-next][PATCH 3/9] tracing: Use TRACE_MAX_PRINT instead of constant

2013-03-15 Thread Steven Rostedt
From: "zhangwei(Jovi)" TRACE_MAX_PRINT macro is defined, but is not used. Link: http://lkml.kernel.org/r/513d8421.4070...@huawei.com Signed-off-by: zhangwei(Jovi) Signed-off-by: Steven Rostedt --- kernel/trace/trace.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[for-next][PATCH 6/9] tracing: Fix comment about prefix in arch_syscall_match_sym_name()

2013-03-15 Thread Steven Rostedt
From: "zhangwei(Jovi)" ppc64 has its own syscall prefix like ".SyS" or ".sys". Make the comment in arch_syscall_match_sym_name() more understandable. Link: http://lkml.kernel.org/r/513d842f.40...@huawei.com Signed-off-by: zhangwei(Jovi) Signed-off-by: Ste

Re: [RFC PATCH] seq_file: Use seq_puts when seq_printf has only a format with no args

2013-03-16 Thread Steven Rostedt
seq_printf(seq, fmt, ##__VA_ARGS__);\ > + else\ > + seq_puts(seq, fmt); \ > +} while (0) BTW, you need to return a value. > + > int se

Re: [RFC PATCH] seq_file: Use seq_puts when seq_printf has only a format with no args

2013-03-16 Thread Steven Rostedt
On Sat, 2013-03-16 at 09:43 -0600, Bjorn Helgaas wrote: > This is certainly a neat trick. Thank you ;-) > > But I don't really like the fact that it complicates things for every > future code reader, especially when a trivial change in the caller > would accomplish the same thing. Do you have

Re: [RFC PATCH] seq_file: Use seq_puts when seq_printf has only a format with no args

2013-03-16 Thread Steven Rostedt
On Sat, 2013-03-16 at 09:15 -0700, Joe Perches wrote: > > > +int (seq_printf)(struct seq_file *m, const char *f, ...) > > > > That's rather ugly. Why not just #undef seq_printf before defining it? > > The whole thing is ugly, nasty and hackish. > I kinda like it. > > But I don't like unnecessar

Re: [RFC PATCH] seq_file: Use seq_puts when seq_printf has only a format with no args

2013-03-16 Thread Steven Rostedt
On Sat, 2013-03-16 at 18:01 +, Al Viro wrote: > On Sat, Mar 16, 2013 at 10:51:18AM -0700, Joe Perches wrote: > > > This is certainly a neat trick. > > > > > > But I don't really like the fact that it complicates things for every > > > future code reader, especially when a trivial change in the

Re: [PATCH v2 3/7] f2fs: add tracepoint for tracing the page i/o operations

2013-03-17 Thread Steven Rostedt
On Sun, 2013-03-17 at 17:40 +0900, Namjae Jeon wrote: > From: Namjae Jeon > > Add tracepoints for page i/o operations and block allocation > tracing during page read operation. > > Signed-off-by: Namjae Jeon > Signed-off-by: Pankaj Kumar > --- > fs/f2fs/data.c | 11 ++-- >

Re: [PATCH 1/2] tracing: syscall_*regfunc() can race with copy_process()

2013-03-17 Thread Steven Rostedt
On Sun, 2013-03-17 at 19:28 +0100, Oleg Nesterov wrote: > syscall_regfunc() and syscall_unregfunc() should set/clear > TIF_SYSCALL_TRACEPOINT system-wide, but do_each_thread() can race > with copy_process() and miss the new child which was not added to > init_task.tasks list yet. > > Change copy_p

[ANNOUNCE] 3.2.28-rt42

2012-08-30 Thread Steven Rostedt
g the incremental patch: http://www.kernel.org/pub/linux/kernel/projects/rt/3.2/incr/patch-3.2.28-rt41-rt42.patch.xz Enjoy, -- Steve Changes from 3.2.28-rt41: --- Frank Rowand (1): fix printk flush of messages Steven Rostedt (1): Linux 3.2.28-rt42 kernel/printk.c |

[ANNOUNCE] 3.0.41-rt62

2012-08-30 Thread Steven Rostedt
g the incremental patch: http://www.kernel.org/pub/linux/kernel/projects/rt/3.0/incr/patch-3.0.41-rt61-rt62.patch.xz Enjoy, -- Steve Changes from 3.0.41-rt61: --- Frank Rowand (1): fix printk flush of messages Steven Rostedt (1): Linux 3.0.41-rt62 kernel/printk.c |

[ANNOUNCE] 3.0.42-rt63

2012-08-30 Thread Steven Rostedt
Dear RT Folks, I'm pleased to announce the 3.0.42-rt63 stable release. This release is just an update to the new stable 3.0.42 version and no RT specific changes have been made. You can get this release via the git tree at: git://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.g

Re: i386, v3.6-rc3: Kernel panic - not syncing: Fatal exception in interrupt

2012-09-04 Thread Steven Rostedt
On Tue, 2012-08-28 at 10:12 +0200, Robert Richter wrote: > On 27.08.12 17:14:27, Robert Richter wrote: > > Steven and Ingo, > > > > while running profiling tests on 32 bit I got a > > > > Kernel panic - not syncing: Fatal exception in interrupt Sorry for the late response, was too busy drinking

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-09-04 Thread Steven Rostedt
On Tue, 2012-08-28 at 19:00 -0400, Mathieu Desnoyers wrote: > Looking again at: > > +#define hash_for_each_size(name, bits, bkt, node, obj, member) > \ > + for (bkt = 0; bkt < HASH_SIZE(bits); bkt++) > \ > + hlist_for_each_entry(ob

Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt

2012-09-04 Thread Steven Rostedt
On Wed, 2012-08-29 at 09:28 +0800, Cong Wang wrote: > On Tue, Aug 28, 2012 at 2:22 AM, Steven Rostedt wrote: > > The file feature-removal-schedule.txt is ignored by most people except > > for people that add to it. It's more of a global TODO list for > > developers tha

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 17:40 +0100, Pedro Alves wrote: > BTW, you can also go a step further and remove the need to close with double > }}, > with something like: > > #define do_for_each_ftrace_rec(pg, rec) >\ > for (pg = ftrace_pages_start, rec

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 18:21 +0100, Pedro Alves wrote: > On 09/04/2012 06:17 PM, Steven Rostedt wrote: > > On Tue, 2012-09-04 at 17:40 +0100, Pedro Alves wrote: > > > >> BTW, you can also go a step further and remove the need to close with > >> do

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 22:51 +0100, Pedro Alves wrote: > On 09/04/2012 09:59 PM, Steven Rostedt wrote: > > On Tue, 2012-09-04 at 18:21 +0100, Pedro Alves wrote: > >> On 09/04/2012 06:17 PM, Steven Rostedt wrote: > >>> On Tue, 2012-09-04 at 17:40 +0100, Pedro Alves wr

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-04 Thread Steven Rostedt
he idle loop. > > > In other environments, RCU_NONIDLE() could be used to properly protect > > > the use of RCU, but RCU_NONIDLE() currently cannot be invoked except > > > from process context. > > > > > > This commit therefore modifies RCU_NONIDLE()

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 16:08 -0700, Josh Triplett wrote: > On Tue, Sep 04, 2012 at 06:51:22PM -0400, Steven Rostedt wrote: > > On Tue, 2012-09-04 at 15:33 -0700, Paul E. McKenney wrote: > > > On Fri, Aug 31, 2012 at 11:00:52AM -0700, Josh Triplett wrote: > > > > On T

Re: [PATCH v3 01/17] hashtable: introduce a small and naive hashtable

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 23:58 +0100, Pedro Alves wrote: > Not true. The comma operator introduces a sequence point. It's the comma > that separates function parameters that doesn't guarantee ordering. Bah! C language is almost as confusing as English. -- Steve -- To unsubscribe from this list

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-04 Thread Steven Rostedt
On Tue, 2012-09-04 at 16:33 -0700, Josh Triplett wrote: > > > > +#ifdef MODULE > > > +#define __DECLARE_TRACE_RCU(name, proto, args, cond, data_proto, > > > data_args) \ > > > + static inline void trace_##name##_rcuidle(proto)\ > > > + {

Re: [PATCH v3] trace: Move trace event enable from fs_initcall to core_initcall

2012-09-11 Thread Steven Rostedt
ameter and is important > to trace boot-up allocations. > > Note that, in order to enable events at core_initcall, > it's necessary to move init_ftrace_syscalls() from > core_initcall to early_initcall. Found another issue... > > Cc: Steven Rostedt > Signed-off-b

Re: [PATCH 1/3] tracing,x86: add a TSC trace_clock; reset buffer on clock change

2012-09-11 Thread Steven Rostedt
On Tue, 2012-09-11 at 19:41 -0700, David Sharp wrote: > In order to promote interoperability between userspace tracers and ftrace, > add a trace_clock that reports raw TSC values which will then be recorded > in the ring buffer. Userspace tracers that also record TSCs are then on > exactly the same

Re: [PATCH] plist: Make plist test announcements KERN_DEBUG

2012-09-12 Thread Steven Rostedt
On Wed, 2012-09-12 at 12:38 +0200, Borislav Petkov wrote: > From: Borislav Petkov > > They show up in dmesg > > [4.041094] start plist test > [4.045804] end plist test > > without a lot of meaning so hide them behind debug loglevel. > Acked-by: Steven

Re: [PATCH v3] trace: Move trace event enable from fs_initcall to core_initcall

2012-09-12 Thread Steven Rostedt
On Wed, 2012-09-12 at 09:29 -0300, Ezequiel Garcia wrote: > Steven, > > On Tue, Sep 11, 2012 at 9:34 PM, Steven Rostedt wrote: > ble(void) > [...] > >> + /* > >> + * Early initialization already enabled ftrace event. > >> + * Now it'

Re: [PATCH -v2] x86, 32-bit: Fix invalid stack address while in softirq

2012-09-12 Thread Steven Rostedt
inter we fall back to regs as stack if no previous stack > + * exists. > + * > + * This is valid only for kernel mode traps. > + */ > +unsigned long kernel_stack_pointer(struct pt_regs *regs) > +{ > + unsigned long context = (unsigned long)regs & ~(THREAD_SIZE - 1); >

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-12 Thread Steven Rostedt
On Tue, 2012-09-11 at 18:07 -0700, Paul E. McKenney wrote: > > > Paul, you just push your changes through tip, right? Then we can just > > let Ingo know. I could even make the patch a separate branch, that Ingo > > can pull into the RCU branch too. > > Yep! But we also need to worry about -next

[PATCH RT] hwlat_detector: Check the outer side of the loop

2012-09-12 Thread Steven Rostedt
rd column shows the delta of the outer part of the loop, and is reported if that is greater than the threshold. Signed-off-by: Steven Rostedt diff --git a/drivers/misc/hwlat_detector.c b/drivers/misc/hwlat_detector.c index b7b7c90..b53bdfa 100644 --- a/drivers/misc/hwlat_detector.c +++

Re: [PATCH tip/core/rcu 04/15] rcu: Permit RCU_NONIDLE() to be used from interrupt context

2012-09-12 Thread Steven Rostedt
[ Added Linus to verify what I'm talking about ] On Wed, 2012-09-12 at 08:03 -0700, Paul E. McKenney wrote: > Doesn't Fengguang pull branches individually? And won't that mean > that as soon as I push my export-removal commit, that he will see > build failures? > > Or are you suggesting that we

[PATCH][GIT PULL][v3.7] trace: Don't declare trace_*_rcuidle functions in modules

2012-09-12 Thread Steven Rostedt
GA14756@leaf Reported-by: Steven Rostedt Acked-by: Mathieu Desnoyers Acked-by: Paul E. McKenney Signed-off-by: Josh Triplett Signed-off-by: Steven Rostedt diff --git a/include/linux/tracepoint.h b/include/linux/tracepoint.h index 802de56..2f322c3 100644 --- a/in

Re: [PATCH v2 1/3] tracing,x86: add a TSC trace_clock

2012-09-12 Thread Steven Rostedt
This change will require an ack from one of the x86 maintainers. For reference to this patch, please see this thread: https://lkml.org/lkml/2012/9/11/572 On Wed, 2012-09-12 at 16:31 -0700, David Sharp wrote: > In order to promote interoperability between userspace tracers and ftrace, > add a tr

Re: [PATCH 1/7] perf: Enable function tracing for perf core

2012-09-13 Thread Steven Rostedt
On Sat, 2012-09-08 at 15:06 +0200, Frederic Weisbecker wrote: > On Sat, Sep 08, 2012 at 07:55:16AM -0400, Steven Rostedt wrote: > > According to Steven R. there is no reason left to not support > > function tracing for the perf core. This makes it easier to debug > > perf.

[PATCH RT 2/2] Linux 3.0.42-rt64-rc1

2012-09-13 Thread Steven Rostedt
From: Steven Rostedt --- localversion-rt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index b0e8dd7..c64934c 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt63 +-rt64-rc1 -- 1.7.10.4 -- To unsubscribe from this

[PATCH RT 1/2] fix printk flush of messages

2012-09-13 Thread Steven Rostedt
Signed-off-by: Steven Rostedt --- drivers/tty/serial/cpm_uart/cpm_uart_core.c |2 +- drivers/tty/sysrq.c | 23 --- include/linux/sysrq.h |5 - kernel/printk.c |5 ++--- lib/Kconfig.debug

[PATCH RT 0/2] [ANNOUNCE] 3.0.42-rt64-rc1 stable review

2012-09-13 Thread Steven Rostedt
u can also build from 3.0.42-rt63 by applying the incremental patch: http://www.kernel.org/pub/linux/kernel/projects/rt/3.0/incr/patch-3.0.42-rt63-rt64-rc1.patch.xz Changes from 3.0.42-rt63: --- Frank Rowand (1): fix printk flush of messages Steven Rostedt (1): Linux 3.0.42-rt6

[PATCH RT 0/2] [ANNOUNCE] 3.2.28-rt43-rc1 stable review

2012-09-13 Thread Steven Rostedt
u can also build from 3.2.28-rt42 by applying the incremental patch: http://www.kernel.org/pub/linux/kernel/projects/rt/3.2/incr/patch-3.2.28-rt42-rt43-rc1.patch.xz Changes from 3.2.28-rt42: --- Frank Rowand (1): fix printk flush of messages Steven Rostedt (1): Linux 3.2.28-rt4

[PATCH RT 2/2] Linux 3.2.28-rt43-rc1

2012-09-13 Thread Steven Rostedt
From: Steven Rostedt --- localversion-rt |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/localversion-rt b/localversion-rt index 8bdfb9a..d5a74ba 100644 --- a/localversion-rt +++ b/localversion-rt @@ -1 +1 @@ --rt42 +-rt43-rc1 -- 1.7.10.4 -- To unsubscribe from this

[PATCH RT 1/2] fix printk flush of messages

2012-09-13 Thread Steven Rostedt
Signed-off-by: Steven Rostedt --- drivers/tty/serial/cpm_uart/cpm_uart_core.c |2 +- drivers/tty/sysrq.c | 23 --- include/linux/sysrq.h |5 - kernel/printk.c |5 ++--- lib/Kconfig.debug

Re: [PATCH RT 0/2] [ANNOUNCE] 3.0.42-rt64-rc1 stable review

2012-09-13 Thread Steven Rostedt
On Thu, 2012-09-13 at 14:52 -0400, Steven Rostedt wrote: > > Note: The patches have not been uploaded to kernel.org yet. I've > recently changed the subkey that I use to sign the patches as the > old subkey is about to expire. But unfortunately, kernel.org has > not updat

Re: [PATCH RT 0/2] [ANNOUNCE] 3.2.28-rt43-rc1 stable review

2012-09-13 Thread Steven Rostedt
On Thu, 2012-09-13 at 14:57 -0400, Steven Rostedt wrote: > > Note: The patches have not been uploaded to kernel.org yet. I've > recently changed the subkey that I use to sign the patches as the > old subkey is about to expire. But unfortunately, kernel.org has > not updat

Re: [RFC] add "enable" to the kconfig language

2012-09-13 Thread Steven Rostedt
On Fri, 2012-09-14 at 10:14 +1000, Stephen Rothwell wrote: > Hi all, > > I have noticed that we use the following paradigm quite a bit theses days: > > configselect HAVE_ > > config > depends on HAVE_ > > or similar > > I was wondering if it would make sense to replace this with

[PATCH 0/6 v2] [GIT PULL][3.7] tracing: cleanups and fixes

2012-09-13 Thread Steven Rostedt
like as x86-64 kprobes/x86: Fix kprobes to collectly handle IP on ftrace kprobes/x86: Fix to support jprobes on ftrace-based kprobe Steven Rostedt (1): ftrace/x86-64: Allow to change RIP in handlers Yuanhan Liu (1): tracing: Skip printing "OK" if failed to dis

[PATCH 5/6 v2] ftrace/x86-64: Allow to change RIP in handlers

2012-09-13 Thread Steven Rostedt
From: Steven Rostedt Allow ftrace handlers to change RIP register (regs->ip) in handlers. This will allow handlers to call another function instead of original function. Link: http://lkml.kernel.org/r/20120905143118.10329.5078.stgit@localhost.localdomain Cc: Peter Zijlstra Cc: Frede

[PATCH 3/6 v2] ftrace/x86: Adjust x86 regs.ip as like as x86-64

2012-09-13 Thread Steven Rostedt
ric Weisbecker Cc: Thomas Gleixner Cc: "H. Peter Anvin" Signed-off-by: Masami Hiramatsu Signed-off-by: Steven Rostedt --- arch/x86/kernel/entry_32.S |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/x86/kernel/entry_32.S b/arch/x86/kernel/entry_32.S index 061

[PATCH 2/6 v2] trace: Stop compiling in trace_clock unconditionally

2012-09-13 Thread Steven Rostedt
Weisbecker Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Andrew Morton Cc: "Eric W. Biederman" Cc: Al Viro Signed-off-by: Josh Triplett Signed-off-by: Steven Rostedt --- kernel/Makefile |2 +- kernel/trace/Kconfig |5 + kernel/trace/Makefile |6 +- 3 files changed

[PATCH 4/6 v2] kprobes/x86: Fix kprobes to collectly handle IP on ftrace

2012-09-13 Thread Steven Rostedt
.72069.stgit@localhost.localdomain Cc: Peter Zijlstra Cc: Frederic Weisbecker Cc: Thomas Gleixner Cc: "H. Peter Anvin" Signed-off-by: Masami Hiramatsu Signed-off-by: Steven Rostedt --- arch/x86/kernel/kprobes.c |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff

[PATCH 1/6 v2] tracing: Skip printing "OK" if failed to disable event

2012-09-13 Thread Steven Rostedt
From: Yuanhan Liu No acutal case found. But logically, we should skip "OK" in case any error met. Link: http://lkml.kernel.org/r/1346051625-25231-1-git-send-email-yuanhan@linux.intel.com Signed-off-by: Yuanhan Liu Signed-off-by: Steven Rostedt --- kernel/trace/trace_event

[PATCH 6/6 v2] kprobes/x86: Fix to support jprobes on ftrace-based kprobe

2012-09-13 Thread Steven Rostedt
c: Peter Zijlstra Cc: Frederic Weisbecker Cc: Thomas Gleixner Cc: "H. Peter Anvin" Signed-off-by: Masami Hiramatsu Signed-off-by: Steven Rostedt --- arch/x86/kernel/kprobes.c | 42 +- kernel/kprobes.c |3 --- 2 files changed,

Re: [PATCH 1/2] Update FSF postal address to be URL's.

2012-09-14 Thread Steven Rostedt
Signed-off-by: Jon Stanley > Cc: Arnaldo Carvalho de Melo > Cc: Steven Rostedt Acked-by: Steven Rostedt Arnaldo, can you pull these patches into your queue. Thanks, -- Steve > --- > tools/lib/traceevent/event-parse.c |3 +-- > tools/lib/traceevent/event-parse.h |3

[PATCH 15/23 -v8] Generic command line storage

2008-01-30 Thread Steven Rostedt
may simply store this information in the trace itself. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- lib/tracing/Kconfig |1 lib/tracing/trace_function.c |2 lib/tracing/trace_sched_switch.c |5 + lib/tracing/tracer.c

[PATCH 13/23 -v8] Make the task State char-string visible to all

2008-01-30 Thread Steven Rostedt
The tracer wants to be able to convert the state number into a user visible character. This patch pulls that conversion string out the scheduler into the header. This way if it were to ever change, other parts of the kernel will know. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- i

[PATCH 12/23 -v8] Add context switch marker to sched.c

2008-01-30 Thread Steven Rostedt
Add marker into context_switch to record the prev and next tasks. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- kernel/sched.c |2 ++ 1 file changed, 2 insertions(+) Index: linux-mcount.git/kernel/sched.c === ---

[PATCH 03/23 -v8] Annotate core code that should not be traced

2008-01-30 Thread Steven Rostedt
Mark with "notrace" functions in core code that should not be traced. The "notrace" attribute will prevent gcc from adding a call to mcount on the annotated funtions. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> Signed-off-by: Steven Rostedt <

[PATCH 01/23 -v8] printk - dont wakeup klogd with interrupts disabled

2008-01-30 Thread Steven Rostedt
hed.c for other files to see if the current runqueue lock is held. This is used in printk to determine whether it is safe or not to wakeup the klogd. And with this patch, my code ran fine ;-) Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- include/linux/sched.h |2 ++ kernel/

[PATCH 14/23 -v8] Add tracing of context switches

2008-01-30 Thread Steven Rostedt
:R --> 3139:120 gdm-bina-3139 0d..3 3us!: 3139:120:S --> 0:140 swapper-0 1d..3 102328us+: 0:140:R --> 2847:120 yum-upda-2847 1d..3 102348us!: 2847:120:S --> 0:140 "sched_switch" is added to /debugfs/tracing/available_tracers Signed-off-by: Steven Rostedt &

[PATCH 07/23 -v8] initialize the clock source to jiffies clock.

2008-01-30 Thread Steven Rostedt
use clocksource_jiffies, so that any early user of clocksource_read will not crash. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> Acked-by: John Stultz <[EMAIL PROTECTED]> --- include/linux/clocksource.h |3 +++ kernel/time/timekeeping.c |9 +++-- 2 files changed, 10 i

[PATCH 10/23 -v8] mcount tracer add preempt_enable/disable notrace macros

2008-01-30 Thread Steven Rostedt
) functions. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- include/linux/clocksource.h |5 +++-- include/linux/preempt.h | 32 2 files changed, 35 insertions(+), 2 deletions(-) Index: linux-mcount.git/include/linux/clockso

[PATCH 00/23 -v8] mcount and latency tracing utility -v8

2008-01-30 Thread Steven Rostedt
[ version 8 of mcount / trace patches: changes include: Added read_barrier_depends to mcount and sched function loops incase anyone is crazy enough to port this to alpha. (Suggested by Peter Ziljstra) Added smp_rmb for the wakeup_sched_switch to match the smp_wmb in the wake

[PATCH 09/23 -v8] add notrace annotations to timing events

2008-01-30 Thread Steven Rostedt
This patch adds notrace annotations to timer functions that will be used by tracing. This helps speed things up and also keeps the ugliness of printing these functions down. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/kernel/apic_32.c |2 +- arch/x86/kernel/

[PATCH 05/23 -v8] add notrace annotations to vsyscall.

2008-01-30 Thread Steven Rostedt
Add the notrace annotations to some of the vsyscall functions. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/kernel/vsyscall_64.c |3 ++- arch/x86/vdso/vclock_gettime.c | 15 --- arch/x86/vdso/vgetcpu.c|3 ++- include/asm-x86/vsyscall.h

[PATCH 21/23 -v8] Add markers to various events

2008-01-30 Thread Steven Rostedt
This patch adds markers to various events in the kernel. (interrupts, task activation and hrtimers) Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/kernel/apic_32.c |2 ++ arch/x86/kernel/irq_32.c |1 + arch/x86/kernel/irq_64.c |2 ++ arch/x86/kernel/trap

[PATCH 16/23 -v8] trace generic call to schedule switch

2008-01-30 Thread Steven Rostedt
This patch adds hooks into the schedule switch tracing to allow other latency traces to hook into the schedule switches. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- lib/tracing/trace_sched_switch.c | 135 +-- lib/tracing/tracer.h

[PATCH 17/23 -v8] Add marker in try_to_wake_up

2008-01-30 Thread Steven Rostedt
Add markers into the wakeup code, to allow the tracer to record wakeup timings. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- kernel/sched.c |8 1 file changed, 8 insertions(+) Index: linux-mcount.git/kernel/s

[PATCH 08/23 -v8] add get_monotonic_cycles

2008-01-30 Thread Steven Rostedt
e in a monotonic fashion. Signed-off-by: John Stultz <[EMAIL PROTECTED]> Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- include/linux/clocksource.h | 54 +--- kernel/time/timekeeping.c | 26 +++-- 2 files changed, 70

[PATCH 04/23 -v8] x86_64: notrace annotations

2008-01-30 Thread Steven Rostedt
Add "notrace" annotation to x86_64 specific files. Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/kernel/head64.c |2 +- arch/x86/kernel/setup64.c|4 ++-- arch/x86/kernel/smpboot_6

[PATCH 02/23 -v8] Add basic support for gcc profiler instrumentation

2008-01-30 Thread Steven Rostedt
pens to be registered. [This code has been highly hacked by Steven Rostedt, so don't blame Arnaldo for all of this ;-) ] Update: It is now possible to register more than one mcount function. If only one mcount function is registered, that will be the function that mcount calls directl

[PATCH 23/23 -v8] Critical latency timings histogram

2008-01-30 Thread Steven Rostedt
This patch adds hooks into the latency tracer to give us histograms of interrupts off, preemption off and wakeup timings. This code was based off of work done by Yi Yang <[EMAIL PROTECTED]> But heavily modified to work with the new tracer, and some clean ups by Steven Rostedt <[EMAIL

[PATCH 18/23 -v8] mcount tracer for wakeup latency timings.

2008-01-30 Thread Steven Rostedt
t+0xe/0x77 ) bash-10653 1d..4 36us : sched_switch_callback+0x73/0x81 (schedule+0x483/0x6d5 ) vim:ft=help The [...] was added here to not waste your email box space. Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- lib/tracing/Kconfig| 14 + lib/tracing/Make

[PATCH 22/23 -v8] Add event tracer.

2008-01-30 Thread Steven Rostedt
This patch adds a event trace that hooks into various events in the kernel. Although it can be used separately, it is mainly to help other traces (wakeup and preempt off) with seeing various events in the traces without having to enable the heavy mcount hooks. Signed-off-by: Steven Rostedt

[PATCH 06/23 -v8] handle accurate time keeping over long delays

2008-01-30 Thread Steven Rostedt
From: John Stultz <[EMAIL PROTECTED]> Handle accurate time even if there's a long delay between accumulated clock cycles. Signed-off-by: John Stultz <[EMAIL PROTECTED]> Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/powerpc/kernel/time.c|3 +- arch/x

[PATCH 19/23 -v8] Trace irq disabled critical timings

2008-01-30 Thread Steven Rostedt
er-0 1dNs3 102us : _spin_unlock_irqrestore+0x32/0x5f (e1000_update_stats+0x641/0x64c [e1000]) swapper-0 1dNs3 102us : trace_hardirqs_on_caller+0x75/0x89 (_spin_unlock_irqrestore+0x32/0x5f) vim:ft=help === Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/kernel/process_64.

[PATCH 11/23 -v8] mcount based trace in the form of a header file library

2008-01-30 Thread Steven Rostedt
lock_functions+0x9/0x24 [ 81.479914] CPU 0: bash:3154 vfs_write+0x11d/0x155 <-- dnotify_parent+0x12/0x78 [ 81.479914] CPU 0: bash:3154 dnotify_parent+0x2d/0x78 <-- _spin_lock+0xe/0x70 [ 81.479914] CPU 0: bash:3154 _spin_lock+0x1b/0x70 <-- add_preempt_count+0xe/0x77 [ 81.479914]

[PATCH 20/23 -v8] trace preempt off critical timings

2008-01-30 Thread Steven Rostedt
l only record the time irqs are disabled, but "preemptirqsoff" will take the total time irqs or preemption are disabled. Runtime switching of these options is now supported by simpling echoing in the appropriate trace name into /debugfs/tracing/current_tracer. Signed-off-by: Steve

[PATCH] fix directory entry in arch-x86-Makefile

2008-01-31 Thread Steven Rostedt
Doing a make randconfig I came across this error in the Makefile. This patch makes a directory out of arch/x86/mach-default for CONFIG_X86_RDC321X Signed-off-by: Steven Rostedt <[EMAIL PROTECTED]> --- arch/x86/Makefile |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index:

Re: [PATCH 02/22 -v7] Add basic support for gcc profiler instrumentation

2008-02-01 Thread Steven Rostedt
On Fri, 1 Feb 2008, Paul E. McKenney wrote: > > > > OK, fair enough. I'll explain it a bit more. > > > > > > > > How's this: > > > > > > > > /* > > > > * We are entering ops into the mcount_list but another > > > > * CPU might be walking that list. We need to make sure > > > > * the ops->

[PATCH] add support for dynamic ticks and preempt rcu

2008-02-01 Thread Steven Rostedt
ff-by: Steven Rostedt <[EMAIL PROTECTED]> Signed-off-by: Paul E. McKenney <[EMAIL PROTECTED]> --- include/linux/hardirq.h| 10 ++ include/linux/rcuclassic.h |3 include/linux/rcupreempt.h | 22 kernel/rcupreempt.c| 224

Re: [PATCH 21/23 -v8] Add markers to various events

2008-02-01 Thread Steven Rostedt
5B On Thu, 31 Jan 2008, Mathieu Desnoyers wrote: > * Steven Rostedt ([EMAIL PROTECTED]) wrote: > > This patch adds markers to various events in the kernel. > > (interrupts, task activation and hrtimers) > > > > Hi Steven, > > I would propose the following

Re: [PATCH 06/23 -v8] handle accurate time keeping over long delays

2008-02-01 Thread Steven Rostedt
On Fri, 1 Feb 2008, Mathieu Desnoyers wrote: > > > > accumulated clock cycles. > > > > > > > > > > About this one.. we talked a lot about the importance of timekeeping at > > > the first Montreal Tracing Summit this week. Actually, someone > > > mentioned a very interesting point : in order to

Re: [PATCH] add support for dynamic ticks and preempt rcu

2008-02-02 Thread Steven Rostedt
On Sat, 2 Feb 2008, Ingo Molnar wrote: > > i've added it a few days ago and it's already in the sched-devel.git > tree: > > git://git.kernel.org/pub/scm/linux/kernel/git/mingo/linux-2.6-sched-devel.git OK, thanks! I know everyone is extremely busy now and I just wanted to make sure this didn't

Re: [PATCH 02/22 -v7] Add basic support for gcc profiler instrumentation

2008-02-04 Thread Steven Rostedt
Hi Paul, First I want to say, "Thank you", for taking the time to explain this in considerable detail. But I still have some minor questions. (Even though you already convinced me, but I still want full understanding ;-) On Sat, 2 Feb 2008, Paul E. McKenney wrote: > Yep, you have dependenc

Re: [PATCH 02/22 -v7] Add basic support for gcc profiler instrumentation

2008-02-04 Thread Steven Rostedt
On Mon, 4 Feb 2008, Paul E. McKenney wrote: > OK, will see what I can do... > > > On Sat, 2 Feb 2008, Paul E. McKenney wrote: > > > > > Yep, you have dependencies, so something like the following: > > > > > > initial state: > > > > > > struct foo { > > > int a; > > > }; > > > struc

[PATCH v2] lib tools traceevent: Add back pevent assignment in __pevent_parse_format()

2012-10-01 Thread Steven Rostedt
d) This happens because the scsi_dispatch_cmd_timeout event format has the following: scsi_trace_parse_cdb(p, __get_dynamic_array(cmnd), REC->cmd_len) which hasn't been defined by the pevent code. Reviewed-by: Namhyung Kim Signed-off-by: Steven Rostedt diff --git a/tools/lib/traceevent/event-p

[PATCH v2] tools lib traceevent: Fix missed freeing of subargs in free_arg() in filter

2012-10-01 Thread Steven Rostedt
Some of args were missed in free_args(), as well as subargs. That is args like FILTER_ARG_NUM have left and right pointers to other args that also need to be freed. Signed-off-by: Steven Rostedt --- tools/lib/traceevent/parse-filter.c | 15 +++ 1 files changed, 15 insertions(+), 0

Re: [PATCH v3 1/3] tracing,x86: add a TSC trace_clock

2012-10-01 Thread Steven Rostedt
On Mon, 2012-10-01 at 17:33 -0700, David Sharp wrote: > > Unfortunately this isn't enough. You need to add a stub in all > > arch/*/include/asm/ directories. That has: > > Oh really? Okay. Sorry, I really don't know the mechanics of the arch > trees. I'll work on that now, and then send you a new

Re: [PATCH -tip ] [BUGFIX] perf probe: Add a workaround for GCC -mfentry

2012-10-03 Thread Steven Rostedt
On Wed, 2012-10-03 at 21:17 +0900, Masami Hiramatsu wrote: > Here are examples: > - > <1><9a58>: Abbrev Number: 86 (DW_TAG_subprogram) > <9a59> DW_AT_external: 1 > <9a59> DW_AT_name: (indirect string, offset: 0xd82): > unregister_di > e_notifier > <9a5d> DW_AT_d

Re: [PATCH 7/10] compiler{,-gcc4}.h: Introduce __flatten function attribute

2012-10-03 Thread Steven Rostedt
On Wed, 2012-10-03 at 06:20 -0500, Daniel Santos wrote: > > Daniel, please introduce __flatten in the patch series that uses it, > > thanks. > That isn't going to work. I split my patches out into three sets > because, otherwise, the list of maintainers that must be CCed exceeds > the allowable s

Re: [PATCH 7/10] compiler{,-gcc4}.h: Introduce __flatten function attribute

2012-10-03 Thread Steven Rostedt
On Wed, 2012-10-03 at 09:46 -0500, Daniel Santos wrote: > > Please move the patch to the patch series where it is used. Otherwise it > > confuses reviewers as it did here. > Ok then, but this would also apply to the addition of these macros as well: > BUILD_BUG_ON_NON_CONST > BUILD_BUG42 > BUILD_B

Re: [PATCH 7/10] compiler{,-gcc4}.h: Introduce __flatten function attribute

2012-10-03 Thread Steven Rostedt
On Wed, 2012-10-03 at 08:38 -0700, Joe Perches wrote: > On Wed, 2012-10-03 at 11:14 -0400, Steven Rostedt wrote: > > I first check > > the MAINTAINERS file. If the subsystem I'm working on exists there, I > > only email those that are listed there, including any

[PATCH RT] softirq: Init softirq local lock after per cpu section is set up

2012-10-04 Thread Steven Rostedt
eas have not been set up yet. Thus initializing a local_irq_lock() before the per_cpu section is set up, may not work as it is initializing the per cpu locks before the per cpu exists. By moving the softirq_early_init() right after setup_per_cpu_areas(), the kernel boots fine. Signed-off-by: Steve

Re: [PATCH RT] softirq: Init softirq local lock after per cpu section is set up

2012-10-04 Thread Steven Rostedt
On Thu, 2012-10-04 at 11:02 -0400, Steven Rostedt wrote: > By moving the softirq_early_init() right after setup_per_cpu_areas(), > the kernel boots fine. > > Signed-off-by: Steven Rostedt > Bah, I forgot to Cc stable-rt. Cc: stable...@vger.kernel.org -- Steve -- To unsubs

Re: [PATCH RT] softirq: Init softirq local lock after per cpu section is set up

2012-10-04 Thread Steven Rostedt
On Thu, 2012-10-04 at 11:02 -0400, Steven Rostedt wrote: > void __init softirq_early_init(void) > { > local_irq_lock_init(local_softirq_lock); > } > > Where: > > #define local_irq_lock_init(lvar)

[STABLE][3.0][3.2][Add PATCH] phy/fixed: use an unique MDIO bus name

2012-10-05 Thread Steven Rostedt
I started testing my 3.2-rt branch on a powerpc box and it would constantly spit out the following warning causing ktest to think the boot failed: sysfs: cannot create duplicate filename '/class/mdio_bus/0 [ cut here ] WARNING: at /work/autotest/nobackup/powerpc-test.git/fs

Re: [PATCH v4] trace,x86: add x86 irq vector tracepoints

2012-10-05 Thread Steven Rostedt
On Tue, 2012-10-02 at 19:10 +, Seiji Aguchi wrote: > > > > > > If I misunderstand something, please let me know. > > > > > > > Quite. > > > > These functions are being invoked from the IDT, which is an indirect > > pointer structure. When not being traced, there is absolutely no > > reason

Re: [PATCHv2.5] trace: reuse kbasename() functionality

2012-10-05 Thread Steven Rostedt
On Wed, 2012-10-03 at 11:53 +0300, Andy Shevchenko wrote: > Signed-off-by: Andy Shevchenko > Cc: Steven Rostedt > Cc: Frederic Weisbecker > --- > kernel/trace/trace_uprobe.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/kerne

Re: [PATCH -tip 2/4] tracing: add a resize function for making one buffer equivalent to the other buffer

2012-10-05 Thread Steven Rostedt
(2) was also implemented in other place. So this patch assembles > them in a new function - resize_buffer_even(). > > Signed-off-by: Hiraku Toyooka > Cc: Steven Rostedt > Cc: Frederic Weisbecker > Cc: Ingo Molnar > Cc: linux-kernel@vger.kernel.org > --- > > kernel/

Re: [PATCHv2.5] trace: reuse kbasename() functionality

2012-10-05 Thread Steven Rostedt
On Fri, 2012-10-05 at 20:02 +0300, Andy Shevchenko wrote: > > I don't see kbasename() anywhere. Is this based off of other patches? > It's introduced by first patch in the series. > Usually a series has the format of: [PATCH x/n] Where x is the patch number and n is the total number of pa

Re: [PATCHv2.5] trace: reuse kbasename() functionality

2012-10-05 Thread Steven Rostedt
On Fri, 2012-10-05 at 13:12 -0400, Steven Rostedt wrote: > -- Seve Error: SIGFAULT! -- Steve ;-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majord

<    1   2   3   4   5   6   7   8   9   10   >