From: Mike Frysinger Date: Sun, 12 Apr 2015 09:49:31 +0000 (-0400) Subject: sim: microblaze: switch to common memory functions X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c85fc61074e9fab1bafbae97ea4628c7bffaaf76;p=binutils-gdb.git sim: microblaze: switch to common memory functions Re-use the existing memory core that handles reads/writes. This drops support for the dumpmem command, but gdb itself has support for dumping memory regions. The verbose command is converted to the common --verbose flag since only two call sites use it now. Support for the clearstats command is dropped entirely, but no other sim really does this, and the same thing can be done by reloading. If it's important (clearing cycle stats) to someone, we can add a common function for it. --- diff --git a/sim/microblaze/ChangeLog b/sim/microblaze/ChangeLog index 8ac3a262a6e..7bfb6577000 100644 --- a/sim/microblaze/ChangeLog +++ b/sim/microblaze/ChangeLog @@ -1,3 +1,23 @@ +2015-04-17 Mike Frysinger + + * interp.c (issue_messages, wbat, wlat, what, rbat, rlat, rhat, + sim_memory_size, MEM_SIZE_FLOOR, sim_size, init_pointers, sim_write, + sim_read, sim_do_command): Delete. + (set_initial_gprs): Delete memsize and call to init_pointers. + (sim_resume): Change rlat to MEM_RD_WORD. Change issue_messages to + STATE_VERBOSE_P. + (sim_store_register, sim_fetch_register): Drop call to init_pointers. + (sim_open): Drop issue_messages setup. Delete osize and call to + sim_size. Call sim_do_commandf to set the memory size. + * microblaze.h (MEM): Delete. + (MEM_RD_BYTE): Change rbat to sim_core_read_1. + (MEM_RD_HALF): Change rhat to sim_core_read_2. + (MEM_RD_WORD): Change rlat to sim_core_read_4. + (MEM_WR_BYTE): Change wbat to sim_core_write_1. + (MEM_WR_HALF): Change what to sim_core_write_2. + (MEM_WR_WORD): Change wlat to sim_core_write_4. + * sim-main.h (microblaze_regset): Delete msize and memory. + 2015-04-17 Mike Frysinger * sim-main.h (CIA_GET, CIA_SET): Delete. diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c index aa8634e3d88..e05c107dc6c 100644 --- a/sim/microblaze/interp.c +++ b/sim/microblaze/interp.c @@ -89,240 +89,16 @@ microblaze_store_unsigned_integer (unsigned char *addr, int len, } } -/* TODO: Convert to common tracing framework. */ -static int issue_messages = 0; - -static void /* INLINE */ -wbat (SIM_CPU *cpu, word x, word v) -{ - if (((uword)x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "byte write to 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - } - else - { - unsigned char *p = CPU.memory + x; - p[0] = v; - } -} - -static void /* INLINE */ -wlat (SIM_CPU *cpu, word x, word v) -{ - if (((uword)x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "word write to 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - } - else - { - if ((x & 3) != 0) - { - if (issue_messages) - fprintf (stderr, "word write to unaligned memory address: 0x%x\n", x); - - CPU.exception = SIGBUS; - } - else if (!target_big_endian) - { - unsigned char *p = CPU.memory + x; - p[3] = v >> 24; - p[2] = v >> 16; - p[1] = v >> 8; - p[0] = v; - } - else - { - unsigned char *p = CPU.memory + x; - p[0] = v >> 24; - p[1] = v >> 16; - p[2] = v >> 8; - p[3] = v; - } - } -} - -static void /* INLINE */ -what (SIM_CPU *cpu, word x, word v) -{ - if (((uword)x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "short write to 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - } - else - { - if ((x & 1) != 0) - { - if (issue_messages) - fprintf (stderr, "short write to unaligned memory address: 0x%x\n", - x); - - CPU.exception = SIGBUS; - } - else if (!target_big_endian) - { - unsigned char *p = CPU.memory + x; - p[1] = v >> 8; - p[0] = v; - } - else - { - unsigned char *p = CPU.memory + x; - p[0] = v >> 8; - p[1] = v; - } - } -} - -/* Read functions. */ -static int /* INLINE */ -rbat (SIM_CPU *cpu, word x) -{ - if (((uword)x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "byte read from 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - return 0; - } - else - { - unsigned char *p = CPU.memory + x; - return p[0]; - } -} - -static int /* INLINE */ -rlat (SIM_CPU *cpu, word x) -{ - if (((uword) x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "word read from 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - return 0; - } - else - { - if ((x & 3) != 0) - { - if (issue_messages) - fprintf (stderr, "word read from unaligned address: 0x%x\n", x); - - CPU.exception = SIGBUS; - return 0; - } - else if (! target_big_endian) - { - unsigned char *p = CPU.memory + x; - return (p[3] << 24) | (p[2] << 16) | (p[1] << 8) | p[0]; - } - else - { - unsigned char *p = CPU.memory + x; - return (p[0] << 24) | (p[1] << 16) | (p[2] << 8) | p[3]; - } - } -} - -static int /* INLINE */ -rhat (SIM_CPU *cpu, word x) -{ - if (((uword)x) >= CPU.msize) - { - if (issue_messages) - fprintf (stderr, "short read from 0x%x outside memory range\n", x); - - CPU.exception = SIGSEGV; - return 0; - } - else - { - if ((x & 1) != 0) - { - if (issue_messages) - fprintf (stderr, "short read from unaligned address: 0x%x\n", x); - - CPU.exception = SIGBUS; - return 0; - } - else if (!target_big_endian) - { - unsigned char *p = CPU.memory + x; - return (p[1] << 8) | p[0]; - } - else - { - unsigned char *p = CPU.memory + x; - return (p[0] << 8) | p[1]; - } - } -} - -/* TODO: Delete all sim_size and use common memory functions. */ -/* Default to a 8 Mbyte (== 2^23) memory space. */ -static int sim_memory_size = 1 << 23; - -#define MEM_SIZE_FLOOR 64 -static void -sim_size (SIM_CPU *cpu, int size) -{ - sim_memory_size = size; - CPU.msize = sim_memory_size; - - if (CPU.memory) - free (CPU.memory); - - CPU.memory = (unsigned char *) calloc (1, CPU.msize); - - if (!CPU.memory) - { - if (issue_messages) - fprintf (stderr, - "Not enough VM for simulation of %ld bytes of RAM\n", - CPU.msize); - - CPU.msize = 1; - CPU.memory = (unsigned char *) calloc (1, 1); - } -} - -static void -init_pointers (SIM_CPU *cpu) -{ - if (CPU.msize != (sim_memory_size)) - sim_size (cpu, sim_memory_size); -} - static void set_initial_gprs (SIM_CPU *cpu) { int i; long space; - unsigned long memsize; - - init_pointers (cpu); /* Set up machine just out of reset. */ PC = 0; MSR = 0; - memsize = CPU.msize / (1024 * 1024); - - if (issue_messages > 1) - fprintf (stderr, "Simulated memory of %ld Mbytes (0x0 .. 0x%08lx)\n", - memsize, CPU.msize - 1); - /* Clean out the GPRs */ for (i = 0; i < 32; i++) CPU.regs[i] = 0; @@ -365,7 +141,7 @@ sim_resume (SIM_DESC sd, int step, int siggnal) do { /* Fetch the initial instructions that we'll decode. */ - inst = rlat (cpu, PC & 0xFFFFFFFC); + inst = MEM_RD_WORD (PC & 0xFFFFFFFC); op = get_insn_microblaze (inst, &imm_unsigned, &insn_type, &num_delay_slot); @@ -439,7 +215,7 @@ sim_resume (SIM_DESC sd, int step, int siggnal) { newpc = PC; PC = oldpc + INST_SIZE; - inst = rlat (cpu, PC & 0xFFFFFFFC); + inst = MEM_RD_WORD (PC & 0xFFFFFFFC); op = get_insn_microblaze (inst, &imm_unsigned, &insn_type, &num_delay_slot); if (op == invalid_inst) @@ -452,14 +228,14 @@ sim_resume (SIM_DESC sd, int step, int siggnal) /* immword = IMM_W; */ if (op == microblaze_brk) { - if (issue_messages) + if (STATE_VERBOSE_P (sd)) fprintf (stderr, "Breakpoint set in delay slot " "(at address 0x%x) will not be honored\n", PC); /* ignore the breakpoint */ } else if (insn_type == branch_inst || insn_type == return_inst) { - if (issue_messages) + if (STATE_VERBOSE_P (sd)) fprintf (stderr, "Cannot have branch or return instructions " "in delay slot (at address 0x%x)\n", PC); CPU.exception = SIGILL; @@ -522,41 +298,11 @@ sim_resume (SIM_DESC sd, int step, int siggnal) CPU.cycles += memops; /* and memop cycle delays */ } - -int -sim_write (SIM_DESC sd, SIM_ADDR addr, const unsigned char *buffer, int size) -{ - SIM_CPU *cpu = STATE_CPU (sd, 0); - int i; - - init_pointers (cpu); - - memcpy (&CPU.memory[addr], buffer, size); - - return size; -} - -int -sim_read (SIM_DESC sd, SIM_ADDR addr, unsigned char *buffer, int size) -{ - SIM_CPU *cpu = STATE_CPU (sd, 0); - int i; - - init_pointers (cpu); - - memcpy (buffer, &CPU.memory[addr], size); - - return size; -} - - int sim_store_register (SIM_DESC sd, int rn, unsigned char *memory, int length) { SIM_CPU *cpu = STATE_CPU (sd, 0); - init_pointers (cpu); - if (rn < NUM_REGS + NUM_SPECIAL && rn >= 0) { if (length == 4) @@ -582,8 +328,6 @@ sim_fetch_register (SIM_DESC sd, int rn, unsigned char *memory, int length) SIM_CPU *cpu = STATE_CPU (sd, 0); long ival; - init_pointers (cpu); - if (rn < NUM_REGS + NUM_SPECIAL && rn >= 0) { if (length == 4) @@ -709,25 +453,20 @@ sim_open (SIM_OPEN_KIND kind, host_callback *cb, struct bfd *abfd, char **argv) return 0; } - if (kind == SIM_OPEN_STANDALONE) - issue_messages = 1; - /* CPU specific initialization. */ for (i = 0; i < MAX_NR_PROCESSORS; ++i) { SIM_CPU *cpu = STATE_CPU (sd, i); - int osize = sim_memory_size; CPU_PC_FETCH (cpu) = microblaze_pc_get; CPU_PC_STORE (cpu) = microblaze_pc_set; set_initial_gprs (cpu); - - /* Discard and reacquire memory -- start with a clean slate. */ - sim_size (cpu, 1); /* small */ - sim_size (cpu, osize); /* and back again */ } + /* Default to a 8 Mbyte (== 2^23) memory space. */ + sim_do_commandf (sd, "memory-size 0x800000"); + return sd; } @@ -746,57 +485,3 @@ sim_create_inferior (SIM_DESC sd, struct bfd *prog_bfd, char **argv, char **env) return SIM_RC_OK; } - -void -sim_do_command (SIM_DESC sd, const char *cmd) -{ - SIM_CPU *cpu = STATE_CPU (sd, 0); - - /* Nothing there yet; it's all an error. */ - - if (cmd != NULL) - { - char ** simargv = buildargv (cmd); - - if (strcmp (simargv[0], "dumpmem") == 0) - { - unsigned char * p; - FILE * dumpfile; - - if (simargv[1] == NULL) - fprintf (stderr, "Error: missing argument to dumpmem cmd.\n"); - - fprintf (stderr, "Writing dumpfile %s...",simargv[1]); - - dumpfile = fopen (simargv[1], "w"); - p = CPU.memory; - fwrite (p, CPU.msize-1, 1, dumpfile); - fclose (dumpfile); - - fprintf (stderr, "done.\n"); - } - else if (strcmp (simargv[0], "clearstats") == 0) - { - CPU.cycles = 0; - CPU.insts = 0; - } - else if (strcmp (simargv[0], "verbose") == 0) - { - issue_messages = 2; - } - else - { - fprintf (stderr,"Error: \"%s\" is not a valid M.CORE simulator command.\n", - cmd); - } - - freeargv (simargv); - } - else - { - fprintf (stderr, "M.CORE sim commands: \n"); - fprintf (stderr, " dumpmem \n"); - fprintf (stderr, " clearstats\n"); - fprintf (stderr, " verbose\n"); - } -} diff --git a/sim/microblaze/microblaze.h b/sim/microblaze/microblaze.h index e08f5a0cf8e..6b10d56cfc0 100644 --- a/sim/microblaze/microblaze.h +++ b/sim/microblaze/microblaze.h @@ -52,18 +52,16 @@ #define RETREG CPU.regs[3] -#define MEM(X) memory[X] - -#define MEM_RD_BYTE(X) rbat(cpu, X) -#define MEM_RD_HALF(X) rhat(cpu, X) -#define MEM_RD_WORD(X) rlat(cpu, X) +#define MEM_RD_BYTE(X) sim_core_read_1 (cpu, 0, read_map, X) +#define MEM_RD_HALF(X) sim_core_read_2 (cpu, 0, read_map, X) +#define MEM_RD_WORD(X) sim_core_read_4 (cpu, 0, read_map, X) #define MEM_RD_UBYTE(X) (ubyte) MEM_RD_BYTE(X) #define MEM_RD_UHALF(X) (uhalf) MEM_RD_HALF(X) #define MEM_RD_UWORD(X) (uword) MEM_RD_WORD(X) -#define MEM_WR_BYTE(X, D) wbat(cpu, X, D) -#define MEM_WR_HALF(X, D) what(cpu, X, D) -#define MEM_WR_WORD(X, D) wlat(cpu, X, D) +#define MEM_WR_BYTE(X, D) sim_core_write_1 (cpu, 0, write_map, X, D) +#define MEM_WR_HALF(X, D) sim_core_write_2 (cpu, 0, write_map, X, D) +#define MEM_WR_WORD(X, D) sim_core_write_4 (cpu, 0, write_map, X, D) #define MICROBLAZE_SEXT8(X) ((char) X) diff --git a/sim/microblaze/sim-main.h b/sim/microblaze/sim-main.h index 1b34f3a0176..e5d2b15754e 100644 --- a/sim/microblaze/sim-main.h +++ b/sim/microblaze/sim-main.h @@ -45,8 +45,6 @@ typedef struct _sim_cpu SIM_CPU; int cycles; int insts; int exception; - unsigned long msize; - unsigned char *memory; ubyte imm_enable; half imm_high; };