diff options
author | Andreas Hansson <andreas.hansson@arm.com> | 2015-07-30 03:41:40 -0400 |
---|---|---|
committer | Andreas Hansson <andreas.hansson@arm.com> | 2015-07-30 03:41:40 -0400 |
commit | 5a18e181ffb8fbef5f4aca8fb9a63ee6a7c9e0d6 (patch) | |
tree | e4d9f26feca3c415dabc4a013cb7cabf36a8c3ac /src/mem/packet.hh | |
parent | 5902e29e84037b6f0dd470c09a73f5590368d755 (diff) | |
download | gem5-5a18e181ffb8fbef5f4aca8fb9a63ee6a7c9e0d6.tar.xz |
mem: Transition away from isSupplyExclusive for writebacks
This patch changes how writebacks communicate whether the line is
passed as modified or owned. Previously we relied on the
isSupplyExclusive mechanism, which was originally designed to avoid
unecessary snoops.
For normal cache requests we use the sharedAsserted mechanism to
determine if a block should be marked writeable or not, and with this
patch we transition the writebacks to also use this
mechanism. Conceptually this is cleaner and more consistent.
Diffstat (limited to 'src/mem/packet.hh')
-rw-r--r-- | src/mem/packet.hh | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/mem/packet.hh b/src/mem/packet.hh index fe5e10d0a..d1774e5a8 100644 --- a/src/mem/packet.hh +++ b/src/mem/packet.hh @@ -500,7 +500,6 @@ class Packet : public Printable void setExpressSnoop() { flags.set(EXPRESS_SNOOP); } bool isExpressSnoop() const { return flags.isSet(EXPRESS_SNOOP); } void setSupplyExclusive() { flags.set(SUPPLY_EXCLUSIVE); } - void clearSupplyExclusive() { flags.clear(SUPPLY_EXCLUSIVE); } bool isSupplyExclusive() const { return flags.isSet(SUPPLY_EXCLUSIVE); } void setSuppressFuncError() { flags.set(SUPPRESS_FUNC_ERROR); } bool suppressFuncError() const { return flags.isSet(SUPPRESS_FUNC_ERROR); } |