Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/27243 )
Change subject: util: Pull argument parsing functions out of m5.c.
......................................................................
util: Pull argument parsing functions out of m5.c.
Make them available in other files as well.
Change-Id: I3ddaed1a06023f929acc95c90f8f9adda7ff429c
---
M util/m5/src/SConscript
A util/m5/src/args.c
A util/m5/src/args.h
M util/m5/src/m5.c
4 files changed, 153 insertions(+), 49 deletions(-)
diff --git a/util/m5/src/SConscript b/util/m5/src/SConscript
index 4ae2f6d..31de053 100644
--- a/util/m5/src/SConscript
+++ b/util/m5/src/SConscript
@@ -29,6 +29,7 @@
# Raw source files.
m5_mmap = 'm5_mmap.c'
+args = 'args.c'
m5 = 'm5.c'
jni = 'jni_gem5Op.c'
lua = 'lua_gem5Op.c'
@@ -61,7 +62,8 @@
# The m5 stand alone command line utility.
#
ct_support = list([ File('%s_call_type.c' % ct.name) for ct in call_types
])
-m5_bin = static_env.Program('out/m5', ct_support + [ m5, m5_mmap, libm5 ])
+m5_bin = static_env.Program('out/m5',
+ ct_support + [ args, m5, m5_mmap, libm5 ])
# The shared version of the m5 op call sights, used by mutliple targets
below.
diff --git a/util/m5/src/args.c b/util/m5/src/args.c
new file mode 100644
index 0000000..f1896c8
--- /dev/null
+++ b/util/m5/src/args.c
@@ -0,0 +1,86 @@
+/*
+ * Copyright (c) 2011, 2017 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder. You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
+ * Copyright (c) 2003-2005 The Regents of The University of Michigan
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include <inttypes.h>
+#include <stdlib.h>
+#include <string.h>
+
+int
+parse_int_args(int argc, char *argv[], uint64_t ints[], int len)
+{
+ if (argc > len)
+ return 0;
+
+// On 32 bit platforms we need to use strtoull to do the conversion
+#ifdef __LP64__
+#define strto64 strtoul
+#else
+#define strto64 strtoull
+#endif
+ int i;
+ for (i = 0; i < len; ++i)
+ ints[i] = (i < argc) ? strto64(argv[i], NULL, 0) : 0;
+
+#undef strto64
+ return 1;
+}
+
+int
+pack_str_into_regs(const char *str, uint64_t regs[], int num_regs)
+{
+ const size_t RegSize = sizeof(regs[0]);
+ const size_t MaxLen = num_regs * RegSize;
+
+ size_t len = strlen(str);
+
+ if (len > MaxLen)
+ return 0;
+
+ memset(regs, 0, MaxLen);
+
+ while (len) {
+ for (int offset = 0; offset < RegSize && len; offset++, len--) {
+ int shift = offset * 8;
+ *regs |= (uint64_t)(uint8_t)*str++ << shift;
+ }
+ regs++;
+ }
+ return 1;
+}
diff --git a/util/m5/src/args.h b/util/m5/src/args.h
new file mode 100644
index 0000000..530462e
--- /dev/null
+++ b/util/m5/src/args.h
@@ -0,0 +1,49 @@
+/*
+ * Copyright (c) 2011, 2017 ARM Limited
+ * All rights reserved
+ *
+ * The license below extends only to copyright in the software and shall
+ * not be construed as granting a license to any other intellectual
+ * property including but not limited to intellectual property relating
+ * to a hardware implementation of the functionality of the software
+ * licensed hereunder. You may use the software subject to the license
+ * terms below provided that you ensure that this notice is replicated
+ * unmodified and in its entirety in all distributions of the software,
+ * modified or unmodified, in source code or in binary form.
+ *
+ * Copyright (c) 2003-2005 The Regents of The University of Michigan
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifndef __ARGS_H__
+#define __ARGS_H__
+
+#include <stdint.h>
+
+int parse_int_args(int argc, char *argv[], uint64_t ints[], int len);
+int pack_str_into_regs(const char *str, uint64_t regs[], int num_regs);
+
+#endif // __ARGS_H__
diff --git a/util/m5/src/m5.c b/util/m5/src/m5.c
index 6ff15d2..5597966 100644
--- a/util/m5/src/m5.c
+++ b/util/m5/src/m5.c
@@ -52,6 +52,7 @@
#include <gem5/asm/generic/m5ops.h>
#include <gem5/m5ops.h>
+#include "args.h"
#include "call_type.h"
#include "dispatch_table.h"
@@ -59,47 +60,6 @@
char *command = "unspecified";
void usage();
-void
-parse_int_args(int argc, char *argv[], uint64_t ints[], int len)
-{
- if (argc > len)
- usage();
-
-// On 32 bit platforms we need to use strtoull to do the conversion
-#ifdef __LP64__
-#define strto64 strtoul
-#else
-#define strto64 strtoull
-#endif
- int i;
- for (i = 0; i < len; ++i)
- ints[i] = (i < argc) ? strto64(argv[i], NULL, 0) : 0;
-
-#undef strto64
-}
-
-void
-pack_str_into_regs(const char *str, uint64_t regs[], int num_regs)
-{
- const size_t RegSize = sizeof(regs[0]);
- const size_t MaxLen = num_regs * RegSize;
-
- size_t len = strlen(str);
-
- if (len > MaxLen)
- usage();
-
- memset(regs, 0, MaxLen);
-
- while (len) {
- for (int offset = 0; offset < RegSize && len; offset++, len--) {
- int shift = offset * 8;
- *regs |= (uint64_t)(uint8_t)*str++ << shift;
- }
- regs++;
- }
-}
-
int
read_file(DispatchTable *dt, int dest_fid)
{
@@ -168,7 +128,8 @@
usage();
uint64_t ints[1];
- parse_int_args(argc, argv, ints, 1);
+ if (!parse_int_args(argc, argv, ints, 1))
+ usage();
(*dt->m5_exit)(ints[0]);
}
@@ -179,7 +140,8 @@
usage();
uint64_t ints[2] = {0,0};
- parse_int_args(argc, argv, ints, argc);
+ if (!parse_int_args(argc, argv, ints, argc))
+ usage();
(*dt->m5_fail)(ints[1], ints[0]);
}
@@ -187,7 +149,8 @@
do_reset_stats(DispatchTable *dt, int argc, char *argv[])
{
uint64_t ints[2];
- parse_int_args(argc, argv, ints, 2);
+ if (!parse_int_args(argc, argv, ints, 2))
+ usage();
(*dt->m5_reset_stats)(ints[0], ints[1]);
}
@@ -195,7 +158,8 @@
do_dump_stats(DispatchTable *dt, int argc, char *argv[])
{
uint64_t ints[2];
- parse_int_args(argc, argv, ints, 2);
+ if (parse_int_args(argc, argv, ints, 2))
+ usage();
(*dt->m5_dump_stats)(ints[0], ints[1]);
}
@@ -203,7 +167,8 @@
do_dump_reset_stats(DispatchTable *dt, int argc, char *argv[])
{
uint64_t ints[2];
- parse_int_args(argc, argv, ints, 2);
+ if (!parse_int_args(argc, argv, ints, 2))
+ usage();
(*dt->m5_dump_reset_stats)(ints[0], ints[1]);
}
@@ -232,7 +197,8 @@
do_checkpoint(DispatchTable *dt, int argc, char *argv[])
{
uint64_t ints[2];
- parse_int_args(argc, argv, ints, 2);
+ if (!parse_int_args(argc, argv, ints, 2))
+ usage();
(*dt->m5_checkpoint)(ints[0], ints[1]);
}
@@ -264,7 +230,8 @@
usage();
uint64_t key_str[2];
- pack_str_into_regs(argc == 0 ? "" : argv[0], key_str, 2);
+ if (!pack_str_into_regs(argc == 0 ? "" : argv[0], key_str, 2))
+ usage();
uint64_t val = (*dt->m5_init_param)(key_str[0], key_str[1]);
printf("%"PRIu64, val);
}
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/27243
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I3ddaed1a06023f929acc95c90f8f9adda7ff429c
Gerrit-Change-Number: 27243
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabebl...@google.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev