Deleted Added
sdiff udiff text old ( 12429:beefb9f5f551 ) new ( 14005:6cad91d6136c )
full compact
1/*
2 * Copyright (c) 2013-2017 ARM Limited
3 * All rights reserved
4 *
5 * The license below extends only to copyright in the software and shall
6 * not be construed as granting a license to any other intellectual
7 * property including but not limited to intellectual property relating
8 * to a hardware implementation of the functionality of the software
9 * licensed hereunder. You may use the software subject to the license
10 * terms below provided that you ensure that this notice is replicated

--- 33 unchanged lines hidden (view full) ---

44
45#include "mem/snoop_filter.hh"
46
47#include "base/logging.hh"
48#include "base/trace.hh"
49#include "debug/SnoopFilter.hh"
50#include "sim/system.hh"
51
52void
53SnoopFilter::eraseIfNullEntry(SnoopFilterCache::iterator& sf_it)
54{
55 SnoopItem& sf_item = sf_it->second;
56 if (!(sf_item.requested | sf_item.holder)) {
57 cachedLocations.erase(sf_it);
58 DPRINTF(SnoopFilter, "%s: Removed SF entry.\n",
59 __func__);
60 }
61}
62
63std::pair<SnoopFilter::SnoopList, Cycles>
64SnoopFilter::lookupRequest(const Packet* cpkt, const SlavePort& slave_port)

--- 26 unchanged lines hidden (view full) ---

91
92 // Store unmodified value of snoop filter item in temp storage in
93 // case we need to revert because of a send retry in
94 // updateRequest.
95 retryItem = sf_item;
96
97 totRequests++;
98 if (is_hit) {
99 // Single bit set -> value is a power of two
100 if (isPow2(interested))
101 hitSingleRequests++;
102 else
103 hitMultiRequests++;
104 }
105
106 DPRINTF(SnoopFilter, "%s: SF value %x.%x\n",
107 __func__, sf_item.requested, sf_item.holder);
108
109 // If we are not allocating, we are done
110 if (!allocate)
111 return snoopSelected(maskToPortList(interested & ~req_port),
112 lookupLatency);
113
114 if (cpkt->needsResponse()) {
115 if (!cpkt->cacheResponding()) {
116 // Max one request per address per port
117 panic_if(sf_item.requested & req_port, "double request :( " \
118 "SF value %x.%x\n", sf_item.requested, sf_item.holder);
119
120 // Mark in-flight requests to distinguish later on
121 sf_item.requested |= req_port;
122 DPRINTF(SnoopFilter, "%s: new SF value %x.%x\n",
123 __func__, sf_item.requested, sf_item.holder);
124 } else {
125 // NOTE: The memInhibit might have been asserted by a cache closer
126 // to the CPU, already -> the response will not be seen by this
127 // filter -> we do not need to keep the in-flight request, but make
128 // sure that we know that that cluster has a copy
129 panic_if(!(sf_item.holder & req_port), "Need to hold the value!");
130 DPRINTF(SnoopFilter,
131 "%s: not marking request. SF value %x.%x\n",
132 __func__, sf_item.requested, sf_item.holder);
133 }
134 } else { // if (!cpkt->needsResponse())
135 assert(cpkt->isEviction());
136 // make sure that the sender actually had the line
137 panic_if(!(sf_item.holder & req_port), "requester %x is not a " \
138 "holder :( SF value %x.%x\n", req_port,
139 sf_item.requested, sf_item.holder);
140 // CleanEvicts and Writebacks -> the sender and all caches above
141 // it may not have the line anymore.
142 if (!cpkt->isBlockCached()) {
143 sf_item.holder &= ~req_port;
144 DPRINTF(SnoopFilter, "%s: new SF value %x.%x\n",
145 __func__, sf_item.requested, sf_item.holder);

--- 55 unchanged lines hidden (view full) ---

201 SnoopItem& sf_item = sf_it->second;
202
203 DPRINTF(SnoopFilter, "%s: old SF value %x.%x\n",
204 __func__, sf_item.requested, sf_item.holder);
205
206 SnoopMask interested = (sf_item.holder | sf_item.requested);
207
208 totSnoops++;
209 // Single bit set -> value is a power of two
210 if (isPow2(interested))
211 hitSingleSnoops++;
212 else
213 hitMultiSnoops++;
214
215 // ReadEx and Writes require both invalidation and exlusivity, while reads
216 // require neither. Writebacks on the other hand require exclusivity but
217 // not the invalidation. Previously Writebacks did not generate upward
218 // snoops so this was never an issue. Now that Writebacks generate snoops
219 // we need a special case for Writebacks. Additionally cache maintenance
220 // operations can generate snoops as they clean and/or invalidate all
221 // caches down to the specified point of reference.
222 assert(cpkt->isWriteback() || cpkt->req->isUncacheable() ||
223 (cpkt->isInvalidate() == cpkt->needsWritable()) ||
224 cpkt->req->isCacheMaintenance());
225 if (cpkt->isInvalidate() && !sf_item.requested) {
226 // Early clear of the holder, if no other request is currently going on
227 // @todo: This should possibly be updated even though we do not filter
228 // upward snoops
229 sf_item.holder = 0;
230 }
231
232 eraseIfNullEntry(sf_it);
233 DPRINTF(SnoopFilter, "%s: new SF value %x.%x interest: %x \n",

--- 27 unchanged lines hidden (view full) ---

261 SnoopMask rsp_mask = portToMask(rsp_port);
262 SnoopMask req_mask = portToMask(req_port);
263 SnoopItem& sf_item = cachedLocations[line_addr];
264
265 DPRINTF(SnoopFilter, "%s: old SF value %x.%x\n",
266 __func__, sf_item.requested, sf_item.holder);
267
268 // The source should have the line
269 panic_if(!(sf_item.holder & rsp_mask), "SF value %x.%x does not have "\
270 "the line\n", sf_item.requested, sf_item.holder);
271
272 // The destination should have had a request in
273 panic_if(!(sf_item.requested & req_mask), "SF value %x.%x missing "\
274 "the original request\n", sf_item.requested, sf_item.holder);
275
276 // If the snoop response has no sharers the line is passed in
277 // Modified state, and we know that there are no other copies, or
278 // they will all be invalidated imminently
279 if (!cpkt->hasSharers()) {
280 DPRINTF(SnoopFilter,
281 "%s: dropping %x because non-shared snoop "
282 "response SF val: %x.%x\n", __func__, rsp_mask,
283 sf_item.requested, sf_item.holder);
284 sf_item.holder = 0;
285 }
286 assert(!cpkt->isWriteback());
287 // @todo Deal with invalidating responses
288 sf_item.holder |= req_mask;
289 sf_item.requested &= ~req_mask;
290 assert(sf_item.requested | sf_item.holder);
291 DPRINTF(SnoopFilter, "%s: new SF value %x.%x\n",
292 __func__, sf_item.requested, sf_item.holder);
293}
294
295void
296SnoopFilter::updateSnoopForward(const Packet* cpkt,
297 const SlavePort& rsp_port, const MasterPort& req_port)
298{

--- 55 unchanged lines hidden (view full) ---

354
355 SnoopMask slave_mask = portToMask(slave_port);
356 SnoopItem& sf_item = sf_it->second;
357
358 DPRINTF(SnoopFilter, "%s: old SF value %x.%x\n",
359 __func__, sf_item.requested, sf_item.holder);
360
361 // Make sure we have seen the actual request, too
362 panic_if(!(sf_item.requested & slave_mask), "SF value %x.%x missing "\
363 "request bit\n", sf_item.requested, sf_item.holder);
364
365 sf_item.requested &= ~slave_mask;
366 // Update the residency of the cache line.
367
368 if (cpkt->req->isCacheMaintenance()) {
369 // A cache clean response does not carry any data so it
370 // shouldn't change the holders, unless it is invalidating.
371 if (cpkt->isInvalidate()) {
372 sf_item.holder &= ~slave_mask;
373 }
374 eraseIfNullEntry(sf_it);
375 } else {
376 // Any other response implies that a cache above will have the
377 // block.
378 sf_item.holder |= slave_mask;
379 assert(sf_item.holder | sf_item.requested);
380 }
381 DPRINTF(SnoopFilter, "%s: new SF value %x.%x\n",
382 __func__, sf_item.requested, sf_item.holder);
383}
384
385void
386SnoopFilter::regStats()
387{

--- 36 unchanged lines hidden ---