On 2/11/20 1:39 AM, Taylor Simpson wrote:
The Hexagon disassembler calls disassemble_hexagon to decode a packet
and format it for printing

Signed-off-by: Taylor Simpson <tsimp...@quicinc.com>
---
  disas/Makefile.objs     |  1 +
  disas/hexagon.c         | 56 +++++++++++++++++++++++++++++++++++++++++++++++++
  include/disas/dis-asm.h |  1 +
  3 files changed, 58 insertions(+)
  create mode 100644 disas/hexagon.c

diff --git a/disas/Makefile.objs b/disas/Makefile.objs
index 3c1cdce..0e86964 100644
--- a/disas/Makefile.objs
+++ b/disas/Makefile.objs
@@ -24,6 +24,7 @@ common-obj-$(CONFIG_SH4_DIS) += sh4.o
  common-obj-$(CONFIG_SPARC_DIS) += sparc.o
  common-obj-$(CONFIG_LM32_DIS) += lm32.o
  common-obj-$(CONFIG_XTENSA_DIS) += xtensa.o
+common-obj-$(CONFIG_HEXAGON_DIS) += hexagon.o
# TODO: As long as the TCG interpreter and its generated code depend
  # on the QEMU target, we cannot compile the disassembler here.
diff --git a/disas/hexagon.c b/disas/hexagon.c
new file mode 100644
index 0000000..fd9af10
--- /dev/null
+++ b/disas/hexagon.c
@@ -0,0 +1,56 @@
+/*
+ *  Copyright (c) 2019 Qualcomm Innovation Center, Inc. All Rights Reserved.
+ *
+ *  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.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program; if not, see <http://www.gnu.org/licenses/>.
+ */
+
+/*
+ * QEMU Hexagon Disassembler
+ */
+
+#include "qemu/osdep.h"
+#include "disas/dis-asm.h"
+#include "target/hexagon/cpu_bits.h"
+
+int print_insn_hexagon(bfd_vma memaddr, struct disassemble_info *info)
+{
+    uint32_t words[PACKET_WORDS_MAX];
+    int len, slen;
+    char buf[1028];

Why 1028?

+    int status;
+    int i;
+
+    for (i = 0; i < PACKET_WORDS_MAX; i++) {
+        status = (*info->read_memory_func)(memaddr + i * sizeof(uint32_t),
+                                           (bfd_byte *)&words[i],
+                                           sizeof(uint32_t), info);
+        if (status) {
+            if (i > 0) {
+                break;
+            }
+            (*info->memory_error_func)(status, memaddr, info);
+            return status;
+        }
+    }
+
+    len = disassemble_hexagon(words, i, buf, 1028);

1028 -> sizeof(buf)

+    slen = strlen(buf);
+    if (buf[slen - 1] == '\n') {
+        buf[slen - 1] = '\0';
+    }
+    (*info->fprintf_func)(info->stream, "%s", buf);
+
+    return len;
+}
+
diff --git a/include/disas/dis-asm.h b/include/disas/dis-asm.h
index f87f468..a7fa82d 100644
--- a/include/disas/dis-asm.h
+++ b/include/disas/dis-asm.h
@@ -436,6 +436,7 @@ int print_insn_little_nios2     (bfd_vma, 
disassemble_info*);
  int print_insn_xtensa           (bfd_vma, disassemble_info*);
  int print_insn_riscv32          (bfd_vma, disassemble_info*);
  int print_insn_riscv64          (bfd_vma, disassemble_info*);
+int print_insn_hexagon          (bfd_vma, disassemble_info*);
#if 0
  /* Fetch the disassembler for a given BFD, if that support is available.  */



Reply via email to