From 15a979c6be704a4bb083b64148d1a25d7fc4e682 Mon Sep 17 00:00:00 2001 From: Andreas Hansson Date: Mon, 7 Jan 2013 13:05:38 -0500 Subject: [PATCH] mem: Tidy up bus addr range debug messages This patch tidies up a number of the bus DPRINTFs related to range manipulation. In particular, it shifts the message about range changes to the start of the member function, and also adds information about when all ranges are received. --- src/mem/bus.cc | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/src/mem/bus.cc b/src/mem/bus.cc index e8a7084b0..a880eca8f 100644 --- a/src/mem/bus.cc +++ b/src/mem/bus.cc @@ -378,6 +378,9 @@ BaseBus::findPort(Addr addr) void BaseBus::recvRangeChange(PortID master_port_id) { + DPRINTF(BusAddrRanges, "Received range change from slave port %s\n", + masterPorts[master_port_id]->getSlavePort().name()); + // remember that we got a range from this master port and thus the // connected slave module gotAddrRanges[master_port_id] = true; @@ -391,15 +394,14 @@ BaseBus::recvRangeChange(PortID master_port_id) while (gotAllAddrRanges && r != gotAddrRanges.end()) { gotAllAddrRanges &= *r++; } + if (gotAllAddrRanges) + DPRINTF(BusAddrRanges, "Got address ranges from all slaves\n"); } // note that we could get the range from the default port at any // point in time, and we cannot assume that the default range is // set before the other ones are, so we do additional checks once // all ranges are provided - DPRINTF(BusAddrRanges, "received RangeChange from slave port %s\n", - masterPorts[master_port_id]->getSlavePort().name()); - if (master_port_id == defaultPortID) { // only update if we are indeed checking ranges for the // default port since the port might not have a valid range @@ -492,17 +494,24 @@ BaseBus::getAddrRanges() const // of the connected devices assert(gotAllAddrRanges); - DPRINTF(BusAddrRanges, "received address range request, returning:\n"); + // at the moment, this never happens, as there are no cycles in + // the range queries and no devices on the master side of a bus + // (CPU, cache, bridge etc) actually care about the ranges of the + // ports they are connected to + + DPRINTF(BusAddrRanges, "Received address range request, returning:\n"); // start out with the default range AddrRangeList ranges; - ranges.push_back(defaultRange); - DPRINTF(BusAddrRanges, " -- %s DEFAULT\n", defaultRange.to_string()); + if (useDefaultRange) { + ranges.push_back(defaultRange); + DPRINTF(BusAddrRanges, " -- Default %s\n", defaultRange.to_string()); + } // add any range that is not a subset of the default range for (PortMapConstIter p = portMap.begin(); p != portMap.end(); ++p) { if (useDefaultRange && p->first.isSubset(defaultRange)) { - DPRINTF(BusAddrRanges, " -- %s is a SUBSET\n", + DPRINTF(BusAddrRanges, " -- %s is a subset of default\n", p->first.to_string()); } else { ranges.push_back(p->first); -- 2.30.2