On 2015-12-20, at 3:21 PM, Helge Deller wrote: > On 20.12.2015 21:19, Helge Deller wrote: >> On 20.12.2015 21:14, John David Anglin wrote: >>> On 2015-12-18, at 6:30 PM, Helge Deller wrote: >>> >>>> + /* Get assembler opcode of code in delay branch */ >>>> + uaddr = (unsigned int *) (regs->gr[31] + 1); >>> >>> Shouldn't increment be 4? >> >> No. >> The first address which is being executed in userspace is regs->gr[31]-3. > > I could (should?) have used > uaddr = (unsigned int *) ((regs->gr[31] & ~3) + 4); Possibly that is better. The former assumes user space is at level 3. Dave -- John David Anglin dave.anglin@xxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html