[obv] Remove tests/backtrace-dwarf.c unused code

2013-12-17 Thread Jan Kratochvil
Hi Mark, checked in as obvious, some leftover. Thanks, Jan --- Begin Message --- Signed-off-by: Jan Kratochvil --- tests/ChangeLog | 5 + tests/backtrace-dwarf.c | 40 2 files changed, 5 insertions(+), 40 deletions(-) diff --git a/tests/Ch

[patch] Use executable_for_core in dwfl_build_id_find_elf

2013-12-17 Thread Jan Kratochvil
Hi Mark, I believe if executable_for_core is set it means the Dwfl is really for a core file and the executable has been explicitly set by the user. Therefore it should override/precede any searches. I was experimenting also with replacing [exe]/[pie] by executable_for_core but I do not see any

Re: [PATCH] libdw: Try .gnu_debugaltlink paths relative to the debug file

2013-12-17 Thread Josh Stone
On 12/17/2013 01:59 AM, Mark Wielaard wrote: > On Mon, 2013-12-16 at 11:16 -0800, Josh Stone wrote: >> On 12/16/2013 06:17 AM, Mark Wielaard wrote: >>> On Sat, 2013-12-14 at 13:51 -0800, Josh Stone wrote: In Fedora, these paths are relative to the debug file itself, so that's a better thi

pkg-config

2013-12-17 Thread Masatake YAMATO
Do you have a plan to provide *.pc files for libraries of elfutils? With the files using the libraries from an application becomes easier. Masatake YAMATO

Re: [patch 4/4] unwinder: s390 and s390x

2013-12-17 Thread Jan Kratochvil
Hi Mark, jankratochvil/unwinds390 attached patch lacks the binary blobs. On Mon, 18 Nov 2013 23:35:25 +0100, Mark Wielaard wrote: > I understand what the new unwind hook is supposed to do. > It gives the backend a fallback for unwinding if DWARF unwinding fails. > Which seems a good thing to hav

[obv] Fix forgotten call of ebl_dwarf_to_regno

2013-12-17 Thread Jan Kratochvil
Hi Mark, it has been posted in [patch 4/4] unwinder: s390 and s390x with your comment OK. Shouldn't this have been part of the previous (ppc) patch? Checked in. Thanks, Jan --- Begin Message --- Signed-off-by: Jan Kratochvil --- libdwfl/ChangeLog | 4 libdwfl/frame_u

Re: [PATCH v2] stack: show binary and source file names where a function is defined

2013-12-17 Thread Masatake YAMATO
Works fine. The output is easier to be read. Tested-by: Masatake YAMATO > On Tue, 2013-12-17 at 12:03 +0900, Masatake YAMATO wrote: >> This patch adds the module and source file information to the >> each stack trace line. `-m' is for module file information >> and `-s' is for source file infor

[patch] Add NEWS items for the new functions in 0.158

2013-12-17 Thread Jan Kratochvil
Hi Mark, OK for check-in? Thanks, Jan --- Begin Message --- Signed-off-by: Jan Kratochvil --- ChangeLog | 9 + NEWS | 6 ++ 2 files changed, 15 insertions(+) diff --git a/ChangeLog b/ChangeLog index 334c769..8ffc1f7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +201

Re: [PATCH v2] stack: show binary and source file names where a function is defined

2013-12-17 Thread Mark Wielaard
On Tue, 2013-12-17 at 12:03 +0900, Masatake YAMATO wrote: > This patch adds the module and source file information to the > each stack trace line. `-m' is for module file information > and `-s' is for source file information. `-v' is for both and more. > > This is based on private discussion with

Re: [PATCH] libdw: Try .gnu_debugaltlink paths relative to the debug file

2013-12-17 Thread Mark Wielaard
On Mon, 2013-12-16 at 11:16 -0800, Josh Stone wrote: > On 12/16/2013 06:17 AM, Mark Wielaard wrote: > > On Sat, 2013-12-14 at 13:51 -0800, Josh Stone wrote: > >> In Fedora, these paths are relative to the debug file itself, so that's > >> a better thing to try first. We don't actually have the deb

Re: [PATCH] Introduce tids cache for dwfl_getthreads

2013-12-17 Thread Masatake YAMATO
I've tested the patch and It works fine. Using gimp as a target process, I've compared my patch and rewinddir. I found there is no big difference in execution time between two patches. I guess getdents used in readdir can read enough entries at once. Thanks. Tested-by: Masatake YAMATO Masatak

Re: [PATCH] Introduce tids cache for dwfl_getthreads

2013-12-17 Thread Mark Wielaard
On Mon, 2013-12-16 at 17:01 +0100, Jan Kratochvil wrote: > On Mon, 16 Dec 2013 16:24:26 +0100, Mark Wielaard wrote: > > --- a/libdwfl/linux-pid-attach.c > > +++ b/libdwfl/linux-pid-attach.c > > @@ -164,6 +164,9 @@ pid_next_thread (Dwfl *dwfl __attribute__ > > ((unused)), void * > > { > >struct