Re: [PATCH v3 0/8] gdb: introduce file_reader_t to read procfs files

[Matthieu Longo <matthieu.longo@arm.com>]

On 22/09/2026 14:50, Matthieu Longo wrote:
> This series introduces target_file_reader, a helper class for reading target files, and migrates existing /proc file handling to use it. It also refactors the /proc mapping parser to operate on bounded views rather than NUL-terminated strings.
> 
> The series is organized as follows:
> - Patch 1 adds gdb::ranges::replace, a C++17-compatible replacement for std::ranges::replace.
> - Patches 2 and 3 contain small cleanups identified during review: removing redundant struct prefixes and styling /proc filenames in warning messages.
> - Patch 4 adds extract_next_field, a helper for extracting separator-delimited views from a buffer.
> - Patch 5 refactors /proc mapping parsing to use gdb::array_view<char> and extract_view_from_buffer, removing the dependency on NUL-terminated strings and strtok_r.
> - Patch 6 introduces target_file_reader, which encapsulates the contents and path of a target file and provides convenient accessors and typed views.
> - Patches 7 and 8 migrate existing procfs parsing to target_file_reader and remove the now-obsolete parse_smaps_data interface.
> 
> == Review status ==
> Most of the patches have been already reviewed once by Andrew Burgess in v2, and others reviewers in v1.
> However, for clarity:
> - Patches 2 and 3 are new but based on comments from Andrew Burgess in v2.
> - Patch 5 contains new or substantially changed code and require more attention. This rework was triggered by the discovery of a bug in patch 4/6 in v2.
> 
> 
> == Changes since v2 ==
> 
> v2: https://inbox.sourceware.org/gdb-patches/20260825100912.514232-1-matthieu.longo@arm.com/
> - Renamed extract_view_from_buffer to extract_next_field, and add unit tests in selftests.
> - Refactored /proc mapping parsing to operate on gdb::array_view<char> instead of NUL-terminated strings. read_mapping now parses bounded ranges and diagnoses malformed mapping fields.
> - Split the cleanup removing redundant struct prefixes from smaps_data, as suggested by Christina Joos and Andrew Burgess.
> - Added filename styling to /proc warning messages, as suggested by Andrew Burgess.
> 
> 
> This series depends on: https://inbox.sourceware.org/gdb-patches/20260917201755.589524-1-simon.marchi@efficios.com/
> 
> Regards,
> Matthieu
> 
> 
> Matthieu Longo (8):
>   gdb support: add gdb::ranges::replace algorithm
>   gdb/linux: remove redundant struct prefixes from smaps_data
>   gdb/linux: style filenames in /proc warning messages
>   gdbsupport: add extract_next_field helper
>   gdb/linux: refactor /proc mapping parsing to use array_view
>   gdb: introduce helper class target_file_reader
>   gdb/linux-tdep: use target_file_reader for procfs parsing
>   gdb/linux-tdep: remove legacy parse_smaps_data overload
> 
>  gdb/Makefile.in                        |   1 +
>  gdb/amd64-linux-tdep.c                 |  12 +-
>  gdb/linux-tdep.c                       | 406 +++++++++++++------------
>  gdb/sparc64-tdep.c                     |  17 +-
>  gdb/target.h                           | 116 +++++++
>  gdb/unittests/common-utils-selftests.c |  32 ++
>  gdb/unittests/ranges-selftests.c       |  63 ++++
>  gdbsupport/common-utils.h              |  50 +++
>  gdbsupport/ranges.h                    |  39 +++
>  9 files changed, 530 insertions(+), 206 deletions(-)
>  create mode 100644 gdb/unittests/ranges-selftests.c
>  create mode 100644 gdbsupport/ranges.h
> 

Ping !

Regards,
Matthieu