Re: Beyond inotify recursive watches

2013-04-05 Thread Ramkumar Ramachandra
Jan Kara wrote: > Hum, I have somewhat hard time to understand what do you mean by > 'magically optimized syscalls'. What should happen in VFS to speedup your > load? In retrospect, I think this is a terrible hack to begin with. Tuning the filesystem specifically for git repositories is inelega

[PATCH] perf tool: simplify a git command in PERF-VERSION-GEN

2013-10-02 Thread Ramkumar Ramachandra
Replace a complex git log invocation with a simpler git rev-parse invocation. Cc: Ingo Molnar Cc: Namhyung Kim Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/util/PERF-VERSION-GEN | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf

[PATCH] perf tool: add example to perf-trace documentation

2013-10-02 Thread Ramkumar Ramachandra
It's not clear that perf trace must be run as a privileged user, so include an example demonstrating its usage with sudo. While at it, update the synopsis to include options. Cc: Ingo Molnar Cc: Namhyung Kim Cc: Arnaldo Carvalho de Melo Cc: David Ahern Signed-off-by: Ramkumar Ramach

[QUERY] Why does perf-trace need me to be root?

2013-10-02 Thread Ramkumar Ramachandra
Hi, All the other perf tooling can be used as a normal user. At one point, I had to touch /proc/sys/kernel/perf_event_paranoid to get me numbers on a PMU event without a symbolic name, but that's about it. perf-trace seems to be the exception though: artagnon|perf-core=:~/src/linux/tools/perf$

[BUG] perf timechart segfault

2013-10-02 Thread Ramkumar Ramachandra
Hi, I tried using the `perf timechart` command after a `sudo perf timechart record -- git status`, and it segfaulted. Backtrace follows: Program received signal SIGSEGV, Segmentation fault. 0x in ?? () (gdb) bt #0 0x in ?? () #1 0x0047d728 in perf_session

Re: [BUG] perf timechart segfault

2013-10-02 Thread Ramkumar Ramachandra
Hi Jiri, Jiri Olsa wrote: > what perf version are you running? I'm running off the one in torvalds/linux.git. $ perf --version perf version 3.12.rc3.g34b22d5 The log of builtin-timechart.c shows that it was last touched by you three months ago: 5936678 (perf timechart: Remove event types framew

Re: [BUG] perf timechart segfault

2013-10-02 Thread Ramkumar Ramachandra
Jiri Olsa wrote: > looks like you're missing this one: > (git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux perf/core) Very messy; doesn't rebase on top of the latest torvalds/linux.git cleanly: I get a merge conflict at patch 9 of 74. Since my work is based on torvalds/linux.git, I was hop

[PATCH] MAINTAINERS: add acme's tree to perf subsystem

2013-10-02 Thread Ramkumar Ramachandra
Cc: Ingo Molnar Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 744a239..9ef82c7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -6403,6 +6403,7 @@ M:Paul

Re: [QUERY] Why does perf-trace need me to be root?

2013-10-02 Thread Ramkumar Ramachandra
David Ahern wrote: > If you trust your users make the debugfs mount point rx by group,world. Thanks David. I can preserve this configuration across reboots by putting an entry in fstab, right? How do I preserve the value of /proc/sys/kernel/perf_event_paranoid across reboots? -- To unsubscribe fro

Re: [QUERY] Why does perf-trace need me to be root?

2013-10-03 Thread Ramkumar Ramachandra
Ingo Molnar wrote: >echo 'kernel.perf_event_paranoid = -1' >> /etc/sysctl.conf Thanks Ingo. It's called /etc/sysctl.d/* now: https://www.archlinux.org/news/deprecation-of-etcsysctlconf/ > The current output: > > comet:~/tip> perf trace > Couldn't read the raw_syscalls tracepoints informat

[PATCH] perf tool: more user-friendly errors from trace

2013-10-03 Thread Ramkumar Ramachandra
t: Change the permissions of debugfs: /sys/kernel/debug The directory will be present if your kernel was compiled with debugfs support. Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/builtin-trace.c | 8 ++-- 1 file changed, 6 i

[PATCH v2] perf tool: add example to perf-trace documentation

2013-10-03 Thread Ramkumar Ramachandra
While at it, update the synopsis to include options. Cc: Ingo Molnar Cc: Namhyung Kim Cc: Arnaldo Carvalho de Melo Cc: David Ahern Signed-off-by: Ramkumar Ramachandra --- tools/perf/Documentation/perf-trace.txt | 29 - 1 file changed, 28 insertions(+), 1 deletion

[PATCH] perf tool: ignore output from timechart

2013-10-03 Thread Ramkumar Ramachandra
Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/perf/.gitignore b/tools/perf/.gitignore index 8f8fbc2..782d86e 100644 --- a/tools/perf/.gitignore +++ b/tools/perf/.gitignore

[PATCH] perf tool: report user-friendly error from timechart

2013-10-03 Thread Ramkumar Ramachandra
of debugfs: /sys/kernel/debug The directory will be present if your kernel was compiled with debugfs support. Cc: Ingo Molnar Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/builtin-timechart.c | 8 1 file changed, 8 insertions(+) d

[PATCH] perf tool: add example to perf-timechart documentation

2013-10-03 Thread Ramkumar Ramachandra
While at it, update the synopsis to show both forms. Cc: Ingo Molnar Cc: Namhyung Kim Cc: David Ahern Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/Documentation/perf-timechart.txt | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff

Re: [PATCH] perf tool: report user-friendly error from timechart

2013-10-03 Thread Ramkumar Ramachandra
Ingo Molnar wrote: >> + /* Perform a quick sanity check */ >> + if (!is_valid_tracepoint("power:cpu_frequency")) { >> + fprintf(stderr, "Error:\tNo permissions to read >> $debugfs/tracing/events/power/cpu_frequency\n"); >> + fprintf(stderr, "Hint:\tChange the permis

Re: [PATCH] perf tool: more user-friendly errors from trace

2013-10-03 Thread Ramkumar Ramachandra
Arnaldo Carvalho de Melo wrote: > [acme@zoo ~]$ mount | grep debugfs > [acme@zoo ~]$ > [acme@zoo ~]$ perf trace usleep 1 > Is debugfs mounted? Try 'sudo mount -t debugfs nodev /sys/kernel/debug' > [acme@zoo ~]$ sudo mkdir /d > [acme@zoo ~]$ sudo mount -t debugfs nodev /d > [acme@zoo ~]$ mount | gre

[PATCH v2] perf tool: report user-friendly errors from trace

2013-10-03 Thread Ramkumar Ramachandra
permissions to read $debugfs/tracing/events/raw_syscalls Hint: Change the permissions of the debugfs filesystem Cc: Ingo Molnar Cc: David Ahern Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- We don't want to spoonfeed the user with an exact command-line to execute

[PATCH] perf tool: don't open-code TEST_EVENTS macro

2013-10-06 Thread Ramkumar Ramachandra
The TEST_EVENTS macro is open-coded towards the end of test__parse_events(). While at it, undef it cleanly after its usage. Cc: Robert Richter Cc: Jiri Olsa Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/tests/parse-events.c | 6 +++--- 1

Re: [PATCH] perf tool: don't open-code TEST_EVENTS macro

2013-10-06 Thread Ramkumar Ramachandra
Ramkumar Ramachandra wrote: > The TEST_EVENTS macro is open-coded towards the end of > test__parse_events(). While at it, undef it cleanly after its usage. Kindly ignore this patch; I made a trivial mistake. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

[BUG] Junk output from perf test

2013-10-06 Thread Ramkumar Ramachandra
Hi, `perf test` doesn't give me a clean run; it seems to be out-of-date. I spent many hours on tracing "parse events tests" with gdb, but preprocessor macros and other complexity make my job very difficult. I haven't determined where the warnings in #5 are coming from exactly, but it seems to be t

[QUERY] Timestamps in dmesg

2013-10-07 Thread Ramkumar Ramachandra
Hi, Probably a very silly question, but why doesn't dmesg output include timestamps in place of the cryptic number prefix? Thanks. -- 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.k

Re: [PATCH v2 0/4] Minor perf build fixes

2013-07-09 Thread Ramkumar Ramachandra
Hi Namhyung, Namhyung Kim wrote: > On Fri, 5 Jul 2013 15:46:13 +0530, Ramkumar Ramachandra wrote: >> [3/4] introduces a util/perf-perl.h to include with #pragma >> statements, hence eliminating duplication. It then updates Context.xs >> and trace-event-perl.c to use t

[PATCH v3 3/5] perf: squelch warnings from perl.h to compile-pass

2013-07-09 Thread Ramkumar Ramachandra
squelch exactly those warnings, making perf compile-pass. Helped-by: Namhyung Kim Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/util/perl.h | 10 ++ 1 file changed, 10 insertions(+) create mode 100644 tools/perf/util/perl.h diff --git a/tools/perf/util

[PATCH v3 0/5] Minor perf build fixes

2013-07-09 Thread Ramkumar Ramachandra
hanks. Ramkumar Ramachandra (5): perf/Makefile: do not open-code shell-sq perf/Perf-Trace-Util: fix broken include in Context.xs perf: squelch warnings from perl.h to compile-pass perf/Perf-Trace-Util: regenerate Context.c perf/Perf-Trace-Util: add .gitignore tools/perf/Mak

[PATCH v3 1/5] perf/Makefile: do not open-code shell-sq

2013-07-09 Thread Ramkumar Ramachandra
s are filled with $(call) invocations now. So, use the shell-sq function introduced in ced465c4 (perf tools: Makefile: PYTHON{,_CONFIG} to bandage Python 3 incompatibility, 2011-04-02) in place of open-coding. Cc: Michael Witten Cc: Ingo Molnar Signed-off-by: Ramkumar Ramachandra --- tools

[PATCH v3 2/5] perf/Perf-Trace-Util: fix broken include in Context.xs

2013-07-09 Thread Ramkumar Ramachandra
765532c8 (perf script: Finish the rename from trace to script, 2010-12-23) made a mistake during find-and-replace replacing "../../../util/trace-event.h" with "../../../util/script-event.h", a non-existent file. Fix this include. Cc: Arnaldo Carvalho de Melo Signed-off-by:

[PATCH v3 5/5] perf/Perf-Trace-Util: add .gitignore

2013-07-09 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- tools/perf/scripts/perl/Perf-Trace-Util/.gitignore | 5 + 1 file changed, 5 insertions(+) create mode 100644 tools/perf/scripts/perl/Perf-Trace-Util/.gitignore diff --git a/tools/perf/scripts/perl/Perf-Trace-Util/.gitignore b/tools/perf/scripts/perl

[PATCH v3 4/5] perf/Perf-Trace-Util: regenerate Context.c

2013-07-09 Thread Ramkumar Ramachandra
>From Context.xs, using the procedure described in the README. Cc: Tom Zanussi Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/scripts/perl/Perf-Trace-Util/Context.c | 211 ++ 1 file changed, 171 insertions(+), 40 deletions(-) diff --gi

Re: [PATCH 2/2] perf: squelch warnings from perl.h to compile-pass

2013-07-09 Thread Ramkumar Ramachandra
Ramkumar Ramachandra wrote: >> This one: https://lkml.org/lkml/2013/6/24/66 ? > > Yeah, I think this is more sensible than my approach. Thanks to Namhyung, I think my v3 is better now: http://article.gmane.org/gmane.linux.kernel/1522137 -- To unsubscribe from this list: send the line

[PATCH] virtio tools: add .gitignore

2013-07-10 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- tools/virtio/.gitignore | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 tools/virtio/.gitignore diff --git a/tools/virtio/.gitignore b/tools/virtio/.gitignore new file mode 100644 index 000..1cfbb01 --- /dev/null +++ b/tools/virtio

[PATCH 0/2] virtio tools: build failure

2013-07-10 Thread Ramkumar Ramachandra
the build, and thought I should atleast submit a bug report. Maybe I'm missing something? Thanks. Ramkumar Ramachandra (2): virtio tools: strip bad include-path from Makefile virtio tools: add .gitignore tools/virtio/.gitignore | 3 +++ tools/virtio/Makefile | 2 +- 2 files

[PATCH 1/2] virtio tools: strip bad include-path from Makefile

2013-07-10 Thread Ramkumar Ramachandra
ned-off-by: Ramkumar Ramachandra --- tools/virtio/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/virtio/Makefile b/tools/virtio/Makefile index 3187c62..53623c6 100644 --- a/tools/virtio/Makefile +++ b/tools/virtio/Makefile @@ -3,7 +3,7 @@ test: virtio_test v

Re: [PATCH 1/2] virtio tools: strip bad include-path from Makefile

2013-07-10 Thread Ramkumar Ramachandra
Michael S. Tsirkin wrote: >> CFLAGS contains a mysterious "-I ../../usr/include", but no such path >> exists in the linux tree. The line was originally introduced in >> 4e53f78e (tools/virtio: virtio_test tool, 2010-11-29), but no such path >> existed in the tree even then. > > It exists if you do

Re: [PATCH 1/2] virtio tools: strip bad include-path from Makefile

2013-07-10 Thread Ramkumar Ramachandra
Michael S. Tsirkin wrote: > This is usespace code so it needs the cleaned-up version from > usr/include, not the internal kernel one. Right, thanks for clearing that up. However, it does seem to depend on some code in /include in the linux tree. From tools/virtio/linux/virtio_ring.h: #include

[PATCH 1/3] menuconfig: factor out reserved letters

2013-07-10 Thread Ramkumar Ramachandra
These letters that should not be used in menu selection. Cc: Michal Marek Signed-off-by: Ramkumar Ramachandra --- scripts/kconfig/lxdialog/menubox.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/kconfig/lxdialog/menubox.c b/scripts/kconfig/lxdialog

[PATCH 3/3] menuconfig: allow j/k to move down/up the menu

2013-07-10 Thread Ramkumar Ramachandra
Like in Vim. Cc: Michal Marek Signed-off-by: Ramkumar Ramachandra --- scripts/kconfig/lxdialog/menubox.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/kconfig/lxdialog/menubox.c b/scripts/kconfig/lxdialog/menubox.c index 698d7c3..151394a 100644 --- a

[PATCH 2/3] menuconfig: remove redundant info from RESERVED_LETTERS

2013-07-10 Thread Ramkumar Ramachandra
first_alpha() checks only against lowercase characters anyway. Cc: Michal Marek Signed-off-by: Ramkumar Ramachandra --- scripts/kconfig/lxdialog/menubox.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/kconfig/lxdialog/menubox.c b/scripts/kconfig/lxdialog

[PATCH v2 0/3] Modernize menuconfig a bit

2013-07-10 Thread Ramkumar Ramachandra
ds to [3/3]. Independent of part 3, I think parts 1 and 2 are obviously correct, and should be merged. Thanks. Ramkumar Ramachandra (3): menuconfig: factor out reserved letters menuconfig: remove redundant info from RESERVED_LETTERS menuconfig: allow j/k to move down/up the menu scrip

Re: [PATCH v3 3/5] perf: squelch warnings from perl.h to compile-pass

2013-07-10 Thread Ramkumar Ramachandra
Arnaldo Carvalho de Melo wrote: > Huh? In what distro? I see, one that has a newer perl (5.18.0) I use Arch. Yeah, all my packages are new :) -- 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 ht

[PATCH] x86/ia32: remove unused argument in macro

2013-07-10 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- Trivial. arch/x86/ia32/ia32entry.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/ia32/ia32entry.S b/arch/x86/ia32/ia32entry.S index 474dc1b..4299eb0 100644 --- a/arch/x86/ia32/ia32entry.S +++ b/arch/x86/ia32/ia32entry.S

[PATCH] scripts: remove unused function in sortextable.c

2013-07-10 Thread Ramkumar Ramachandra
Cc: David Daney Cc: Michal Marek Signed-off-by: Ramkumar Ramachandra --- Trivial. I wonder why make didn't tell us? scripts/sortextable.c | 8 1 file changed, 8 deletions(-) diff --git a/scripts/sortextable.c b/scripts/sortextable.c index f9ce116..7c2310c 100644 --- a/sc

Re: [PATCH 4/7] perf completion: modernize style

2013-09-18 Thread Ramkumar Ramachandra
Hi Arnaldo, Arnaldo Carvalho de Melo wrote: > Can you check how is this in the current perf/core branch so that we can > move forward while I process some other patches? perf/completion is now 3 months old, and my more recent patches have already been merged into Linus' tree. Why is perf/completi

Re: [PATCH v7 1/4] spinlock: A new lockref structure for lockless update of refcount

2013-09-08 Thread Ramkumar Ramachandra
Linus Torvalds wrote: > On Sun, Sep 8, 2013 at 5:03 PM, Al Viro wrote: >> >> Well... unlazy_walk() is always followed by terminate_walk() very shortly, >> but there's a minor problem - terminate_walk() uses "are we in RCU >> mode?" for two things: >> a) do we need to do path_put() here? >

Re: [PATCH] staging: Disable lustre file system for MIPS, SH, and XTENSA

2013-09-08 Thread Ramkumar Ramachandra
Greg Kroah-Hartman wrote: > On Sun, Sep 08, 2013 at 06:03:00PM -0700, Guenter Roeck wrote: >> mips allmodconfig fails with >> >> ERROR: "copy_from_user_page" [drivers/staging/lustre/lustre/libcfs/libcfs.ko] >> undefined! >> >> which is due to LUSTRE using copy_from_user_page which is not exported b

Re: soft lockup in sysvipc code.

2013-09-08 Thread Ramkumar Ramachandra
Dave Jones wrote: > Haven't seen this before. > Tree based on v3.11-3104-gf357a82 > > BUG: soft lockup - CPU#0 stuck for 22s! [trinity-child0:25479] A combination of hard and soft locks is the solution to the problem, ultimately. > Modules linked in: sctp snd_seq_dummy fuse dlci rfcomm tun bnep h

Re: [PATCH] staging: Disable lustre file system for MIPS, SH, and XTENSA

2013-09-08 Thread Ramkumar Ramachandra
Greg Kroah-Hartman wrote: > What do you mean by this? What "initial cost"? You should be able to > cross-compile almost all arches on your desktop machine today with the > compilers we have on kernel.org. If I can get them set up and working, > they can't be that hard for anyone else :) Won't y

Re: [PATCH] staging: Disable lustre file system for MIPS, SH, and XTENSA

2013-09-08 Thread Ramkumar Ramachandra
Greg Kroah-Hartman wrote: >> How will the hardware industry be driven otherwise? > > I have no idea what you are referring to here, please explain. For stability, hardware needs to be present. When I started out a couple of days ago, I blamed my monitor for the data corruption: it's one extra poin

Re: [PATCH 3/7] perf lock: plug some memleaks

2013-09-08 Thread Ramkumar Ramachandra
Davidlohr Bueso wrote: > diff --git a/tools/perf/builtin-lock.c b/tools/perf/builtin-lock.c > index d318862..7784347 100644 > --- a/tools/perf/builtin-lock.c > +++ b/tools/perf/builtin-lock.c > @@ -321,10 +321,12 @@ static struct lock_stat *lock_stat_findnew(void *addr, > const char *name) > >

Re: [PATCH] xen: remove deprecated IRQF_DISABLED

2013-09-08 Thread Ramkumar Ramachandra
Michael Opdenacker wrote: > diff --git a/drivers/xen/platform-pci.c b/drivers/xen/platform-pci.c > index 99db9e1..2f3528e 100644 > --- a/drivers/xen/platform-pci.c > +++ b/drivers/xen/platform-pci.c > @@ -84,7 +84,7 @@ static irqreturn_t do_hvm_evtchn_intr(int irq, void *dev_id) > static int xen_a

Re: [PATCH] staging: Disable lustre file system for MIPS, SH, and XTENSA

2013-09-08 Thread Ramkumar Ramachandra
Guenter Roeck wrote: >> For stability, hardware needs to be present. When I started out a >> couple of days ago, I blamed my monitor for the data corruption: it's >> one extra point of leverage. > > I have no idea whatsoever what you are talking about. Sorry, my bad. I haven't done hardware driver

Re: [PATCH v7 1/4] spinlock: A new lockref structure for lockless update of refcount

2013-09-08 Thread Ramkumar Ramachandra
Linus Torvalds wrote: > On Sun, Sep 8, 2013 at 5:03 PM, Al Viro wrote: >> >> There's one exception - basically, we decide to put duplicates of >> reference(s) we hold into (a bunch of) structures being created. If >> we decide that we'd failed and need to roll back, the structures >> need to be t

Re: [PATCH 2/5] xen/spinlock: We don't need the old structure anymore

2013-09-09 Thread Ramkumar Ramachandra
Konrad Rzeszutek Wilk wrote: > diff --git a/arch/x86/xen/spinlock.c b/arch/x86/xen/spinlock.c > index 0438b93..71db82c 100644 > --- a/arch/x86/xen/spinlock.c > +++ b/arch/x86/xen/spinlock.c > @@ -81,7 +81,6 @@ static inline void spin_time_accum_blocked(u64 start) > spinlock_stats.time_block

Re: [PATCH v3 2/2] Documentation: dt: hwmon: add OF document for LM90

2013-09-09 Thread Ramkumar Ramachandra
Wei Ni wrote: > diff --git a/Documentation/devicetree/bindings/hwmon/lm90.txt > b/Documentation/devicetree/bindings/hwmon/lm90.txt > new file mode 100644 > index 000..5570875 > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/lm90.txt While at it, please update and rename ads1015

Re: [PATCH 4/5] xen/spinlock: Don't setup xen spinlock IPI kicker if disabled.

2013-09-09 Thread Ramkumar Ramachandra
Konrad Rzeszutek Wilk wrote: > There is no need to setup this IPI kicker if we are never going > to use the paravirtualized ticketlock mechanism. Excellent patch; the important takeaway is that paravirtualization is still quite useful in the real world. -- To unsubscribe from this list: send the l

Segfault in running aarch64

2013-09-09 Thread Ramkumar Ramachandra
Pointers appreciated. % ./linux Locating the bottom of the address space ... 0x1000 Locating the top of the address space ... 0xd000 Core dump limits : soft - 0 hard - NONE Checking that ptrace can change system call numbers...OK Checking syscall emulation patch for ptrace...O

Re: [PATCH v4 1/1] dcache: Translating dentry into pathname without taking rename_lock

2013-09-09 Thread Ramkumar Ramachandra
Al Viro wrote: > _What_ "pathname translations"? Pathname resolution doesn't fall back to > seq_writelock() at all. Maybe it should then? -- 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:/

Re: [PATCH v4 1/1] dcache: Translating dentry into pathname without taking rename_lock

2013-09-09 Thread Ramkumar Ramachandra
Linus Torvalds wrote: > It doesn't need to. The RCU lookup looks at individual dentry sequence > numbers and doesn't care about the bigger rename sequence number at > all. Right; it's sequential. > The fallback (if you hit one of the very very rare races, or if you > hit a symlink) ends up doing

Re: PEBS bug on HSW: "Unexpected number of pebs records 10" (was: Re: [GIT PULL] perf changes for v3.12)

2013-09-10 Thread Ramkumar Ramachandra
Stephane Eranian wrote: > [ 2229.021966] Call Trace: > [ 2229.021967][] dump_stack+0x46/0x58 > [ 2229.021976] [] warn_slowpath_common+0x8c/0xc0 > [ 2229.021979] [] warn_slowpath_fmt+0x46/0x50 > [ 2229.021982] [] intel_pmu_drain_pebs_hsw+0xa8/0xc0 > [ 2229.021986] [] intel_pmu_handle_irq+0x2

Re: PEBS bug on HSW: "Unexpected number of pebs records 10" (was: Re: [GIT PULL] perf changes for v3.12)

2013-09-10 Thread Ramkumar Ramachandra
Stephane Eranian wrote: > a simple multithreaded program where > #threads >> #CPUs To put it another way, does Intel's HT work for CPU intensive and IO minimal tasks? I think HT assumes some amount of inefficient IO coupled with pure CPU usage. -- To unsubscribe from this list: send the line "unsu

[PATCH] sched: remove bogus parameter in structured comment

2013-10-10 Thread Ramkumar Ramachandra
The balance parameter was removed by 23f0d20 (sched: Factor out code to should_we_balance(), 2013-08-06). Cc: Joonsoo Kim Cc: Ingo Molnar Cc: Peter Zijlstra Signed-off-by: Ramkumar Ramachandra --- kernel/sched/fair.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/sched/fair.c b

[PATCH] tools/perf: remove unused MANIFEST

2013-09-26 Thread Ramkumar Ramachandra
Cc: Peter Zijlstra Cc: Arnaldo Carvalho de Melo Cc: Paul Mackerras Cc: Ingo Molnar Signed-off-by: Ramkumar Ramachandra --- This file was last touched 7 months ago. A lot seems to have changed since then. Time to get rid of it? tools/perf/MANIFEST | 32 1

Re: [PATCH 1/8] um: Create defconfigs for i386 and x86_64

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > This patch is based on: https://lkml.org/lkml/2013/7/4/396 > > Cc: Ramkumar Ramachandra > Signed-off-by: Richard Weinberger > --- > arch/um/configs/i386_defconfig | 954 > +++ > arch/um/config

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > This patch is based on: https://lkml.org/lkml/2013/7/4/396 This is the original patch I sent across in July. > diff --git a/arch/um/Makefile b/arch/um/Makefile > index 133f7de..5bc7892 100644 > --- a/arch/um/Makefile > +++ b/arch/um/Makefile > @@ -8,6 +8,8 @@ > > ARCH

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > So, what exactly is broken in upstream? > make defconfig works as it always did. Auto-detection of SUBARCH, which can be done with a simple call to uname -m (the 90% case). The second patch I submitted prevented spawning xterms unnecessarily, which we discussed was a go

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: >> Auto-detection of SUBARCH, which can be done with a simple call to >> uname -m (the 90% case). The second patch I submitted prevented >> spawning xterms unnecessarily, which we discussed was a good move. > > Covering only 90% of all cases is not enough. > We must not br

Re: [PATCH 1/8] um: Create defconfigs for i386 and x86_64

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: >> $ file linux >> linux: ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), >> dynamically linked (uses shared libs), for GNU/Linux 2.6.32, not >> stripped >> $ ./linux ubd0=busybox-rootfs >> [...] >> Kernel panic - not syncing: No init found. Try passing init= o

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > I told you already that "make defconfig ARCH=um SUBARCH=x86" will spuriously > create a x86_64 config on x86_64. > This breaks existing setups. I'll fix this and resubmit soon. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the bo

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Ramkumar Ramachandra wrote: > Richard Weinberger wrote: >> I told you already that "make defconfig ARCH=um SUBARCH=x86" will spuriously >> create a x86_64 config on x86_64. >> This breaks existing setups. > > I'll fix this and resubmit soon. Wait a mi

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Geert Uytterhoeven wrote: > Sorry for chiming in, but... what about cross compiling? > SUBARCH=x86 should give you a 32-bit ia32 kernel, right? User-Mode Linux only supports two host architectures (called $SUBARCH) at the moment: i386 and x86_64. If you leave out the $SUBARCH on either an i386 or

[PATCH] tools/perf: remove non-existent entries in MANIFEST

2013-09-26 Thread Ramkumar Ramachandra
le entries (non-existent paths) by running: $ while read l; do file $l; done Cc: Peter Zijlstra Cc: Arnaldo Carvalho de Melo Cc: Paul Mackerras Cc: Ingo Molnar Signed-off-by: Ramkumar Ramachandra --- Thanks to David for pointing out that the MANIFEST is used by scripts/package. This pa

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: >> Sorry for chiming in, but... what about cross compiling? >> SUBARCH=x86 should give you a 32-bit ia32 kernel, right? > > Correct. > Users expect from SUBARCH=x86 a i386 32bit UML kernel. This is an insane expectation. This is kernel convention (it has nothing to do wit

Re: [PATCH] tools/perf: remove non-existent entries in MANIFEST

2013-09-26 Thread Ramkumar Ramachandra
David Ahern wrote: > Did you create a tarfile after this change, unpack it somewhere out of three > and then verify perf builds? Yeah, it builds fine. In fact, the archive generated from the current upstream is broken. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in t

Re: [PATCH 2/8] um: Do not use SUBARCH

2013-09-26 Thread Ramkumar Ramachandra
Richard Weinberger wrote: > And, of course, this makes your patch valid. > Can you also please ensure that your new defconfigs are minimal? Yeah, it's close to a minimal configuration for the 3.10 kernel (latest at the time of patch submission). I was aiming to minimize the diff between the curren

Re: [PATCH] tools/perf: remove non-existent entries in MANIFEST

2013-09-26 Thread Ramkumar Ramachandra
Arnaldo Carvalho de Melo wrote: > Could you please describe how it is broken? > > [acme@zoo linux]$ git branch | grep '*' > * perf/urgent > [acme@zoo linux]$ git describe > v3.12-rc1-51-gc1bf214 Oh, I meant the torvalds/linux.git master. $ git describe v3.12-rc2-83-g4b97280 $ make perf-tarb

[PATCH 0/2] More plesant User-Mode Linux build

2013-09-26 Thread Ramkumar Ramachandra
with an x86_64 userland, as well as the i386 kernel with an i386 userland. The second patch is just a related "while we're there". Thanks. Ramkumar Ramachandra (2): arch/um: make it work with defconfig and x86_64 um/vdso: add .gitignore for a couple of targets ar

[PATCH 2/2] um/vdso: add .gitignore for a couple of targets

2013-09-26 Thread Ramkumar Ramachandra
Cc: Richard Weinberger Signed-off-by: Ramkumar Ramachandra --- arch/x86/um/vdso/.gitignore | 2 ++ 1 file changed, 2 insertions(+) create mode 100644 arch/x86/um/vdso/.gitignore diff --git a/arch/x86/um/vdso/.gitignore b/arch/x86/um/vdso/.gitignore new file mode 100644 index 000..9cac6d0

[PATCH 1/2] arch/um: make it work with defconfig and x86_64

2013-09-26 Thread Ramkumar Ramachandra
CH=um make and successfully build User-Mode Linux on an x86_64 box in default configuration. Cc: Richard Weinberger Cc: Jeff Dike Signed-off-by: Ramkumar Ramachandra --- arch/um/Kconfig.common | 5 - arch/um/Makefile | 11 + arch/um/configs/i386_defconfig | 7

[QUERY] Building and testing Aarch64

2013-09-27 Thread Ramkumar Ramachandra
Hi, I'm interested in cross-compiling for ARM v8, and fixing some early papercuts to begin with; my host machine is a normal x86_64 box. Although I managed to use Linaro's toolchain [1] to build an Aarch64 kernel, and found a suitable rootfs [2] to use it with, I am not able to find an ARM64 machi

Re: [QUERY] Building and testing Aarch64

2013-09-27 Thread Ramkumar Ramachandra
Hi Catalin, Catalin Marinas wrote: > ARM provides an emulator, ARMv8 Foundation Model (free as in free beer > but you need to register to be able to download, just like the > architecture documentation): > > http://www.arm.com/products/tools/models/fast-models/foundation-model.php I downloaded th

Re: [QUERY] Building and testing Aarch64

2013-09-28 Thread Ramkumar Ramachandra
Catalin Marinas wrote: > git://git.kernel.org/pub/scm/linux/kernel/git/cmarinas/boot-wrapper-aarch64.git Thanks, I made some progress. Now I'm stuck trying to specify the rootfs. I boot with the following arguments (full fork of your repository here [1]): console=ttyAMA0 mem=2048M devtmpfs.moun

Re: [QUERY] Building and testing Aarch64

2013-09-28 Thread Ramkumar Ramachandra
Andreas Schwab wrote: >> I'm not exactly sure what went wrong: I made sure to compile my kernel >> with virtio support. > > Make sure you have CONFIG_VIRTIO_MMIO enabled. Now, I get the following build error: LD init/built-in.o drivers/built-in.o: In function `vgacon_init': :(.text+0xbc4):

Re: [QUERY] Building and testing Aarch64

2013-09-28 Thread Ramkumar Ramachandra
Andreas Schwab wrote: > Ramkumar Ramachandra writes: >> Is there a bug in the Kconfig dependency tree? Sounds like VIRTIO_MMIO >> requires some vga (?) drivers to be compiled in as well. I hit this bug when I do `make menuconfig` directly (which configuration does it start out

Re: [QUERY] Building and testing Aarch64

2013-09-28 Thread Ramkumar Ramachandra
Andreas Schwab wrote: > You need all the other relevant virtio options built-in, too, like > VIRTIO_BLK. Thanks! VIRTIO_BLK was the missing feature. I'll submit patches shortly. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.ke

[PATCH 1/2] arm64: include EXT4 in defconfig

2013-09-28 Thread Ramkumar Ramachandra
Most readily available root filesystems are formatted as EXT4 these days. For example, see the raring rootfs that the Debian folk is preparing [1]. [1]: http://people.debian.org/~wookey/bootstrap/rootfs/ Cc: Catalin Marinas Cc: Will Deacon Cc: Andreas Schwab Signed-off-by: Ramkumar

[PATCH 0/2] Lower entry barrier for arm64 development

2013-09-28 Thread Ramkumar Ramachandra
icantly [1], and it works well for me. I intend to address more papercuts in future patches. Thanks. [1]: https://github.com/artagnon/boot-wrapper-aarch64 Ramkumar Ramachandra (2): arm64: include EXT4 in defconfig arm64: include VIRTIO_{MMIO,BLK} in defconfig arch/arm64/configs/defconfig |

[PATCH 2/2] arm64: include VIRTIO_{MMIO,BLK} in defconfig

2013-09-28 Thread Ramkumar Ramachandra
: Ramkumar Ramachandra --- arch/arm64/configs/defconfig | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 798bcbb..31c81e9 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -42,7 +42,7

[PATCH 0/2] Toying with the perf tool

2013-09-30 Thread Ramkumar Ramachandra
Hi, I was curiously poking around the perf tool this morning. The hacking session resulted in two minor patches: [1/2] is a minor non-functional cleanup of the Makefile. [2/2] is more important: it omits printing bogus data in an edge case. Thanks for reading. Ramkumar Ramachandra (2): perf

[PATCH 1/2] perf tool: simplify ARCH code in Makefile

2013-09-30 Thread Ramkumar Ramachandra
: Arnaldo Carvalho de Melo Cc: Namhyung Kim Signed-off-by: Ramkumar Ramachandra --- tools/perf/config/Makefile | 47 ++ 1 file changed, 22 insertions(+), 25 deletions(-) diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile index 5f6f9b3

[PATCH 2/2] perf tool: don't print bogus data on -e cycles

2013-09-30 Thread Ramkumar Ramachandra
a SW_TASK_CLOCK. In our case, since we have only requested HW_CPU_CYCLES, runtime_nsecs_stats is unavailable. So, omit printing the comment altogether. Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Paul Mackerras Cc: Arnaldo Carvalho de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/bui

[PATCH] perf tool: don't print bogus data on -e instructions

2013-10-01 Thread Ramkumar Ramachandra
de Melo Signed-off-by: Ramkumar Ramachandra --- tools/perf/builtin-stat.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c index cc167ae..0b747a5 100644 --- a/tools/perf/builtin-stat.c +++ b/tools/perf/builtin-st

Re: [PATCH 0/3] Trivial sparse acquire/release annotations

2013-09-23 Thread Ramkumar Ramachandra
Ramkumar Ramachandra wrote: > Ramkumar Ramachandra (3): > nfs: add lock annotations to squelch sparse warnings > apic: add lock annotations to squelch sparse warnings > irq: add lock annotation to squelch a sparse warning Ping. Can someone pick these up? -- To unsubscribe fr

[PATCH 3/7] perf completion: strip dependency on _filedir

2013-07-04 Thread Ramkumar Ramachandra
_filedir is defined in the bash-completion package, but there is no need to depend on it. Instead, call complete with multiple -o arguments before the -F argument like in git.git's completion script. Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_compl

[PATCH 6/7] perf completion: strip dependency on bash-completion

2013-07-04 Thread Ramkumar Ramachandra
The bash-completion package defines the _get_comp_words_by_ref function. There is no need to depend on it, as we can reimplement it like git.git has. Cc: Namhyung Kim Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 83

[PATCH 7/7] perf completion: use more comp words

2013-07-04 Thread Ramkumar Ramachandra
The completion words $words and $cword are available, so we might as well use them instead of directly accessing COMP_WORDS. Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 2/7] perf completion: update __ltrim_colon_completions

2013-07-04 Thread Ramkumar Ramachandra
The function is taken from the bash-completion package; update it to use the latest version where colon_word doesn't miss quoting. Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

[PATCH 4/7] perf completion: modernize style

2013-07-04 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/perf/bash_completion b/tools/perf/bash_completion index d2598be..8c04f5d 100644 --- a/tools/perf/bash_completion +++ b/tools/perf/bash_completion

[PATCH 5/7] perf completion: strip function_exists ()

2013-07-04 Thread Ramkumar Ramachandra
Use "type" to check existence consistently. Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/tools/perf/bash_completion b/tools/perf/bash_completion index 8c04f5

[PATCH 1/7] perf completion: don't dictate perf install location

2013-07-04 Thread Ramkumar Ramachandra
The statement have perf limits the locations in which to look for the perf program. Moreover, it depends on the bash-completion package to be installed. Replace it with a call to `type perf`. Cc: Frederic Weisbecker Signed-off-by: Ramkumar Ramachandra --- tools/perf/bash_completion | 2

[PATCH 0/7] Minor perf completion improvements

2013-07-04 Thread Ramkumar Ramachandra
dency on the bash-completion package. Thanks. Ramkumar Ramachandra (7): perf completion: don't dictate perf install location perf completion: update __ltrim_colon_completions perf completion: strip dependency on _filedir perf completion: modernize style perf completion: strip funct

  1   2   3   4   >