Re: Re: [PATCH] kprobes: bugfix: set kprobes_all_disarmed earlier to enable re-optimization.

2015-02-18 Thread Ingo Molnar
* Masami Hiramatsu wrote: > Ingo, > Could you also pull this patch to your -tip tree? Yeah, as long as you use Signed-off-by instead of Acked-by, which is really how patches written by others should be routed. Please re-send any still pending patches that are not yet upstream. Thanks,

Re: Re: [PATCH] kprobes: bugfix: set kprobes_all_disarmed earlier to enable re-optimization.

2015-01-19 Thread Masami Hiramatsu
(2015/01/20 12:02), Wang Nan wrote: > On 2015/1/12 19:24, Masami Hiramatsu wrote: >> (2015/01/05 21:31), Wang Nan wrote: >>> In original code, the probed instruction doesn't get optimized after >>> >>> echo 0 > /sys/kernel/debug/kprobes/enabled >>> echo 1 > /sys/kernel/debug/kprobes/enabled >>> >>>

Re: [PATCH] kprobes: bugfix: set kprobes_all_disarmed earlier to enable re-optimization.

2015-01-19 Thread Wang Nan
On 2015/1/12 19:24, Masami Hiramatsu wrote: > (2015/01/05 21:31), Wang Nan wrote: >> In original code, the probed instruction doesn't get optimized after >> >> echo 0 > /sys/kernel/debug/kprobes/enabled >> echo 1 > /sys/kernel/debug/kprobes/enabled >> >> This is because original code checks kprobes

Re: [PATCH] kprobes: bugfix: set kprobes_all_disarmed earlier to enable re-optimization.

2015-01-12 Thread Masami Hiramatsu
(2015/01/05 21:31), Wang Nan wrote: > In original code, the probed instruction doesn't get optimized after > > echo 0 > /sys/kernel/debug/kprobes/enabled > echo 1 > /sys/kernel/debug/kprobes/enabled > > This is because original code checks kprobes_all_disarmed in > optimize_kprobe(), but this fla

[PATCH] kprobes: bugfix: set kprobes_all_disarmed earlier to enable re-optimization.

2015-01-05 Thread Wang Nan
In original code, the probed instruction doesn't get optimized after echo 0 > /sys/kernel/debug/kprobes/enabled echo 1 > /sys/kernel/debug/kprobes/enabled This is because original code checks kprobes_all_disarmed in optimize_kprobe(), but this flag is turned off after calling that function. There