Hello Pau Cabre,

I'd like you to reexamine a change. Please visit

    https://gem5-review.googlesource.com/7481

to look at the new patch set (#2).

Change subject: cpu-o3: Missing freeing the heads of DepGraph in IQ squashing
......................................................................

cpu-o3: Missing freeing the heads of DepGraph in IQ squashing

Free the squahsed instructions' heads of DepGraph in IQ squashing

In a system with large register file (ex.2048), the number of
DynInst hits the hardcoded limit (1500). This is caused by
missing freeing the heads of DepGraph in IQ. IQ only clears
out the heads when instructions reach writeback stage.
If a instruction is squashed before writeback stage, its head of
dependency graph, which holds the instruction's DynInstPtr,
would not be clared out. This prevents freeing the DynInst of the
squahsed instruction even after it is committed.

Change-Id: I05b3db93cb6ad8960183d7ae765149c7f292e5b3
---
M src/cpu/o3/inst_queue_impl.hh
1 file changed, 19 insertions(+), 0 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/7481
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I05b3db93cb6ad8960183d7ae765149c7f292e5b3
Gerrit-Change-Number: 7481
Gerrit-PatchSet: 2
Gerrit-Owner: Hanhwi Jang <jang.han...@gmail.com>
Gerrit-Reviewer: Pau Cabre <pau.ca...@metempsy.com>
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to