From: Andrew Cagney <ac131313@redhat.com>
To: davidm@hpl.hp.com
Cc: Andrew Cagney <ac131313@redhat.com>,
Kevin Buettner <kevinb@redhat.com>,
"J. Johnston" <jjohnstn@redhat.com>,
gdb-patches@sources.redhat.com
Subject: Re: RFA: ia64 portion of libunwind patch
Date: Mon, 10 Nov 2003 22:43:00 -0000 [thread overview]
Message-ID: <3FB0149C.1060908@redhat.com> (raw)
In-Reply-To: <16304.3297.662733.250523@napali.hpl.hp.com>
> Andrew> If we look at GDB with its 128k of unwind data. At 14*28
> Andrew> byte requests per unwind, it would take ~300 unwinds before
> Andrew> GDB was required to xfer 128k (yes I'm pushing the numbers a
> Andrew> little here, but then I'm also ignoring the very significant
> Andrew> locality of the searches).
>
> Oh, but you're ignoring the latency effects. N 1-byte transfers can
> easily be much slower than a single N-byte transfer.
It's easy to play with the numbers here. Ex: The remote protocol
typically caps each transfer at ~1k. So that would be 128 (xfer all) vs
14 (xfer needed) so it would still be faster.
More seriously. If problems are identified in the remote protocol, GDB
should fix them. It is important though, that its clients don't program
around perceived performance problems and in doing so create artifical
loads. As my previous e-mail mentioned, GDB's already seen one example
of that - a library demanding ever increasing amounts of data in attempt
to work around an io throughput bottle neck.
> Andrew> Scary as it is, GDB's already got a requrest to feltch a
> Andrew> shared library image from the target's memory :-/.
>
> That kind of throws your speed argument out of the water, though,
> doesn't it? ;-)
The extraction will need to be done very carefully so only required data
is read.
> Andrew> Provided the remote target knows the address of the unwind
> Andrew> table, GDB should be able to find a way of getting it to
> Andrew> libunwind.
>
> OK, I still don't quite understand why this is a common and important
> scenario. It strikes me as a corner-case which _occasionally_ may be
> useful, but if that's true, a bit of extra latency doesn't seem like a
> huge deal.
>
> In any case, perhaps it is possible to add incremental reading support
> by stealing a bit from one of the members in the "unw_dyn_table_info".
> All we really need is a single bit to indicate whether the table-data
> should be fetched from remote-memory. I'll think about it some more.
It would be appreciated. My suggestion was to use memory reads when the
unwind table pointer was NULL. However, anything would help.
Andrew
next prev parent reply other threads:[~2003-11-10 22:43 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-10-31 19:25 J. Johnston
2003-10-31 20:46 ` Andrew Cagney
2003-10-31 22:55 ` David Mosberger
2003-11-07 21:47 ` Andrew Cagney
2003-11-07 22:43 ` David Mosberger
2003-11-07 23:01 ` Andrew Cagney
2003-11-07 23:12 ` David Mosberger
2003-11-07 23:38 ` Andrew Cagney
2003-11-07 23:55 ` David Mosberger
2003-11-08 0:07 ` Andrew Cagney
2003-11-08 0:13 ` Kevin Buettner
2003-11-08 0:27 ` Andrew Cagney
2003-11-08 7:21 ` David Mosberger
2003-11-09 0:13 ` Andrew Cagney
2003-11-10 22:10 ` David Mosberger
2003-11-10 22:43 ` Andrew Cagney [this message]
2003-11-10 23:01 ` David Mosberger
2003-11-26 0:11 ` David Mosberger
2003-12-04 2:15 ` David Mosberger
2003-12-04 3:15 ` Kevin Buettner
2003-12-04 23:57 ` J. Johnston
2003-12-05 0:39 ` David Mosberger
2003-12-10 20:58 ` J. Johnston
2003-12-10 22:15 ` David Mosberger
2003-12-12 22:25 ` Kevin Buettner
[not found] ` <davidm@napali.hpl.hp.com>
2003-12-13 4:01 ` Kevin Buettner
2003-12-31 20:19 ` make inferior calls work on ia64 even when syscall is pending David Mosberger
2003-12-31 23:37 ` Mark Kettenis
2004-01-01 2:43 ` David Mosberger
2004-02-13 1:14 ` David Mosberger
2004-02-13 15:00 ` Mark Kettenis
2004-02-13 15:09 ` Andrew Cagney
2004-02-13 15:12 ` Andrew Cagney
2004-02-13 22:07 ` David Mosberger
2004-02-17 16:21 ` Andrew Cagney
2004-02-23 19:58 ` Kevin Buettner
2004-02-23 21:15 ` Kevin Buettner
2003-11-09 1:34 ` RFA: ia64 portion of libunwind patch Marcel Moolenaar
2003-11-10 21:54 ` David Mosberger
2003-11-10 23:18 ` Marcel Moolenaar
2003-10-31 21:36 ` Marcel Moolenaar
2003-10-31 23:00 ` David Mosberger
2003-10-31 23:42 ` Andrew Cagney
2003-10-31 23:59 ` David Mosberger
-- strict thread matches above, loose matches on Subject: below --
2003-10-24 0:11 J. Johnston
2003-10-24 17:57 ` Kevin Buettner
2003-10-24 18:20 ` J. Johnston
2003-10-24 18:56 ` Kevin Buettner
2003-10-24 21:53 ` Marcel Moolenaar
2003-10-24 23:58 ` Kevin Buettner
2003-10-28 23:53 ` J. Johnston
2003-10-29 1:28 ` Daniel Jacobowitz
2003-10-29 4:48 ` Kevin Buettner
2003-10-29 18:43 ` J. Johnston
2003-10-29 22:48 ` Andrew Cagney
2003-11-04 19:09 ` J. Johnston
2003-11-04 20:48 ` Kevin Buettner
2003-11-14 0:26 ` J. Johnston
2003-11-14 1:17 ` Kevin Buettner
2003-11-14 20:49 ` J. Johnston
2003-10-29 23:28 ` Andrew Cagney
2003-11-02 20:39 ` Elena Zannoni
2003-10-29 15:18 ` Andrew Cagney
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=3FB0149C.1060908@redhat.com \
--to=ac131313@redhat.com \
--cc=davidm@hpl.hp.com \
--cc=gdb-patches@sources.redhat.com \
--cc=jjohnstn@redhat.com \
--cc=kevinb@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