Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [rfa] break-entry.exp:  suppress test if remote target
@ 2010-12-07 19:36 Michael Snyder
  2010-12-09 21:41 ` Jan Kratochvil
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Snyder @ 2010-12-07 19:36 UTC (permalink / raw)
  To: gdb-patches

[-- Attachment #1: Type: text/plain, Size: 190 bytes --]

This test doesn't work on remote targets (gdbserver), because it sets a 
breakpoint at _start, which is also the current pc.  We never hit a 
breakpoint at the current pc when we continue.


[-- Attachment #2: break-entry.txt --]
[-- Type: text/plain, Size: 671 bytes --]

2010-12-07  Michael Snyder  <msnyder@vmware.com>

	* gdb.base/break-entry.exp: Skip test if remote target.

Index: break-entry.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/break-entry.exp,v
retrieving revision 1.1
diff -u -p -r1.1 break-entry.exp
--- break-entry.exp	4 Apr 2010 22:12:14 -0000	1.1
+++ break-entry.exp	7 Dec 2010 19:34:26 -0000
@@ -21,6 +21,10 @@ if { [prepare_for_testing ${testfile}.ex
     return -1
 }
 
+if { [target_info exists use_gdb_stub] || [target_info exists gdb_protocol]} then {
+    return -1
+}
+
 set test "info files"
 set entry ""
 gdb_test_multiple $test $test {

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [rfa] break-entry.exp:  suppress test if remote target
  2010-12-07 19:36 [rfa] break-entry.exp: suppress test if remote target Michael Snyder
@ 2010-12-09 21:41 ` Jan Kratochvil
  2010-12-09 22:13   ` Michael Snyder
  0 siblings, 1 reply; 3+ messages in thread
From: Jan Kratochvil @ 2010-12-09 21:41 UTC (permalink / raw)
  To: Michael Snyder; +Cc: gdb-patches

On Tue, 07 Dec 2010 20:35:57 +0100, Michael Snyder wrote:
> This test doesn't work on remote targets (gdbserver), because it
> sets a breakpoint at _start, which is also the current pc.  We never
> hit a breakpoint at the current pc when we continue.

Your patch is out of sync with HEAD so you may not need this patch:

commit 787ced139be0421e8c5e0fbfe7ff5429d8b67601
Author: Nathan Froyd <froydnj@codesourcery.com>
Date:   Fri Nov 12 21:47:47 2010 +0000

    gdb/testsuite/
        * gdb.base/break-entry.exp: Skip if using a stub.

As this testcase uses `-static' it is probably OK.  (Otherwise it would
probably make sense to stop at _start, which is only after ld.so gets
executed.)


Thanks,
Jan


> 2010-12-07  Michael Snyder  <msnyder@vmware.com>
> 
> 	* gdb.base/break-entry.exp: Skip test if remote target.
> 
> Index: break-entry.exp
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/gdb.base/break-entry.exp,v
> retrieving revision 1.1
> diff -u -p -r1.1 break-entry.exp
> --- break-entry.exp	4 Apr 2010 22:12:14 -0000	1.1
> +++ break-entry.exp	7 Dec 2010 19:34:26 -0000
> @@ -21,6 +21,10 @@ if { [prepare_for_testing ${testfile}.ex
>      return -1
>  }
>  
> +if { [target_info exists use_gdb_stub] || [target_info exists gdb_protocol]} then {
> +    return -1
> +}
> +
>  set test "info files"
>  set entry ""
>  gdb_test_multiple $test $test {


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [rfa] break-entry.exp:  suppress test if remote target
  2010-12-09 21:41 ` Jan Kratochvil
@ 2010-12-09 22:13   ` Michael Snyder
  0 siblings, 0 replies; 3+ messages in thread
From: Michael Snyder @ 2010-12-09 22:13 UTC (permalink / raw)
  To: Jan Kratochvil; +Cc: gdb-patches

Jan Kratochvil wrote:
> On Tue, 07 Dec 2010 20:35:57 +0100, Michael Snyder wrote:
>> This test doesn't work on remote targets (gdbserver), because it
>> sets a breakpoint at _start, which is also the current pc.  We never
>> hit a breakpoint at the current pc when we continue.
> 
> Your patch is out of sync with HEAD so you may not need this patch:
> 
> commit 787ced139be0421e8c5e0fbfe7ff5429d8b67601
> Author: Nathan Froyd <froydnj@codesourcery.com>
> Date:   Fri Nov 12 21:47:47 2010 +0000
> 
>     gdb/testsuite/
>         * gdb.base/break-entry.exp: Skip if using a stub.

I see.  I was on the 7.2 branch.
Thanks.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-12-09 22:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-07 19:36 [rfa] break-entry.exp: suppress test if remote target Michael Snyder
2010-12-09 21:41 ` Jan Kratochvil
2010-12-09 22:13   ` Michael Snyder

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox