Messages in this thread | | | Date | Tue, 04 Feb 2014 00:42:44 +0900 | From | Masami Hiramatsu <> | Subject | Re: [PATCH] kernel/kprobes.c: move cleanup_rp_inst() to where CONFIG_KRETPROBES enabled |
| |
(2014/02/03 20:48), Chen Gang wrote: > On 02/02/2014 10:40 AM, Masami Hiramatsu wrote: >> (2014/02/01 21:17), Chen Gang wrote: >>> When CONFIG_KRETPROBES disabled, cleanup_rp_inst() is useless too. It >>> is only called by unregister_kretprobes() which is in CONFIG_KRETPROBES >>> enabled area. >>> >>> The related warning (allmodconfig under avr32): >>> >>> kernel/kprobes.c:1181: warning: 'cleanup_rp_inst' defined but not used >> >> This patch itself looks good to me. >> And it seems that not only the cleanup_rp_inst, but also other >> kretprobe related functions should be moved (free_rp_inst,etc) >> > > OK, thanks, need/should I check them again and send patch v2 for them?
Yes, I'm happy to review it :)
Thank you!
> > > Thanks. >
-- Masami HIRAMATSU IT Management Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
| |