public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc/fw/unwind-link] m68k: Implement backtrace on top of <unwind-link.h>
@ 2020-02-10 17:09 Florian Weimer
0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2020-02-10 17:09 UTC (permalink / raw)
To: glibc-cvs
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=8a1dc6368693fcd05685d1985afe405ebfe1c8d0
commit 8a1dc6368693fcd05685d1985afe405ebfe1c8d0
Author: Florian Weimer <fweimer@redhat.com>
Date: Mon Feb 10 16:01:29 2020 +0100
m68k: Implement backtrace on top of <unwind-link.h>
Diff:
---
sysdeps/m68k/backtrace.c | 82 ++++++++++------------------------------------
sysdeps/m68k/unwind-link.h | 31 ++++++++++++++++++
2 files changed, 49 insertions(+), 64 deletions(-)
diff --git a/sysdeps/m68k/backtrace.c b/sysdeps/m68k/backtrace.c
index 6020aca..16f8327 100644
--- a/sysdeps/m68k/backtrace.c
+++ b/sysdeps/m68k/backtrace.c
@@ -16,52 +16,18 @@
License along with the GNU C Library; if not, see
<https://www.gnu.org/licenses/>. */
-#include <libc-lock.h>
-#include <dlfcn.h>
#include <execinfo.h>
#include <stdlib.h>
-#include <unwind.h>
+#include <unwind-link.h>
struct trace_arg
{
void **array;
+ struct unwind_link *unwind_link;
int cnt, size;
void *lastfp, *lastsp;
};
-#ifdef SHARED
-static _Unwind_Reason_Code (*unwind_backtrace) (_Unwind_Trace_Fn, void *);
-static _Unwind_Ptr (*unwind_getip) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getcfa) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getgr) (struct _Unwind_Context *, int);
-static void *libgcc_handle;
-
-static void
-init (void)
-{
- libgcc_handle = __libc_dlopen ("libgcc_s.so.2");
-
- if (libgcc_handle == NULL)
- return;
-
- unwind_backtrace = __libc_dlsym (libgcc_handle, "_Unwind_Backtrace");
- unwind_getip = __libc_dlsym (libgcc_handle, "_Unwind_GetIP");
- unwind_getcfa = __libc_dlsym (libgcc_handle, "_Unwind_GetCFA");
- unwind_getgr = __libc_dlsym (libgcc_handle, "_Unwind_GetGR");
- if (unwind_getip == NULL || unwind_getgr == NULL || unwind_getcfa == NULL)
- {
- unwind_backtrace = NULL;
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#else
-# define unwind_backtrace _Unwind_Backtrace
-# define unwind_getip _Unwind_GetIP
-# define unwind_getcfa _Unwind_GetCFA
-# define unwind_getgr _Unwind_GetGR
-#endif
-
static _Unwind_Reason_Code
backtrace_helper (struct _Unwind_Context *ctx, void *a)
{
@@ -70,13 +36,16 @@ backtrace_helper (struct _Unwind_Context *ctx, void *a)
/* We are first called with address in the __backtrace function.
Skip it. */
if (arg->cnt != -1)
- arg->array[arg->cnt] = (void *) unwind_getip (ctx);
+ arg->array[arg->cnt]
+ = (void *) UNWIND_LINK_CALL (arg->unwind_link, _Unwind_GetIP, ctx);
if (++arg->cnt == arg->size)
return _URC_END_OF_STACK;
/* %fp is DWARF2 register 14 on M68K. */
- arg->lastfp = (void *) unwind_getgr (ctx, 14);
- arg->lastsp = (void *) unwind_getcfa (ctx);
+ arg->lastfp
+ = (void *) UNWIND_LINK_CALL (arg->unwind_link, _Unwind_GetGR, ctx, 14);
+ arg->lastsp
+ = (void *) UNWIND_LINK_CALL (arg->unwind_link, _Unwind_GetCFA, ctx);
return _URC_NO_REASON;
}
@@ -110,20 +79,19 @@ struct layout
int
__backtrace (void **array, int size)
{
- struct trace_arg arg = { .array = array, .size = size, .cnt = -1 };
-
- if (size <= 0)
- return 0;
-
-#ifdef SHARED
- __libc_once_define (static, once);
+ struct trace_arg arg =
+ {
+ .array = array,
+ .unwind_link = __libc_unwind_link_get (),
+ .size = size,
+ .cnt = -1,
+ };
- __libc_once (once, init);
- if (unwind_backtrace == NULL)
+ if (size <= 0 || arg.unwind_link == NULL)
return 0;
-#endif
- unwind_backtrace (backtrace_helper, &arg);
+ UNWIND_LINK_CALL (arg.unwind_link, _Unwind_Backtrace,
+ backtrace_helper, &arg);
if (arg.cnt > 1 && arg.array[arg.cnt - 1] == NULL)
--arg.cnt;
@@ -146,17 +114,3 @@ __backtrace (void **array, int size)
}
weak_alias (__backtrace, backtrace)
libc_hidden_def (__backtrace)
-
-
-#ifdef SHARED
-/* Free all resources if necessary. */
-libc_freeres_fn (free_mem)
-{
- unwind_backtrace = NULL;
- if (libgcc_handle != NULL)
- {
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#endif
diff --git a/sysdeps/m68k/unwind-link.h b/sysdeps/m68k/unwind-link.h
new file mode 100644
index 0000000..4e28221
--- /dev/null
+++ b/sysdeps/m68k/unwind-link.h
@@ -0,0 +1,31 @@
+/* Dynamic loading of the libgcc unwinder. m68k overrides.
+ Copyright (C) 2020 Free Software Foundation, Inc.
+ This file is part of the GNU C Library.
+
+ The GNU C Library is free software; you can redistribute it and/or
+ modify it under the terms of the GNU Lesser General Public
+ License as published by the Free Software Foundation; either
+ version 2.1 of the License, or (at your option) any later version.
+
+ The GNU C Library 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
+ Lesser General Public License for more details.
+
+ You should have received a copy of the GNU Lesser General Public
+ License along with the GNU C Library; if not, see
+ <https://www.gnu.org/licenses/>. */
+
+#ifndef _UNWIND_LINK_H
+
+#define UNWIND_LINK_EXTRA_FIELDS \
+ __typeof (_Unwind_GetGR) *ptr__Unwind_GetGR;
+#define UNWIND_LINK_EXTRA_INIT \
+ result->ptr__Unwind_GetGR \
+ = __libc_dlsym (result->libgcc_handle, "_Unwind_GetGR"); \
+ assert (result->ptr__Unwind_GetGR != NULL); \
+ PTR_MANGLE (result->ptr__Unwind_GetGR);
+
+#include <sysdeps/generic/unwind-link.h>
+
+#endif /* _UNWIND_LINK_H */
^ permalink raw reply [flat|nested] 3+ messages in thread
* [glibc/fw/unwind-link] m68k: Implement backtrace on top of <unwind-link.h>
@ 2020-02-13 16:22 Florian Weimer
0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2020-02-13 16:22 UTC (permalink / raw)
To: glibc-cvs
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=089a9d42a3f941afb2129906bf6b359346e354e4
commit 089a9d42a3f941afb2129906bf6b359346e354e4
Author: Florian Weimer <fweimer@redhat.com>
Date: Mon Feb 10 16:01:29 2020 +0100
m68k: Implement backtrace on top of <unwind-link.h>
Diff:
---
sysdeps/m68k/backtrace.c | 82 +++++++++++-------------------------------------
1 file changed, 18 insertions(+), 64 deletions(-)
diff --git a/sysdeps/m68k/backtrace.c b/sysdeps/m68k/backtrace.c
index 6020aca..c586a14 100644
--- a/sysdeps/m68k/backtrace.c
+++ b/sysdeps/m68k/backtrace.c
@@ -16,52 +16,18 @@
License along with the GNU C Library; if not, see
<https://www.gnu.org/licenses/>. */
-#include <libc-lock.h>
-#include <dlfcn.h>
#include <execinfo.h>
#include <stdlib.h>
-#include <unwind.h>
+#include <unwind-link.h>
struct trace_arg
{
void **array;
+ struct unwind_link *unwind_link;
int cnt, size;
void *lastfp, *lastsp;
};
-#ifdef SHARED
-static _Unwind_Reason_Code (*unwind_backtrace) (_Unwind_Trace_Fn, void *);
-static _Unwind_Ptr (*unwind_getip) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getcfa) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getgr) (struct _Unwind_Context *, int);
-static void *libgcc_handle;
-
-static void
-init (void)
-{
- libgcc_handle = __libc_dlopen ("libgcc_s.so.2");
-
- if (libgcc_handle == NULL)
- return;
-
- unwind_backtrace = __libc_dlsym (libgcc_handle, "_Unwind_Backtrace");
- unwind_getip = __libc_dlsym (libgcc_handle, "_Unwind_GetIP");
- unwind_getcfa = __libc_dlsym (libgcc_handle, "_Unwind_GetCFA");
- unwind_getgr = __libc_dlsym (libgcc_handle, "_Unwind_GetGR");
- if (unwind_getip == NULL || unwind_getgr == NULL || unwind_getcfa == NULL)
- {
- unwind_backtrace = NULL;
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#else
-# define unwind_backtrace _Unwind_Backtrace
-# define unwind_getip _Unwind_GetIP
-# define unwind_getcfa _Unwind_GetCFA
-# define unwind_getgr _Unwind_GetGR
-#endif
-
static _Unwind_Reason_Code
backtrace_helper (struct _Unwind_Context *ctx, void *a)
{
@@ -70,13 +36,16 @@ backtrace_helper (struct _Unwind_Context *ctx, void *a)
/* We are first called with address in the __backtrace function.
Skip it. */
if (arg->cnt != -1)
- arg->array[arg->cnt] = (void *) unwind_getip (ctx);
+ arg->array[arg->cnt]
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetIP) (ctx);
if (++arg->cnt == arg->size)
return _URC_END_OF_STACK;
/* %fp is DWARF2 register 14 on M68K. */
- arg->lastfp = (void *) unwind_getgr (ctx, 14);
- arg->lastsp = (void *) unwind_getcfa (ctx);
+ arg->lastfp
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetGR) (ctx, 14);
+ arg->lastsp
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetCFA) (ctx);
return _URC_NO_REASON;
}
@@ -110,20 +79,19 @@ struct layout
int
__backtrace (void **array, int size)
{
- struct trace_arg arg = { .array = array, .size = size, .cnt = -1 };
-
- if (size <= 0)
- return 0;
-
-#ifdef SHARED
- __libc_once_define (static, once);
+ struct trace_arg arg =
+ {
+ .array = array,
+ .unwind_link = __libc_unwind_link_get (),
+ .size = size,
+ .cnt = -1,
+ };
- __libc_once (once, init);
- if (unwind_backtrace == NULL)
+ if (size <= 0 || arg.unwind_link == NULL)
return 0;
-#endif
- unwind_backtrace (backtrace_helper, &arg);
+ UNWIND_LINK_PTR (arg.unwind_link, _Unwind_Backtrace)
+ (backtrace_helper, &arg);
if (arg.cnt > 1 && arg.array[arg.cnt - 1] == NULL)
--arg.cnt;
@@ -146,17 +114,3 @@ __backtrace (void **array, int size)
}
weak_alias (__backtrace, backtrace)
libc_hidden_def (__backtrace)
-
-
-#ifdef SHARED
-/* Free all resources if necessary. */
-libc_freeres_fn (free_mem)
-{
- unwind_backtrace = NULL;
- if (libgcc_handle != NULL)
- {
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#endif
^ permalink raw reply [flat|nested] 3+ messages in thread
* [glibc/fw/unwind-link] m68k: Implement backtrace on top of <unwind-link.h>
@ 2020-02-13 10:46 Florian Weimer
0 siblings, 0 replies; 3+ messages in thread
From: Florian Weimer @ 2020-02-13 10:46 UTC (permalink / raw)
To: glibc-cvs
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=80188847b5e4cb93bf70aa61fa15946c50c4b7aa
commit 80188847b5e4cb93bf70aa61fa15946c50c4b7aa
Author: Florian Weimer <fweimer@redhat.com>
Date: Mon Feb 10 16:01:29 2020 +0100
m68k: Implement backtrace on top of <unwind-link.h>
Diff:
---
sysdeps/m68k/backtrace.c | 82 +++++++++++-------------------------------------
1 file changed, 18 insertions(+), 64 deletions(-)
diff --git a/sysdeps/m68k/backtrace.c b/sysdeps/m68k/backtrace.c
index 6020aca..c586a14 100644
--- a/sysdeps/m68k/backtrace.c
+++ b/sysdeps/m68k/backtrace.c
@@ -16,52 +16,18 @@
License along with the GNU C Library; if not, see
<https://www.gnu.org/licenses/>. */
-#include <libc-lock.h>
-#include <dlfcn.h>
#include <execinfo.h>
#include <stdlib.h>
-#include <unwind.h>
+#include <unwind-link.h>
struct trace_arg
{
void **array;
+ struct unwind_link *unwind_link;
int cnt, size;
void *lastfp, *lastsp;
};
-#ifdef SHARED
-static _Unwind_Reason_Code (*unwind_backtrace) (_Unwind_Trace_Fn, void *);
-static _Unwind_Ptr (*unwind_getip) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getcfa) (struct _Unwind_Context *);
-static _Unwind_Ptr (*unwind_getgr) (struct _Unwind_Context *, int);
-static void *libgcc_handle;
-
-static void
-init (void)
-{
- libgcc_handle = __libc_dlopen ("libgcc_s.so.2");
-
- if (libgcc_handle == NULL)
- return;
-
- unwind_backtrace = __libc_dlsym (libgcc_handle, "_Unwind_Backtrace");
- unwind_getip = __libc_dlsym (libgcc_handle, "_Unwind_GetIP");
- unwind_getcfa = __libc_dlsym (libgcc_handle, "_Unwind_GetCFA");
- unwind_getgr = __libc_dlsym (libgcc_handle, "_Unwind_GetGR");
- if (unwind_getip == NULL || unwind_getgr == NULL || unwind_getcfa == NULL)
- {
- unwind_backtrace = NULL;
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#else
-# define unwind_backtrace _Unwind_Backtrace
-# define unwind_getip _Unwind_GetIP
-# define unwind_getcfa _Unwind_GetCFA
-# define unwind_getgr _Unwind_GetGR
-#endif
-
static _Unwind_Reason_Code
backtrace_helper (struct _Unwind_Context *ctx, void *a)
{
@@ -70,13 +36,16 @@ backtrace_helper (struct _Unwind_Context *ctx, void *a)
/* We are first called with address in the __backtrace function.
Skip it. */
if (arg->cnt != -1)
- arg->array[arg->cnt] = (void *) unwind_getip (ctx);
+ arg->array[arg->cnt]
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetIP) (ctx);
if (++arg->cnt == arg->size)
return _URC_END_OF_STACK;
/* %fp is DWARF2 register 14 on M68K. */
- arg->lastfp = (void *) unwind_getgr (ctx, 14);
- arg->lastsp = (void *) unwind_getcfa (ctx);
+ arg->lastfp
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetGR) (ctx, 14);
+ arg->lastsp
+ = (void *) UNWIND_LINK_PTR (arg->unwind_link, _Unwind_GetCFA) (ctx);
return _URC_NO_REASON;
}
@@ -110,20 +79,19 @@ struct layout
int
__backtrace (void **array, int size)
{
- struct trace_arg arg = { .array = array, .size = size, .cnt = -1 };
-
- if (size <= 0)
- return 0;
-
-#ifdef SHARED
- __libc_once_define (static, once);
+ struct trace_arg arg =
+ {
+ .array = array,
+ .unwind_link = __libc_unwind_link_get (),
+ .size = size,
+ .cnt = -1,
+ };
- __libc_once (once, init);
- if (unwind_backtrace == NULL)
+ if (size <= 0 || arg.unwind_link == NULL)
return 0;
-#endif
- unwind_backtrace (backtrace_helper, &arg);
+ UNWIND_LINK_PTR (arg.unwind_link, _Unwind_Backtrace)
+ (backtrace_helper, &arg);
if (arg.cnt > 1 && arg.array[arg.cnt - 1] == NULL)
--arg.cnt;
@@ -146,17 +114,3 @@ __backtrace (void **array, int size)
}
weak_alias (__backtrace, backtrace)
libc_hidden_def (__backtrace)
-
-
-#ifdef SHARED
-/* Free all resources if necessary. */
-libc_freeres_fn (free_mem)
-{
- unwind_backtrace = NULL;
- if (libgcc_handle != NULL)
- {
- __libc_dlclose (libgcc_handle);
- libgcc_handle = NULL;
- }
-}
-#endif
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-02-13 16:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-10 17:09 [glibc/fw/unwind-link] m68k: Implement backtrace on top of <unwind-link.h> Florian Weimer
2020-02-13 10:46 Florian Weimer
2020-02-13 16:22 Florian Weimer
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).