Messages in this thread | | | Date | Tue, 12 Mar 2013 20:07:32 +0900 | From | Masami Hiramatsu <> | Subject | Re: [PATCH -tip ] [BUGFIX] kprobes: Move hash_64() into .text.kprobe section |
| |
(2013/03/12 17:21), Ingo Molnar wrote: > > * Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote: > >> @@ -31,7 +32,7 @@ >> #error Wordsize not 32 or 64 >> #endif >> >> -static inline u64 hash_64(u64 val, unsigned int bits) >> +static __kprobes inline u64 hash_64(u64 val, unsigned int bits) >> { >> u64 hash = val; > > We should also, really, really fix the '__kprobes' misnomer and switch to > the '__noprobe' pattern or so. The naming does not make it obvious at all > that what we do here is to turn _off_ kprobing of select functions...
Agreed.
> The only complication is that __kprobes is now present in 600+ places, > which will create merge conflicts. If you remind me during the next merge > window I can generate the rename on the spot and send it to Linus without > anyone having to carry the patch for too long.
Let me confirm that I just move __kprobes definition into compiler.h this time, and ping you when the next merge window opens, is that correct?
Thank you,
-- Masami HIRAMATSU IT Management Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
| |