From: Kevin Buettner <kevinb@redhat.com>
To: gdb-patches@sourceware.org
Subject: Re: [PATCH 0/8] Non-contiguous address range support
Date: Wed, 11 Jul 2018 15:27:00 -0000 [thread overview]
Message-ID: <20180711082740.05e94750@pinnacle.lan> (raw)
In-Reply-To: <20180625233239.49dc52ea@pinnacle.lan>
Ping.
On Mon, 25 Jun 2018 23:32:39 -0700
Kevin Buettner <kevinb@redhat.com> wrote:
> This eight part patch sequence adds (further) support for
> non-contiguous address ranges to GDB.
>
> This sequence of patches was motivated by GCC bug 84550:
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84550
>
> There is a test case posted to that bug along with some analysis of
> the underlying problem.
>
> There is also a GDB bug for the same issue; it's 23021, but at the
> moment, there is little there aside from a link to the GCC bug
> mentioned above. But here's a link anyway:
>
> https://sourceware.org/bugzilla/show_bug.cgi?id=23021
>
> A quick synopsis of the problem is as follows...
>
> Recent versions of gcc can generate code in which a function is split
> into at least two non-contiguous address ranges. As I understand it,
> the idea here is to separate code which gcc does not expect to execute
> in normal operation from the rest of the code. Doing this may result
> in better cache locality for the normal case. The generated code for
> the example in GCC bug 84550 separated a call to abort() from the rest
> of the code comprising the function.
>
> In the course of my investigation, I identified at least four
> problems:
>
> 1) Stepping into a function from a function which occupies non-contiguous
> address ranges does not always work. It was not uncommon to see the
> program run to completion when attempting to do a step.
>
> 2) Setting a breakpoint on a function with non-contiguous address ranges
> causes a breakpoint to be placed on more than one location. When a
> breakpoint is set on the "cold" address range, this is almost certainly
> incorrect. The breakpoint should instead be set only on code near the
> entry point(s).
>
> 3) The disassemble command did not work correctly. E.g. here is what I
> found during my analysis of 84550:
>
> (gdb) x/i 'main.cold.0'
> 0x4010e0 <main()>: mov %rax,%rdi
> (gdb) x/i main
> 0x4011a0 <main>: push %r12
> (gdb) disassemble main
> Dump of assembler code for function main():
> 0x00000000004010e0 <+0>: mov %rax,%rdi
> ...
> [No addresses starting at 0x4011a0 are shown]
>
> 4) Display of addresses associated with the non-contiguous function are
> confusing. E.g. in the above example, note that GDB thinks that
> the address associated with main.cold.0 is <main()>, but that there's
> also a minsym called main which is displayed as <main>.
>
> There are probably several other problems which are related those
> identified above.
>
> I discovered that the stepping problem could be "fixed" by disabling
> the find_pc_partial_function cache. This cache keeps track of the
> most recent result (of calling find_pc_partial_function). If
> find_pc_partial_function is called with an address which falls within
> the cache range, then that's considered to be a cache hit and the most
> recent result is returned. Obviously, this won't work correctly for
> functions which occupy non-contiguous (disjoint) address ranges where
> other functions might be placed in the gap.
>
> So one of the problems that needed to be solved was to make the
> caching code work correctly. It is interesting to note that stepping
> _did_ work when the cache was disabled. This is/was due to GDB
> already having some (albeit incomplete) support for non-contiguous
> addresses in the form of blockvector address maps. Code responsible
> for mapping addresses to blocks (which form the lower levels of
> find_pc_partial_function) handle this case correctly.
>
> To solve the problem of incorrect disassembly, we need to be able
> to iterate over all of the ranges associated with a block.
>
> Finally, we need to distinguish between the entry pc and the lowest
> address in a block. I discovered that this lack of distinction was
> the cause of the remainder of the bugs including some which seemed to
> be introduced by fixing the problems noted above. Once this
> distinction is made, it will be straightforward to add full support for
> DW_AT_entry_pc. I considered adding this support as part of this
> patch series, but decided to wait until the community weighs in on my
> work thus far...
next prev parent reply other threads:[~2018-07-11 15:27 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-26 6:32 Kevin Buettner
2018-06-26 6:42 ` [PATCH 1/8] Add block range data structure for blocks with non-contiguous address ranges Kevin Buettner
2018-08-01 1:36 ` Simon Marchi
2018-08-01 23:57 ` Kevin Buettner
2018-08-01 1:40 ` Simon Marchi
2018-06-26 6:44 ` [PATCH 2/8] Record explicit block ranges from dwarf2read.c Kevin Buettner
2018-08-01 1:41 ` Simon Marchi
2018-06-26 6:47 ` [PATCH 3/8] Add support for non-contiguous blocks to find_pc_partial_function Kevin Buettner
2018-07-19 18:52 ` Kevin Buettner
2018-08-01 2:01 ` Simon Marchi
2018-08-01 23:40 ` Kevin Buettner
2018-06-26 6:49 ` [PATCH 4/8] Disassemble blocks with non-contiguous ranges Kevin Buettner
2018-08-01 2:08 ` Simon Marchi
2018-06-26 6:51 ` [PATCH 5/8] Use BLOCK_ENTRY_PC in place of most uses of BLOCK_START Kevin Buettner
2018-08-01 2:22 ` Simon Marchi
2018-08-02 0:07 ` Kevin Buettner
2018-06-26 6:53 ` [PATCH 6/8] Use BLOCK_ENTRY_PC to find function entry pc in infrun.c Kevin Buettner
2018-08-01 2:28 ` Simon Marchi
2018-06-26 6:55 ` [PATCH 7/8] Relocate block range start and end addresses Kevin Buettner
2018-08-01 2:30 ` Simon Marchi
2018-06-26 6:57 ` [PATCH 8/8] Test case for functions with non-contiguous ranges Kevin Buettner
2018-08-01 2:56 ` Simon Marchi
2018-07-11 15:27 ` Kevin Buettner [this message]
2018-07-11 15:32 ` [PATCH 0/8] Non-contiguous address range support Keith Seitz
2018-07-12 19:12 ` Simon Marchi
2018-07-17 2:00 ` Kevin Buettner
2018-07-19 15:55 ` Simon Marchi
2018-07-19 19:07 ` Kevin Buettner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180711082740.05e94750@pinnacle.lan \
--to=kevinb@redhat.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).