Use 2-byte instead of 4-byte NOP on S390 in 'bp-permanent' test case
authorAndreas Arnez <arnez@linux.vnet.ibm.com>
Mon, 17 Nov 2014 15:22:48 +0000 (15:22 +0000)
committerAndreas Krebbel <krebbel@linux.vnet.ibm.com>
Wed, 19 Nov 2014 09:03:32 +0000 (10:03 +0100)
commit0d7b2549024f66cb98dd173b6f339673df1b0ea1
tree9812c48f2c6416e37cb5b2957506f28069480bbc
parent2b0f535a446c682c3dc7c1276e2cbc747bfae163
Use 2-byte instead of 4-byte NOP on S390 in 'bp-permanent' test case

The bp-permanent test case assumes that a NOP is exactly as long as a
software breakpoint.  This is not the case for the S390 "nop"
instruction, which is 4 bytes long, while a software breakpoint is
just 2 bytes long.  The "nopr" instruction has the right size and can
be used instead.

Without this patch the test case fails on S390 when trying to continue
after SIGTRAP on the permanent breakpoint:

  ...
  Continuing.

  Program received signal SIGILL, Illegal instruction.
  test () at /home/arnez/src/binutils-gdb/gdb/testsuite/gdb.base/bp-permanent.c:40
  40   NOP; /* after permanent bp */
  (gdb)
  FAIL: gdb.base/bp-permanent.exp: always_inserted=off, sw_watchpoint=0:
    basics: stop at permanent breakpoint

With this patch the test case succeeds without any FAILs.

gdb/testsuite/ChangeLog:

* gdb.base/bp-permanent.c (NOP): Define as 2-byte instead of
4-byte instruction on S390.
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/bp-permanent.c