Bobby R. Bruce has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/44086 )
Change subject: misc,mem-ruby: Fixing unused variable error for fast builds
......................................................................
misc,mem-ruby: Fixing unused variable error for fast builds
This fixes the broken compiler tests for .fast builds:
https://www.mail-archive.com/[email protected]/msg38412.html
Change-Id: Ibc377a57ce6455ca709003f326b0ca8d4c01377b
---
M src/mem/ruby/protocol/chi/CHI-cache-actions.sm
M src/mem/ruby/protocol/chi/CHI-cache-ports.sm
M src/mem/ruby/protocol/chi/CHI-mem.sm
3 files changed, 9 insertions(+), 10 deletions(-)
diff --git a/src/mem/ruby/protocol/chi/CHI-cache-actions.sm
b/src/mem/ruby/protocol/chi/CHI-cache-actions.sm
index ea5eaff..32fdff1 100644
--- a/src/mem/ruby/protocol/chi/CHI-cache-actions.sm
+++ b/src/mem/ruby/protocol/chi/CHI-cache-actions.sm
@@ -1844,10 +1844,10 @@
action(UpdateDataState_FromWUDataResp, desc="") {
assert(is_valid(tbe));
- int offset := addressOffset(tbe.accAddr, tbe.addr);
if (tbe.expected_req_resp.hasReceivedData()) {
- assert(tbe.dataBlkValid.test(offset));
- assert(tbe.dataBlkValid.test(offset + tbe.accSize - 1));
+ assert(tbe.dataBlkValid.test(addressOffset(tbe.accAddr, tbe.addr)));
+ assert(tbe.dataBlkValid.test(addressOffset(tbe.accAddr, tbe.addr)
+ + tbe.accSize - 1));
peek(datInPort, CHIDataMsg) {
assert(in_msg.type == CHIDataType:NCBWrData);
tbe.dataDirty := true;
@@ -2678,13 +2678,11 @@
// pick a victim to deallocate
Addr victim_addr := cache.cacheProbe(address);
- CacheEntry victim_entry := getCacheEntry(victim_addr);
- assert(is_valid(victim_entry));
TBE victim_tbe := getCurrentActiveTBE(victim_addr);
if (is_invalid(victim_tbe)) {
DPRINTF(RubySlicc, "Eviction for %#x victim: %#x state=%s\n",
- address, victim_addr, victim_entry.state);
+ address, victim_addr,
getCacheEntry(victim_addr));
enqueue(replTriggerOutPort, ReplacementMsg, 0) {
out_msg.addr := victim_addr;
out_msg.from_addr := address;
diff --git a/src/mem/ruby/protocol/chi/CHI-cache-ports.sm
b/src/mem/ruby/protocol/chi/CHI-cache-ports.sm
index 6a4fe5b..efba9bc 100644
--- a/src/mem/ruby/protocol/chi/CHI-cache-ports.sm
+++ b/src/mem/ruby/protocol/chi/CHI-cache-ports.sm
@@ -96,8 +96,8 @@
if (datInPort.isReady(clockEdge())) {
printResources();
peek(datInPort, CHIDataMsg) {
- int received := in_msg.bitMask.count();
- assert((received <= data_channel_size) && (received > 0));
+ assert((in_msg.bitMask.count() <= data_channel_size)
+ && (in_msg.bitMask.count() > 0));
trigger(dataToEvent(in_msg.type), in_msg.addr,
getCacheEntry(in_msg.addr),
getCurrentActiveTBE(in_msg.addr));
}
diff --git a/src/mem/ruby/protocol/chi/CHI-mem.sm
b/src/mem/ruby/protocol/chi/CHI-mem.sm
index 954a449..a3d04eb 100644
--- a/src/mem/ruby/protocol/chi/CHI-mem.sm
+++ b/src/mem/ruby/protocol/chi/CHI-mem.sm
@@ -365,8 +365,9 @@
if (datInPort.isReady(clockEdge())) {
printResources();
peek(datInPort, CHIDataMsg) {
- int received := in_msg.bitMask.count();
- assert((received <= data_channel_size) && (received > 0));
+ //int received := in_msg.bitMask.count();
+ assert((in_msg.bitMask.count() <= data_channel_size)
+ && (in_msg.bitMask.count() > 0));
trigger(dataToEvent(in_msg.type), in_msg.addr, TBEs[in_msg.addr]);
}
}
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44086
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: Ibc377a57ce6455ca709003f326b0ca8d4c01377b
Gerrit-Change-Number: 44086
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
