From: Andreas Sandberg <andreas.sandberg@arm.com>
Date: Wed, 26 Apr 2017 16:44:23 +0000 (+0000)
Subject: util: Fix incorrect use of m5_loadsymbol
X-Git-Tag: v19.0.0.0~2818
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d43d59595da7e8bf7be522d06ab9dd0943c3c03a;p=gem5.git

util: Fix incorrect use of m5_loadsymbol

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>
---

diff --git a/util/m5/m5.c b/util/m5/m5.c
index 8e6f4fc59..ee921347d 100644
--- a/util/m5/m5.c
+++ b/util/m5/m5.c
@@ -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