Clean up many C99 integer types format specifiers
[openocd.git] / src / target / breakpoints.c
index 16ab7e0caa672c03d5c9dbd4ebb6625d80904ccc..5143afb7bca46f9d7a9995fe182900710d635a3a 100644 (file)
@@ -2,6 +2,9 @@
  *   Copyright (C) 2005 by Dominic Rath                                    *
  *   Dominic.Rath@gmx.de                                                   *
  *                                                                         *
+ *   Copyright (C) ST-Ericsson SA 2011                                     *
+ *   michel.jaouen@stericsson.com : smp minimum support                    *
+ *                                                                         *
  *   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 2 of the License, or     *
  *   You should have received a copy of the GNU General Public License     *
  *   along with this program; if not, write to the                         *
  *   Free Software Foundation, Inc.,                                       *
- *   59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.             *
+ *   51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.           *
  ***************************************************************************/
+
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 
 #include "target.h"
-#include "log.h"
+#include <helper/log.h>
 #include "breakpoints.h"
 
-
-static char *breakpoint_type_strings[] =
-{
+static char *breakpoint_type_strings[] = {
        "hardware",
        "software"
 };
 
-static char *watchpoint_rw_strings[] =
-{
+static char *watchpoint_rw_strings[] = {
        "read",
        "write",
        "access"
 };
 
-// monotonic counter/id-number for breakpoints and watch points
+/* monotonic counter/id-number for breakpoints and watch points */
 static int bpwp_unique_id;
 
-int breakpoint_add(struct target *target, uint32_t address, uint32_t length, enum breakpoint_type type)
+int breakpoint_add_internal(struct target *target,
+       uint32_t address,
+       uint32_t length,
+       enum breakpoint_type type)
 {
        struct breakpoint *breakpoint = target->breakpoints;
        struct breakpoint **breakpoint_p = &target->breakpoints;
@@ -51,16 +55,15 @@ int breakpoint_add(struct target *target, uint32_t address, uint32_t length, enu
        int n;
 
        n = 0;
-       while (breakpoint)
-       {
+       while (breakpoint) {
                n++;
                if (breakpoint->address == address) {
                        /* FIXME don't assume "same address" means "same
                         * breakpoint" ... check all the parameters before
                         * succeeding.
                         */
-                       LOG_DEBUG("Duplicate Breakpoint address: 0x%08" PRIx32 " (BP %d)",
-                                 address, breakpoint->unique_id );
+                       LOG_DEBUG("Duplicate Breakpoint address: 0x%08" PRIx32 " (BP %" PRIu32 ")",
+                               address, breakpoint->unique_id);
                        return ERROR_OK;
                }
                breakpoint_p = &breakpoint->next;
@@ -69,6 +72,7 @@ int breakpoint_add(struct target *target, uint32_t address, uint32_t length, enu
 
        (*breakpoint_p) = malloc(sizeof(struct breakpoint));
        (*breakpoint_p)->address = address;
+       (*breakpoint_p)->asid = 0;
        (*breakpoint_p)->length = length;
        (*breakpoint_p)->type = type;
        (*breakpoint_p)->set = 0;
@@ -78,41 +82,220 @@ int breakpoint_add(struct target *target, uint32_t address, uint32_t length, enu
 
        retval = target_add_breakpoint(target, *breakpoint_p);
        switch (retval) {
-       case ERROR_OK:
-               break;
-       case ERROR_TARGET_RESOURCE_NOT_AVAILABLE:
-               reason = "resource not available";
-               goto fail;
-       case ERROR_TARGET_NOT_HALTED:
-               reason = "target running";
-               goto fail;
-       default:
-               reason = "unknown reason";
+               case ERROR_OK:
+                       break;
+               case ERROR_TARGET_RESOURCE_NOT_AVAILABLE:
+                       reason = "resource not available";
+                       goto fail;
+               case ERROR_TARGET_NOT_HALTED:
+                       reason = "target running";
+                       goto fail;
+               default:
+                       reason = "unknown reason";
 fail:
-               LOG_ERROR("can't add breakpoint: %s", reason);
+                       LOG_ERROR("can't add breakpoint: %s", reason);
+                       free((*breakpoint_p)->orig_instr);
+                       free(*breakpoint_p);
+                       *breakpoint_p = NULL;
+                       return retval;
+       }
+
+       LOG_DEBUG("added %s breakpoint at 0x%8.8" PRIx32 " of length 0x%8.8x, (BPID: %" PRIu32 ")",
+               breakpoint_type_strings[(*breakpoint_p)->type],
+               (*breakpoint_p)->address, (*breakpoint_p)->length,
+               (*breakpoint_p)->unique_id);
+
+       return ERROR_OK;
+}
+
+int context_breakpoint_add_internal(struct target *target,
+       uint32_t asid,
+       uint32_t length,
+       enum breakpoint_type type)
+{
+       struct breakpoint *breakpoint = target->breakpoints;
+       struct breakpoint **breakpoint_p = &target->breakpoints;
+       int retval;
+       int n;
+
+       n = 0;
+       while (breakpoint) {
+               n++;
+               if (breakpoint->asid == asid) {
+                       /* FIXME don't assume "same address" means "same
+                        * breakpoint" ... check all the parameters before
+                        * succeeding.
+                        */
+                       LOG_DEBUG("Duplicate Breakpoint asid: 0x%08" PRIx32 " (BP %" PRIu32 ")",
+                               asid, breakpoint->unique_id);
+                       return -1;
+               }
+               breakpoint_p = &breakpoint->next;
+               breakpoint = breakpoint->next;
+       }
+
+       (*breakpoint_p) = malloc(sizeof(struct breakpoint));
+       (*breakpoint_p)->address = 0;
+       (*breakpoint_p)->asid = asid;
+       (*breakpoint_p)->length = length;
+       (*breakpoint_p)->type = type;
+       (*breakpoint_p)->set = 0;
+       (*breakpoint_p)->orig_instr = malloc(length);
+       (*breakpoint_p)->next = NULL;
+       (*breakpoint_p)->unique_id = bpwp_unique_id++;
+       retval = target_add_context_breakpoint(target, *breakpoint_p);
+       if (retval != ERROR_OK) {
+               LOG_ERROR("could not add breakpoint");
                free((*breakpoint_p)->orig_instr);
                free(*breakpoint_p);
                *breakpoint_p = NULL;
                return retval;
        }
 
-       LOG_DEBUG("added %s breakpoint at 0x%8.8" PRIx32 " of length 0x%8.8x, (BPID: %d)",
-                         breakpoint_type_strings[(*breakpoint_p)->type],
-                         (*breakpoint_p)->address, (*breakpoint_p)->length,
-                         (*breakpoint_p)->unique_id  );
+       LOG_DEBUG("added %s Context breakpoint at 0x%8.8" PRIx32 " of length 0x%8.8x, (BPID: %" PRIu32 ")",
+               breakpoint_type_strings[(*breakpoint_p)->type],
+               (*breakpoint_p)->asid, (*breakpoint_p)->length,
+               (*breakpoint_p)->unique_id);
+
+       return ERROR_OK;
+}
+
+int hybrid_breakpoint_add_internal(struct target *target,
+       uint32_t address,
+       uint32_t asid,
+       uint32_t length,
+       enum breakpoint_type type)
+{
+       struct breakpoint *breakpoint = target->breakpoints;
+       struct breakpoint **breakpoint_p = &target->breakpoints;
+       int retval;
+       int n;
+       n = 0;
+       while (breakpoint) {
+               n++;
+               if ((breakpoint->asid == asid) && (breakpoint->address == address)) {
+                       /* FIXME don't assume "same address" means "same
+                        * breakpoint" ... check all the parameters before
+                        * succeeding.
+                        */
+                       LOG_DEBUG("Duplicate Hybrid Breakpoint asid: 0x%08" PRIx32 " (BP %" PRIu32 ")",
+                               asid, breakpoint->unique_id);
+                       return -1;
+               } else if ((breakpoint->address == address) && (breakpoint->asid == 0)) {
+                       LOG_DEBUG("Duplicate Breakpoint IVA: 0x%08" PRIx32 " (BP %" PRIu32 ")",
+                               address, breakpoint->unique_id);
+                       return -1;
+
+               }
+               breakpoint_p = &breakpoint->next;
+               breakpoint = breakpoint->next;
+       }
+       (*breakpoint_p) = malloc(sizeof(struct breakpoint));
+       (*breakpoint_p)->address = address;
+       (*breakpoint_p)->asid = asid;
+       (*breakpoint_p)->length = length;
+       (*breakpoint_p)->type = type;
+       (*breakpoint_p)->set = 0;
+       (*breakpoint_p)->orig_instr = malloc(length);
+       (*breakpoint_p)->next = NULL;
+       (*breakpoint_p)->unique_id = bpwp_unique_id++;
+
+
+       retval = target_add_hybrid_breakpoint(target, *breakpoint_p);
+       if (retval != ERROR_OK) {
+               LOG_ERROR("could not add breakpoint");
+               free((*breakpoint_p)->orig_instr);
+               free(*breakpoint_p);
+               *breakpoint_p = NULL;
+               return retval;
+       }
+       LOG_DEBUG(
+               "added %s Hybrid breakpoint at address 0x%8.8" PRIx32 " of length 0x%8.8x, (BPID: %" PRIu32 ")",
+               breakpoint_type_strings[(*breakpoint_p)->type],
+               (*breakpoint_p)->address,
+               (*breakpoint_p)->length,
+               (*breakpoint_p)->unique_id);
 
        return ERROR_OK;
 }
 
+int breakpoint_add(struct target *target,
+       uint32_t address,
+       uint32_t length,
+       enum breakpoint_type type)
+{
+       int retval = ERROR_OK;
+       if (target->smp) {
+               struct target_list *head;
+               struct target *curr;
+               head = target->head;
+               if (type == BKPT_SOFT)
+                       return breakpoint_add_internal(head->target, address, length, type);
+
+               while (head != (struct target_list *)NULL) {
+                       curr = head->target;
+                       retval = breakpoint_add_internal(curr, address, length, type);
+                       if (retval != ERROR_OK)
+                               return retval;
+                       head = head->next;
+               }
+               return retval;
+       } else
+               return breakpoint_add_internal(target, address, length, type);
+}
+int context_breakpoint_add(struct target *target,
+       uint32_t asid,
+       uint32_t length,
+       enum breakpoint_type type)
+{
+       int retval = ERROR_OK;
+       if (target->smp) {
+               struct target_list *head;
+               struct target *curr;
+               head = target->head;
+               while (head != (struct target_list *)NULL) {
+                       curr = head->target;
+                       retval = context_breakpoint_add_internal(curr, asid, length, type);
+                       if (retval != ERROR_OK)
+                               return retval;
+                       head = head->next;
+               }
+               return retval;
+       } else
+               return context_breakpoint_add_internal(target, asid, length, type);
+}
+int hybrid_breakpoint_add(struct target *target,
+       uint32_t address,
+       uint32_t asid,
+       uint32_t length,
+       enum breakpoint_type type)
+{
+       int retval = ERROR_OK;
+       if (target->smp) {
+               struct target_list *head;
+               struct target *curr;
+               head = target->head;
+               while (head != (struct target_list *)NULL) {
+                       curr = head->target;
+                       retval = hybrid_breakpoint_add_internal(curr, address, asid, length, type);
+                       if (retval != ERROR_OK)
+                               return retval;
+                       head = head->next;
+               }
+               return retval;
+       } else
+               return hybrid_breakpoint_add_internal(target, address, asid, length, type);
+}
+
 /* free up a breakpoint */
-static void breakpoint_free(struct target *target, struct breakpoint *breakpoint_remove)
+static void breakpoint_free(struct target *target, struct breakpoint *breakpoint_to_remove)
 {
        struct breakpoint *breakpoint = target->breakpoints;
        struct breakpoint **breakpoint_p = &target->breakpoints;
+       int retval;
 
-       while (breakpoint)
-       {
-               if (breakpoint == breakpoint_remove)
+       while (breakpoint) {
+               if (breakpoint == breakpoint_to_remove)
                        break;
                breakpoint_p = &breakpoint->next;
                breakpoint = breakpoint->next;
@@ -121,55 +304,84 @@ static void breakpoint_free(struct target *target, struct breakpoint *breakpoint
        if (breakpoint == NULL)
                return;
 
-       target_remove_breakpoint(target, breakpoint);
+       retval = target_remove_breakpoint(target, breakpoint);
 
-       LOG_DEBUG("BPID: %d", breakpoint->unique_id );
+       LOG_DEBUG("free BPID: %" PRIu32 " --> %d", breakpoint->unique_id, retval);
        (*breakpoint_p) = breakpoint->next;
        free(breakpoint->orig_instr);
        free(breakpoint);
 }
 
-void breakpoint_remove(struct target *target, uint32_t address)
+int breakpoint_remove_internal(struct target *target, uint32_t address)
 {
        struct breakpoint *breakpoint = target->breakpoints;
-       struct breakpoint **breakpoint_p = &target->breakpoints;
 
-       while (breakpoint)
-       {
-               if (breakpoint->address == address)
+       while (breakpoint) {
+               if ((breakpoint->address == address) && (breakpoint->asid == 0))
+                       break;
+               else if ((breakpoint->address == 0) && (breakpoint->asid == address))
+                       break;
+               else if ((breakpoint->address == address) && (breakpoint->asid != 0))
                        break;
-               breakpoint_p = &breakpoint->next;
                breakpoint = breakpoint->next;
        }
 
-       if (breakpoint)
-       {
+       if (breakpoint) {
                breakpoint_free(target, breakpoint);
-       }
-       else
-       {
-               LOG_ERROR("no breakpoint at address 0x%8.8" PRIx32 " found", address);
+               return 1;
+       } else {
+               if (!target->smp)
+                       LOG_ERROR("no breakpoint at address 0x%8.8" PRIx32 " found", address);
+               return 0;
        }
 }
+void breakpoint_remove(struct target *target, uint32_t address)
+{
+       int found = 0;
+       if (target->smp) {
+               struct target_list *head;
+               struct target *curr;
+               head = target->head;
+               while (head != (struct target_list *)NULL) {
+                       curr = head->target;
+                       found += breakpoint_remove_internal(curr, address);
+                       head = head->next;
+               }
+               if (found == 0)
+                       LOG_ERROR("no breakpoint at address 0x%8.8" PRIx32 " found", address);
+       } else
+               breakpoint_remove_internal(target, address);
+}
+
+void breakpoint_clear_target_internal(struct target *target)
+{
+       LOG_DEBUG("Delete all breakpoints for target: %s",
+               target_name(target));
+       while (target->breakpoints != NULL)
+               breakpoint_free(target, target->breakpoints);
+}
 
 void breakpoint_clear_target(struct target *target)
 {
-       struct breakpoint *breakpoint;
+       if (target->smp) {
+               struct target_list *head;
+               struct target *curr;
+               head = target->head;
+               while (head != (struct target_list *)NULL) {
+                       curr = head->target;
+                       breakpoint_clear_target_internal(curr);
+                       head = head->next;
+               }
+       } else
+               breakpoint_clear_target_internal(target);
 
-       LOG_DEBUG("Delete all breakpoints for target: %s",
-                       target_name(target));
-       while ((breakpoint = target->breakpoints) != NULL)
-       {
-               breakpoint_free(target, breakpoint);
-       }
 }
 
-struct breakpointbreakpoint_find(struct target *target, uint32_t address)
+struct breakpoint *breakpoint_find(struct target *target, uint32_t address)
 {
        struct breakpoint *breakpoint = target->breakpoints;
 
-       while (breakpoint)
-       {
+       while (breakpoint) {
                if (breakpoint->address == address)
                        return breakpoint;
                breakpoint = breakpoint->next;
@@ -179,23 +391,22 @@ struct breakpoint* breakpoint_find(struct target *target, uint32_t address)
 }
 
 int watchpoint_add(struct target *target, uint32_t address, uint32_t length,
-               enum watchpoint_rw rw, uint32_t value, uint32_t mask)
+       enum watchpoint_rw rw, uint32_t value, uint32_t mask)
 {
        struct watchpoint *watchpoint = target->watchpoints;
        struct watchpoint **watchpoint_p = &target->watchpoints;
        int retval;
        char *reason;
 
-       while (watchpoint)
-       {
+       while (watchpoint) {
                if (watchpoint->address == address) {
                        if (watchpoint->length != length
-                                       || watchpoint->value != value
-                                       || watchpoint->mask != mask
-                                       || watchpoint->rw != rw) {
+                               || watchpoint->value != value
+                               || watchpoint->mask != mask
+                               || watchpoint->rw != rw) {
                                LOG_ERROR("address 0x%8.8" PRIx32
-                                               "already has watchpoint %d",
-                                               address, watchpoint->unique_id);
+                                       "already has watchpoint %d",
+                                       address, watchpoint->unique_id);
                                return ERROR_FAIL;
                        }
 
@@ -216,43 +427,43 @@ int watchpoint_add(struct target *target, uint32_t address, uint32_t length,
 
        retval = target_add_watchpoint(target, *watchpoint_p);
        switch (retval) {
-       case ERROR_OK:
-               break;
-       case ERROR_TARGET_RESOURCE_NOT_AVAILABLE:
-               reason = "resource not available";
-               goto bye;
-       case ERROR_TARGET_NOT_HALTED:
-               reason = "target running";
-               goto bye;
-       default:
-               reason = "unrecognized error";
+               case ERROR_OK:
+                       break;
+               case ERROR_TARGET_RESOURCE_NOT_AVAILABLE:
+                       reason = "resource not available";
+                       goto bye;
+               case ERROR_TARGET_NOT_HALTED:
+                       reason = "target running";
+                       goto bye;
+               default:
+                       reason = "unrecognized error";
 bye:
-               LOG_ERROR("can't add %s watchpoint at 0x%8.8" PRIx32 ", %s",
-                        watchpoint_rw_strings[(*watchpoint_p)->rw],
-                        address, reason);
-               free (*watchpoint_p);
-               *watchpoint_p = NULL;
-               return retval;
+                       LOG_ERROR("can't add %s watchpoint at 0x%8.8" PRIx32 ", %s",
+                               watchpoint_rw_strings[(*watchpoint_p)->rw],
+                               address, reason);
+                       free(*watchpoint_p);
+                       *watchpoint_p = NULL;
+                       return retval;
        }
 
        LOG_DEBUG("added %s watchpoint at 0x%8.8" PRIx32
-                       " of length 0x%8.8" PRIx32 " (WPID: %d)",
-                       watchpoint_rw_strings[(*watchpoint_p)->rw],
-                       (*watchpoint_p)->address,
-                       (*watchpoint_p)->length,
-                       (*watchpoint_p)->unique_id );
+               " of length 0x%8.8" PRIx32 " (WPID: %d)",
+               watchpoint_rw_strings[(*watchpoint_p)->rw],
+               (*watchpoint_p)->address,
+               (*watchpoint_p)->length,
+               (*watchpoint_p)->unique_id);
 
        return ERROR_OK;
 }
 
-static void watchpoint_free(struct target *target, struct watchpoint *watchpoint_remove)
+static void watchpoint_free(struct target *target, struct watchpoint *watchpoint_to_remove)
 {
        struct watchpoint *watchpoint = target->watchpoints;
        struct watchpoint **watchpoint_p = &target->watchpoints;
+       int retval;
 
-       while (watchpoint)
-       {
-               if (watchpoint == watchpoint_remove)
+       while (watchpoint) {
+               if (watchpoint == watchpoint_to_remove)
                        break;
                watchpoint_p = &watchpoint->next;
                watchpoint = watchpoint->next;
@@ -260,8 +471,8 @@ static void watchpoint_free(struct target *target, struct watchpoint *watchpoint
 
        if (watchpoint == NULL)
                return;
-       target_remove_watchpoint(target, watchpoint);
-       LOG_DEBUG("WPID: %d", watchpoint->unique_id );
+       retval = target_remove_watchpoint(target, watchpoint);
+       LOG_DEBUG("free WPID: %d --> %d", watchpoint->unique_id, retval);
        (*watchpoint_p) = watchpoint->next;
        free(watchpoint);
 }
@@ -269,34 +480,42 @@ static void watchpoint_free(struct target *target, struct watchpoint *watchpoint
 void watchpoint_remove(struct target *target, uint32_t address)
 {
        struct watchpoint *watchpoint = target->watchpoints;
-       struct watchpoint **watchpoint_p = &target->watchpoints;
 
-       while (watchpoint)
-       {
+       while (watchpoint) {
                if (watchpoint->address == address)
                        break;
-               watchpoint_p = &watchpoint->next;
                watchpoint = watchpoint->next;
        }
 
        if (watchpoint)
-       {
                watchpoint_free(target, watchpoint);
-       }
        else
-       {
                LOG_ERROR("no watchpoint at address 0x%8.8" PRIx32 " found", address);
-       }
 }
 
 void watchpoint_clear_target(struct target *target)
 {
-       struct watchpoint *watchpoint;
-
        LOG_DEBUG("Delete all watchpoints for target: %s",
-                       target_name(target));
-       while ((watchpoint = target->watchpoints) != NULL)
-       {
-               watchpoint_free(target, watchpoint);
-       }
+               target_name(target));
+       while (target->watchpoints != NULL)
+               watchpoint_free(target, target->watchpoints);
+}
+
+int watchpoint_hit(struct target *target, enum watchpoint_rw *rw, uint32_t *address)
+{
+       int retval;
+       struct watchpoint *hit_watchpoint;
+
+       retval = target_hit_watchpoint(target, &hit_watchpoint);
+       if (retval != ERROR_OK)
+               return ERROR_FAIL;
+
+       *rw = hit_watchpoint->rw;
+       *address = hit_watchpoint->address;
+
+       LOG_DEBUG("Found hit watchpoint at 0x%8.8" PRIx32 " (WPID: %d)",
+               hit_watchpoint->address,
+               hit_watchpoint->unique_id);
+
+       return ERROR_OK;
 }

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)