On Thu, Mar 12, 2015 at 3:07 PM, Roland McGrath
<roland(a)hack.frob.com> wrote:
> GETREGS/SETREGS works fine everywhere. Just use it unconditionally.
This patch uses PTRACE_GETREGS/PTRACE_SETREGS to update rip.
On Thu, 2015-03-12 at 16:28 -0700, H.J. Lu wrote:
> On Thu, Mar 12, 2015 at 3:07 PM, Roland McGrath <roland(a)hack.frob.com> wrote:
> > GETREGS/SETREGS works fine everywhere. Just use it unconditionally.
>
> This patch uses PTRACE_GETREGS/PTRACE_SETREGS to update rip.
That looks fine.
Would you mind reading the CONTRIBUTING document and adding a
Signed-off-by line to your commit if you agree with that?
https://git.fedorahosted.org/cgit/elfutils.git/plain/CONTRIBUTING
On Fri, Mar 13, 2015 at 6:03 AM, Mark Wielaard <mjw(a)redhat.com>
wrote:
> On Thu, 2015-03-12 at 16:28 -0700, H.J. Lu wrote:
>> On Thu, Mar 12, 2015 at 3:07 PM, Roland McGrath <roland(a)hack.frob.com>
wrote:
>> > GETREGS/SETREGS works fine everywhere. Just use it unconditionally.
>>
>> This patch uses PTRACE_GETREGS/PTRACE_SETREGS to update rip.
>
> That looks fine.
>
> Would you mind reading the CONTRIBUTING document and adding a
> Signed-off-by line to your commit if you agree with that?
> https://git.fedorahosted.org/cgit/elfutils.git/plain/CONTRIBUTING
>
Here it is.