Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] Don't use thread_db on corefiles
Date: Sun, 16 Dec 2001 17:58:00 -0000	[thread overview]
Message-ID: <20011216205642.A4358@nevyn.them.org> (raw)
In-Reply-To: <3C18FA17.6030603@cygnus.com>

On Thu, Dec 13, 2001 at 10:57:27AM -0800, Andrew Cagney wrote:
> >This patch fixes a really frustrating internal error when you open the
> >coredump of a multithreaded application.  Depending on your kernel, either
> >the core has threads (corefile.c supports this just fine) or it doesn't. 
> >Neither way will opening libthread_db work right.

corelow.c, rather.

> Daniel, can you explain the problem?  Thread-db should work on core 
> files (or realize it is silly and not open its self).

Right now, even after the event reporting issues, thread-db will not
work with core files.  The problem is that corefile.c adds the threads
to the thread list itself - and then thread_db finds them all and adds
them again.  One or the other needs to go.  If both try to add the
threads they'll step all over each other.

My instinct was "there is support in corefile-specific code for
handling threads.  It works just fine; use it."  It's been there since
1993.  Does it have any current consumers?  I think it must, since
Kevin touched it in May and mentioned some sort of regressions.  But
Linux has never produced these before and Solaris has its own handling. 
Does AIX maybe?

Since the overwhelming response seems to be that thread-db should be
doing this, and I can't make that happen without gutting corelow and I
have no idea what platforms that use that support, I'm going to leave
this alone unless someone else has a good idea.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer


  parent reply	other threads:[~2001-12-17  1:58 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-13  8:50 Daniel Jacobowitz
2001-12-13 10:57 ` Andrew Cagney
2001-12-13 11:37   ` Daniel Jacobowitz
2001-12-16 17:58   ` Daniel Jacobowitz [this message]
2001-12-16 21:32     ` Kevin Buettner
2001-12-17  8:34       ` Daniel Jacobowitz
2001-12-13 12:26 ` Michael Snyder
2001-12-13 12:31   ` Daniel Jacobowitz
2001-12-13 14:59     ` Michael Snyder
2001-12-13 15:04       ` Daniel Jacobowitz
2001-12-13 15:08         ` Michael Snyder
2001-12-13 15:11           ` Daniel Jacobowitz
2001-12-13 15:37             ` Andrew Cagney
2001-12-13 15:46               ` Daniel Jacobowitz
2001-12-13 17:14                 ` Michael Snyder
2001-12-13 20:29                   ` Daniel Jacobowitz
2001-12-14 18:12                     ` Andrew Cagney
2001-12-14 18:25                       ` Daniel Jacobowitz
2001-12-13 15:47             ` Michael Snyder
2001-12-13 15:57               ` Daniel Jacobowitz
2001-12-13 16:06                 ` Daniel Jacobowitz
2001-12-13 17:31                   ` Michael Snyder
2001-12-13 20:23                     ` Daniel Jacobowitz
2001-12-14 15:43                       ` Michael Snyder
2001-12-14 17:14                         ` Daniel Jacobowitz
2001-12-17 11:40                           ` Michael Snyder
2001-12-17 11:51                             ` Daniel Jacobowitz
2001-12-13 17:26                 ` Michael Snyder
2001-12-13 20:27                   ` Daniel Jacobowitz
2001-12-14 18:31                     ` Andrew Cagney
2001-12-14 18:36                       ` Andrew Cagney
2001-12-14 18:42                       ` Daniel Jacobowitz
2001-12-15  9:16                         ` Andrew Cagney
2001-12-16 12:26                           ` Daniel Jacobowitz
2001-12-16 13:29                             ` Andrew Cagney
2001-12-16 17:02                               ` Daniel Jacobowitz
2002-01-03 17:11 ` Michael Snyder
2002-01-04 10:25   ` Daniel Jacobowitz
2002-01-04 14:49     ` Michael Snyder
2002-01-04 17:28     ` Michael Snyder
2002-01-04 17:47       ` Daniel Jacobowitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20011216205642.A4358@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=ac131313@cygnus.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox