Skip to content

Commit

Permalink
Fix MegaBoomConfig by increasing icache associativity
Browse files Browse the repository at this point in the history
There is a requirement that we can only support 64 sets in the icache due to
aliasing bugs. Increase associativity and reduce the sets for icache to fix the
default MegaBoomConfig.
  • Loading branch information
pranith authored and jerryz123 committed Dec 21, 2018
1 parent 124ea89 commit 252c67a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/scala/common/configs.scala
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ class WithMegaBooms extends Config((site, here, up) => {
tage = Some(TageParameters())),
dcache = Some(DCacheParams(rowBits = site(SystemBusKey).beatBytes*8,
nSets=64, nWays=16, nMSHRs=8, nTLBEntries=32)),
icache = Some(ICacheParams(fetchBytes = 8*4, rowBits = site(SystemBusKey).beatBytes*8, nSets=128, nWays=4))
icache = Some(ICacheParams(fetchBytes = 8*4, rowBits = site(SystemBusKey).beatBytes*8, nSets=64, nWays=8))
)}
// Set TL network to 128bits wide
case SystemBusKey => up(SystemBusKey, site).copy(beatBytes = 16)
Expand Down

0 comments on commit 252c67a

Please sign in to comment.