mem: Only track snooping ports in the snoop filter

Information
Submitter:	Andreas Hansson
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 11064:900d7faf3522
---------------------------
mem: Only track snooping ports in the snoop filter

This patch changes the tracking of ports in the snoop filter to use
local dense port IDs so that we can have 64 snooping ports (rather
than crossbar slave ports). This is achieved by adding a simple
remapping vector that translates the actal port IDs into the local
slave IDs used in the SnoopMask.

Ultimately this patch allows us to scale to much larger systems
without introducing a hierarchy of crossbars.

Testing Done

So you are assuming that most ports do not need snooping.

Andreas Hansson Sept. 23, 2015, 10:07 a.m. (Sept. 23, 2015, 10:07 a.m.)

The assumption (and common case) is that some ports do not need snooping.

src/mem/coherent_xbar.cc (Diff revision 1)

Does the placement of these lines matter?

Andreas Hansson Sept. 23, 2015, 10:07 a.m. (Sept. 23, 2015, 10:07 a.m.)

Not beyond the fact that it has to happen in init when the ports are connected...

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Status: Closed (submitted)