Messages in this thread | | | Date | Sat, 29 Aug 2009 13:02:36 +0200 | From | Ingo Molnar <> | Subject | Re: [GIT PULL v2] tracing/kprobes: v1 + two fixes |
| |
* Masami Hiramatsu <mhiramat@redhat.com> wrote:
> Ingo Molnar wrote: >> >> * Masami Hiramatsu<mhiramat@redhat.com> wrote: >> >>> >>> >>> Ingo Molnar wrote: >>>> >>>> * Frederic Weisbecker<fweisbec@gmail.com> wrote: >>>> >>>>> Ingo, >>>>> >>>>> This v2 can be either pulled after tracing/kprobes (the v1) or can >>>>> be pulled alone, it is based on the previous one and addresses >>>>> reviews from Li Zefan. >>>>> >>>>> Thanks, >>>>> Frederic. >>>>> >>>>> The following changes since commit 24851d2447830e6cba4c4b641cb73e713f312373: >>>>> Frederic Weisbecker (1): >>>>> tracing/kprobes: Dump the culprit kprobe in case of kprobe recursion >>>>> >>>>> are available in the git repository at: >>>>> >>>>> git://git.kernel.org/pub/scm/linux/kernel/git/frederic/random-tracing.git \ >>>>> tracing/kprobes-2 >>>>> >>>>> Frederic Weisbecker (2): >>>>> tracing: Restore the const qualifier for field names and types definition >>>>> tracing: Remove unneeded pointer casts >>>>> >>>>> include/linux/ftrace_event.h | 6 +++--- >>>>> kernel/trace/trace_events.c | 4 ++-- >>>>> kernel/trace/trace_syscalls.c | 8 ++++---- >>>>> 3 files changed, 9 insertions(+), 9 deletions(-) >>>> >>>> I've pulled it into a staging branch (not yet in tip:master either >>>> and not yet on the road to upstream either), thanks guys! >>>> >>>> I have done some brief testing. One hickup is this build failure on >>>> x86, caused by the kprobes instruction decoder self-test: >>>> >>>> CHK include/linux/compile.h >>>> TEST posttest >>>> Error: 41001024: a1 08 30 c9 41 mov >>>> 0x41c93008,%eax >>>> Error: objdump says 5 bytes, but insn_get_length() says 9 (attr:40000) >>> >>> Oops, it should be fixed. Hmm, it seems that addr_bytes or >>> opnd_bytes is broken, would you did a cross compiling? (compiling >>> x86-32 on x86-64) >> >> yeah, i did that. > > I see, so current selftest is not enough supporting the cross > compiling yet. I think I can fix that. Could you tell me how you > did that? make ARCH=i386 ?
Yeah, make ARCH=i386 and a CROSS= pointer to the cross-build toolchain.
Ingo
| |