Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Maciej W. Rozycki" <macro@mips.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: gdb-patches@sourceware.org, "Maciej W. Rozycki" <macro@linux-mips.org>
Subject: Re: mips-tdep.c: Fix sw watchpoint-out-of-scope events
Date: Wed, 19 Sep 2007 12:31:00 -0000	[thread overview]
Message-ID: <Pine.LNX.4.61.0709191219020.27377@perivale.mips.com> (raw)
In-Reply-To: <20070918155400.GA1128@caradoc.them.org>

On Tue, 18 Sep 2007, Daniel Jacobowitz wrote:

> This patch is fine.  I wish it wasn't necessary, but for now it is.

 Well, actually then, I have thought, rather than trying to fix the 
unfixable, why don't we mimic the behaviour of hardware watchpoints?  As 
they do not single-step (the whole point of them, at least for the write 
watchpoints) a scope software breakpoint is placed at the return target of 
the scope covered.  We could do the same for software watchpoints -- 
insert a scope software breakpoint, single-step till the watchpoints goes 
out of scope and then resume at the full speed till the scope breakpoint 
is hit.  Was there any particular reason for not doing so in the first 
place?

 I'll go forward with the patch regardless as it would become obsolete 
automatically if we accepted the hw watchpoint model for sw ones too as 
in_function_epilogue_p() would become irrelevant then as far as I can 
tell.

> Hmm, do you think we should accept that line as a pass then?  The same
> will happen on other architectures too.

 Well, it depends on the policy we have (if we have any in this area) -- 
the patch fixes non-PIC o32 and presumably new ABIs (I'll try to see 
whether I have a way to test it), but does not quite so for PIC o32 
(beacuse of the load to $gp after the return).  There may be other 
architectures with a similar problem, but if MIPS o32 was to be the only 
one affected, then a PASS would be reasonable, as stated originally.  
Though I have no idea how to take care of the load to $gp at the moment -- 
perhaps some logic from the scope breakpoint calculation could be reused.

  Maciej


  reply	other threads:[~2007-09-19 12:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-11 16:24 Maciej W. Rozycki
2007-09-18 15:54 ` Daniel Jacobowitz
2007-09-19 12:31   ` Maciej W. Rozycki [this message]
2007-09-19 12:38     ` Daniel Jacobowitz
2007-09-19 13:05       ` Maciej W. Rozycki
2007-09-19 13:16         ` Daniel Jacobowitz

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=Pine.LNX.4.61.0709191219020.27377@perivale.mips.com \
    --to=macro@mips.com \
    --cc=drow@false.org \
    --cc=gdb-patches@sourceware.org \
    --cc=macro@linux-mips.org \
    /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