Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tromey@redhat.com>
To: Giuseppe MONTALTO <giuseppe.montalto@st.com>
Cc: "gdb-patches\@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [PATCH] new MI command for pattern filling of memory regions
Date: Wed, 09 May 2012 18:11:00 -0000	[thread overview]
Message-ID: <87sjf9rzr0.fsf@fleche.redhat.com> (raw)
In-Reply-To: <76FE3225DF13124EA2D05B290B624C95E65F2D466E@SAFEX1MAIL1.st.com>	(Giuseppe MONTALTO's message of "Wed, 9 May 2012 18:18:27 +0200")

>>>>> "Giuseppe" == Giuseppe MONTALTO <giuseppe.montalto@st.com> writes:

Giuseppe> it's the first time I attempt a contribution to gdb, so feel
Giuseppe> free to ask for details and, please, let me know if I'm doing
Giuseppe> anything wrong!

I was told off-list that you are covered by your employer's assignment.
So, good.

Giuseppe> the proposed enhancement is about memory filling.

Giuseppe> Filling a memory region with a pattern of bytes so far, can
Giuseppe> only be done by sending a potentially high number of repeated
Giuseppe> commands to gdb (depending on the size of the region that we
Giuseppe> need to fill).  If gdb is controlled through a GUI
Giuseppe> (e.g. Eclipse/CDT), this may cause the UI to freeze for a
Giuseppe> quite long time.  This is especially true when you're
Giuseppe> debugging a remote target, due to the amount of generated
Giuseppe> network traffic.

I started to review this, but I wondered why a new command is necessary.

It seems like you could send any hex string to want to
-data-write-memory-bytes.

Alternatively, if you really want the repeat count, why not just add an
optional argument to -data-write-memory-bytes?  That would seem to be
simpler.

Tom


  parent reply	other threads:[~2012-05-09 18:11 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-09 16:18 Giuseppe MONTALTO
2012-05-09 16:30 ` Tom Tromey
2012-05-09 18:11 ` Tom Tromey [this message]
2012-05-10 11:41   ` Giuseppe MONTALTO
2012-05-10 13:57     ` Tom Tromey
2012-05-11  8:53       ` Giuseppe MONTALTO
2012-05-11 14:36         ` Tom Tromey
2012-05-11 16:06           ` Giuseppe MONTALTO
2012-05-25 14:35           ` Giuseppe MONTALTO
2012-06-12  9:07           ` Giuseppe MONTALTO
2012-06-12 10:01             ` Abid, Hafiz
2012-06-12 14:13               ` Giuseppe MONTALTO
2012-06-12 14:22               ` [PATCH] enhancement of mi_cmd_data_write_memory_bytes for filling memory regions (was [PATCH] new MI command for pattern filling of memory regions) Giuseppe MONTALTO
2012-09-14 16:30                 ` Tom Tromey
2012-09-18 15:19                   ` Giuseppe MONTALTO
2012-09-26 20:44                     ` Tom Tromey
2012-09-27 15:27                       ` Giuseppe MONTALTO
2012-09-28 20:07                         ` Tom Tromey
2012-10-01  9:29                           ` Giuseppe MONTALTO
2012-10-17 20:59                             ` Tom Tromey
2012-10-18 13:40                               ` Giuseppe MONTALTO
2012-10-18 15:41                         ` Pedro Alves
2012-10-18 16:16                           ` Giuseppe MONTALTO
2012-10-18 16:28                             ` [+docs] " Pedro Alves
2012-10-18 17:31                               ` Eli Zaretskii
2012-10-18 17:30                             ` Eli Zaretskii
2012-10-19 14:12                               ` Giuseppe MONTALTO
2012-10-19 16:34                                 ` Eli Zaretskii
2012-10-19 16:56                                 ` Pedro Alves
2012-10-22  7:02                                   ` Giuseppe MONTALTO
2012-11-08 20:37                                     ` Tom Tromey
2012-11-09  8:59                                       ` Giuseppe MONTALTO
2012-11-12 16:52                                         ` Tom Tromey
2012-11-13 10:50                                           ` Giuseppe MONTALTO
2012-11-13 21:19                                             ` Tom Tromey
2012-10-18 16:38                           ` Giuseppe MONTALTO
2012-10-18 19:45                           ` Tom Tromey

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=87sjf9rzr0.fsf@fleche.redhat.com \
    --to=tromey@redhat.com \
    --cc=gdb-patches@sourceware.org \
    --cc=giuseppe.montalto@st.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