From e4086878f62f5eacb84b9bf2183975e92ff564f5 Mon Sep 17 00:00:00 2001 From: Mitch Hayenga Date: Tue, 1 Apr 2014 14:22:06 -0500 Subject: cpu: Fix case where o3 lsq could print out uninitialized data In the O3 LSQ, data read/written is printed out in DPRINTFs. However, the data field is treated as a character string with a null terminated. However the data field is not encoded this way. This patch removes that possibility by removing the data part of the print. --- src/cpu/o3/lsq_unit.hh | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'src/cpu/o3/lsq_unit.hh') diff --git a/src/cpu/o3/lsq_unit.hh b/src/cpu/o3/lsq_unit.hh index 724d2cb0b..0978e721b 100644 --- a/src/cpu/o3/lsq_unit.hh +++ b/src/cpu/o3/lsq_unit.hh @@ -711,8 +711,7 @@ LSQUnit::read(Request *req, Request *sreqLow, Request *sreqHigh, storeQueue[store_idx].data + shift_amt, req->getSize()); DPRINTF(LSQUnit, "Forwarding from store idx %i to load to " - "addr %#x, data %#x\n", - store_idx, req->getVaddr(), data); + "addr %#x\n", store_idx, req->getVaddr()); PacketPtr data_pkt = new Packet(req, MemCmd::ReadReq); data_pkt->dataStatic(load_inst->memData); @@ -916,9 +915,9 @@ LSQUnit::write(Request *req, Request *sreqLow, Request *sreqHigh, { assert(storeQueue[store_idx].inst); - DPRINTF(LSQUnit, "Doing write to store idx %i, addr %#x data %#x" + DPRINTF(LSQUnit, "Doing write to store idx %i, addr %#x" " | storeHead:%i [sn:%i]\n", - store_idx, req->getPaddr(), data, storeHead, + store_idx, req->getPaddr(), storeHead, storeQueue[store_idx].inst->seqNum); storeQueue[store_idx].req = req; -- cgit v1.2.3