return zero in arm_linux_can_use_hw_breakpoint if HW point isn't supported
authorYao Qi <yao.qi@linaro.org>
Fri, 17 Apr 2015 12:52:44 +0000 (13:52 +0100)
committerYao Qi <yao.qi@linaro.org>
Fri, 17 Apr 2015 12:52:44 +0000 (13:52 +0100)
This patch is to cherry-pick part of Pedro's patch here
https://sourceware.org/ml/gdb-patches/2015-04/msg00527.html in which
zero is returned if the HW point isn't supported.

In arm-linux native gdb testing on a board doesn't support HW breakpoint,
without this patch, the output in gdb.base/breakpoint-in-ro-region.exp is like:

(gdb) hbreak *0x83bc^M
Hardware breakpoints used exceeds limit.^M
(gdb) PASS: gdb.base/breakpoint-in-ro-region.exp: probe hbreak support (support)

with this patch, the output becomes:

(gdb) hbreak *0x83bc^M
No hardware breakpoint support in the target.^M
(gdb) PASS: gdb.base/breakpoint-in-ro-region.exp: probe hbreak support (no support)

As a result, the following fails are fixed.

-FAIL: gdb.base/breakpoint-in-ro-region.exp: always-inserted off: auto-hw on: step in ro region (cannot insert hw break)
-FAIL: gdb.base/breakpoint-in-ro-region.exp: always-inserted off: auto-hw on: thread advanced
-FAIL: gdb.base/breakpoint-in-ro-region.exp: always-inserted on: auto-hw on: step in ro region (cannot insert hw break)
-FAIL: gdb.base/breakpoint-in-ro-region.exp: always-inserted on: auto-hw on: thread advanced

gdb:

2015-04-17  Pedro Alves  <palves@redhat.com>

* arm-linux-nat.c (arm_linux_can_use_hw_breakpoint): Return zero
if HW point of TYPE isn't supported.

gdb/ChangeLog
gdb/arm-linux-nat.c

index 295ef34e6c714f2b8b520fd5e42ffe6ef6f10a17..273c6b3386f6a65ce06837530f6411b6a7e30b87 100644 (file)
@@ -1,3 +1,8 @@
+2015-04-17  Pedro Alves  <palves@redhat.com>
+
+       * arm-linux-nat.c (arm_linux_can_use_hw_breakpoint): Return zero
+       if HW point of TYPE isn't supported.
+
 2015-04-17  Yao Qi  <yao.qi@linaro.org>
            Pedro Alves  <palves@redhat.com>
 
index bb8358c1bf7b23a4f66f152b186a473929e16907..afc58178908dca3d603446b106b2524d09975c6f 100644 (file)
@@ -771,12 +771,20 @@ arm_linux_can_use_hw_breakpoint (struct target_ops *self,
   if (type == bp_hardware_watchpoint || type == bp_read_watchpoint
       || type == bp_access_watchpoint || type == bp_watchpoint)
     {
-      if (cnt + ot > arm_linux_get_hw_watchpoint_count ())
+      int count = arm_linux_get_hw_watchpoint_count ();
+
+      if (count == 0)
+       return 0;
+      else if (cnt + ot > count)
        return -1;
     }
   else if (type == bp_hardware_breakpoint)
     {
-      if (cnt > arm_linux_get_hw_breakpoint_count ())
+      int count = arm_linux_get_hw_breakpoint_count ();
+
+      if (count == 0)
+       return 0;
+      else if (cnt > count)
        return -1;
     }
   else