cpu: Clarify meaning of cachePorts variable in lsq_unit.hh of O3
authorArthur Perais <arthur.perais@inria.fr>
Wed, 21 Dec 2016 21:04:06 +0000 (15:04 -0600)
committerArthur Perais <arthur.perais@inria.fr>
Wed, 21 Dec 2016 21:04:06 +0000 (15:04 -0600)
commite5fb6752d613a6f85e2f93b4c01836ac59a8c90c
tree7bec60d7645ed4a1d7e20dc8071c0dafd288b786
parent3a656da1a64f08d5e4c755e94cefda5a4e985a50
cpu: Clarify meaning of cachePorts variable in lsq_unit.hh of O3

cachePorts currently constrains the number of store packets written to the
D-Cache each cycle), but loads currently affect this variable. This leads
to unexpected congestion (e.g., setting cachePorts to a realistic 1 will
in fact allow a store to WB only if no loads have accessed the D-Cache
this cycle). In the absence of arbitration, this patch decouples how many
loads can be done per cycle from how many stores can be done per cycle.

Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
src/cpu/o3/O3CPU.py
src/cpu/o3/lsq_unit.hh
src/cpu/o3/lsq_unit_impl.hh