[cairo-commit] 3 commits - configure.ac Makefile.am meson.build meson-cc-tests/bfd-section-flags.c util/backtrace-symbols.c util/cairo-trace util/Makefile.am util/meson.build util/README

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Tue Feb 23 13:19:01 UTC 2021


 Makefile.am                        |    1 
 configure.ac                       |    8 
 meson-cc-tests/bfd-section-flags.c |    9 
 meson.build                        |   11 -
 util/Makefile.am                   |    8 
 util/README                        |   23 --
 util/backtrace-symbols.c           |  377 -------------------------------------
 util/cairo-trace/lookup-symbol.c   |    7 
 util/meson.build                   |    2 
 9 files changed, 28 insertions(+), 418 deletions(-)

New commits:
commit d85738b39c2fe01ec09b83d2e1b88fa918939a01
Merge: d6c1787ac 0ed1053d8
Author: Uli Schlachter <psychon at znc.in>
Date:   Tue Feb 23 13:19:00 2021 +0000

    Merge branch 'utils-cairo-trace-update-for-new-bfd' into 'master'
    
    cairo-trace: fix build with newer bfd and remove backtrace-symbols.c
    
    Closes #391 and #460
    
    See merge request cairo/cairo!128

commit 0ed1053d8f9751d20b313491adc0117b8addd804
Author: Tim-Philipp Müller <tim at centricular.com>
Date:   Tue Feb 23 12:12:47 2021 +0000

    utils: remove hacky replacement for backtrace_symbols in glibc
    
    Doesn't build any more, is very much non-essential, and hasn't
    been touched in any meaningful way since it was added 13 years
    ago, so just remove it for now until someone steps up. Chances
    are the glibc version has improved since then.

diff --git a/util/Makefile.am b/util/Makefile.am
index 82d0a804c..3b25f6159 100644
--- a/util/Makefile.am
+++ b/util/Makefile.am
@@ -70,7 +70,7 @@ show_events_CFLAGS = $(gtk_CFLAGS)
 #show_events_LDADD = $(top_builddir)/src/libcairo.la $(gtk_LIBS)
 show_events_LDADD = $(gtk_LIBS)
 
-util: malloc-stats.so backtrace-symbols.so
+util: malloc-stats.so
 
 .la.so:
 	$(RM) $@
@@ -82,11 +82,7 @@ CLEANFILES += *.so
 # ie. with EXTRA_LTLIBRARIES
 AM_LDFLAGS = -module -avoid-version -export-dynamic -rpath /dev/null
 
-EXTRA_LTLIBRARIES += malloc-stats.la backtrace-symbols.la
-
-backtrace_symbols_la_LIBADD = -lbfd -liberty
-
-#malloc_stats_la_LIBADD  = $(backtrace_symbols_la_LIBADD) backtrace-symbols.lo
+EXTRA_LTLIBRARIES += malloc-stats.la
 
 if HAVE_GTK
 EXTRA_PROGRAMS += font-view
diff --git a/util/README b/util/README
index 39560a8c3..146180ded 100644
--- a/util/README
+++ b/util/README
@@ -4,24 +4,6 @@ Cairo Utilities
 There are a varieties of utilities we use with cairo.
 
 
-backtrace-symbols
------------------
-
-This is a small shared library designed to be preloaded by the
-linker and its purpose is to make the backtrace_symbols() function
-of glibc produce more useful source reference information.
-
-Build by:
-
-	make backtrace-symbols.so
-
-and use by:
-
-	LD_PRELOAD=$PWD/backtrace-symbols.so app-to-run
-
-This code should be contributed back to glibc at some point.
-
-
 malloc-stats
 ------------
 
@@ -37,11 +19,6 @@ and use by:
 
 	LD_PRELOAD=$PWD/malloc-stats.so app-to-run
 
-This works best when backtrace-symbols is in use.  That is:
-
-	LD_PRELOAD="$PWD/backtrace-symbols.so $PWD/malloc-stats.so" app-to-run
-
-
 cairo-trace
 -----------
 
diff --git a/util/backtrace-symbols.c b/util/backtrace-symbols.c
deleted file mode 100644
index 9931413de..000000000
--- a/util/backtrace-symbols.c
+++ /dev/null
@@ -1,377 +0,0 @@
-/*
-  A hacky replacement for backtrace_symbols in glibc
-
-  backtrace_symbols in glibc looks up symbols using dladdr which is limited in
-  the symbols that it sees. libbacktracesymbols opens the executable and shared
-  libraries using libbfd and will look up backtrace information using the symbol
-  table and the dwarf line information.
-
-  It may make more sense for this program to use libelf instead of libbfd.
-  However, I have not investigated that yet.
-
-  Derived from addr2line.c from GNU Binutils by Jeff Muizelaar
-
-  Copyright 2007 Jeff Muizelaar
-*/
-
-/* addr2line.c -- convert addresses to line number and function name
-   Copyright 1997, 1998, 1999, 2000, 2001, 2002 Free Software Foundation, Inc.
-   Contributed by Ulrich Lauther <Ulrich.Lauther at mchp.siemens.de>
-
-   This file was part of GNU Binutils.
-
-   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 2, 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, write to the Free Software
-   Foundation, 51 Franklin Street, Suite 500, Boston, MA 02110-1335, USA.  */
-
-#define fatal(a, b) exit(1)
-#define bfd_fatal(a) exit(1)
-#define bfd_nonfatal(a) exit(1)
-#define list_matching_formats(a) exit(1)
-
-/* 2 characters for each byte, plus 1 each for 0, x, and NULL */
-#define PTRSTR_LEN (sizeof(void *) * 2 + 3)
-#define true 1
-#define false 0
-
-#define _GNU_SOURCE
-#include <string.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <execinfo.h>
-#include <bfd.h>
-#include <libiberty.h>
-#include <dlfcn.h>
-#include <link.h>
-#if 0
-
-void (*dbfd_init)(void);
-bfd_vma (*dbfd_scan_vma)(const char *string, const char **end, int base);
-bfd* (*dbfd_openr)(const char *filename, const char *target);
-bfd_boolean (*dbfd_check_format)(bfd *abfd, bfd_format format);
-bfd_boolean (*dbfd_check_format_matches)(bfd *abfd, bfd_format format, char ***matching);
-bfd_boolean (*dbfd_close)(bfd *abfd);
-bfd_boolean (*dbfd_map_over_sections)(bfd *abfd, void (*func)(bfd *abfd, asection *sect, void *obj),
-		void *obj);
-#define bfd_init dbfd_init
-
-static void load_funcs(void)
-{
-	void * handle = dlopen("libbfd.so", RTLD_NOW);
-	dbfd_init = dlsym(handle, "bfd_init");
-	dbfd_scan_vma = dlsym(handle, "bfd_scan_vma");
-	dbfd_openr = dlsym(handle, "bfd_openr");
-	dbfd_check_format = dlsym(handle, "bfd_check_format");
-	dbfd_check_format_matches = dlsym(handle, "bfd_check_format_matches");
-	dbfd_close = dlsym(handle, "bfd_close");
-	dbfd_map_over_sections = dlsym(handle, "bfd_map_over_sections");
-}
-
-#endif
-
-
-static asymbol **syms;		/* Symbol table.  */
-
-/* 150 isn't special; it's just an arbitrary non-ASCII char value.  */
-#define OPTION_DEMANGLER	(150)
-
-static void slurp_symtab(bfd * abfd);
-static void find_address_in_section(bfd *abfd, asection *section, void *data);
-
-/* Read in the symbol table.  */
-
-static void slurp_symtab(bfd * abfd)
-{
-	long symcount;
-	unsigned int size;
-
-	if ((bfd_get_file_flags(abfd) & HAS_SYMS) == 0)
-		return;
-
-	symcount = bfd_read_minisymbols(abfd, false, (PTR) & syms, &size);
-	if (symcount == 0)
-		symcount = bfd_read_minisymbols(abfd, true /* dynamic */ ,
-						(PTR) & syms, &size);
-
-	if (symcount < 0)
-		bfd_fatal(bfd_get_filename(abfd));
-}
-
-/* These global variables are used to pass information between
-   translate_addresses and find_address_in_section.  */
-
-static bfd_vma pc;
-static const char *filename;
-static const char *functionname;
-static unsigned int line;
-static int found;
-
-/* Look for an address in a section.  This is called via
-   bfd_map_over_sections.  */
-
-static void find_address_in_section(bfd *abfd, asection *section, void *data __attribute__ ((__unused__)) )
-{
-	bfd_vma vma;
-	bfd_size_type size;
-
-	if (found)
-		return;
-
-	if ((bfd_get_section_flags(abfd, section) & SEC_ALLOC) == 0)
-		return;
-
-	vma = bfd_get_section_vma(abfd, section);
-	if (pc < vma)
-		return;
-
-	size = bfd_section_size(abfd, section);
-	if (pc >= vma + size)
-		return;
-
-	found = bfd_find_nearest_line(abfd, section, syms, pc - vma,
-				      &filename, &functionname, &line);
-}
-
-/* Read hexadecimal addresses from stdin, translate into
-   file_name:line_number and optionally function name.  */
-#if 0
-static void translate_addresses(bfd * abfd, char (*addr)[PTRSTR_LEN], int naddr)
-{
-	while (naddr) {
-		pc = bfd_scan_vma(addr[naddr-1], NULL, 16);
-
-		found = false;
-		bfd_map_over_sections(abfd, find_address_in_section,
-		(PTR) NULL);
-
-		if (!found) {
-			printf("[%s] \?\?() \?\?:0\n",addr[naddr-1]);
-		} else {
-			const char *name;
-
-			name = functionname;
-			if (name == NULL || *name == '\0')
-				name = "??";
-			if (filename != NULL) {
-				char *h;
-
-				h = strrchr(filename, '/');
-				if (h != NULL)
-					filename = h + 1;
-			}
-
-			printf("\t%s:%u\t", filename ? filename : "??",
-			       line);
-
-			printf("%s()\n", name);
-
-		}
-
-		/* fflush() is essential for using this command as a server
-		   child process that reads addresses from a pipe and responds
-		   with line number information, processing one address at a
-		   time.  */
-		fflush(stdout);
-		naddr--;
-	}
-}
-#endif
-
-static char** translate_addresses_buf(bfd * abfd, bfd_vma *addr, int naddr)
-{
-	int naddr_orig = naddr;
-	char b;
-	int total  = 0;
-	enum { Count, Print } state;
-	char *buf = &b;
-	int len = 0;
-	char **ret_buf = NULL;
-	/* iterate over the formatting twice.
-	 * the first time we count how much space we need
-	 * the second time we do the actual printing */
-	for (state=Count; state<=Print; state++) {
-	if (state == Print) {
-		ret_buf = malloc(total + sizeof(char*)*naddr);
-		buf = (char*)(ret_buf + naddr);
-		len = total;
-	}
-	while (naddr) {
-		if (state == Print)
-			ret_buf[naddr-1] = buf;
-		pc = addr[naddr-1];
-
-		found = false;
-		bfd_map_over_sections(abfd, find_address_in_section,
-		(PTR) NULL);
-
-		if (!found) {
-			total += snprintf(buf, len, "[0x%llx] \?\?() \?\?:0",(long long unsigned int) addr[naddr-1]) + 1;
-		} else {
-			const char *name;
-
-			name = functionname;
-			if (name == NULL || *name == '\0')
-				name = "??";
-			if (filename != NULL) {
-				char *h;
-
-				h = strrchr(filename, '/');
-				if (h != NULL)
-					filename = h + 1;
-			}
-			total += snprintf(buf, len, "%s:%u\t%s()", filename ? filename : "??",
-			       line, name) + 1;
-
-		}
-		if (state == Print) {
-			/* set buf just past the end of string */
-			buf = buf + total + 1;
-		}
-		naddr--;
-	}
-	naddr = naddr_orig;
-	}
-	return ret_buf;
-}
-/* Process a file.  */
-
-static char **process_file(const char *file_name, bfd_vma *addr, int naddr)
-{
-	bfd *abfd;
-	char **matching;
-	char **ret_buf;
-
-	abfd = bfd_openr(file_name, NULL);
-
-	if (abfd == NULL)
-		bfd_fatal(file_name);
-
-	if (bfd_check_format(abfd, bfd_archive))
-		fatal("%s: can not get addresses from archive", file_name);
-
-	if (!bfd_check_format_matches(abfd, bfd_object, &matching)) {
-		bfd_nonfatal(bfd_get_filename(abfd));
-		if (bfd_get_error() ==
-		    bfd_error_file_ambiguously_recognized) {
-			list_matching_formats(matching);
-			free(matching);
-		}
-		xexit(1);
-	}
-
-	slurp_symtab(abfd);
-
-	ret_buf = translate_addresses_buf(abfd, addr, naddr);
-
-	free (syms);
-	syms = NULL;
-
-	bfd_close(abfd);
-	return ret_buf;
-}
-
-#define MAX_DEPTH 16
-
-struct file_match {
-	const char *file;
-	void *address;
-	void *base;
-	void *hdr;
-};
-
-static int find_matching_file(struct dl_phdr_info *info,
-		size_t size, void *data)
-{
-	struct file_match *match = data;
-	/* This code is modeled from Gfind_proc_info-lsb.c:callback() from libunwind */
-	long n;
-	const ElfW(Phdr) *phdr;
-	ElfW(Addr) load_base = info->dlpi_addr;
-	phdr = info->dlpi_phdr;
-	for (n = info->dlpi_phnum; --n >= 0; phdr++) {
-		if (phdr->p_type == PT_LOAD) {
-			ElfW(Addr) vaddr = phdr->p_vaddr + load_base;
-			if (match->address >= vaddr && match->address < vaddr + phdr->p_memsz) {
-				/* we found a match */
-				match->file = info->dlpi_name;
-				match->base = info->dlpi_addr;
-			}
-		}
-	}
-	return 0;
-}
-
-char **backtrace_symbols(void *const *buffer, int size)
-{
-	int stack_depth = size - 1;
-	int x,y;
-	/* discard calling function */
-	int total = 0;
-
-	char ***locations;
-	char **final;
-	char *f_strings;
-
-	locations = malloc(sizeof(char**) * (stack_depth+1));
-
-	bfd_init();
-	for(x=stack_depth, y=0; x>=0; x--, y++){
-		struct file_match match = { .address = buffer[x] };
-		char **ret_buf;
-		bfd_vma addr;
-		dl_iterate_phdr(find_matching_file, &match);
-		addr = buffer[x] - match.base;
-		if (match.file && strlen(match.file))
-			ret_buf = process_file(match.file, &addr, 1);
-		else
-			ret_buf = process_file("/proc/self/exe", &addr, 1);
-		locations[x] = ret_buf;
-		total += strlen(ret_buf[0]) + 1;
-	}
-
-	/* allocate the array of char* we are going to return and extra space for
-	 * all of the strings */
-	final = malloc(total + (stack_depth + 1) * sizeof(char*));
-	/* get a pointer to the extra space */
-	f_strings = (char*)(final + stack_depth + 1);
-
-	/* fill in all of strings and pointers */
-	for(x=stack_depth; x>=0; x--){
-		strcpy(f_strings, locations[x][0]);
-		free(locations[x]);
-		final[x] = f_strings;
-		f_strings += strlen(f_strings) + 1;
-	}
-
-	free(locations);
-
-	return final;
-}
-
-void
-backtrace_symbols_fd(void *const *buffer, int size, int fd)
-{
-        int j;
-        char **strings;
-
-        strings = backtrace_symbols(buffer, size);
-        if (strings == NULL) {
-		perror("backtrace_symbols");
-		exit(EXIT_FAILURE);
-        }
-
-        for (j = 0; j < size; j++)
-		printf("%s\n", strings[j]);
-
-        free(strings);
-}
diff --git a/util/meson.build b/util/meson.build
index 016955cea..55362b6d4 100644
--- a/util/meson.build
+++ b/util/meson.build
@@ -61,5 +61,3 @@ endforeach
 if cc.has_header_symbol('malloc.h', '__malloc_hook') and cc.has_header('execinfo.h')
   libmallocstats = library('malloc-stats', 'malloc-stats.c')
 endif
-
-# backtrace-symbols does not build
commit e30259f6237571c61992433c110bc6e1ef900244
Author: Tim-Philipp Müller <tim at centricular.com>
Date:   Tue Feb 23 11:36:24 2021 +0000

    cairo-trace: fix build with newer versions of bfd
    
    And update configure/meson checks to check for the new function.
    
    Drop libiberty.h check since it's only needed by backtrace-symbols.c
    which we're about to remove.
    
    Closes #391, #460

diff --git a/Makefile.am b/Makefile.am
index 2caa7352f..1b7c59bf0 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -71,6 +71,7 @@ EXTRA_DIST += \
 	util/cairo-trace/meson.build \
 	util/meson.build \
 	meson-cc-tests/atomic-ops-cxx11.c \
+	meson-cc-tests/bfd-section-flags.c \
 	meson-cc-tests/ipc_rmid_deferred_release.c \
 	meson-cc-tests/ft_has_color.c \
 	meson-cc-tests/mkdir-variant-1.c \
diff --git a/configure.ac b/configure.ac
index 18e4a305f..6444a3da3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -826,7 +826,13 @@ CAIRO_ENABLE(interpreter, cairo-script-interpreter, yes, [
 AC_CHECK_LIB(bfd, bfd_openr,
 	 [AC_CHECK_HEADER(bfd.h, [have_bfd=yes],
 	 [have_bfd=no])], [have_bfd=no])
-AC_CHECK_HEADER(libiberty.h,, [have_bfd=no])
+dnl bfd_section_flags is an inline func so we don't bother with linking the lib in
+AC_LINK_IFELSE([AC_LANG_PROGRAM([
+  #include <bfd.h>
+  asection *s;
+],[
+  return bfd_section_flags(s) == 0;
+])],[],[have_bfd=no])
 if test "x$have_bfd" = "xyes"; then
     AC_DEFINE([HAVE_BFD], [1], [Define to 1 if you have the binutils development files installed])
     BFD_LIBS=-lbfd
diff --git a/meson-cc-tests/bfd-section-flags.c b/meson-cc-tests/bfd-section-flags.c
new file mode 100644
index 000000000..b58118d06
--- /dev/null
+++ b/meson-cc-tests/bfd-section-flags.c
@@ -0,0 +1,9 @@
+#include <bfd.h>
+
+int
+f (asection *s)
+{
+  return bfd_section_flags(s) == 0;
+}
+
+int main (void) { return 0; }
diff --git a/meson.build b/meson.build
index 5cbeda913..52945db52 100644
--- a/meson.build
+++ b/meson.build
@@ -673,15 +673,16 @@ if zlib_dep.found() and png_dep.found()
   }]
 endif
 
-# Untested, libiberty.h is in a libiberty subfolder for me
-# Also, the bfd API seems to have changed at some point
-bfd_dep = cc.find_library('bfd', has_headers: ['bfd.h', 'libiberty.h'], required: get_option('symbol-lookup'))
-if bfd_dep.found() and cc.has_function('bfd_openr', dependencies: [bfd_dep])
+bfd_dep = cc.find_library('bfd', has_headers: ['bfd.h'], required: get_option('symbol-lookup'))
+if bfd_dep.found() and \
+   cc.has_function('bfd_openr', dependencies: [bfd_dep]) and \
+   cc.links(files('meson-cc-tests/bfd-section-flags.c'), name: 'bfd_section_flags', dependencies: bfd_dep)
   conf.set('HAVE_BFD', 1)
   deps += [bfd_dep]
+elif get_option('symbol-lookup').enabled()
+  error('symbol lookup via bfd was enabled via options but is not available (bfd might be too old)')
 endif
 
-# Untested, see above
 if conf.get('HAVE_BFD', 0) == 1
   conf.set('CAIRO_HAS_SYMBOL_LOOKUP', 1)
 endif
diff --git a/util/cairo-trace/lookup-symbol.c b/util/cairo-trace/lookup-symbol.c
index f9665b36f..9af0b5944 100644
--- a/util/cairo-trace/lookup-symbol.c
+++ b/util/cairo-trace/lookup-symbol.c
@@ -65,7 +65,6 @@
 
 #if HAVE_BFD
 #include <bfd.h>
-#include <libiberty.h>
 
 struct symtab {
     bfd *bfd;
@@ -145,14 +144,14 @@ find_address_in_section (bfd *abfd,
     if (symbol->found)
 	return;
 
-    if ((bfd_get_section_flags (symtab->bfd, section) & SEC_ALLOC) == 0)
+    if ((bfd_section_flags (section) & SEC_ALLOC) == 0)
 	return;
 
-    vma = bfd_get_section_vma (symtab->bfd, section);
+    vma = bfd_section_vma (section);
     if (symbol->pc < vma)
 	return;
 
-    size = bfd_section_size (symtab->bfd, section);
+    size = bfd_section_size (section);
     if (symbol->pc >= vma + size)
 	return;
 


More information about the cairo-commit mailing list