From: "Sangamesh Mallayya" <sangamesh.swamy@in.ibm.com>
To: "Ulrich Weigand" <uweigand@de.ibm.com>,
kevinb@redhat.com (Kevin Buettner)
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH] Adding support for reding signal handler frame in AIX
Date: Tue, 30 Oct 2018 07:16:00 -0000 [thread overview]
Message-ID: <OF16846FC6.4A0E2CCE-ON65258336.002671AD-65258336.0027F2EB@notes.na.collabserv.com> (raw)
In-Reply-To: <OF4C5ED06A.3C846B3C-ON65258315.002EABF7-65258315.002F01D3@LocalDomain>
[-- Attachment #1: Type: text/plain, Size: 3729 bytes --]
Hi Ulrich/Kevin,
I have modified the patch and implemented it using the trad frame cache.
Please review and let me know the comments.
Here is the dwarf test results summary.
Right now gdb isn't properly able to debug stabs binary which i will be
debugging.
Without patch
=============
# of expected passes 18912
# of unexpected failures 2289
# of expected failures 14
# of known failures 17
# of unresolved testcases 29
# of untested testcases 64
# of unsupported tests 37
With patch
==========
# of expected passes 18936
# of unexpected failures 2265
# of expected failures 14
# of known failures 17
# of unresolved testcases 27
# of untested testcases 65
# of unsupported tests 37
Thanks,
Sangamesh
From: Sangamesh Mallayya/India/IBM
To: "Ulrich Weigand" <uweigand@de.ibm.com>
Cc: gdb-patches@sourceware.org, kevinb@redhat.com (Kevin Buettner)
Date: 09/27/2018 02:03 PM
Subject: Re: [PATCH] Adding support for reding signal handler frame
in AIX
Thanks Ulrich and kevin for the review and comments.
Sorry for the late reply as i was off due to personal emergency.
I will have look at the suggestions and implement the new changes.
Thanks,
Sangamesh
From: "Ulrich Weigand" <uweigand@de.ibm.com>
To: Sangamesh Mallayya/India/IBM@IBMIN
Cc: kevinb@redhat.com (Kevin Buettner), gdb-patches@sourceware.org
Date: 09/12/2018 07:23 PM
Subject: Re: [PATCH] Adding support for reding signal handler frame
in AIX
Sangamesh Mallayya wrote:
> Yes. Thanks!
> Earlier code was calling validate function twice which wasn't required.
> We can remove that AIX ifdef and i have made the below changes. Rest
all=20
> are same.
> Let me know your view on this.
>
> # diff -u tramp-frame.c_orig tramp-frame.c
> --- tramp-frame.c_orig 2018-08-27 03:25:49 +0000
> +++ tramp-frame.c 2018-09-07 10:20:09 +0000
> @@ -86,11 +86,15 @@
> struct gdbarch *gdbarch =3D get_frame_arch (this_frame);
> enum bfd_endian byte_order =3D gdbarch_byte_order (gdbarch);
> int ti;
> + CORE_ADDR old_pc =3D pc;
> =20
> /* Check if we can use this trampoline. */
> if (tramp->validate && !tramp->validate (tramp, this_frame, &pc))
> return 0;
> -
> + if ((tramp->insn[0].bytes =3D=3D TRAMP_SENTINEL_INSN) &&
> + (tramp->insn[1].bytes =3D=3D AIX_TRAMP_SENTINEL_INSN) &&
> + (old_pc < 0x1000000))
> + return pc;
I agree with Kevin that code like this shouldn't be in common code.
It looks like the underlying problem is that tramp-frame isn't a
good match for what you're trying to do: tramp-frame tries to
detect trampolines by matching well-known *code sequences*.
However, you don't actually have any code sequence to match,
but want to identify trampolines solely by their PC. Since you
pass no code sequence to the tramp-frame matcher, it will actually
never match.
I'd suggest the best way forward is to not actually use tramp-frame
at all then, but just write your own matcher based directly on a
trad-frame cache.
An example to look at might be s390_stub_frame_unwind. Along those
lines, you can implement a sniffer that checks for special PC value
(and possibly a backchain zero check in addition), and then implement
this_id and prev_register routines based on a trad-frame register
cache (you should be able to use the aix_sigtramp_cache routine in
your patch as-is for that part).
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU/Linux compilers and toolchain
Ulrich.Weigand@de.ibm.com
[-- Attachment #2: aix-sighandle.patch --]
[-- Type: application/octet-stream, Size: 4800 bytes --]
--- ./gdb/rs6000-aix-tdep.c_orig 2018-10-27 06:23:41 +0000
+++ ./gdb/rs6000-aix-tdep.c 2018-10-29 12:29:26 +0000
@@ -38,6 +38,8 @@
#include "solib-aix.h"
#include "target-float.h"
#include "xml-utils.h"
+#include "trad-frame.h"
+#include "frame-unwind.h"
/* If the kernel has to deliver a signal, it pushes a sigcontext
structure on the stack and then calls the signal handler, passing
@@ -45,11 +47,121 @@
the signal handler doesn't save this register, so we have to
access the sigcontext structure via an offset from the signal handler
frame.
- The following constants were determined by experimentation on AIX 3.2. */
+ The following constants were determined by experimentation on AIX 3.2.
+
+ sigconext structure have the mstsave saved under the
+ sc_jmpbuf.jmp_context. STKMIN(minimum stack size) is 56 for 32-bit
+ processes, and iar offset under sc_jmpbuf.jmp_context is 40.
+ ie offsetof(struct sigcontext, sc_jmpbuf.jmp_context.iar).
+ so PC offset in this case is STKMIN+iar offset, which is 96 */
+
#define SIG_FRAME_PC_OFFSET 96
#define SIG_FRAME_LR_OFFSET 108
+/* STKMIN+grp1 offset, which is 56+228=284 */
#define SIG_FRAME_FP_OFFSET 284
+/* 64 bit process.
+ STKMIN64 is 112 and iar offset is 312. So 112+312=424 */
+#define SIG_FRAME_LR_OFFSET64 424
+/* STKMIN64+grp1 offset. 112+56=168 */
+#define SIG_FRAME_FP_OFFSET64 168
+
+static struct trad_frame_cache *
+aix_sighandle_frame_cache (struct frame_info *this_frame,
+ void **this_cache)
+{
+ LONGEST backchain;
+ CORE_ADDR base, base_orig, func;
+ struct gdbarch *gdbarch = get_frame_arch (this_frame);
+ struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch);
+ int wordsize = tdep->wordsize;
+ enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
+ struct trad_frame_cache *this_trad_cache;
+
+ if ((*this_cache) != NULL)
+ return (struct trad_frame_cache *) (*this_cache);
+
+ this_trad_cache = trad_frame_cache_zalloc (this_frame);
+ (*this_cache) = this_trad_cache;
+
+ base = get_frame_register_unsigned (this_frame,
+ gdbarch_sp_regnum (gdbarch));
+ base_orig = base;
+
+ if (wordsize == 4) {
+ func = read_memory_unsigned_integer (base_orig +
+ SIG_FRAME_PC_OFFSET + 8,
+ tdep->wordsize, byte_order);
+ safe_read_memory_integer (base_orig + SIG_FRAME_FP_OFFSET + 8,
+ wordsize, byte_order, &backchain);
+ base = (CORE_ADDR)backchain;
+ } else {
+ func = read_memory_unsigned_integer (base_orig +
+ SIG_FRAME_LR_OFFSET64,
+ tdep->wordsize, byte_order);
+ safe_read_memory_integer (base_orig + SIG_FRAME_FP_OFFSET64,
+ wordsize, byte_order, &backchain);
+ base = (CORE_ADDR)backchain;
+ }
+
+ trad_frame_set_reg_value (this_trad_cache, gdbarch_pc_regnum (gdbarch), func);
+ trad_frame_set_reg_value (this_trad_cache, gdbarch_sp_regnum (gdbarch), base);
+
+ if (wordsize == 4) {
+ trad_frame_set_reg_addr (this_trad_cache, tdep->ppc_lr_regnum,
+ base_orig + 0x38 + 52 + 8);
+ } else {
+ trad_frame_set_reg_addr (this_trad_cache, tdep->ppc_lr_regnum,
+ base_orig + 0x70 + 320);
+ }
+
+ trad_frame_set_id (this_trad_cache, frame_id_build (base, func));
+ trad_frame_set_this_base (this_trad_cache, base);
+
+ return this_trad_cache;
+}
+
+static void
+aix_sighandle_frame_this_id (struct frame_info *this_frame,
+ void **this_prologue_cache,
+ struct frame_id *this_id)
+{
+ struct trad_frame_cache *this_trad_cache
+ = aix_sighandle_frame_cache (this_frame, this_prologue_cache);
+ trad_frame_get_id (this_trad_cache, this_id);
+}
+
+static struct value *
+aix_sighandle_frame_prev_register (struct frame_info *this_frame,
+ void **this_prologue_cache, int regnum)
+{
+ struct trad_frame_cache *this_trad_cache
+ = aix_sighandle_frame_cache (this_frame, this_prologue_cache);
+ return trad_frame_get_register (this_trad_cache, this_frame, regnum);
+}
+
+int
+aix_sighandle_frame_sniffer (const struct frame_unwind *self,
+ struct frame_info *this_frame,
+ void **this_prologue_cache)
+{
+ CORE_ADDR pc = get_frame_pc (this_frame);
+ if (pc && pc < AIX_TEXT_SEGMENT_BASE)
+ return 1;
+
+ return 0;
+}
+
+/* AIX signal handler frame unwinder */
+
+static const struct frame_unwind aix_sighandle_frame_unwind = {
+ SIGTRAMP_FRAME,
+ default_frame_unwind_stop_reason,
+ aix_sighandle_frame_this_id,
+ aix_sighandle_frame_prev_register,
+ NULL,
+ aix_sighandle_frame_sniffer
+};
/* Core file support. */
@@ -1061,6 +1173,7 @@
set_gdbarch_auto_wide_charset (gdbarch, rs6000_aix_auto_wide_charset);
set_solib_ops (gdbarch, &solib_aix_so_ops);
+ frame_unwind_append_unwinder (gdbarch, &aix_sighandle_frame_unwind);
}
void
[-- Attachment #3: ChangeLog_gdb --]
[-- Type: application/octet-stream, Size: 317 bytes --]
2018-10-30 Sangamesh Mallayya <sangamesh.swamy@in.ibm.com>
* rs6000-aix-tdep.c: New signal handler frame_unwinder structure.
(aix_sighandle_frame_cache): New Function.
(aix_sighandle_frame_this_id): New Function.
(aix_sighandle_frame_prev_register): New Function.
(aix_sighandle_frame_sniffer): New Function.
[-- Attachment #4: ChangeLog_testsuites --]
[-- Type: application/octet-stream, Size: 151 bytes --]
2018-10-30 Sangamesh Mallayya <sangamesh.swamy@in.ibm.com>
* gdb.base/aix-sighandle_test.c: New file.
* gdb.base/aix-sighandle_test.exp: New file.
[-- Attachment #5: aix-sighandle_test.c --]
[-- Type: application/octet-stream, Size: 952 bytes --]
/* Copyright 2018 Free Software Foundation, Inc.
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 3 of the License, or
(at your option) any later version.
This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
#include <stdio.h>
#include <signal.h>
#include <string.h>
void sig_handle_aix (int signo)
{
printf ("signal is: %d\n", signo);
}
void foo()
{
char *ptr;
signal (SIGSEGV, sig_handle_aix);
strcpy (ptr, "signal");
}
int main()
{
foo ();
}
[-- Attachment #6: aix-sighandle_test.exp --]
[-- Type: application/octet-stream, Size: 1600 bytes --]
# Copyright 2018 Free Software Foundation, Inc.
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 3 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program. If not, see <http://www.gnu.org/licenses/>.
if {![istarget "powerpc*-*-aix*"]} {
return
}
if { [prepare_for_testing "failed to prepare" aix-sighandle_test aix-sighandle_test.c] } {
return -1
}
set srcfile aix-sighandle_test.c
set binfile aix-sighandle_test
gdb_test "break sig_handle_aix" \
"Breakpoint.1.at.*:.file.*$srcfile,.line.22." \
"breakpoint sig_handle_aix"
gdb_test "run" \
"Starting.program:.*$binfile.*\r\nProgram.received.signal.SIGSEGV,.*\r\n.*.in.foo.*.at.*$srcfile:29.*" \
"run to breakpoint sig_handle_aix"
gdb_test "continue" \
"Continuing.*Breakpoint.1,.sig_handle_aix..signo=11..at.*$srcfile:22.*" \
"continue to breakpoint sig_handle_aix"
gdb_test_sequence "backtrace" "backtrace for sighandle" {
"\[\r\n\]+#0 .* sig_handle_aix \\(signo=11\\) at "
"\[\r\n\]+#1 .* .signal.handler.called."
"\[\r\n\]+#2 .* foo \\(\\) at "
"\[\r\n\]+#3 .* main \\(\\) at "
}
next prev parent reply other threads:[~2018-10-30 7:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-29 5:54 Sangamesh Mallayya
2018-09-04 23:18 ` Kevin Buettner
2018-09-10 6:32 ` Sangamesh Mallayya
2018-09-10 17:34 ` Kevin Buettner
2018-09-12 13:53 ` Ulrich Weigand
2018-09-27 8:33 ` Sangamesh Mallayya
[not found] ` <OF4C5ED06A.3C846B3C-ON65258315.002EABF7-65258315.002F01D3@LocalDomain>
2018-10-30 7:16 ` Sangamesh Mallayya [this message]
2018-10-30 10:20 ` Ulrich Weigand
2018-10-31 10:18 ` Sangamesh Mallayya
2018-10-31 10:37 ` Ulrich Weigand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=OF16846FC6.4A0E2CCE-ON65258336.002671AD-65258336.0027F2EB@notes.na.collabserv.com \
--to=sangamesh.swamy@in.ibm.com \
--cc=gdb-patches@sourceware.org \
--cc=kevinb@redhat.com \
--cc=uweigand@de.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).