On Thu, 19 May 2016 13:19:42 +0530
Anju T <a...@linux.vnet.ibm.com> wrote:

> >> +void arch_unoptimize_kprobes(struct list_head *oplist,
> >> +                       struct list_head *done_list)
> >> +{
> >> +  struct optimized_kprobe *op;
> >> +  struct optimized_kprobe *tmp;
> >> +
> >> +  list_for_each_entry_safe(op, tmp, oplist, list) {
> >> +          arch_unoptimize_kprobe(op);
> >> +          list_move(&op->list, done_list);
> >> +  }
> >> +}
> >> +
> >> +int arch_within_optimized_kprobe(struct optimized_kprobe *op,
> >> +                           unsigned long addr)
> >> +{
> > Please make sure addr != op->kp.addr and addr is aligned.
> 
> The only case this check will succeed is if kp.addr is not a multiple of 4, 
> which is not a valid address at all on
> Power.So should we again check here for that?

Ah, right. OK, so we may not need that.

Thank you,
-- 
Masami Hiramatsu <mhira...@kernel.org>
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to