[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Libunwind-devel] [PATCH 3/5] Rework the lookup of the ARM specific
From: |
Arun Sharma |
Subject: |
Re: [Libunwind-devel] [PATCH 3/5] Rework the lookup of the ARM specific unwind entry for local unwinding |
Date: |
Mon, 8 Aug 2011 09:08:11 -0700 |
On Mon, Aug 8, 2011 at 7:41 AM, Ken Werner <address@hidden> wrote:
>> I'm a bit concerned about the arch-specific stuff getting added to
>> callback_data. What are the minimal arch specific hooks you need to
>> get this to work without an #ifdef UNW_TARGET_ARM? Something along the
>> lines of:
>>
>> unw_dyn_info_t *di_tdep;
>
> I'm not sure I can follow. I agree that these ifdefs aren't very nice but
> what are the alternatives given the design of the libunwind API? The
> libunwind ARM backend could define its own tdep_find_proc_info but almost
> all of the code would be identical. It would walk the program headers using
> dl_iterate_phdr to collect the unw_dyn_info_t information and then call
> tdep_search_unwind_table to fill unw_proc_info_t.
>
Could we make just the callback from dl_iterate_phdr target dependent?
-Arun
- [Libunwind-devel] [PATCH 0/5] ARM: Initial support for remote unwinding using libunwind-ptrace, Ken Werner, 2011/08/03
- [Libunwind-devel] [PATCH 2/5] Change address type from void* to unw_word_t., Ken Werner, 2011/08/03
- [Libunwind-devel] [PATCH 4/5] Change libunwind-ptrace to also look for .debug_frame if nothing found in .eh_frame., Ken Werner, 2011/08/03
- [Libunwind-devel] [PATCH 1/5] Move arm_exidx_entry and arm_exidx_table structs from ex_tables.h into libunwind-arm.h., Ken Werner, 2011/08/03
- [Libunwind-devel] [PATCH 5/5] ARM: Initial support for remote unwinding using libunwind-ptrace, Ken Werner, 2011/08/03
- [Libunwind-devel] [PATCH 3/5] Rework the lookup of the ARM specific unwind entry for local unwinding, Ken Werner, 2011/08/03
Re: [Libunwind-devel] [PATCH 0/5] ARM: Initial support for remote unwinding using libunwind-ptrace, Ken Werner, 2011/08/03