[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[jan.kratochvil at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Jan Kratochvil <jan.kratochvil at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |12575

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[drepper.fsp at gmail dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Bug 2328 depends on bug 12575, which changed state.

Bug 12575 Summary: dlopen RTLD_NOW does not apply to GNU-IFUNCs
http://sourceware.org/bugzilla/show_bug.cgi?id=12575

           What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WORKSFORME

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Gary Benson <gbenson at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gbenson at redhat dot com

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[jan.kratochvil at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Jan Kratochvil <jan.kratochvil at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |ASSIGNED

--- Comment #7 from Jan Kratochvil <jan.kratochvil at redhat dot com> 2011-08-11 11:38:42 UTC ---
(In reply to comment #6)
> The breakpoint could be added into the appropriate code in the dynamic linker
> (i.e., the place calling the constructor for the object.  That code is
> already relocated.

Such breakpoint is put there by the SystemTap probes, as implemented by Gary
Benson for glibc + gdb.

> It is wrong to change the semantics of RT_CONSISTENT.

This patch does not do so.

> And I think it is a bad approach trying to extend this strange interface

Neither this one.

> If we need more state information it is better to expose some APIs.  We have
> libthread_db, maybe we need librtld_db as well.

These SystemTap probes are such alternative interface but SystemTap probes
were never replied to in the mailing list:
        merging roland/systemtap branch
        http://sourceware.org/ml/libc-alpha/2011-03/msg00001.html

> If we'd have a specification of what information of the implementation gdb
> currently contains and what it still wants we can design such a library.

SystemTap probes work well for this purpose.

> I guess I am waiting for gdb people to provide this information.

This patch contains the needed features.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

--- Comment #8 from Gary Benson <gbenson at redhat dot com> 2011-08-11 12:11:47 UTC ---
Created attachment 5895
  --> http://sourceware.org/bugzilla/attachment.cgi?id=5895
Proposed patch for Fedora

To elaborate on Jan's comment above, this is the fix we are proposing for
Fedora.  It adds a new function, _dl_debug_notify, which is called wherever
_dl_debug_state was previously called, and in a couple of other places.  For
backwards compatibility, _dl_debug_notify calls _dl_debug_state whenever it was
previously called, so applications using _dl_debug_state will see no change.
The new interface is in the form of a number of SystemTap probes in
_dl_debug_notify which applications can locate and set breakpoints on if
necessary.  The new interface has the following advantages over the existing
one:

* Applications using the new interface have much more control over the events
they are notified about.  They can set breakpoints or whatever on the probes of
interest and ignore probes that are not of interest.

* Applications using the new interface are able to be notified about other
events than the ones supported by the existing interface.  Currently there are
two new events, for the start and end of object relocation, but the interface
is extensible in the sense that new probes can be added for other events of
interest.

* Applications using the new interface can handle libraries loaded into
namespaces other than the default, eg with dlmopen.  Under the existing
interface _dl_debug_state is called but the data structure for the namespace
(with the list of loaded libraries) is unavailable.  This allows, for example,
for a fix to bug 11839.

In addition to the addition of new probes, the new interface is also extensible
in the sense that new arguments may be added to existing probes without
breaking compatibility.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug libc/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

--- Comment #9 from Gary Benson <gbenson at redhat dot com> 2011-08-16 15:10:13 UTC ---
(In reply to comment #6)
> If we need more state information it is better to expose some APIs.  We have
> libthread_db, maybe we need librtld_db as well.  If we'd have a specification
> of what information of the implementation gdb currently contains and what it
> still wants we can design such a library.
> 
> I guess I am waiting for gdb people to provide this information.

What gdb needs is not so much more information per se as more (and more
finely-grained notifications).  I would say as a minimum a new interface should
allow debuggers to do everything that the existing interface does, so the new
interface needs to be able to notify the debugger at every point the existing
interface does.  Then, to be able to fix this particular issue (which
incidentally prevents gdb from being able to debug programs that dlopen
libpthread) we need a notification after relocation is complete -- basically in
the location John Reiser proposed moving the RT_CONSISTENT notification to.  In
the probes-based interface I proposed I also added a notification before
relocation occurs, for symmetry with all the other notifications which all have
before and after versions.

We also need the ability to be able to enable and disable the various
notifications.  In the probes-based interface this is done on the gdb side:
each notification has its own address, so debuggers can install breakpoints on
only the set of notifications they care about.  Another way to do this would be
to have one breakpoint address but calls into glibc to enable/disable the
various notifications individually.  I didn't go down that route as it would
mean debuggers altering the controlled program more than necessary, but it's
not unfeasable.

Also, to allow debugging of programs that use dlmopen the interface needs to
provide some way for debuggers to inspect namespaces other than the default. 
The existing interface only exposes the initial namespace so any others are
effectively invisible.

Finally, it is desirable to minimise the amount of data than needs to flow
between the inferior and the debugger, as this has performance issues when
debugging applications with large numbers of shared libraries, especially when
debugging remotely.  The probes-based interface as it stands does not do this
yet, but it could be extended to do so without breaking compatibility.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug ld.so|libdl/2328] _dl_debug_state() RT_CONSISTENT called too early

[jsm28 at gcc dot gnu.org]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Joseph Myers <jsm28 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|libc                        |ld.so|libdl

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Gary Benson <gbenson at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #5895|0                           |1
        is obsolete|                            |

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Gary Benson <gbenson at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Depends on|                            |14298

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Gary Benson <gbenson at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|drepper.fsp at gmail dot    |gbenson at redhat dot com
                   |com                         |

--- Comment #10 from Gary Benson <gbenson at redhat dot com> 2012-06-27 10:03:01 UTC ---
The tracker for the glibc side of this is bug 14298.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Bug 2328 depends on bug 14298, which changed state.

Bug 14298 Summary: Add static probes to runtime linker
http://sourceware.org/bugzilla/show_bug.cgi?id=14298

           What    |Old Value                   |New Value
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

--- Comment #11 from Gary Benson <gbenson at redhat dot com> ---
The GDB side of this interface was committed here:
http://cygwin.com/ml/gdb-patches/2013-06/msg00046.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[gbenson at redhat dot com]

http://sourceware.org/bugzilla/show_bug.cgi?id=2328

Gary Benson <gbenson at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #12 from Gary Benson <gbenson at redhat dot com> ---
Fixed with this glibc commit:
http://cygwin.com/ml/libc-alpha/2012-07/msg00557.html

and this GDB commit:
http://cygwin.com/ml/gdb-patches/2013-06/msg00046.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[jackie.rosen at hushmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=2328

Jackie Rosen <jackie.rosen at hushmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jackie.rosen at hushmail dot com

--- Comment #13 from Jackie Rosen <jackie.rosen at hushmail dot com> ---
*** Bug 260998 has been marked as a duplicate of this bug. ***
Seen from the domain http://volichat.com
Page where seen: http://volichat.com/adult-chat-rooms
Marked for reference. Resolved as fixed @bugzilla.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug dynamic-link/2328] _dl_debug_state() RT_CONSISTENT called too early

[schwab at sourceware dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=2328

Andreas Schwab <schwab at sourceware dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|jackie.rosen at hushmail dot com   |

-- 
You are receiving this mail because:
You are on the CC list for the bug.