This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: pending patches ping#2 [patchv3] Fix dwfl_report_elf BASE alignment
- From: Mark Wielaard <mjw at redhat dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Fri, 15 Mar 2013 14:37:53 +0100
- Subject: Re: pending patches ping#2 [patchv3] Fix dwfl_report_elf BASE alignment
On Thu, 2013-02-28 at 15:30 +0100, Jan Kratochvil wrote:
> [patchv3] Fix dwfl_report_elf BASE alignment
> https://lists.fedorahosted.org/pipermail/elfutils-devel/2012-November/002759.html
> Message-ID: <20121116161051.GA2042@host2.jankratochvil.net>
I think we agree the actual change is fine. Roland reviewed your glibc
comment change to l_addr in link.h. So I assume he didn't see anything
odd either in this change.
The only reason I was hesitant about the patch was the testcase. I
assume this mimics what dwfl_linux_proc_report () would do, and it was
easier to write it using dwfl_report_elf ()?
If you could add a comment where the constants for base (0x7f3560c92000)
and funcaddr (0x7f3560c92585) come from (and/or how to calculate your
own in case someone wants to extend the testcase) that would be
appreciated.
Thanks,
Mark