Review Board 2.0.15


mem: Update mostly exclusive policy even further

Review Request #3561 - Created July 12, 2016 and submitted

Information
Andreas Sandberg
gem5
default
Reviewers
Default
Changeset 11563:7bdfd47d54da
---------------------------
mem: Update mostly exclusive policy even further

This patch takes yet another step in maintaining the clusivity, in
that it allows a mostly-inclusive cache to hold on to blocks even when
responding to a ReadExReq or UpgradeReq. Previously the cache simply
invalidated these blocks, but there is no strict need to do so.

The most important part of this patch is that we simply mark the block
clean when satisfying the upstream request where the cache is allowed
to keep the block. The only tricky part of the patch is in the memory
management of deferred snoops, where we need to distinguish the cases
where only the packet was copied (we expected to respond), and the
cases where we created an entirely new packet and request (we kept it
only to replay later).

The code in satisfyRequest is definitely ready for some refactoring
after this.

Change-Id: I201ddc7b2582eaa46fb8cff0c7ad09e02d64b0fc
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>

   
Review request changed
Updated (Aug. 12, 2016, 6:07 a.m.)

Status: Closed (submitted)