Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <palves@redhat.com>
To: Mike Frysinger <vapier@gentoo.org>
Cc: gdb-patches@sourceware.org, Jan Kratochvil <jan.kratochvil@redhat.com>
Subject: Re: [patch+7.5.1] Work around PR libc/13097 "linux-vdso.so.1" #3
Date: Wed, 28 Nov 2012 01:12:00 -0000	[thread overview]
Message-ID: <50B558BF.5040300@redhat.com> (raw)
In-Reply-To: <201211271659.11975.vapier@gentoo.org>

On 11/27/2012 09:59 PM, Mike Frysinger wrote:
> On Friday 23 November 2012 11:01:03 Pedro Alves wrote:
>> On 11/23/2012 12:39 PM, Jan Kratochvil wrote:
>>> Besides that there is no standard for any such rule and I consider this
>>> patch only as a workaround of a WONTFIXed glibc PR.
>>
>> Given the new order of glibc maintenance, if we believe glibc's WONTFIX was
>> wrong, then we should reopen the PR, and retry discussing the issue.
> 
> i think the glibc behavior is actually nice and we don't want to revert it.  
> if i'm single stepping through code that enters the vdso, if gdb doesn't know 
> about it, it looks like i just stepped off into the weeds and the program is 
> doing something wrong.  if, instead, it told me i was in the vdso (and we had 
> some way of communicating symbol information), that's a lot more useful.

Indeed.  GDB already loads the vDSO 's elf (symfile-mem.c).  GDB finds the vdso address
in the auxv.  I think in Fedora, the idea is to find the debug info file through
build id matching (I haven't checked if that's actually implemented, but in previous
online searches I found references to that).  In
<http://sourceware.org/ml/gdb-patches/2012-11/msg00628.html> I mentioned that this change
could make it possible to find the separate debug file by name.  I think it'd be
nice to turn things around, and list the vDSO in the shared libraries list instead of
hiding it.

BTW, before that change, when the vDSO was nameless, what made sure it didn't appear
in the DSO list?  Where was it skipped?

-- 
Pedro Alves


  parent reply	other threads:[~2012-11-28  1:12 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-22 20:17 Jan Kratochvil
2012-11-23 11:30 ` Luis Gustavo
2012-11-23 11:41   ` Jan Kratochvil
2012-11-23 11:59     ` Luis Machado
2012-11-23 12:03 ` Pedro Alves
2012-11-23 12:40   ` Jan Kratochvil
2012-11-23 14:05     ` Luis Machado
2012-11-23 14:07       ` Jan Kratochvil
2012-11-23 16:01     ` Pedro Alves
2012-11-23 16:12       ` Mark Kettenis
2012-11-23 16:17         ` Jan Kratochvil
2012-11-23 16:23         ` Pedro Alves
2012-11-23 17:28       ` Joel Brobecker
2012-11-23 18:17         ` Jan Kratochvil
2012-11-23 18:22           ` Pedro Alves
2012-11-27 21:58       ` Mike Frysinger
2012-11-27 23:00         ` Matt Rice
2012-11-28  1:12         ` Pedro Alves [this message]
2012-11-28 20:39           ` Mike Frysinger
2012-11-28 22:44           ` Jan Kratochvil
2012-11-29  1:07             ` Mike Frysinger
2012-11-23 12:43   ` Mark Kettenis
2012-11-23 16:30 ` H.J. Lu
2012-11-23 18:19   ` Pedro Alves
2012-11-23 18:43     ` Jan Kratochvil
2012-11-25 18:15       ` Jan Kratochvil

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=50B558BF.5040300@redhat.com \
    --to=palves@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=jan.kratochvil@redhat.com \
    --cc=vapier@gentoo.org \
    /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