Code Review / openocd.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
target/xtensa: avoid IHI for writes to non-executable memory
[openocd.git] / src / target / mem_ap.c
diff --git a/src/target/mem_ap.c b/src/target/mem_ap.c
index 584e19d4c9247aad7783cb2ce99a714b6a1ea731..61a9475e64a428d41b985dffa9f861a897e416b8 100644 (file)
--- a/src/target/mem_ap.c
+++ b/src/target/mem_ap.c
@@ -1,16 +1,8 @@
-/*****************************************************************************
- *   Copyright (C) 2016 by Matthias Welwarsky <matthias.welwarsky@sysgo.com> *
- *                                                                           *
- *   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       *
- *   (at your option) any later version.                                     *
- *                                                                           *
- *   This program is distributed in the hope that it will be useful,         *
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of          *
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the           *
- *   GNU General Public License for more details.                            *
- ****************************************************************************/
+// SPDX-License-Identifier: GPL-2.0-or-later
+
+/*
+ * Copyright (C) 2016 by Matthias Welwarsky <matthias.welwarsky@sysgo.com>
+ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -18,15 +10,18 @@
 
 #include "target.h"
 #include "target_type.h"
-#include "arm.h"
 #include "arm_adi_v5.h"
+#include "register.h"
 
 #include <jtag/jtag.h>
 
+#define MEM_AP_COMMON_MAGIC 0x4DE4DA50
+
 struct mem_ap {
-       struct arm arm;
+       int common_magic;
+       struct adiv5_dap *dap;
        struct adiv5_ap *ap;
-       int ap_num;
+       uint64_t ap_num;
 };
 
 static int mem_ap_target_create(struct target *target, Jim_Interp *interp)
@@ -35,7 +30,7 @@ static int mem_ap_target_create(struct target *target, Jim_Interp *interp)
        struct adiv5_private_config *pc;
 
        pc = (struct adiv5_private_config *)target->private_config;
-       if (pc == NULL)
+       if (!pc)
                return ERROR_FAIL;
 
        if (pc->ap_num == DP_APSEL_INVALID) {
@@ -44,17 +39,20 @@ static int mem_ap_target_create(struct target *target, Jim_Interp *interp)
        }
 
        mem_ap = calloc(1, sizeof(struct mem_ap));
-       if (mem_ap == NULL) {
+       if (!mem_ap) {
                LOG_ERROR("Out of memory");
                return ERROR_FAIL;
        }
 
        mem_ap->ap_num = pc->ap_num;
-       mem_ap->arm.common_magic = ARM_COMMON_MAGIC;
-       mem_ap->arm.dap = pc->dap;
+       mem_ap->common_magic = MEM_AP_COMMON_MAGIC;
+       mem_ap->dap = pc->dap;
 
        target->arch_info = mem_ap;
 
+       if (!target->gdb_port_override)
+               target->gdb_port_override = strdup("disabled");
+
        return ERROR_OK;
 }
 
@@ -62,9 +60,24 @@ static int mem_ap_init_target(struct command_context *cmd_ctx, struct target *ta
 {
        LOG_DEBUG("%s", __func__);
        target->state = TARGET_UNKNOWN;
+       target->debug_reason = DBG_REASON_UNDEFINED;
        return ERROR_OK;
 }
 
+static void mem_ap_deinit_target(struct target *target)
+{
+       struct mem_ap *mem_ap = target->arch_info;
+
+       LOG_DEBUG("%s", __func__);
+
+       if (mem_ap->ap)
+               dap_put_ap(mem_ap->ap);
+
+       free(target->private_config);
+       free(target->arch_info);
+       return;
+}
+
 static int mem_ap_arch_state(struct target *target)
 {
        LOG_DEBUG("%s", __func__);
@@ -73,8 +86,10 @@ static int mem_ap_arch_state(struct target *target)
 
 static int mem_ap_poll(struct target *target)
 {
-       if (target->state == TARGET_UNKNOWN)
+       if (target->state == TARGET_UNKNOWN) {
                target->state = TARGET_RUNNING;
+               target->debug_reason = DBG_REASON_NOTHALTED;
+       }
 
        return ERROR_OK;
 }
@@ -83,6 +98,8 @@ static int mem_ap_halt(struct target *target)
 {
        LOG_DEBUG("%s", __func__);
        target->state = TARGET_HALTED;
+       target->debug_reason = DBG_REASON_DBGRQ;
+       target_call_event_callbacks(target, TARGET_EVENT_HALTED);
        return ERROR_OK;
 }
 
@@ -91,6 +108,7 @@ static int mem_ap_resume(struct target *target, int current, target_addr_t addre
 {
        LOG_DEBUG("%s", __func__);
        target->state = TARGET_RUNNING;
+       target->debug_reason = DBG_REASON_NOTHALTED;
        return ERROR_OK;
 }
 
@@ -99,12 +117,15 @@ static int mem_ap_step(struct target *target, int current, target_addr_t address
 {
        LOG_DEBUG("%s", __func__);
        target->state = TARGET_HALTED;
+       target->debug_reason = DBG_REASON_DBGRQ;
+       target_call_event_callbacks(target, TARGET_EVENT_HALTED);
        return ERROR_OK;
 }
 
 static int mem_ap_assert_reset(struct target *target)
 {
        target->state = TARGET_RESET;
+       target->debug_reason = DBG_REASON_UNDEFINED;
 
        LOG_DEBUG("%s", __func__);
        return ERROR_OK;
@@ -115,9 +136,16 @@ static int mem_ap_examine(struct target *target)
        struct mem_ap *mem_ap = target->arch_info;
 
        if (!target_was_examined(target)) {
-               mem_ap->ap = dap_ap(mem_ap->arm.dap, mem_ap->ap_num);
+               if (!mem_ap->ap) {
+                       mem_ap->ap = dap_get_ap(mem_ap->dap, mem_ap->ap_num);
+                       if (!mem_ap->ap) {
+                               LOG_ERROR("Cannot get AP");
+                               return ERROR_FAIL;
+                       }
+               }
                target_set_examined(target);
                target->state = TARGET_UNKNOWN;
+               target->debug_reason = DBG_REASON_UNDEFINED;
                return mem_ap_init(mem_ap->ap);
        }
 
@@ -126,24 +154,94 @@ static int mem_ap_examine(struct target *target)
 
 static int mem_ap_deassert_reset(struct target *target)
 {
-       if (target->reset_halt)
+       if (target->reset_halt) {
                target->state = TARGET_HALTED;
-       else
+               target->debug_reason = DBG_REASON_DBGRQ;
+               target_call_event_callbacks(target, TARGET_EVENT_HALTED);
+       } else {
                target->state = TARGET_RUNNING;
+               target->debug_reason = DBG_REASON_NOTHALTED;
+       }
 
        LOG_DEBUG("%s", __func__);
        return ERROR_OK;
 }
 
+static int mem_ap_reg_get(struct reg *reg)
+{
+       return ERROR_OK;
+}
+
+static int mem_ap_reg_set(struct reg *reg, uint8_t *buf)
+{
+       return ERROR_OK;
+}
+
+static struct reg_arch_type mem_ap_reg_arch_type = {
+       .get = mem_ap_reg_get,
+       .set = mem_ap_reg_set,
+};
+
+static const char *mem_ap_get_gdb_arch(const struct target *target)
+{
+       return "arm";
+}
+
+/*
+ * Dummy ARM register emulation:
+ * reg[0..15]:  32 bits, r0~r12, sp, lr, pc
+ * reg[16..23]: 96 bits, f0~f7
+ * reg[24]:     32 bits, fps
+ * reg[25]:     32 bits, cpsr
+ *
+ * GDB requires only reg[0..15]
+ */
+#define NUM_REGS     26
+#define NUM_GDB_REGS 16
+#define MAX_REG_SIZE 96
+#define REG_SIZE(n)  ((((n) >= 16) && ((n) < 24)) ? 96 : 32)
+
+struct mem_ap_alloc_reg_list {
+       /* reg_list must be the first field */
+       struct reg *reg_list[NUM_REGS];
+       struct reg regs[NUM_REGS];
+       uint8_t regs_value[MAX_REG_SIZE / 8];
+};
+
+static int mem_ap_get_gdb_reg_list(struct target *target, struct reg **reg_list[],
+                               int *reg_list_size, enum target_register_class reg_class)
+{
+       struct mem_ap_alloc_reg_list *mem_ap_alloc = calloc(1, sizeof(struct mem_ap_alloc_reg_list));
+       if (!mem_ap_alloc) {
+               LOG_ERROR("Out of memory");
+               return ERROR_FAIL;
+       }
+
+       *reg_list = mem_ap_alloc->reg_list;
+       *reg_list_size = (reg_class == REG_CLASS_ALL) ? NUM_REGS : NUM_GDB_REGS;
+       struct reg *regs = mem_ap_alloc->regs;
+
+       for (int i = 0; i < NUM_REGS; i++) {
+               regs[i].number = i;
+               regs[i].value = mem_ap_alloc->regs_value;
+               regs[i].size = REG_SIZE(i);
+               regs[i].exist = true;
+               regs[i].type = &mem_ap_reg_arch_type;
+               (*reg_list)[i] = &regs[i];
+       }
+
+       return ERROR_OK;
+}
+
 static int mem_ap_read_memory(struct target *target, target_addr_t address,
                               uint32_t size, uint32_t count, uint8_t *buffer)
 {
        struct mem_ap *mem_ap = target->arch_info;
 
-       LOG_DEBUG("Reading memory at physical address 0x" TARGET_ADDR_FMT
-                 "; size %" PRId32 "; count %" PRId32, address, size, count);
+       LOG_DEBUG("Reading memory at physical address " TARGET_ADDR_FMT
+                 "; size %" PRIu32 "; count %" PRIu32, address, size, count);
 
-       if (count == 0 || buffer == NULL)
+       if (count == 0 || !buffer)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        return mem_ap_read_buf(mem_ap->ap, buffer, size, count, address);
@@ -155,10 +253,10 @@ static int mem_ap_write_memory(struct target *target, target_addr_t address,
 {
        struct mem_ap *mem_ap = target->arch_info;
 
-       LOG_DEBUG("Writing memory at physical address 0x" TARGET_ADDR_FMT
-                 "; size %" PRId32 "; count %" PRId32, address, size, count);
+       LOG_DEBUG("Writing memory at physical address " TARGET_ADDR_FMT
+                 "; size %" PRIu32 "; count %" PRIu32, address, size, count);
 
-       if (count == 0 || buffer == NULL)
+       if (count == 0 || !buffer)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        return mem_ap_write_buf(mem_ap->ap, buffer, size, count, address);
@@ -169,6 +267,7 @@ struct target_type mem_ap_target = {
 
        .target_create = mem_ap_target_create,
        .init_target = mem_ap_init_target,
+       .deinit_target = mem_ap_deinit_target,
        .examine = mem_ap_examine,
        .target_jim_configure = adiv5_jim_configure,
 
@@ -182,6 +281,9 @@ struct target_type mem_ap_target = {
        .assert_reset = mem_ap_assert_reset,
        .deassert_reset = mem_ap_deassert_reset,
 
+       .get_gdb_arch = mem_ap_get_gdb_arch,
+       .get_gdb_reg_list = mem_ap_get_gdb_reg_list,
+
        .read_memory = mem_ap_read_memory,
        .write_memory = mem_ap_write_memory,
 };
Open On-Chip Debugger

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)