On 11/26, Suzuki K. Poulose wrote:Yes, we do. Thanks for catching that. I will fix it.
@@ -121,8 +125,11 @@ int arch_uprobe_post_xol(struct arch_uprobe *auprobe, struct pt_regs *regs)
* to be executed.
*/
regs->nip = utask->vaddr + MAX_UINSN_BYTES;
+ regs->msr = utask->autask.saved_msr;
+#ifdef CONFIG_PPC_ADV_DEBUG_REGS
+ mtspr(SPRN_DBCR0, utask->autask.saved_dbcr);
+#endif
- user_disable_single_step(current);
Don't we need the same change in arch_uprobe_abort_xol() ?