Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: toddpw@wrs.com (Todd Whitesel)
To: jtc@redbacknetworks.com (J.T. Conklin)
Cc: gdb@cygnus.com
Subject: Re: Any way to avoid inserting & removing breakpoints?
Date: Thu, 17 Dec 1998 19:20:00 -0000	[thread overview]
Message-ID: <199812180320.TAA02708@alabama.wrs.com> (raw)
In-Reply-To: <199812150126.RAA24778@jtc.redbacknetworks.com>

> Although I'm not surprised by this behavior, it would be preferable if
> the target itself was responsible for managing, inserting and removing
> breakpoints.  Then a lot of the traffic on the debugging channel could
> be avoided.  I didn't see anything in the GDB source that suggests
> this is possible; but I'm hoping I'm missing something.

At Wind River we use GDB with targets that manage their own breakpoints.

When I took over maintenance of the local GDB last August, I inherited a
fair amount of not-so-clean code sprinked throughout breakpoint.c and our
backend, and I don't like the way infrun.c interacts with it. For some
reason, wait_for_inferior occasionally wants to pull out all the breakpoints
and do something specific, and let them get pushed back in later. This
seems to me to never be a good idea.

I've already written one "breakpoint management layer" at a previous job.
It seems to me that factoring out such a thing would be the correct model
for GDB also. However I would prefer to put a single target_manage_breakpoint
item in the target vector so that all breakpoint functions are switched at
once, rather than exposing lots of little primitives and bloating the target
vector struct. Is there an established GDB rule about this sort of thing?

At work I now have authorization to rework+submit our local GDB patches,
but some accumulated project fires must be put out first and our office
is moving during the holidays so I am taking a vacation during that time.

So if you don't need this improvement for at least a month or two, then
you should just be able to pick up what I submit (assuming I can get it
past Stan that is :) ).

Todd Whitesel
toddpw @ wrs.com


  reply	other threads:[~1998-12-17 19:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
1998-12-14 17:26 J.T. Conklin
1998-12-17 19:20 ` Todd Whitesel [this message]
     [not found] <199812150126.RAA24778.cygnus.gdb@jtc.redbacknetworks.com>
1998-12-17 17:39 ` Stan Shebs
1998-12-17 20:40   ` Todd Whitesel
1998-12-18 11:15   ` J.T. Conklin

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=199812180320.TAA02708@alabama.wrs.com \
    --to=toddpw@wrs.com \
    --cc=gdb@cygnus.com \
    --cc=jtc@redbacknetworks.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