Re: [PATCH v3 5/6] gdb/progspace: add solib_ops pointer in program_space

[Tom de Vries <tdevries@suse.de>]

On 9/2/25 18:05, Simon Marchi wrote:
> On 9/2/25 10:49 AM, Tom de Vries wrote:
>> On 6/16/25 21:33, Simon Marchi wrote:
>>> +     The solib provider must be unset prior to call this method.  */
>>> +  void set_solib_ops (const struct solib_ops &ops)
>>> +  {
>>> +    gdb_assert (m_solib_ops == nullptr);
>>> +    m_solib_ops = &ops;
>>> +  };
>>
>> This assert triggers with test-case gdb.server/extended-remote-restart.exp and target board native-extended-gdbserver, starting with the commit that introduced it.
>>
>> I've filed a PR about this ( https://sourceware.org/bugzilla/show_bug.cgi?id=33357 ).
>>
>> Thanks,
>> - Tom
> 
> This is fixed by this pending series, which I had forgotten about:
> 
> https://inbox.sourceware.org/gdb-patches/20250708205619.3582933-1-simon.marchi@polymtl.ca/
> 
> Can you can confirm that it fixes the issue on your side?

It does.

After applying the series and rebuilding, we're back to the gdbserver 
assertion failure mentioned in the PR:
...
		=== gdb Summary ===

# of unexpected core files	1
# of expected passes		28
...

Thanks,
- Tom