From: Jan Kratochvil <jan.kratochvil@redhat.com>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org, Paul Pluzhnikov <ppluzhnikov@google.com>
Subject: Re: [patch 3/3] Implement qXfer:libraries for Linux/gdbserver #2
Date: Fri, 21 Oct 2011 11:05:00 -0000 [thread overview]
Message-ID: <20111021094258.GA23101@host1.jankratochvil.net> (raw)
In-Reply-To: <201110062009.24796.pedro@codesourcery.com>
On Thu, 06 Oct 2011 21:09:24 +0200, Pedro Alves wrote:
> but this reuse of <library-list> is a mistake.
<library-list/> already supports the "segment" format and "section" format.
SVR4 attributes are just another format of <library-list/>.
Or do you agree there should rather have been <library-list-segments/> and
<library-list-sections/> and it is kept this way for backward compatibility?
> If we're not going to use
> an solib-target.c at all, and the semantics of what goes over the
> wire is not the same as <library-list>/TARGET_OBJECT_LIBRARIES, then
> let's come up with a completely independent new target object + dtd
> instead. Let's call it for example TARGET_OBJECT_SVR4_LIBRARIES.
> We should not prevent the possibility of _both_ using solib-svr4.c and
> solib-target.c at the same time, or the possibility of having a completely
> target-side implementation of svr4 libraries in the future,
I find this implementation as the fully target-side one. SVR4 needs some
information to match symbol files <-> target memory wrt prelinked files and to
aid libthread_db.
Maybe DYNAMIC segment offset could be sent the other way (host->target) to
match the prelink displacement. And also struct link_map address for
libthread_db may not be transferred and TLS variables could be read by special
packets instead. I do not find any of the cases to have advantage, though.
> using <library-list> as is (and having gdb be aware that support is present
> from qXfer:read:libraries+ in qSupported).
That is you suggest to have also new qXfer:read:svr4libraries+?
I do not follow how differently / more target-side could be the libraries
implemented on SVR4 systems, I already tried to implement it that way.
Thanks,
Jan
next prev parent reply other threads:[~2011-10-21 9:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-03 21:55 Jan Kratochvil
2011-10-04 6:11 ` Eli Zaretskii
2011-10-18 17:49 ` Messages localization in gdbserver [Re: [patch 3/3] Implement qXfer:libraries for Linux/gdbserver #2] Jan Kratochvil
2011-10-06 19:09 ` [patch 3/3] Implement qXfer:libraries for Linux/gdbserver #2 Pedro Alves
2011-10-21 11:05 ` Jan Kratochvil [this message]
2011-10-21 13:32 ` Pedro Alves
2011-11-03 21:30 ` [patch] Implement qXfer:libraries-svr4 for Linux/gdbserver #4 Jan Kratochvil
2011-12-02 22:33 ` [commit] " Jan Kratochvil
2011-12-03 17:03 ` Doug Evans
2011-12-03 18:34 ` [commit] Fix compilation --without-expat [Re: [commit] [patch] Implement qXfer:libraries-svr4 for Linux/gdbserver #4] Jan Kratochvil
2011-10-10 2:44 ` [patch 3/3] Implement qXfer:libraries for Linux/gdbserver #2 Daniel Jacobowitz
2011-10-12 20:22 ` 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=20111021094258.GA23101@host1.jankratochvil.net \
--to=jan.kratochvil@redhat.com \
--cc=gdb-patches@sourceware.org \
--cc=pedro@codesourcery.com \
--cc=ppluzhnikov@google.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