Re: Re: Re: [PATCH -tip ] [BUGFIX] kprobes: Move hash_64() into.text.kprobe section
From: Masami Hiramatsu
Date: Thu Mar 21 2013 - 09:23:34 EST
(2013/03/21 20:39), Ingo Molnar wrote:
> * Masami Hiramatsu <masami.hiramatsu.pt@xxxxxxxxxxx> wrote:
>> (2013/03/19 5:57), Timo Juhani Lindfors wrote:
>>> Masami Hiramatsu <masami.hiramatsu.pt@xxxxxxxxxxx> writes:
>>>> Thank you for reporting!!
>>> Thanks for fixing these! I spent some time trying to automate the
>>> process of finding sensitive functions and eventually resorted into
>>> booting a kvm instance with a minimal initrd to test every single
>>> function in a clean and reproducible environment.
>>> I found 7 more cases where calling register_kprobe() leads to an instant
>>> kernel panic:
>> Ah, right and Great! these native_* things are too fundamental one.
>> Hmm, curiously, those are defined as inline functions, and
>> I also couldn't find some of those symbols even with your previous
>>> You can see full kernel console output for each function at
>> As you can see, your panic messages, most of them caused GFP.
>> This may mean that int3 software exception must not happened
>> on those sites. Not the recursive call.
>> Perhaps, I'd better add those native_* things into symbol-name
>> based blacklist, instead of adding __kprobes, because those
>> are not related to kprobes recursion.
> Blacklists are not really good in general - it's easy for a symbol to be
> renamed and the blacklist misses them silently ...
> symbol name and annotation should go hand in hand.
Thus, I think we'd better moving __kprobes into compiler.h first.
Anyway, I'm still waiting for the actual kconfig from Timo,
because I couldn't reproduce the problem yet (no such symbols).
IT Management Research Dept. Linux Technology Center
Hitachi, Ltd., Yokohama Research Laboratory
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/