[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090829110236.GB6812@elte.hu>
Date: Sat, 29 Aug 2009 13:02:36 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Masami Hiramatsu <mhiramat@...hat.com>
Cc: Frederic Weisbecker <fweisbec@...il.com>,
LKML <linux-kernel@...r.kernel.org>,
Li Zefan <lizf@...fujitsu.com>,
Steven Rostedt <rostedt@...dmis.org>
Subject: Re: [GIT PULL v2] tracing/kprobes: v1 + two fixes
* Masami Hiramatsu <mhiramat@...hat.com> wrote:
> Ingo Molnar wrote:
>>
>> * Masami Hiramatsu<mhiramat@...hat.com> wrote:
>>
>>>
>>>
>>> Ingo Molnar wrote:
>>>>
>>>> * Frederic Weisbecker<fweisbec@...il.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
--
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/
Powered by blists - more mailing lists