diff options
| author | Andreas Sandberg <andreas.sandberg@arm.com> | 2017-04-26 16:44:23 +0000 |
|---|---|---|
| committer | Andreas Sandberg <andreas.sandberg@arm.com> | 2017-05-09 09:24:06 +0000 |
| commit | d43d59595da7e8bf7be522d06ab9dd0943c3c03a (patch) | |
| tree | ed7dee7e977ca9c067f7b578f7c71f33bcfd9d1f /util | |
| parent | b525d9509e68cd510b5d00b33ad98ae59338f291 (diff) | |
| download | gem5-d43d59595da7e8bf7be522d06ab9dd0943c3c03a.tar.xz | |
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>
Diffstat (limited to 'util')
| -rw-r--r-- | util/m5/m5.c | 6 |
1 files changed, 3 insertions, 3 deletions
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 |