summaryrefslogtreecommitdiff
path: root/src/base/channel_addr.test.cc
diff options
context:
space:
mode:
authorAndreas Sandberg <andreas.sandberg@arm.com>2019-10-09 09:45:51 +0100
committerAndreas Sandberg <andreas.sandberg@arm.com>2019-10-18 09:45:01 +0000
commite0419b2530d2a63af9c55cc3d370719e65baca1f (patch)
tree4645794362f9cabf4d5f246cd0dc93425b3b6941 /src/base/channel_addr.test.cc
parent85e7a59cf0fa0faf4ba6ce61207a863d8840c453 (diff)
downloadgem5-e0419b2530d2a63af9c55cc3d370719e65baca1f.tar.xz
base: Add classes that encapsulate a channel address
There are cases where the memory system needs to reason about channel-local addresses. These are currently represented using the Addr and AddrRange classes. This is not ideal since it doesn't provide any type safety when working with global addresses and channel-local addresses. This is particularly problematic when porting existing components to work in multi-channel configurations. This changeset introduces the new ChannelAddr and ChannelAddrRange classes. These classes encapsulate channel-local addresses in a contiguous address space. These can, for example, be used in a memory controller to represent a flat address space when calculating timings or in a sectored cache. Change-Id: I45d4061ebc8507a10d0a4577b28796dc5ec7a469 Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com> Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/21600 Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu> Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br> Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com> Reviewed-by: Jason Lowe-Power <jason@lowepower.com> Maintainer: Jason Lowe-Power <jason@lowepower.com> Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/base/channel_addr.test.cc')
-rw-r--r--src/base/channel_addr.test.cc66
1 files changed, 66 insertions, 0 deletions
diff --git a/src/base/channel_addr.test.cc b/src/base/channel_addr.test.cc
new file mode 100644
index 000000000..877b9211e
--- /dev/null
+++ b/src/base/channel_addr.test.cc
@@ -0,0 +1,66 @@
+/*
+ * Copyright (c) 2019 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.
+ *
+ * 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: Andreas Sandberg
+ */
+
+#include <gtest/gtest.h>
+
+#include "base/channel_addr.hh"
+
+/* Default range should be invalid */
+TEST(ChannelAddrRange, DefaultInvalid)
+{
+ ChannelAddrRange default_range;
+
+ EXPECT_FALSE(default_range.valid());
+}
+
+/* Ensure that range bounds are inclusive */
+TEST(ChannelAddrRange, Range)
+{
+ ChannelAddrRange range(ChannelAddr(1), ChannelAddr(3));
+
+ EXPECT_FALSE(range.contains(ChannelAddr(0)));
+ EXPECT_TRUE(range.contains(ChannelAddr(1)));
+ EXPECT_TRUE(range.contains(ChannelAddr(2)));
+ EXPECT_TRUE(range.contains(ChannelAddr(3)));
+ EXPECT_FALSE(range.contains(ChannelAddr(4)));
+
+ EXPECT_EQ(range.start(), ChannelAddr(1));
+ EXPECT_EQ(range.end(), ChannelAddr(3));
+ EXPECT_EQ(range.size(), ChannelAddr(3));
+}