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 29229588b05f48165d97bb39d6e5444d01fd9ec3..61a9475e64a428d41b985dffa9f861a897e416b8 100644 (file)
--- a/src/target/mem_ap.c
+++ b/src/target/mem_ap.c
@@ -1,4 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
+// SPDX-License-Identifier: GPL-2.0-or-later
 
 /*
  * Copyright (C) 2016 by Matthias Welwarsky <matthias.welwarsky@sysgo.com>
@@ -136,15 +136,12 @@ static int mem_ap_examine(struct target *target)
        struct mem_ap *mem_ap = target->arch_info;
 
        if (!target_was_examined(target)) {
-               if (mem_ap->ap) {
-                       dap_put_ap(mem_ap->ap);
-                       mem_ap->ap = NULL;
-               }
-
-               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;
+                       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;
@@ -185,7 +182,7 @@ static struct reg_arch_type mem_ap_reg_arch_type = {
        .set = mem_ap_reg_set,
 };
 
-const char *mem_ap_get_gdb_arch(struct target *target)
+static const char *mem_ap_get_gdb_arch(const struct target *target)
 {
        return "arm";
 }
@@ -197,11 +194,11 @@ const char *mem_ap_get_gdb_arch(struct target *target)
  * reg[24]:     32 bits, fps
  * reg[25]:     32 bits, cpsr
  *
- * Set 'exist' only to reg[0..15], so initial response to GDB is correct
+ * GDB requires only reg[0..15]
  */
 #define NUM_REGS     26
+#define NUM_GDB_REGS 16
 #define MAX_REG_SIZE 96
-#define REG_EXIST(n) ((n) < 16)
 #define REG_SIZE(n)  ((((n) >= 16) && ((n) < 24)) ? 96 : 32)
 
 struct mem_ap_alloc_reg_list {
@@ -221,14 +218,14 @@ static int mem_ap_get_gdb_reg_list(struct target *target, struct reg **reg_list[
        }
 
        *reg_list = mem_ap_alloc->reg_list;
-       *reg_list_size = NUM_REGS;
+       *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 = REG_EXIST(i);
+               regs[i].exist = true;
                regs[i].type = &mem_ap_reg_arch_type;
                (*reg_list)[i] = &regs[i];
        }