Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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



  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