Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: Tom Tromey <tromey@redhat.com>
Cc: <gdb-patches@sourceware.org>
Subject: Re: [PATCH 1/3] New test case for PR12929.
Date: Wed, 31 Jul 2013 07:53:00 -0000	[thread overview]
Message-ID: <51F8C229.50103@codesourcery.com> (raw)
In-Reply-To: <87a9l3ltft.fsf@fleche.redhat.com>

On 07/31/2013 03:49 AM, Tom Tromey wrote:
> I applied this patch to my branch, to see what would happen.  On my
> system (x86-64 Fedora 18, with glibc debuginfo installed), I see this
> backtrace in the log:
> 
> python gdb.execute("attach 11985"); gdb.execute("where")
> 0x0000003e422bb4e0 in __nanosleep_nocancel () at ../sysdeps/unix/syscall-template.S:81
> 81	T_PSEUDO (SYSCALL_SYMBOL, SYSCALL_NAME, SYSCALL_NARGS)
> #0  0x0000003e422bb4e0 in __nanosleep_nocancel () at ../sysdeps/unix/syscall-template.S:81
> #1  0x0000003e422bb3a1 in __sleep (seconds=0) at ../sysdeps/unix/sysv/linux/sleep.c:137
> #2  0x00000000004005da in foo1 () at ../../../archer/gdb/testsuite/gdb.python/py-sync-interp.c:26
> #3  0x00000000004005e5 in foo2 () at ../../../archer/gdb/testsuite/gdb.python/py-sync-interp.c:32
> #4  0x00000000004005f0 in foo3 () at ../../../archer/gdb/testsuite/gdb.python/py-sync-interp.c:38
> #5  0x00000000004005fb in main () at ../../../archer/gdb/testsuite/gdb.python/py-sync-interp.c:45
> 
> 
> This yielded a FAIL:
> 
> FAIL: gdb.python/py-sync-interp.exp: attach and where (pattern 1)

Tom,
The pattern is updated to ".*sleep \\(.*\\)" to match the output.

-- 
Yao (齐尧)

gdb/testsuite:

	* gdb.python/py-sync-interp.c: New.
	* gdb.python/py-sync-interp.exp: New.
---
 gdb/testsuite/gdb.python/py-sync-interp.c   |   46 +++++++++++++++++++++
 gdb/testsuite/gdb.python/py-sync-interp.exp |   59 +++++++++++++++++++++++++++
 2 files changed, 105 insertions(+), 0 deletions(-)
 create mode 100644 gdb/testsuite/gdb.python/py-sync-interp.c
 create mode 100644 gdb/testsuite/gdb.python/py-sync-interp.exp

diff --git a/gdb/testsuite/gdb.python/py-sync-interp.c b/gdb/testsuite/gdb.python/py-sync-interp.c
new file mode 100644
index 0000000..5ae770a
--- /dev/null
+++ b/gdb/testsuite/gdb.python/py-sync-interp.c
@@ -0,0 +1,46 @@
+/* This testcase is part of GDB, the GNU debugger.
+
+   Copyright 2013 Free Software Foundation, Inc.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
+
+/* This program is intended to be started outside of gdb, and then
+   attached to by gdb.  It loops for a while, but not forever.  */
+
+#include <unistd.h>
+
+static void
+foo1 (void)
+{
+  sleep (100);
+}
+
+static void
+foo2 (void)
+{
+  foo1 ();
+}
+
+static void
+foo3 (void)
+{
+  foo2 ();
+}
+
+int
+main (void)
+{
+  foo3 ();
+  return 0;
+}
diff --git a/gdb/testsuite/gdb.python/py-sync-interp.exp b/gdb/testsuite/gdb.python/py-sync-interp.exp
new file mode 100644
index 0000000..cfa75db
--- /dev/null
+++ b/gdb/testsuite/gdb.python/py-sync-interp.exp
@@ -0,0 +1,59 @@
+# Copyright (C) 2013 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# This file is part of the GDB testsuite.
+
+# This test case is copied from PR12929.
+# http://sourceware.org/bugzilla/show_bug.cgi?id=12929
+
+standard_testfile
+
+# We need to use TCL's exec to get the pid.
+if [is_remote target] then {
+    return 0
+}
+
+load_lib gdb-python.exp
+
+# Start with a fresh gdb.
+
+gdb_exit
+gdb_start
+gdb_reinitialize_dir $srcdir/$subdir
+
+# Skip all tests if Python scripting is not enabled.
+if { [skip_python_tests] } { continue }
+gdb_exit
+
+if { [prepare_for_testing ${testfile}.exp ${testfile} ${srcfile}] } {
+    return -1
+}
+
+# Start the program running and then wait for a bit, to be sure
+# that it can be attached to.
+set testpid [eval exec $binfile &]
+exec sleep 2
+if { [istarget "*-*-cygwin*"] } {
+    # testpid is the Cygwin PID, GDB uses the Windows PID, which might be
+    # different due to the way fork/exec works.
+    set testpid [ exec ps -e | gawk "{ if (\$1 == $testpid) print \$4; }" ]
+}
+
+# Test command 'where' is executed when command 'attach' is done, otherwise
+# function 'sleep' may not show up in backtrace.
+
+gdb_test "python gdb.execute(\"attach $testpid\"); gdb.execute(\"where\")" \
+    "in .*sleep \\(.*\\) .* in foo1 \\(\\) at .* in foo2 \\(\\) at .*" \
+    "attach and where"
-- 
1.7.7.6


  parent reply	other threads:[~2013-07-31  7:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-25  9:56 Fix PR14135 Yao Qi
2012-05-25  9:56 ` [PATCH 2/3] Don't force interpreter sync mode in execute_gdb_command Yao Qi
2013-07-30 19:55   ` Tom Tromey
2013-07-30 20:13     ` Tom Tromey
2012-05-25  9:56 ` [PATCH 3/3] Match output in async mode Yao Qi
2013-07-30 20:06   ` Tom Tromey
2013-07-31  7:45     ` Yao Qi
2012-05-25  9:56 ` [PATCH 1/3] New test case for PR12929 Yao Qi
2013-07-30 19:49   ` Tom Tromey
2013-07-30 20:11     ` Tom Tromey
2013-07-31  7:53     ` Yao Qi [this message]
2013-07-31 15:19       ` Tom Tromey
2013-08-01  6:39         ` Yao Qi
2012-07-17  2:51 ` Fix PR14135 Yao Qi

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=51F8C229.50103@codesourcery.com \
    --to=yao@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=tromey@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