This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [PATCH v2 1/3] libdw: Add dwarf_debugaltlink function
- From: Mark Wielaard <mjw at redhat dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Fri, 11 Apr 2014 11:36:18 +0200
- Subject: Re: [PATCH v2 1/3] libdw: Add dwarf_debugaltlink function
On Thu, 2014-04-10 at 20:41 +0200, Florian Weimer wrote:
> On 04/10/2014 08:29 PM, Mark Wielaard wrote:
> >> We could add them at the ELF layer, but then we'd have to iterate over
> >> all the ELF section headers in each of these functions. The section
> >> header array in libdw avoids this. I'm also not sure where to put such
> >> convenience functions.
> >
> > Yeah, I might be designing things. You are right having it in libdw and
^ over-
> > using the debug section array is the right thing to use. I am mostly
> > concerned about the naming because I think this functionality is at a
> > different level than the rest.
>
> So what about dwarf_elf_debugaltlink or even dwarf_elf_gnu_debugaltlink
> as a name? This shows the relationship to the underlying ELF data.
I like it. Maybe we could shorten it to dwelf_gnu_debugaltlink.
And then (if I am over-designing anyway) have a separate header file
libdwelf.h which is like libdwfl.h. But instead of having Dwarf Frontend
Library calls with dwfl_..., it would have Dwarf Elf Library Function
calls with dwelf_... Like the libdwfl.h functions they would be part of
libdw.so, but the user would know these are for different, low-level,
Elf-based Dwarf functionality. It would be good to have an explicit
place to put these low-level helper functions, it has come up before and
neither libelf nor libebl is really the right place for them. libdw
would be and with a separate header it would be clear the are
"different"/low-level.
I'll see if I can setup the build-infrastructure to add the new
libdwelf.h header, get it installed properly and make any functions
added part of libdw.so.
Cheers,
Mark