AddrRange: Transition from Range<T> to AddrRange
authorAndreas Hansson <andreas.hansson@arm.com>
Wed, 19 Sep 2012 10:15:44 +0000 (06:15 -0400)
committerAndreas Hansson <andreas.hansson@arm.com>
Wed, 19 Sep 2012 10:15:44 +0000 (06:15 -0400)
This patch takes the final plunge and transitions from the templated
Range class to the more specific AddrRange. In doing so it changes the
obvious Range<Addr> to AddrRange, and also bumps the range_map to be
AddrRangeMap.

In addition to the obvious changes, including the removal of redundant
includes, this patch also does some house keeping in preparing for the
introduction of address interleaving support in the ranges. The Range
class is also stripped of all the functionality that is never used.

--HG--
rename : src/base/range.hh => src/base/addr_range.hh
rename : src/base/range_map.hh => src/base/addr_range_map.hh

49 files changed:
src/arch/x86/interrupts.cc
src/base/addr_range.hh [new file with mode: 0644]
src/base/addr_range_map.hh [new file with mode: 0644]
src/base/inet.hh
src/base/random.hh
src/base/range.hh [deleted file]
src/base/range_map.hh [deleted file]
src/cpu/simple/base.cc
src/dev/alpha/backdoor.hh
src/dev/alpha/tsunami_cchip.hh
src/dev/alpha/tsunami_io.hh
src/dev/alpha/tsunami_pchip.hh
src/dev/arm/a9scu.hh
src/dev/arm/amba_device.hh
src/dev/arm/amba_fake.hh
src/dev/arm/gic.hh
src/dev/arm/kmi.hh
src/dev/arm/pl011.hh
src/dev/arm/pl111.hh
src/dev/arm/rtc_pl031.hh
src/dev/arm/rv_ctrl.hh
src/dev/arm/timer_cpulocal.hh
src/dev/arm/timer_sp804.hh
src/dev/baddev.hh
src/dev/isa_fake.hh
src/dev/mc146818.hh
src/dev/mips/malta_cchip.hh
src/dev/mips/malta_io.hh
src/dev/mips/malta_pchip.hh
src/dev/pciconfigall.hh
src/dev/sparc/dtod.hh
src/dev/sparc/iob.hh
src/dev/sparc/mm_disk.hh
src/dev/uart.hh
src/dev/uart8250.hh
src/dev/x86/i82094aa.hh
src/mem/abstract_mem.cc
src/mem/abstract_mem.hh
src/mem/bridge.cc
src/mem/bridge.hh
src/mem/bus.cc
src/mem/bus.hh
src/mem/cache/cache_impl.hh
src/mem/physical.cc
src/mem/physical.hh
src/mem/port.hh
src/python/m5/params.py
src/python/swig/range.i
src/unittest/rangemaptest.cc

index 906903b8b9cba9271558052d885ea643188813b6..b34124ce7e87eaf5187c1cc3c0cb5c997c272dae 100644 (file)
@@ -371,9 +371,9 @@ AddrRangeList
 X86ISA::Interrupts::getAddrRanges() const
 {
     AddrRangeList ranges;
-    Range<Addr> range = RangeEx(x86LocalAPICAddress(initialApicId, 0),
-                                x86LocalAPICAddress(initialApicId, 0) + 
-                                PageBytes);
+    AddrRange range = RangeEx(x86LocalAPICAddress(initialApicId, 0),
+                              x86LocalAPICAddress(initialApicId, 0) +
+                              PageBytes);
     ranges.push_back(range);
     return ranges;
 }
diff --git a/src/base/addr_range.hh b/src/base/addr_range.hh
new file mode 100644 (file)
index 0000000..2159335
--- /dev/null
@@ -0,0 +1,126 @@
+/*
+ * Copyright (c) 2012 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) 2002-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.
+ *
+ * Authors: Nathan Binkert
+ *          Steve Reinhardt
+ *          Andreas Hansson
+ */
+
+#ifndef __BASE_ADDR_RANGE_HH__
+#define __BASE_ADDR_RANGE_HH__
+
+#include "base/types.hh"
+
+class AddrRange
+{
+
+  public:
+
+    Addr start;
+    Addr end;
+
+    AddrRange()
+        : start(1), end(0)
+    {}
+
+    AddrRange(Addr _start, Addr _end)
+        : start(_start), end(_end)
+    {}
+
+    AddrRange(const std::pair<Addr, Addr> &r)
+        : start(r.first), end(r.second)
+    {}
+
+    Addr size() const { return end - start + 1; }
+    bool valid() const { return start < end; }
+};
+
+/**
+ * Keep the operators away from SWIG.
+ */
+#ifndef SWIG
+
+/**
+ * @param range1 is a range.
+ * @param range2 is a range.
+ * @return if range1 is less than range2 and does not overlap range1.
+ */
+inline bool
+operator<(const AddrRange& range1, const AddrRange& range2)
+{
+    return range1.start < range2.start;
+}
+
+/**
+ * @param addr address in the range
+ * @param range range compared against.
+ * @return indicates that the address is not within the range.
+ */
+inline bool
+operator!=(const Addr& addr, const AddrRange& range)
+{
+    return addr < range.start || addr > range.end;
+}
+
+/**
+ * @param range range compared against.
+ * @param pos position compared to the range.
+ * @return indicates that position pos is within the range.
+ */
+inline bool
+operator==(const AddrRange& range, const Addr& addr)
+{
+    return addr >= range.start && addr <= range.end;
+}
+
+inline AddrRange
+RangeEx(Addr start, Addr end)
+{ return std::make_pair(start, end - 1); }
+
+inline AddrRange
+RangeIn(Addr start, Addr end)
+{ return std::make_pair(start, end); }
+
+inline AddrRange
+RangeSize(Addr start, Addr size)
+{ return std::make_pair(start, start + size - 1); }
+
+#endif // SWIG
+
+#endif // __BASE_ADDR_RANGE_HH__
diff --git a/src/base/addr_range_map.hh b/src/base/addr_range_map.hh
new file mode 100644 (file)
index 0000000..c35befd
--- /dev/null
@@ -0,0 +1,206 @@
+/*
+ * Copyright (c) 2012 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) 2006 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.
+ *
+ * Authors: Ali Saidi
+ *          Andreas Hansson
+ */
+
+#ifndef __BASE_ADDR_RANGE_MAP_HH__
+#define __BASE_ADDR_RANGE_MAP_HH__
+
+#include <map>
+#include <utility>
+
+#include "base/addr_range.hh"
+
+/**
+ * The AddrRangeMap uses an STL map to implement an interval tree for
+ * address decoding. The value stored is a template type and can be
+ * e.g. a port identifier, or a pointer.
+ */
+template <typename V>
+class AddrRangeMap
+{
+  private:
+    typedef std::map<AddrRange, V> RangeMap;
+    RangeMap tree;
+
+  public:
+    typedef typename RangeMap::iterator iterator;
+    typedef typename RangeMap::const_iterator const_iterator;
+
+    const_iterator
+    find(const AddrRange &r) const
+    {
+        const_iterator i;
+
+        i = tree.upper_bound(r);
+
+        if (i == tree.begin()) {
+            if (i->first.start <= r.end && i->first.end >= r.start)
+                return i;
+            else
+                // Nothing could match, so return end()
+                return tree.end();
+        }
+
+        --i;
+
+        if (i->first.start <= r.end && i->first.end >= r.start)
+            return i;
+
+        return tree.end();
+    }
+
+    iterator
+    find(const AddrRange &r)
+    {
+        iterator i;
+
+        i = tree.upper_bound(r);
+
+        if (i == tree.begin()) {
+            if (i->first.start <= r.end && i->first.end >= r.start)
+                return i;
+            else
+                // Nothing could match, so return end()
+                return tree.end();
+        }
+
+        --i;
+
+        if (i->first.start <= r.end && i->first.end >= r.start)
+            return i;
+
+        return tree.end();
+    }
+
+    const_iterator
+    find(const Addr &r) const
+    {
+        return find(RangeSize(r, 1));
+    }
+
+    iterator
+    find(const Addr &r)
+    {
+        return find(RangeSize(r, 1));
+    }
+
+    bool
+    intersect(const AddrRange &r)
+    {
+        iterator i;
+        i = find(r);
+        if (i != tree.end())
+            return true;
+        return false;
+    }
+
+    iterator
+    insert(const AddrRange &r, const V& d)
+    {
+        if (intersect(r))
+            return tree.end();
+
+        return tree.insert(std::make_pair(r, d)).first;
+    }
+
+    std::size_t
+    erase(Addr k)
+    {
+        return tree.erase(k);
+    }
+
+    void
+    erase(iterator p)
+    {
+        tree.erase(p);
+    }
+
+    void
+    erase(iterator p, iterator q)
+    {
+        tree.erase(p,q);
+    }
+
+    void
+    clear()
+    {
+        tree.erase(tree.begin(), tree.end());
+    }
+
+    const_iterator
+    begin() const
+    {
+        return tree.begin();
+    }
+
+    iterator
+    begin()
+    {
+        return tree.begin();
+    }
+
+    const_iterator
+    end() const
+    {
+        return tree.end();
+    }
+
+    iterator
+    end()
+    {
+        return tree.end();
+    }
+
+    std::size_t
+    size() const
+    {
+        return tree.size();
+    }
+
+    bool
+    empty() const
+    {
+        return tree.empty();
+    }
+};
+
+#endif //__BASE_ADDR_RANGE_MAP_HH__
index 4b73355910f8cda73ec8638eeb0bdc4abfeb160b..1df175c1ea1df56cb845f7be2272be65d00befb6 100644 (file)
@@ -39,7 +39,6 @@
 #include <utility>
 #include <vector>
 
-#include "base/range.hh"
 #include "base/types.hh"
 #include "dev/etherpkt.hh"
 #include "dnet/os.h"
index b7e2a5073588dcf03ecb25e73e3de9ad67026ffa..34107c76f322a9ff1dc180316f78156146e8fb69 100644 (file)
@@ -42,7 +42,6 @@
 #include <ios>
 #include <string>
 
-#include "base/range.hh"
 #include "base/types.hh"
 
 class Checkpoint;
@@ -210,13 +209,6 @@ class Random
         return _random(min, max);
     }
 
-    template <typename T>
-    T
-    random(const Range<T> &range)
-    {
-        return _random(range.start, range.end);
-    }
-
     // [0,1]
     double
     gen_real1()
diff --git a/src/base/range.hh b/src/base/range.hh
deleted file mode 100644 (file)
index 3b1a927..0000000
+++ /dev/null
@@ -1,328 +0,0 @@
-/*
- * Copyright (c) 2002-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.
- *
- * Authors: Nathan Binkert
- *          Steve Reinhardt
- */
-
-#ifndef __BASE_RANGE_HH__
-#define __BASE_RANGE_HH__
-
-template <class T>
-struct Range
-{
-    T start;
-    T end;
-
-    Range() { invalidate(); }
-
-    template <class U>
-    Range(const std::pair<U, U> &r)
-        : start(r.first), end(r.second)
-    {}
-
-    template <class U>
-    Range(const Range<U> &r)
-        : start(r.start), end(r.end)
-    {}
-
-    template <class U>
-    const Range<T> &operator=(const Range<U> &r)
-    {
-        start = r.start;
-        end = r.end;
-        return *this;
-    }
-
-    template <class U>
-    const Range<T> &operator=(const std::pair<U, U> &r)
-    {
-        start = r.first;
-        end = r.second;
-        return *this;
-    }
-
-    void invalidate() { start = 1; end = 0; }
-    T size() const { return end - start + 1; }
-    bool valid() const { return start < end; }
-};
-
-template <class T>
-inline Range<T>
-RangeEx(T start, T end)
-{ return std::make_pair(start, end - 1); }
-
-template <class T>
-inline Range<T>
-RangeIn(T start, T end)
-{ return std::make_pair(start, end); }
-
-template <class T, class U>
-inline Range<T>
-RangeSize(T start, U size)
-{ return std::make_pair(start, start + size - 1); }
-
-////////////////////////////////////////////////////////////////////////
-//
-// Range to Range Comparisons
-//
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 and range2 are identical.
- */
-template <class T, class U>
-inline bool
-operator==(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start == range2.start && range1.end == range2.end;
-}
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 and range2 are not identical.
- */
-template <class T, class U>
-inline bool
-operator!=(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start != range2.start || range1.end != range2.end;
-}
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 is less than range2 and does not overlap range1.
- */
-template <class T, class U>
-inline bool
-operator<(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start < range2.start;
-}
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 is less than range2.  range1 may overlap range2,
- * but not extend beyond the end of range2.
- */
-template <class T, class U>
-inline bool
-operator<=(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start <= range2.start;
-}
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 is greater than range2 and does not overlap range2.
- */
-template <class T, class U>
-inline bool
-operator>(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start > range2.start;
-}
-
-/**
- * @param range1 is a range.
- * @param range2 is a range.
- * @return if range1 is greater than range2.  range1 may overlap range2,
- * but not extend beyond the beginning of range2.
- */
-template <class T, class U>
-inline bool
-operator>=(const Range<T> &range1, const Range<U> &range2)
-{
-    return range1.start >= range2.start;
-}
-
-////////////////////////////////////////////////////////////////////////
-//
-// Position to Range Comparisons
-//
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is within the range.
- */
-template <class T, class U>
-inline bool
-operator==(const T &pos, const Range<U> &range)
-{
-    return pos >= range.start && pos <= range.end;
-}
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is not within the range.
- */
-template <class T, class U>
-inline bool
-operator!=(const T &pos, const Range<U> &range)
-{
-    return pos < range.start || pos > range.end;
-}
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is below the range.
- */
-template <class T, class U>
-inline bool
-operator<(const T &pos, const Range<U> &range)
-{
-    return pos < range.start;
-}
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is below or in the range.
- */
-template <class T, class U>
-inline bool
-operator<=(const T &pos, const Range<U> &range)
-{
-    return pos <= range.end;
-}
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is above the range.
- */
-template <class T, class U>
-inline bool
-operator>(const T &pos, const Range<U> &range)
-{
-    return pos > range.end;
-}
-
-/**
- * @param pos position compared to the range.
- * @param range range compared against.
- * @return indicates that position pos is above or in the range.
- */
-template <class T, class U>
-inline bool
-operator>=(const T &pos, const Range<U> &range)
-{
-    return pos >= range.start;
-}
-
-////////////////////////////////////////////////////////////////////////
-//
-// Range to Position Comparisons (for symmetry)
-//
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * @return indicates that position pos is within the range.
- */
-template <class T, class U>
-inline bool
-operator==(const Range<T> &range, const U &pos)
-{
-    return pos >= range.start && pos <= range.end;
-}
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * @return indicates that position pos is not within the range.
- */
-template <class T, class U>
-inline bool
-operator!=(const Range<T> &range, const U &pos)
-{
-    return pos < range.start || pos > range.end;
-}
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * @return indicates that position pos is above the range.
- */
-template <class T, class U>
-inline bool
-operator<(const Range<T> &range, const U &pos)
-{
-  // with -std=gnu++0x, gcc and clang get confused when range.end is
-  // compared to pos using the operator "<", and the parser expects it
-  // to be the opening bracket for a template parameter,
-  // i.e. range.end<pos>(...);, the reason seems to be the range-type
-  // iteration introduced in c++11 where begin and end are members
-  // that return iterators
-    return operator<(range.end, pos);
-}
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * @return indicates that position pos is above or in the range.
- */
-template <class T, class U>
-inline bool
-operator<=(const Range<T> &range, const U &pos)
-{
-    return range.start <= pos;
-}
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * 'range > pos' indicates that position pos is below the range.
- */
-template <class T, class U>
-inline bool
-operator>(const Range<T> &range, const U &pos)
-{
-    return range.start > pos;
-}
-
-/**
- * @param range range compared against.
- * @param pos position compared to the range.
- * 'range >= pos' indicates that position pos is below or in the range.
- */
-template <class T, class U>
-inline bool
-operator>=(const Range<T> &range, const U &pos)
-{
-    return range.end >= pos;
-}
-
-#endif // __BASE_RANGE_HH__
diff --git a/src/base/range_map.hh b/src/base/range_map.hh
deleted file mode 100644 (file)
index a977427..0000000
+++ /dev/null
@@ -1,200 +0,0 @@
-/*
- * Copyright (c) 2006 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.
- *
- * Authors: Ali Saidi
- */
-
-#ifndef __BASE_RANGE_MAP_HH__
-#define __BASE_RANGE_MAP_HH__
-
-#include <map>
-#include <utility>
-
-#include "base/range.hh"
-
-/**
- * The range_map uses an STL map to implement an interval tree. The
- * type of both the key (range) and the value are template
- * parameters. It can, for example, be used for address decoding,
- * using a range of addresses to map to ports.
- */
-template <class T,class V>
-class range_map
-{
-  private:
-    typedef std::map<Range<T>,V> RangeMap;
-    RangeMap tree;
-
-  public:
-    typedef typename RangeMap::iterator iterator;
-    typedef typename RangeMap::const_iterator const_iterator;
-
-    template <class U>
-    const_iterator
-    find(const Range<U> &r) const
-    {
-        const_iterator i;
-
-        i = tree.upper_bound(r);
-
-        if (i == tree.begin()) {
-            if (i->first.start <= r.end && i->first.end >= r.start)
-                return i;
-            else
-                // Nothing could match, so return end()
-                return tree.end();
-        }
-
-        --i;
-
-        if (i->first.start <= r.end && i->first.end >= r.start)
-            return i;
-
-        return tree.end();
-    }
-
-    template <class U>
-    iterator
-    find(const Range<U> &r)
-    {
-        iterator i;
-
-        i = tree.upper_bound(r);
-
-        if (i == tree.begin()) {
-            if (i->first.start <= r.end && i->first.end >= r.start)
-                return i;
-            else
-                // Nothing could match, so return end()
-                return tree.end();
-        }
-
-        --i;
-
-        if (i->first.start <= r.end && i->first.end >= r.start)
-            return i;
-
-        return tree.end();
-    }
-
-    template <class U>
-    const_iterator
-    find(const U &r) const
-    {
-        return find(RangeSize(r, 1));
-    }
-
-    template <class U>
-    iterator
-    find(const U &r)
-    {
-        return find(RangeSize(r, 1));
-    }
-
-    template <class U>
-    bool
-    intersect(const Range<U> &r)
-    {
-        iterator i;
-        i = find(r);
-        if (i != tree.end())
-            return true;
-        return false;
-    }
-
-    template <class U,class W>
-    iterator
-    insert(const Range<U> &r, const W d)
-    {
-        if (intersect(r))
-            return tree.end();
-
-        return tree.insert(std::make_pair(r, d)).first;
-    }
-
-    size_t
-    erase(T k)
-    {
-        return tree.erase(k);
-    }
-
-    void
-    erase(iterator p)
-    {
-        tree.erase(p);
-    }
-
-    void
-    erase(iterator p, iterator q)
-    {
-        tree.erase(p,q);
-    }
-
-    void
-    clear()
-    {
-        tree.erase(tree.begin(), tree.end());
-    }
-
-    const_iterator
-    begin() const
-    {
-        return tree.begin();
-    }
-
-    iterator
-    begin()
-    {
-        return tree.begin();
-    }
-
-    const_iterator
-    end() const
-    {
-        return tree.end();
-    }
-
-    iterator
-    end()
-    {
-        return tree.end();
-    }
-
-    size_t
-    size() const
-    {
-        return tree.size();
-    }
-
-    bool
-    empty() const
-    {
-        return tree.empty();
-    }
-};
-
-#endif //__BASE_RANGE_MAP_HH__
index bdc4b0f44e869eab55485a0aadaa0a517b92975f..5a9499333fef4477bc633c97b12878e60a3abc10 100644 (file)
@@ -51,7 +51,6 @@
 #include "base/inifile.hh"
 #include "base/misc.hh"
 #include "base/pollevent.hh"
-#include "base/range.hh"
 #include "base/trace.hh"
 #include "base/types.hh"
 #include "config/the_isa.hh"
index 2acaba9a3696970a014bf41c9dcfaa8272f98762..b9d04c7c0874e6144f9c56f2db33b7c8d8c749ef 100644 (file)
@@ -35,7 +35,6 @@
 #ifndef __DEV_ALPHA_BACKDOOR_HH__
 #define __DEV_ALPHA_BACKDOOR_HH__
 
-#include "base/range.hh"
 #include "base/types.hh"
 #include "dev/alpha/access.h"
 #include "dev/io_device.hh"
index 1265c2e80255a502103fbd0f66bab30d56f8e1a7..e9aca5d5ceaa7980023fd43d91ee491797edafbf 100644 (file)
@@ -35,7 +35,6 @@
 #ifndef __TSUNAMI_CCHIP_HH__
 #define __TSUNAMI_CCHIP_HH__
 
-#include "base/range.hh"
 #include "dev/alpha/tsunami.hh"
 #include "dev/io_device.hh"
 #include "params/TsunamiCChip.hh"
index f88cf5a6cf30d4ac618f6e0645d2b8714238ce4a..212e2a3d5c783a2b24690fdc30aa5b79f509e083 100644 (file)
@@ -37,7 +37,6 @@
 #ifndef __DEV_TSUNAMI_IO_HH__
 #define __DEV_TSUNAMI_IO_HH__
 
-#include "base/range.hh"
 #include "dev/alpha/tsunami.hh"
 #include "dev/intel_8254_timer.hh"
 #include "dev/io_device.hh"
index d31a28dbe66ddb3721a593dc1edb2852b0fe8a7e..3e32db989b41f53076f395f2a2fd9318ae8aeff9 100644 (file)
@@ -35,7 +35,6 @@
 #ifndef __TSUNAMI_PCHIP_HH__
 #define __TSUNAMI_PCHIP_HH__
 
-#include "base/range.hh"
 #include "dev/alpha/tsunami.hh"
 #include "dev/io_device.hh"
 #include "params/TsunamiPChip.hh"
index 881401ca699bcc5668afcc919a9f9ed41e19a2e6..10428d91e05f446da248dfa142999fda2dfd630d 100644 (file)
@@ -40,7 +40,6 @@
 #ifndef __DEV_ARM_A9SCU_HH__
 #define __DEV_ARM_A9SCU_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/A9SCU.hh"
 
index 4bea1e0758f9d122caf2976cd05c136dacb26560..92dfed541d5c6568647610a53273aae6a1ff9858 100644 (file)
@@ -49,7 +49,6 @@
 #ifndef __DEV_ARM_AMBA_DEVICE_HH__
 #define __DEV_ARM_AMBA_DEVICE_HH__
 
-#include "base/range.hh"
 #include "dev/arm/gic.hh"
 #include "dev/dma_device.hh"
 #include "dev/io_device.hh"
index 4a67ab9d5d7c0aa0a257638fb1322ccd8104030d..24b326e8aa442d3100ade2c83121504d07f9a4ea 100644 (file)
@@ -51,7 +51,6 @@
 #ifndef __DEV_ARM_AMBA_FAKE_H__
 #define __DEV_ARM_AMBA_FAKE_H__
 
-#include "base/range.hh"
 #include "dev/arm/amba_device.hh"
 #include "params/AmbaFake.hh"
 
index 9d93bbedf6324b65f5a128fbc3f3681f604d7637..02448f651112ab6f7325f3d4185bb5b2efedc374 100644 (file)
@@ -49,7 +49,6 @@
 #define __DEV_ARM_GIC_H__
 
 #include "base/bitunion.hh"
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "dev/platform.hh"
 #include "cpu/intr_control.hh"
index dc488ccce0df72cdaf09fcdac13b35f3044ae039..e769a8a468e06d68d18b1b73249d1d1887281969 100644 (file)
@@ -51,7 +51,6 @@
 #include <list>
 
 #include "base/vnc/vncserver.hh"
-#include "base/range.hh"
 #include "dev/arm/amba_device.hh"
 #include "params/Pl050.hh"
 
index ddfd8305bcbc09fe482fe8d3051134013233abf1..dbd8bd539e89010f2c0f3d2352112115265887ca 100644 (file)
@@ -48,7 +48,6 @@
 #ifndef __DEV_ARM_PL011_H__
 #define __DEV_ARM_PL011_H__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "dev/uart.hh"
 #include "params/Pl011.hh"
index 599d4fa3ee641bd73be71ec90081c2ad09a95056..5776f199ce721d1c931d22a2ed1f0c4564ecb8ee 100644 (file)
@@ -48,7 +48,6 @@
 
 #include <fstream>
 
-#include "base/range.hh"
 #include "dev/arm/amba_device.hh"
 #include "params/Pl111.hh"
 #include "sim/serialize.hh"
index f5615dd55ba35e5fe101a70401da07e8030f0fae..0f1929d29ca3e229948c7d31c050e2ae2bd5a665 100644 (file)
@@ -40,7 +40,6 @@
 #ifndef __DEV_ARM_RTC_PL310_HH__
 #define __DEV_ARM_RTC_PL310_HH__
 
-#include "base/range.hh"
 #include "dev/arm/amba_device.hh"
 #include "params/PL031.hh"
 
index cf14f6bcd29adfd9b62e74fdf44effe368e236e6..c6cf40f96ca213103e8c41895c7615227e80ff5f 100644 (file)
@@ -41,7 +41,6 @@
 #define __DEV_ARM_RV_HH__
 
 #include "base/bitunion.hh"
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/RealViewCtrl.hh"
 
index 144e0b8070d4105bc938b1ac4ef709eaed21b42a..cf7e4649640acdb0c78678f8296a91eec0f01420 100644 (file)
@@ -41,7 +41,6 @@
 #ifndef __DEV_ARM_LOCALTIMER_HH__
 #define __DEV_ARM_LOCALTIMER_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/CpuLocalTimer.hh"
 
index afb6e29ed2499ccf10aed869d0d4e304e99797d7..9f137001dab4ccef6dd89fe05283cdb366e81283 100644 (file)
@@ -40,7 +40,6 @@
 #ifndef __DEV_ARM_SP804_HH__
 #define __DEV_ARM_SP804_HH__
 
-#include "base/range.hh"
 #include "dev/arm/amba_device.hh"
 #include "params/Sp804.hh"
 
index 9cf592c0ef5fb5cb90d6bf4388eceff5d68bcc57..ea902152ea8e242c7ab8da7d685aa7d978e7f1d4 100644 (file)
@@ -36,7 +36,6 @@
 #ifndef __DEV_BADDEV_HH__
 #define __DEV_BADDEV_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/BadDevice.hh"
 
index 07657ad7dfc141a4128ddd484018fba7b7b8eadb..1223e3b22ec589e49153bbeb5b47821b07486574 100644 (file)
@@ -37,7 +37,6 @@
 
 #include <string>
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 // #include "dev/alpha/tsunami.hh"
 #include "mem/packet.hh"
index 576c4ab9fedb5042f7775989a9d7bafa685d0981..c2f1e2dd896dd33bbbbd8dcf404b7eb9fb1e5b20 100644 (file)
@@ -33,7 +33,6 @@
 #ifndef __DEV_MC146818_HH__
 #define __DEV_MC146818_HH__
 
-#include "base/range.hh"
 #include "sim/eventq.hh"
 
 /** Real-Time Clock (MC146818) */
index 4841551c4ba71e754be5d9e33d20f68165ea7259..9a17f632f54433ac075864b1428ec752a0fc3a1f 100755 (executable)
@@ -36,7 +36,6 @@
 #ifndef __MALTA_CCHIP_HH__
 #define __MALTA_CCHIP_HH__
 
-#include "base/range.hh"
 #include "dev/mips/malta.hh"
 #include "dev/io_device.hh"
 #include "params/MaltaCChip.hh"
index 38da5adeac55db705ead55d67ac72d94f25dec73..9311d7c225269a01effc5b8328945877a18ae943 100755 (executable)
@@ -37,7 +37,6 @@
 #ifndef __DEV_MALTA_IO_HH__
 #define __DEV_MALTA_IO_HH__
 
-#include "base/range.hh"
 #include "dev/mips/malta.hh"
 #include "dev/intel_8254_timer.hh"
 #include "dev/io_device.hh"
index a554e253e248c5003c5fce550db8a072836727ee..a6145515a9b42f6b6bd01d496bd257b2c9a35d9f 100755 (executable)
@@ -35,7 +35,6 @@
 #ifndef __MALTA_PCHIP_HH__
 #define __MALTA_PCHIP_HH__
 
-#include "base/range.hh"
 #include "dev/mips/malta.hh"
 #include "dev/io_device.hh"
 #include "params/MaltaPChip.hh"
index eb480ad1662b7e62e70e52b5197375557b24be0e..4df36f0b3d728acdd8a067711936c11fbe8ccf88 100644 (file)
@@ -37,7 +37,6 @@
 #ifndef __PCICONFIGALL_HH__
 #define __PCICONFIGALL_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "dev/pcireg.h"
 #include "params/PciConfigAll.hh"
index 325bce90a1f61c3050fdf5abed4f675ac0942b7c..a5b2dfaffe24a6ff0357614ff2a5be28c2512c34 100644 (file)
@@ -38,7 +38,6 @@
 
 #include <vector>
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/DumbTOD.hh"
 
index b92d3cb2a76b41fecd14b377426a3dc8de2e7101..fc5e610925dfd3e904eca97de02a6214b23eb8da 100644 (file)
@@ -36,7 +36,6 @@
 #ifndef __DEV_SPARC_IOB_HH__
 #define __DEV_SPARC_IOB_HH__
 
-#include "base/range.hh"
 #include "dev/disk_image.hh"
 #include "dev/io_device.hh"
 #include "params/Iob.hh"
index 0e43449a1406aeb2295f875f2b22379acbe6f991..d14e1d4a4c7a07049cdd25d1de1a6418819a1186 100644 (file)
@@ -36,7 +36,6 @@
 #ifndef __DEV_SPARC_MM_DISK_HH__
 #define __DEV_SPARC_MM_DISK_HH__
 
-#include "base/range.hh"
 #include "dev/disk_image.hh"
 #include "dev/io_device.hh"
 #include "params/MmDisk.hh"
index ba10c204c382c982fdfe33a716fac3a5b49c3bfe..eac70bf1f75dbd12a9e2828115bb6537c969b247 100644 (file)
@@ -35,7 +35,6 @@
 #ifndef __UART_HH__
 #define __UART_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "params/Uart.hh"
 
index e2fb043c19b6c614838541de9ef7a6938fe143b2..7d577954cf6c3b1157e185cc327afe911242400f 100644 (file)
@@ -35,7 +35,6 @@
 #ifndef __DEV_UART8250_HH__
 #define __DEV_UART8250_HH__
 
-#include "base/range.hh"
 #include "dev/io_device.hh"
 #include "dev/uart.hh"
 #include "params/Uart8250.hh"
index c90a5b812b853f61216adc21a6bc35364dbf7b0d..76a8f9c004eeb9ab34f0a833d07f56fbe87d5381 100644 (file)
@@ -34,7 +34,6 @@
 #include <map>
 
 #include "base/bitunion.hh"
-#include "base/range_map.hh"
 #include "dev/x86/intdev.hh"
 #include "dev/io_device.hh"
 #include "params/I82094AA.hh"
index 775517e3b26a50156b7e6bf22360a54c21374a3a..ebe4a64b5e5e63d8517cd8b467c9cb664b882564 100644 (file)
@@ -85,9 +85,8 @@ AbstractMemory::AbstractMemory(const Params *p) :
         int fd = open(params()->file.c_str(), O_RDONLY);
         long _size = lseek(fd, 0, SEEK_END);
         if (_size != range.size()) {
-            warn("Specified size %d does not match file %s %d\n", range.size(),
-                 params()->file, _size);
-            range = RangeSize(range.start, _size);
+            fatal("Specified size %d does not match file %s %d\n",
+                  range.size(), params()->file, _size);
         }
         lseek(fd, 0, SEEK_SET);
         pmemAddr = (uint8_t *)mmap(NULL, roundUp(_size, sysconf(_SC_PAGESIZE)),
@@ -222,7 +221,7 @@ AbstractMemory::regStats()
     bwTotal = (bytesRead + bytesWritten) / simSeconds;
 }
 
-Range<Addr>
+AddrRange
 AbstractMemory::getAddrRange() const
 {
     return range;
index 43d9656dad2f9440bad1420cf9ee9ce6a3c7a8c3..66d4a1f16ce583b973e8f5fb774ed3fafa74fc43 100644 (file)
@@ -68,7 +68,7 @@ class AbstractMemory : public MemObject
   protected:
 
     // Address range of this memory
-    Range<Addr> range;
+    AddrRange range;
 
     // Pointer to host memory used to implement this memory
     uint8_t* pmemAddr;
@@ -209,7 +209,7 @@ class AbstractMemory : public MemObject
      *
      * @return a single contigous address range
      */
-    Range<Addr> getAddrRange() const;
+    AddrRange getAddrRange() const;
 
     /**
      * Get the memory size.
index 3a185a8eb50e74316675194307b63dc3e405f38a..8bc34e12e70f37c2d5e7c6c428ead47bc4d3d3e0 100644 (file)
@@ -57,7 +57,7 @@ Bridge::BridgeSlavePort::BridgeSlavePort(const std::string& _name,
                                          Bridge& _bridge,
                                          BridgeMasterPort& _masterPort,
                                          Cycles _delay, int _resp_limit,
-                                         std::vector<Range<Addr> > _ranges)
+                                         std::vector<AddrRange> _ranges)
     : SlavePort(_name, &_bridge), bridge(_bridge), masterPort(_masterPort),
       delay(_delay), ranges(_ranges.begin(), _ranges.end()),
       outstandingResponses(0), retryReq(false),
index c5214646335448f3f865e74cc37a825bebc3e65c..eb0b2434faa9a7d375cf28e82e792a63a669c1c5 100644 (file)
@@ -193,7 +193,7 @@ class Bridge : public MemObject
          */
         BridgeSlavePort(const std::string& _name, Bridge& _bridge,
                         BridgeMasterPort& _masterPort, Cycles _delay,
-                        int _resp_limit, std::vector<Range<Addr> > _ranges);
+                        int _resp_limit, std::vector<AddrRange> _ranges);
 
         /**
          * Queue a response packet to be sent out later and also schedule
index 829d694de3bfdfff1cb41c5c5b2e688f2f9d1526..75ece9bc85df132a8564c277abf8ee37a652e090 100644 (file)
@@ -355,7 +355,6 @@ BaseBus::findPort(Addr addr)
 void
 BaseBus::recvRangeChange(PortID master_port_id)
 {
-    AddrRangeList ranges;
     AddrRangeIter iter;
 
     if (inRecvRangeChange.count(master_port_id))
@@ -394,7 +393,7 @@ BaseBus::recvRangeChange(PortID master_port_id)
         }
 
         // get the address ranges of the connected slave port
-        ranges = port->getAddrRanges();
+        AddrRangeList ranges = port->getAddrRanges();
 
         for (iter = ranges.begin(); iter != ranges.end(); iter++) {
             DPRINTF(BusAddrRanges, "Adding range %#llx - %#llx for id %d\n",
index ac35581b1da6f1f5097991f5c6f9d710400ae150..541e2f3635185d3ac5d56c9b7ef1031ee4d3eefa 100644 (file)
@@ -54,8 +54,7 @@
 #include <list>
 #include <set>
 
-#include "base/range.hh"
-#include "base/range_map.hh"
+#include "base/addr_range_map.hh"
 #include "base/types.hh"
 #include "mem/mem_object.hh"
 #include "params/BaseBus.hh"
@@ -233,9 +232,9 @@ class BaseBus : public MemObject
     /** the width of the bus in bytes */
     int width;
 
-    typedef range_map<Addr, PortID>::iterator PortMapIter;
-    typedef range_map<Addr, PortID>::const_iterator PortMapConstIter;
-    range_map<Addr, PortID> portMap;
+    typedef AddrRangeMap<PortID>::iterator PortMapIter;
+    typedef AddrRangeMap<PortID>::const_iterator PortMapConstIter;
+    AddrRangeMap<PortID> portMap;
 
     AddrRangeList defaultRange;
 
index 563160ac18f3d0965e4576f3131777dd0fcc8057..9b9010d344b6ef7a38cf912ff61379e95af44951 100644 (file)
@@ -51,7 +51,6 @@
  */
 
 #include "base/misc.hh"
-#include "base/range.hh"
 #include "base/types.hh"
 #include "debug/Cache.hh"
 #include "debug/CachePort.hh"
index 5f92976f9dedc35ed7b45c32bd6d91dd8cb23eef..23556f0ab11585ed046454c14899c6ce297d2ce9 100644 (file)
@@ -64,7 +64,6 @@ PhysicalMemory::PhysicalMemory(const vector<AbstractMemory*>& _memories) :
                 "Skipping memory %s that is not in global address map\n",
                 (*m)->name());
     }
-    rangeCache.invalidate();
 }
 
 bool
@@ -73,8 +72,7 @@ PhysicalMemory::isMemAddr(Addr addr) const
     // see if the address is within the last matched range
     if (addr != rangeCache) {
         // lookup in the interval tree
-        range_map<Addr, AbstractMemory*>::const_iterator r =
-            addrMap.find(addr);
+        AddrRangeMap<AbstractMemory*>::const_iterator r = addrMap.find(addr);
         if (r == addrMap.end()) {
             // not in the cache, and not in the tree
             return false;
@@ -110,7 +108,7 @@ PhysicalMemory::access(PacketPtr pkt)
 {
     assert(pkt->isRequest());
     Addr addr = pkt->getAddr();
-    range_map<Addr, AbstractMemory*>::const_iterator m = addrMap.find(addr);
+    AddrRangeMap<AbstractMemory*>::const_iterator m = addrMap.find(addr);
     assert(m != addrMap.end());
     m->second->access(pkt);
 }
@@ -120,7 +118,7 @@ PhysicalMemory::functionalAccess(PacketPtr pkt)
 {
     assert(pkt->isRequest());
     Addr addr = pkt->getAddr();
-    range_map<Addr, AbstractMemory*>::const_iterator m = addrMap.find(addr);
+    AddrRangeMap<AbstractMemory*>::const_iterator m = addrMap.find(addr);
     assert(m != addrMap.end());
     m->second->functionalAccess(pkt);
 }
index e78b1d2da3342ee52b88827310f2a173bf4ab499..fb9969a348a267e994b81769a4c1d9b0cc2a94b5 100644 (file)
@@ -40,7 +40,7 @@
 #ifndef __PHYSICAL_MEMORY_HH__
 #define __PHYSICAL_MEMORY_HH__
 
-#include "base/range_map.hh"
+#include "base/addr_range_map.hh"
 #include "mem/abstract_mem.hh"
 #include "mem/packet.hh"
 
@@ -55,10 +55,10 @@ class PhysicalMemory
   private:
 
     // Global address map
-    range_map<Addr, AbstractMemory* > addrMap;
+    AddrRangeMap<AbstractMemory*> addrMap;
 
     // a mutable cache for the last range that matched an address
-    mutable Range<Addr> rangeCache;
+    mutable AddrRange rangeCache;
 
     // All address-mapped memories
     std::vector<AbstractMemory*> memories;
index 631725ce18dbc5dcd3056cf8ed0622c2b74adce8..eaad9668a04b0408cc9caaaa2cc4990b5bc026a4 100644 (file)
@@ -52,7 +52,7 @@
 
 #include <list>
 
-#include "base/range.hh"
+#include "base/addr_range.hh"
 #include "mem/packet.hh"
 
 /**
@@ -62,9 +62,9 @@
  * defined.
  */
 
-typedef std::list<Range<Addr> > AddrRangeList;
-typedef std::list<Range<Addr> >::iterator AddrRangeIter;
-typedef std::list<Range<Addr> >::const_iterator AddrRangeConstIter;
+typedef std::list<AddrRange> AddrRangeList;
+typedef std::list<AddrRange>::iterator AddrRangeIter;
+typedef std::list<AddrRange>::const_iterator AddrRangeConstIter;
 
 class MemObject;
 
index 46c3d028c5447fdaeb53f5ff0f335834a3eeccd8..cabb91b28a9e4639aa28cee3b006dd236517506b 100644 (file)
@@ -550,7 +550,7 @@ class Addr(CheckedInt):
             return self.value + other
 
 class AddrRange(ParamValue):
-    cxx_type = 'Range<Addr>'
+    cxx_type = 'AddrRange'
 
     def __init__(self, *args, **kwargs):
         def handle_kwargs(self, kwargs):
@@ -594,20 +594,18 @@ class AddrRange(ParamValue):
     @classmethod
     def cxx_predecls(cls, code):
         Addr.cxx_predecls(code)
-        code('#include "base/range.hh"')
+        code('#include "base/addr_range.hh"')
 
     @classmethod
     def swig_predecls(cls, code):
         Addr.swig_predecls(code)
-        code('%import "python/swig/range.i"')
 
     def getValue(self):
+        # Go from the Python class to the wrapped C++ class generated
+        # by swig
         from m5.internal.range import AddrRange
 
-        value = AddrRange()
-        value.start = long(self.start)
-        value.end = long(self.end)
-        return value
+        return AddrRange(long(self.start), long(self.end))
 
 # Boolean parameter type.  Python doesn't let you subclass bool, since
 # it doesn't want to let you create multiple instances of True and
index d8da677bbb21a016be30384d12bfe7678872da6d..e3a79431005d438ac1aa427c58843c7e5c61b180 100644 (file)
 %module(package="m5.internal") range
 
 %{
-#include "base/range.hh"
 #include "base/types.hh"
+#include "base/addr_range.hh"
 %}
 
 %include <stdint.i>
 
 %rename(assign) *::operator=;
-%include "base/range.hh"
 %include "base/types.hh"
-
-%template(AddrRange) Range<Addr>;
-%template(TickRange) Range<Tick>;
+%include "base/addr_range.hh"
index af00e4e5881aec8ce7fe9445e9cff060175a2585..57b954b0ad7bfde71f4f3edb96bdaa18c23b98f1 100644 (file)
@@ -1,4 +1,16 @@
 /*
+ * Copyright (c) 2012 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) 2006 The Regents of The University of Michigan
  * All rights reserved.
  *
 #include <cassert>
 #include <iostream>
 
-#include "base/range_map.hh"
-#include "base/types.hh"
+#include "base/addr_range_map.hh"
 
 using namespace std;
 
 int
 main()
 {
-    range_map<Addr,int> r;
+    AddrRangeMap<int> r;
 
-    range_map<Addr,int>::iterator i;
+    AddrRangeMap<int>::iterator i;
 
-    i = r.insert(RangeIn<Addr>(10,40),5);
+    i = r.insert(RangeIn(10, 40), 5);
     assert(i != r.end());
-    i = r.insert(RangeIn<Addr>(60,90),3);
+    i = r.insert(RangeIn(60, 90), 3);
     assert(i != r.end());
 
-    i = r.find(RangeIn(20,30));
+    i = r.find(RangeIn(20, 30));
     assert(i != r.end());
     cout << i->first << " " << i->second << endl;
 
-    i = r.find(RangeIn(55,55));
+    i = r.find(RangeIn(55, 55));
     assert(i == r.end());
 
-    i = r.insert(RangeIn<Addr>(0,12),1);
+    i = r.insert(RangeIn(0, 12), 1);
     assert(i == r.end());
 
-    i = r.insert(RangeIn<Addr>(0,9),1);
+    i = r.insert(RangeIn(0, 9), 1);
     assert(i != r.end());
 
-    i = r.find(RangeIn(20,30));
+    i = r.find(RangeIn(20, 30));
     assert(i != r.end());
     cout << i->first << " " << i->second << endl;
 
+    return 0;
 }
-
-
-
-
-
-
-
-