gdb, solib-svr4: support namespaces in DSO iteration
authorMarkus Metzger <markus.t.metzger@intel.com>
Mon, 28 Mar 2022 09:20:10 +0000 (11:20 +0200)
committerMarkus Metzger <markus.t.metzger@intel.com>
Tue, 18 Oct 2022 12:16:11 +0000 (14:16 +0200)
commitaebb370bae3f511df8afb68a01a79c54e2066650
tree8e9e25276a2768eb2352ea514377f08c662c7c4f
parent1dc9084f5e95fd9a1f2f0e9baf9c6e52c5a5ee29
gdb, solib-svr4: support namespaces in DSO iteration

When looking up names, GDB needs to stay within one linker namespace to
find the correct instance in case the same name is provided in more than
one namespace.

Modify svr4_iterate_over_objfiles_in_search_order() to stay within the
namespace of the current_objfile argument.  If no current_objfile is
provided (i.e. it is nullptr), iterate over objfiles in the initial
namespace.

For objfiles that do not have a corresponding so_list to provide the
namespace, assume that the objfile was loaded into the initial namespace.
This would cover the main executable objfile (which is indeed loaded into
the initial namespace) as well as manually added symbol files.

Expected fails:

  - gdb.base/non-lazy-array-index.exp: the expression parser may lookup
    global symbols, which may result in xfers to read auxv for determining
    the debug base as part of svr4_iterate_over_objfiles_in_search_order().

  - gdb.server/non-lazy-array-index.exp: symbol lookup may access the
    target to read AUXV in order to determine the debug base for SVR4
    linker namespaces.

Known issues:

  - get_symbol_address() and get_msymbol_address() search objfiles for a
    'better' match.  This was introduced by

        4b610737f02 Handle copy relocations

    to handle copy relocations but it now causes a wrong address to be
    read after symbol lookup actually cound the correct symbol.  This can
    be seen, for example, with gdb.base/dlmopen.exp when compiled with
    clang.

  - gnu ifuncs are only looked up in the initial namespace.

  - lookup_minimal_symbol() and lookup_minimal_symbol_text() directly
    iterate over objfiles and are not aware of linker namespaces.
gdb/solib-svr4.c
gdb/testsuite/gdb.base/dlmopen-lib-dep.c [new file with mode: 0644]
gdb/testsuite/gdb.base/dlmopen-lib.c
gdb/testsuite/gdb.base/dlmopen.exp
gdb/testsuite/gdb.base/non-lazy-array-index.exp
gdb/testsuite/gdb.server/bkpt-other-inferior.exp