RE: [PATCH v3] Support glibc multiple namespace extension

["Metzger, Markus T" <markus.t.metzger@intel.com>]

Hello Kevin,

>They're not, but this code ...
>
>  /* Walk to the end of the list.  */
>  for (tail = info->solib_list; tail->next != NULL; tail = tail->next)
>    /* Nothing.  */;
>
>  /* Don't check shared libraries in other namespaces when updating
>     shared libraries in a new namespace.  */
>  if (debug_base == info->debug_base)
>    {
>      lm_info_svr4 *li = (lm_info_svr4 *) tail->lm_info;
>      prev_lm = li->lm_addr;
>    }
>  else
>    prev_lm = 0;
>
>... looks a bit odd to me.  We're adding to the same solib_list, yet we use
>different prev_lm.  Let me look into this some more.

This is indeed wrong and leads to

      if (li->l_prev != prev_lm)
	{
	  warning (_("Corrupted shared library list: %s != %s"),
		   paddress (target_gdbarch (), prev_lm),
		   paddress (target_gdbarch (), li->l_prev));
	  return 0;
	}

in svr4_read_so_list().  Assigning prev_lm to zero avoided the warning for
the first library in another namespace.  I added more test cases to detect
this reliably.

I'll try organizing SVr4 DSOs in per-namespace lists.

Regards,
Markus.
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928