Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: jose.marchesi@oracle.com (Jose E. Marchesi)
To: "Tedeschi\, Walfred" <walfred.tedeschi@intel.com>
Cc: "gdb-patches\@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [PATCH][SPARC64] Fix breakpointing in syscalls other than sigreturn
Date: Mon, 23 Sep 2013 14:02:00 -0000	[thread overview]
Message-ID: <871u4fab3d.fsf@oracle.com> (raw)
In-Reply-To: <AC542571535E904D8E8ADAE745D60B191B197923@IRSMSX104.ger.corp.intel.com>	(Walfred Tedeschi's message of "Mon, 23 Sep 2013 13:33:37 +0000")


Hi.
    
    Have you observed some failure to submit this patch? If so could you
    also provide a test case?

I noticed that watchpoints were not surviving ld.so.  Consider this test
program:

  int jorl;

  int main ()
  {
    jorl = 20;
    return 0;
  }

If you set a watchpoint to watch writes to the variable 'jorl' and then

  [jemarch@jemarch1 ~]$ gdb foo
  GNU gdb (GDB) Red Hat Enterprise Linux (7.2-60.0.2.el6)
  Copyright (C) 2010 Free Software Foundation, Inc.
  License GPLv3+: GNU GPL version 3 or later
  <http://gnu.org/licenses/gpl.html>
  This is free software: you are free to change and redistribute it.
  There is NO WARRANTY, to the extent permitted by law.  Type "show
  copying"
  and "show warranty" for details.
  This GDB was configured as "sparc64-redhat-linux-gnu".
  For bug reporting instructions, please see:
  <http://www.gnu.org/software/gdb/bugs/>...
  Reading symbols from /home/jemarch/foo...(no debugging symbols
  found)...done.
  (gdb) watch jorl
  Watchpoint 1: jorl
  (gdb) run
  Starting program: /home/jemarch/foo
  0xfffff8010001c444 in mmap64 () from /lib64/ld-linux.so.2
  Could not insert single-step breakpoint at 0x1
  (gdb)

As described in the patch the problem is that single-stepping over 'ta
0x6d' instructions is not working properly.  And the reason is that
sparc64_linux_step_trap is only intended to be useful when
single-stepping sigreturn syscalls and must return zero in any other
case.

Do you want a test for gdb/testsuite?  In that case, what would be the
best place where to add the test?  gdb.base?


  reply	other threads:[~2013-09-23 14:02 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-23 13:04 Jose E. Marchesi
2013-09-23 13:33 ` Tedeschi, Walfred
2013-09-23 14:02   ` Jose E. Marchesi [this message]
2013-10-04 11:22     ` Pedro Alves
2013-09-23 14:36 ` Mark Kettenis
  -- strict thread matches above, loose matches on Subject: below --
2013-09-16  9:46 Jose E. Marchesi
2013-09-10 18:28 Jose E. Marchesi

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=871u4fab3d.fsf@oracle.com \
    --to=jose.marchesi@oracle.com \
    --cc=gdb-patches@sourceware.org \
    --cc=walfred.tedeschi@intel.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