diff options
author | Andreas Sandberg <andreas.sandberg@arm.com> | 2019-10-09 09:45:51 +0100 |
---|---|---|
committer | Andreas Sandberg <andreas.sandberg@arm.com> | 2019-10-18 09:45:01 +0000 |
commit | e0419b2530d2a63af9c55cc3d370719e65baca1f (patch) | |
tree | 4645794362f9cabf4d5f246cd0dc93425b3b6941 /src/base/channel_addr.test.cc | |
parent | 85e7a59cf0fa0faf4ba6ce61207a863d8840c453 (diff) | |
download | gem5-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.cc | 66 |
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)); +} |