mem: Do not rely on the NeedsWritable flag for responses
authorAndreas Hansson <andreas.hansson@arm.com>
Thu, 31 Dec 2015 14:34:18 +0000 (09:34 -0500)
committerAndreas Hansson <andreas.hansson@arm.com>
Thu, 31 Dec 2015 14:34:18 +0000 (09:34 -0500)
commitc153b669fd5b19b45fe5a37484a64b88ee4905db
treed89fc2bf339392aac4780be7e99152a010a53c3d
parent7fca994d04230f1e41b9443b33f891460fc5368d
mem: Do not rely on the NeedsWritable flag for responses

This patch removes the NeedsWritable flag for all responses, as it is
really only the request that needs a writable response. The response,
on the other hand, should in these cases always provide the line in a
writable state, as indicated by the hasSharers flag not being set.

When we send requests that has NeedsWritable set, the response will
always have the hasSharers flag not set. Additionally, there are cases
where the request did not have NeedsWritable set, and we still get a
writable response with the hasSharers flag not set. This never happens
on snoops, but is used by downstream caches to pass ownership
upstream.

As part of this patch, the affected response types are updated, and
the snoop filter is similarly modified to check only the hasSharers
flag (as it should). A sanity check is also added to the packet class,
asserting that we never look at the NeedsWritable flag for responses.

No regressions are affected.
src/mem/packet.cc
src/mem/packet.hh
src/mem/snoop_filter.cc