[PATCH] Remove setup_kfail server/13796 in disp-step-syscall.exp

[PATCH] Remove setup_kfail server/13796 in disp-step-syscall.exp

[Yao Qi]

This patch series add fork support in target remote,

  [PATCH v2 0/3] Target remote mode fork and exec support
  https://sourceware.org/ml/gdb-patches/2015-12/msg00144.html

so GDB can be informed about the child, and adjust child correctly in
displaced stepping.  The PR server/13796 was fixed by this patch
series actually.  Test results on buildbot show this KFAIL->KPASS
change https://sourceware.org/ml/gdb-testers/2015-q4/msg10128.html

gdb/testsuite:

2016-02-18  Yao Qi  <yao.qi@linaro.org>

	* gdb.base/disp-step-syscall.exp (disp_step_cross_syscall):
	Don't call setup_kfail.
---
 gdb/testsuite/gdb.base/disp-step-syscall.exp | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/gdb/testsuite/gdb.base/disp-step-syscall.exp b/gdb/testsuite/gdb.base/disp-step-syscall.exp
index 7426ee7..80cdc3b 100644
--- a/gdb/testsuite/gdb.base/disp-step-syscall.exp
+++ b/gdb/testsuite/gdb.base/disp-step-syscall.exp
@@ -49,8 +49,6 @@ proc disp_step_cross_syscall { syscall } {
 	    return
 	}
 
-	set is_target_remote [gdb_is_target_remote]
-
 	# Delete the breakpoint on main.
 	gdb_test_no_output "delete break 1"
 
@@ -130,10 +128,6 @@ proc disp_step_cross_syscall { syscall } {
 	gdb_test_no_output "set displaced-stepping on"
 
 	# Check the address of next instruction of syscall.
-	if {$syscall == "vfork" && $is_target_remote} {
-	    setup_kfail server/13796 "*-*-*"
-	}
-
 	if {[gdb_test "stepi" "x/i .*=>.*" "single step over $syscall"] != 0} {
 	    return -1
 	}
-- 
1.9.1

Re: [PATCH] Remove setup_kfail server/13796 in disp-step-syscall.exp

[Pedro Alves]

On 02/18/2016 12:00 PM, Yao Qi wrote:
> This patch series add fork support in target remote,
> 
>   [PATCH v2 0/3] Target remote mode fork and exec support
>   https://sourceware.org/ml/gdb-patches/2015-12/msg00144.html
> 
> so GDB can be informed about the child, and adjust child correctly in
> displaced stepping.  The PR server/13796 was fixed by this patch
> series actually.  Test results on buildbot show this KFAIL->KPASS
> change https://sourceware.org/ml/gdb-testers/2015-q4/msg10128.html
> 
> gdb/testsuite:
> 
> 2016-02-18  Yao Qi  <yao.qi@linaro.org>
> 
> 	* gdb.base/disp-step-syscall.exp (disp_step_cross_syscall):
> 	Don't call setup_kfail.

OK.

Thanks,
Pedro Alves