On Fri, Jun 04, 2010 at 12:21:45PM +0530, K.Prasad wrote: > Meanwhile I tested the per-cpu breakpoints with the new emulate_step > patch (refer linuxppc-dev message-id: > 20100602112903.gb30...@brick.ozlabs.ibm.com) and they continue to fail > due to emulate_step() failure, in my case, on a "lwz r0,0(r28)" > instruction.
Strange, what was in r28? The emulator should handle that instruction. > About the latest patchset, given that we chose to ignore extraneous > interrupts for non-ptrace breakpoints, I thought that re-using > current->thread.ptrace_bps as a flag would be efficient than introducing > a new member in 'struct thread_struct' to do the same. I'm not sure if > you foresee any issues with that. I just wonder what provides exclusion between its use as a flag and its use to hold a real ptrace breakpoint. As far as I can see nothing does. If there is something, it's off in some other source file, unless I'm missing something. And in that case there should be a bit fat comment explaining why it's safe. > If so, I'd like to send a new patch (rather than a new version of the > complete patchset) to fix it along with the dangling put_cpu() in > arch_unregister_hw_breakpoint() (I forgot to remove parts of the code > between versions XIX and XX). OK. Paul. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev