From 3cabaf66d6ec7343bd3badc5c44c2ddd06221913 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Thu, 24 Dec 2015 16:41:55 -0500 Subject: [PATCH] sim: delete SIM_HAVE_FLATMEM support No target has used this, and it's a cheap hack in place in using the common memory module. We want everyone using that though, so drop support for flatmem entirely. --- sim/common/ChangeLog | 9 +++++++++ sim/common/sim-base.h | 10 ---------- sim/common/sim-module.c | 3 --- sim/common/sim-options.c | 38 -------------------------------------- sim/mips/ChangeLog | 4 ++++ sim/mips/interp.c | 3 --- 6 files changed, 13 insertions(+), 54 deletions(-) diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog index 10ad02930ba..80fc1644c6a 100644 --- a/sim/common/ChangeLog +++ b/sim/common/ChangeLog @@ -1,3 +1,12 @@ +2015-12-24 Mike Frysinger + + * sim-base.h [SIM_HAVE_FLATMEM] (sim_state_base): Delete flatmem code. + * sim-module.c [SIM_HAVE_FLATMEM] (modules): Always call + sim_memopt_install. + * sim-options.c (OPTION_MEM_SIZE): Delete. + [SIM_HAVE_FLATMEM] (standard_options): Delete flatmem code. + (standard_option_handler): Likewise. + 2015-12-24 Mike Frysinger * tconfig.h (SIM_HAVE_SIMCACHE): Delete. diff --git a/sim/common/sim-base.h b/sim/common/sim-base.h index 21f61f4dc1f..cfef6ecddc0 100644 --- a/sim/common/sim-base.h +++ b/sim/common/sim-base.h @@ -202,16 +202,6 @@ typedef struct { unsigned int scache_size; #define STATE_SCACHE_SIZE(sd) ((sd)->base.scache_size) - /* FIXME: Move to top level sim_state struct (as some struct)? */ -#ifdef SIM_HAVE_FLATMEM - unsigned int mem_size; -#define STATE_MEM_SIZE(sd) ((sd)->base.mem_size) - unsigned int mem_base; -#define STATE_MEM_BASE(sd) ((sd)->base.mem_base) - unsigned char *memory; -#define STATE_MEMORY(sd) ((sd)->base.memory) -#endif - /* core memory bus */ #define STATE_CORE(sd) (&(sd)->base.core) sim_core core; diff --git a/sim/common/sim-module.c b/sim/common/sim-module.c index 2d8e7dc3729..afadb07b796 100644 --- a/sim/common/sim-module.c +++ b/sim/common/sim-module.c @@ -53,10 +53,7 @@ static MODULE_INSTALL_FN * const modules[] = { profile_install, #endif sim_core_install, -#ifndef SIM_HAVE_FLATMEM - /* FIXME: should handle flatmem as well FLATMEM */ sim_memopt_install, -#endif #if WITH_WATCHPOINTS sim_watchpoint_install, #endif diff --git a/sim/common/sim-options.c b/sim/common/sim-options.c index f8231430441..5e04573e0c6 100644 --- a/sim/common/sim-options.c +++ b/sim/common/sim-options.c @@ -102,9 +102,6 @@ typedef enum { OPTION_VERBOSE, OPTION_ENDIAN, OPTION_DEBUG, -#ifdef SIM_HAVE_FLATMEM - OPTION_MEM_SIZE, -#endif OPTION_HELP, OPTION_VERSION, #ifdef SIM_H8300 /* FIXME: Should be movable to h8300 dir. */ @@ -162,12 +159,6 @@ static const OPTION standard_options[] = standard_option_handler }, #endif -#ifdef SIM_HAVE_FLATMEM - { {"mem-size", required_argument, NULL, OPTION_MEM_SIZE}, - 'm', "[in bytes, Kb (k suffix), Mb (m suffix) or Gb (g suffix)]", - "Specify memory size", standard_option_handler }, -#endif - { {"do-command", required_argument, NULL, OPTION_DO_COMMAND}, '\0', "COMMAND", ""/*undocumented*/, standard_option_handler }, @@ -376,35 +367,6 @@ standard_option_handler (SIM_DESC sd, sim_cpu *cpu, int opt, break; #endif -#ifdef SIM_HAVE_FLATMEM - case OPTION_MEM_SIZE: - { - char * endp; - unsigned long ul = strtol (arg, &endp, 0); - - switch (* endp) - { - case 'k': case 'K': size <<= 10; break; - case 'm': case 'M': size <<= 20; break; - case 'g': case 'G': size <<= 30; break; - case ' ': case '\0': case '\t': break; - default: - if (ul > 0) - sim_io_eprintf (sd, "Ignoring strange character at end of memory size: %c\n", * endp); - break; - } - - /* 16384: some minimal amount */ - if (! isdigit (arg[0]) || ul < 16384) - { - sim_io_eprintf (sd, "Invalid memory size `%s'", arg); - return SIM_RC_FAIL; - } - STATE_MEM_SIZE (sd) = ul; - } - break; -#endif - case OPTION_DO_COMMAND: sim_do_command (sd, arg); break; diff --git a/sim/mips/ChangeLog b/sim/mips/ChangeLog index b48f69f38bb..34ab06d0442 100644 --- a/sim/mips/ChangeLog +++ b/sim/mips/ChangeLog @@ -1,3 +1,7 @@ +2015-12-24 Mike Frysinger + + * interp.c [SIM_HAVE_FLATMEM] (sim_open): Delete flatmem code. + 2015-12-24 Mike Frysinger * tconfig.h (SIM_HAVE_SIMCACHE): Delete. diff --git a/sim/mips/interp.c b/sim/mips/interp.c index 8a584d3cd24..524f5bec8e4 100644 --- a/sim/mips/interp.c +++ b/sim/mips/interp.c @@ -399,9 +399,6 @@ sim_open (SIM_OPEN_KIND kind, host_callback *cb, struct bfd *abfd, char **argv) /* Look for largest memory region defined on command-line at phys address 0. */ -#ifdef SIM_HAVE_FLATMEM - mem_size = STATE_MEM_SIZE (sd); -#endif for (entry = STATE_MEMOPT (sd); entry != NULL; entry = entry->next) { /* If we find an entry at address 0, then we will end up