From: Tom de Vries <tdevries@suse.de>
To: gdb-patches@sourceware.org
Subject: [PATCH v4 00/11] [gdb/tui] Some section hole handling fixes
Date: Wed, 7 Oct 2026 11:40:46 +0200 [thread overview]
Message-ID: <20261007094057.1811886-1-tdevries@suse.de> (raw)
This series consists of 11 patches.
The first four deal with tui_find_disassembly_address:
- the first two do refactoring
- the third is an optimization
- the fourth fixes a buglet.
The fifth factors out update_section_map.
The sixth adds section_addrmap.
The seventh extends addrmap::find with low/high results.
The eight makes gdb_disassembly return the number of disassembled insns.
The ninth patch fixes PR34617, for both TUI and CLI.
The tenth patch prevents stopping disassembly at section holes for CLI
and TUI, fixing PR34399.
The eleventh patch skips over data sections. This is an RFC.
Changes in v2:
- add patch fixing tui_disassemble comment after review comment
- drop restrictive fix for PR tui/34399 after review comments
- add patches implementing an enabling fix for PR tui/34399.
Changes in v3:
- committed the first three patches of v2
- drop the find_pc_section refactoring approach of the 4th and 5th patches of
v2
- instead add three refactoring/infrastructure patches:
- factor out update_section_map
- add section_addrmap
- extend addrmap::find with low/high results
- rework the last two patches into these two patches:
- handle sections when disassembling
- don't stop disassembly at section hole
Changes in v4:
- add first four patches dealing with tui_find_disassembly_address
- factor out "makes gdb_disassembly return the number of disassembled
insns" patch out of another patch
- add RFC patch
- add test-case gdb.base/disasm-section-sync.exp in
"Handle sections when disassembling"
- replace disassemble_skip_sections with disassemble_section_p
- fix some style nits
- do less re-calculation of address map
Versions:
- v1 https://sourceware.org/pipermail/gdb-patches/2026-August/229584.html
- v2 https://sourceware.org/pipermail/gdb-patches/2026-September/230207.html
- v3 https://sourceware.org/pipermail/gdb-patches/2026-September/230515.html
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34399
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=34617
Tom de Vries (11):
[gdb/tui] Use early exit in tui_find_disassembly_address
[gdb/tui] Make low_prev loop-local in tui_find_disassembly_address
[gdb/tui] Disassemble less often in tui_find_disassembly_address
[gdb/tui] Fix buglet in tui_find_disassembly_address
[gdb] Factor out update_section_map
[gdb] Add section_addrmap
[gdb] Extend addrmap::find with low/high results
[gdb] Make gdb_disassembly return number of insns
[gdb] Handle sections when disassembling
[gdb] Don't stop disassembly at section hole
[RFC, gdb/tui] Don't scroll into non-code sections
gdb/addrmap.c | 78 +++-
gdb/addrmap.h | 23 +-
gdb/cli/cli-cmds.c | 11 +-
gdb/disasm.c | 125 ++++++-
gdb/disasm.h | 20 +-
gdb/objfiles.c | 62 +++-
gdb/objfiles.h | 6 +
gdb/testsuite/gdb.base/disasm-section-sync.S | 47 +++
.../gdb.base/disasm-section-sync.exp | 28 ++
gdb/tui/tui-disasm.c | 338 +++++++++++-------
gdb/tui/tui-disasm.h | 3 +-
gdb/tui/tui-source.c | 2 +-
gdb/tui/tui-source.h | 3 +-
gdb/tui/tui-winsource.c | 13 +-
gdb/tui/tui-winsource.h | 12 +-
gdb/tui/tui.c | 4 +-
gdb/tui/tui.h | 3 +-
17 files changed, 583 insertions(+), 195 deletions(-)
create mode 100644 gdb/testsuite/gdb.base/disasm-section-sync.S
create mode 100644 gdb/testsuite/gdb.base/disasm-section-sync.exp
base-commit: b8a700f22bfa79ed841ec350938d0b310a5048b9
--
2.51.0
next reply other threads:[~2026-10-07 9:42 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-07 9:40 Tom de Vries [this message]
2026-10-07 9:40 ` [PATCH v4 01/11] [gdb/tui] Use early exit in tui_find_disassembly_address Tom de Vries
2026-10-07 9:40 ` [PATCH v4 02/11] [gdb/tui] Make low_prev loop-local " Tom de Vries
2026-10-07 9:40 ` [PATCH v4 03/11] [gdb/tui] Disassemble less often " Tom de Vries
2026-10-07 9:40 ` [PATCH v4 04/11] [gdb/tui] Fix buglet " Tom de Vries
2026-10-07 9:40 ` [PATCH v4 05/11] [gdb] Factor out update_section_map Tom de Vries
2026-10-07 9:40 ` [PATCH v4 06/11] [gdb] Add section_addrmap Tom de Vries
2026-10-07 9:40 ` [PATCH v4 07/11] [gdb] Extend addrmap::find with low/high results Tom de Vries
2026-10-07 9:40 ` [PATCH v4 08/11] [gdb] Make gdb_disassembly return number of insns Tom de Vries
2026-10-07 9:40 ` [PATCH v4 09/11] [gdb] Handle sections when disassembling Tom de Vries
2026-10-07 9:40 ` [PATCH v4 10/11] [gdb] Don't stop disassembly at section hole Tom de Vries
2026-10-07 9:40 ` [PATCH v4 11/11] [RFC, gdb/tui] Don't scroll into non-code sections Tom de Vries
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=20261007094057.1811886-1-tdevries@suse.de \
--to=tdevries@suse.de \
--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