On Mon, 2007-11-19 at 12:45 +0100, Mark Wielaard wrote:
> I will clean up the libunwind interfaces up next to purge the stuff we
> aren't using anymore and out custom patches to some of the symbol lookup
> stuff in libunwind itself.

Here is the bulk. Now we really aren't doing any ProcName symbol lookups
through libunwind anymore. Hurray! This also gets rid of some of our
local libunwind patches that aren't needed anymore.

frysk-imports/libunwind/ChangeLog
2007-11-23  Mark Wielaard  <mwielaard@redhat.com>

    Revert:
    2006-09-16  Alexandre Oliva  <aoliva@redhat.com>

    * src/x86/Gget_proc_info.c (unw_get_proc_info): Use
    unw_get_proc_name to build a proper proc_info_t.
    * src/x86_64/Gget_proc_info.c (unw_get_proc_info): Likewise.
    * src/hppa/Gget_proc_info.c (unw_get_proc_info): Likewise.

    2006-09-27  Alexandre Oliva  <aoliva@redhat.com>

    * src/elfxx.c (lookup_symbol): Cope with NULL buf and zero buf_len.
    * src/mi/Gget_proc_name.c (intern_string, get_proc_name): Likewise.
    * src/hppa/Gget_proc_info.c (unw_get_proc_info): Use it.
    * src/x86/Gget_proc_info.c (unw_get_proc_info): Likewise.
    * src/x86_64/Gget_proc_info.c (unw_get_proc_info): Likewise.
    * doc/unw_get_proc_name.tex: Document NULL buf and zero buf_len.
    * doc/unw_create_addr_space.tex (get_proc_name): Likewise.
    * doc/unw_get_proc_name.man: Rebuilt.
    * doc/unw_create_addr_space.man: Likewise.

frysk-core/frysk/stack/ChangeLog
2007-11-23  Mark Wielaard  <mwielaard@redhat.com>

    * LibunwindAddressSpace.java (getModuleFromAddress): Removed.
    (getProcName): Removed.

frysk-sys/lib/unwind/ChangeLog
2007-11-23  Mark Wielaard  <mwielaard@redhat.com>

    * AddressSpace.java (getProcName): Removed.
    * Cursor.java (getProcName): Removed.
    * ProcName.java: Removed.
    * TestUnwind.java (getProcName): Removed.
    * Unwind.java (getProcName): Removed.
    * cnu/UnwindH.hxx (min): Removed.
    (get_proc_name): Return UNW_ENOMEM.
    (getProcName): Removed.

I extensively tested this on both x86 and x86_64 (Fedora 8) and no
regressions at all (I suspect there might actually be some unresolved
bugs that got fixed by this, but haven't checked yet). But since this is
actually a big change (as in a huge code path is now never taken) please
let me know if you see anything bad.

Cheers,

Mark