diff options
| author | Andreas Hansson <andreas.hansson@arm.com> | 2015-11-06 03:26:33 -0500 |
|---|---|---|
| committer | Andreas Hansson <andreas.hansson@arm.com> | 2015-11-06 03:26:33 -0500 |
| commit | 8e55d51aaa71d71c7058e8ee15c89d3482991ba2 (patch) | |
| tree | 10da79e95be55efe6bf2bf0e18a103ea8997aae3 /src/mem/packet.cc | |
| parent | ac1368df50af123b32b41d7115ea4a0f15f7c97f (diff) | |
| download | gem5-8e55d51aaa71d71c7058e8ee15c89d3482991ba2.tar.xz | |
mem: Do not treat CleanEvict as a write operation
This patch changes the CleanEvict command type to not be considered a
write. Initially it was made a zero-sized write to match the writeback
command, but as things developed it became clear that it causes more
problems than it solves. For example, the memory modules (and bridge)
should not consider the CleanEvict as a write, but instead discard
it. With this patch it will be neither a read, nor write, and as it
does not need a response the slave will simply sink it.
Diffstat (limited to 'src/mem/packet.cc')
| -rw-r--r-- | src/mem/packet.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mem/packet.cc b/src/mem/packet.cc index b1e6dcd63..80b079138 100644 --- a/src/mem/packet.cc +++ b/src/mem/packet.cc @@ -88,7 +88,7 @@ MemCmd::commandInfo[] = { SET4(IsWrite, NeedsExclusive, IsRequest, HasData), InvalidCmd, "Writeback" }, /* CleanEvict */ - { SET2(IsWrite, IsRequest), InvalidCmd, "CleanEvict" }, + { SET1(IsRequest), InvalidCmd, "CleanEvict" }, /* SoftPFReq */ { SET4(IsRead, IsRequest, IsSWPrefetch, NeedsResponse), SoftPFResp, "SoftPFReq" }, |