util: Fix incorrect use of m5_loadsymbol
authorAndreas Sandberg <andreas.sandberg@arm.com>
Wed, 26 Apr 2017 16:44:23 +0000 (16:44 +0000)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Tue, 9 May 2017 09:24:06 +0000 (09:24 +0000)
It seems like the m5 utility incorrectly called m5_loadsymbol instead
of m5_addsymbol. Judging by the signature of the loadsymbol command,
the expected behavior is to add a new symbol to gem5's symbol
table. This is behavior is implemented by m5_addsymbol.

Change-Id: I83b61c48d6f8d7b1e8b57d884dfca00481c83c3a
Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-by: Curtis Dunham <curtis.dunham@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/2961
Reviewed-by: Gabe Black <gabeblack@google.com>
util/m5/m5.c

index 8e6f4fc59bc74fe4f6708fc3c05fe841c2e6d075..ee921347d433fb3373063d386618620d2ba9e374 100644 (file)
@@ -241,14 +241,14 @@ do_checkpoint(int argc, char *argv[])
 }
 
 void
-do_load_symbol(int argc, char *argv[])
+do_addsymbol(int argc, char *argv[])
 {
     if (argc != 2)
         usage();
 
     uint64_t addr = strtoul(argv[0], NULL, 0);
     char *symbol = argv[1];
-    m5_loadsymbol(addr, symbol);
+    m5_addsymbol(addr, symbol);
 }
 
 void
@@ -320,7 +320,7 @@ struct MainFunc mainfuncs[] = {
     { "writefile",      do_write_file,       "<filename>" },
     { "execfile",       do_exec_file,        "" },
     { "checkpoint",     do_checkpoint,       "[delay [period]]" },
-    { "loadsymbol",     do_load_symbol,      "<address> <symbol>" },
+    { "addsymbol",      do_addsymbol,        "<address> <symbol>" },
     { "initparam",      do_initparam,        "[key] // key must be shorter than 16 chars" },
     { "sw99param",      do_sw99param,        "" },
 #ifdef linux