sim: Get rid of the Arguments class.
authorGabe Black <gabeblack@google.com>
Sat, 28 Dec 2019 09:05:57 +0000 (01:05 -0800)
committerGabe Black <gabeblack@google.com>
Fri, 20 Mar 2020 10:04:18 +0000 (10:04 +0000)
This class read arguments using the arch specific getArgument function
and then presented the arguments as an array. The problem with that
approach is that it's not possible to tell where different arguments
are without knowing the types of previous arguments, and not all
arguments can be simply represented as a native sized integer.

This class has been phased out and is no longer needed.

Change-Id: Ibb4c529fe8c51fd0ae15ed3b6ea30543ad9c23e0
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/24115
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/arch/mips/linux/system.cc
src/base/cp_annotate.cc
src/kern/freebsd/events.cc
src/kern/linux/events.cc
src/sim/SConscript
src/sim/arguments.cc [deleted file]
src/sim/arguments.hh [deleted file]

index cff1cefcf8c650b6368fef37dce8e245798a83df..b2a2a4d7fcef1ae59de73637c9894a8a9e664eac 100644 (file)
@@ -49,7 +49,6 @@
 #include "kern/linux/printk.hh"
 #include "mem/physical.hh"
 #include "mem/port.hh"
-#include "sim/arguments.hh"
 #include "sim/byteswap.hh"
 
 using namespace std;
index 472efa7e90c824e244e000900177f8f0feeb9e18..59c32ab83869a34244e1dcb85b9f87faab75a2bd 100644 (file)
@@ -38,7 +38,6 @@
 #include "cpu/thread_context.hh"
 #include "debug/Annotate.hh"
 #include "debug/AnnotateVerbose.hh"
-#include "sim/arguments.hh"
 #include "sim/core.hh"
 #include "sim/sim_exit.hh"
 #include "sim/system.hh"
index 1367e5e83f32bf3f7bf68a16e36efcab285ff73d..0c4c613291ce596e1ff559bc73cd8bc4d0ee65c3 100644 (file)
@@ -39,7 +39,6 @@
 #include "cpu/thread_context.hh"
 #include "debug/DebugPrintf.hh"
 #include "kern/system_events.hh"
-#include "sim/arguments.hh"
 #include "sim/system.hh"
 
 namespace FreeBSD
index e576a39b2a26b8c964794565bc76be411bacec3e..db487e822d9bc4630006d3c9c3850894fc7e4b70 100644 (file)
@@ -49,7 +49,6 @@
 #include "cpu/thread_context.hh"
 #include "kern/linux/helpers.hh"
 #include "kern/system_events.hh"
-#include "sim/arguments.hh"
 #include "sim/core.hh"
 #include "sim/system.hh"
 
index 06b0822c2ece7a3597c65c443f5fde3cbe4649e8..53239b8a02d76c9beea3ddb3104d876d8037c612 100644 (file)
@@ -39,7 +39,6 @@ SimObject('DVFSHandler.py')
 SimObject('SubSystem.py')
 SimObject('RedirectPath.py')
 
-Source('arguments.cc')
 Source('async.cc')
 Source('backtrace_%s.cc' % env['BACKTRACE_IMPL'])
 Source('core.cc')
diff --git a/src/sim/arguments.cc b/src/sim/arguments.cc
deleted file mode 100644 (file)
index 4162ffe..0000000
+++ /dev/null
@@ -1,55 +0,0 @@
-/*
- * 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 "sim/arguments.hh"
-
-#include "arch/utility.hh"
-#include "config/the_isa.hh"
-
-Arguments::Data::~Data()
-{
-    while (!data.empty()) {
-        delete [] data.front();
-        data.pop_front();
-    }
-}
-
-char *
-Arguments::Data::alloc(size_t size)
-{
-    char *buf = new char[size];
-    data.push_back(buf);
-    return buf;
-}
-
-uint64_t
-Arguments::getArg(uint16_t size, bool fp)
-{
-    return TheISA::getArgument(tc, number, size, fp);
-}
-
diff --git a/src/sim/arguments.hh b/src/sim/arguments.hh
deleted file mode 100644 (file)
index 2861e4d..0000000
+++ /dev/null
@@ -1,148 +0,0 @@
-/*
- * 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 __SIM_ARGUMENTS_HH__
-#define __SIM_ARGUMENTS_HH__
-
-#include <cassert>
-#include <memory>
-
-#include "cpu/thread_context.hh"
-#include "mem/port_proxy.hh"
-
-class Arguments
-{
-  protected:
-    ThreadContext *tc;
-    int number;
-    uint64_t getArg(uint16_t size = (uint16_t)(-1), bool fp = false);
-
-  protected:
-    class Data
-    {
-      public:
-        Data(){}
-        ~Data();
-
-      private:
-        std::list<char *> data;
-
-      public:
-        char *alloc(size_t size);
-    };
-
-    std::shared_ptr<Data> data;
-
-  public:
-    Arguments(ThreadContext *ctx, int n = 0)
-        : tc(ctx), number(n), data(new Data())
-    { assert(number >= 0); }
-    Arguments(const Arguments &args)
-        : tc(args.tc), number(args.number), data(args.data) {}
-    ~Arguments() {}
-
-    ThreadContext *getThreadContext() const { return tc; }
-
-    const Arguments &operator=(const Arguments &args) {
-        if (this != &args) {
-            tc = args.tc;
-            number = args.number;
-            data = args.data;
-        }
-        return *this;
-    }
-
-    // for checking if an argument is NULL
-    bool operator!() {
-        return getArg() == 0;
-    }
-
-    Arguments &operator++() {
-        ++number;
-        assert(number >= 0);
-        return *this;
-    }
-
-    Arguments operator++(int) {
-        Arguments args = *this;
-        ++number;
-        assert(number >= 0);
-        return args;
-    }
-
-    Arguments &operator--() {
-        --number;
-        assert(number >= 0);
-        return *this;
-    }
-
-    Arguments operator--(int) {
-        Arguments args = *this;
-        --number;
-        assert(number >= 0);
-        return args;
-    }
-
-    const Arguments &operator+=(int index) {
-        number += index;
-        assert(number >= 0);
-        return *this;
-    }
-
-    const Arguments &operator-=(int index) {
-        number -= index;
-        assert(number >= 0);
-        return *this;
-    }
-
-    Arguments operator[](int index) {
-        return Arguments(tc, index);
-    }
-
-    template <class T>
-    operator T() {
-        assert(sizeof(T) <= sizeof(uint64_t));
-        T d = static_cast<T>(getArg(sizeof(T)));
-        return d;
-    }
-
-    template <class T>
-    operator T *() {
-        T *buf = (T *)data->alloc(sizeof(T));
-        tc->getVirtProxy().readBlob(getArg(sizeof(T)), buf, sizeof(T));
-        return buf;
-    }
-
-    operator char *() {
-        char *buf = data->alloc(2048);
-        tc->getVirtProxy().readString(buf, getArg(), 2048);
-        return buf;
-    }
-};
-
-#endif // __SIM_ARGUMENTS_HH__