Re: [PATCH v2] ARM: Add SystemTap probes to longjmp and setjmp.

[Will Newton <will.newton@linaro.org>]

On 6 February 2014 16:26, Carlos O'Donell <carlos@redhat.com> wrote:
> On 02/05/2014 04:56 AM, Will Newton wrote:
>> Now the ARM port implements pointer encryption for jmpbufs, gdb needs
>> a SystemTap probe point in longjmp to determine the target PC of
>> a call to longjmp. This patch implements the probe points in longjmp
>> and a similar probe point in setjmp.
>>
>> In order to have all the appropriate registers available to pass to the
>> probe this reorders the layout of jmpbuf, putting the sp and lr registers
>> at the start rather than the end.
>>
>> Tested on armv7, no new failures in the glibc testsuite and confirmed
>> that this fixes the gdb.base/longjmp.exp failures in the gdb testsuite.
>
> This looks good to me.
>
> If it's considered a bug, please check this in immediately and CC Allan
> to keep him in the loop for these last minute bug fixes.
>
> We are about to freeze so the 2.19 branch can be cut. If there is anything
> else like this please bring it to his attention immediately.

Sorry I should have been clearer, I am proposing this patch for 2.20.

It is a regression of functionality to add pointer encryption without
this patch as gdb now cannot deal with longjmp calls as well as it
could previously but I agree with Jospeh that it's not appropriate to
push a change of this nature so late in the freeze.

-- 
Will Newton
Toolchain Working Group, Linaro