Tage way tunning and enlarge L3 size to 32MB#811
Conversation
Change-Id: Ibe23fc57b667b597292eec6847b4760242aa372d
Change-Id: Ie3e72b9396ad0ca0a25d332ef2b1b4cffcac0c5a
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughConfiguration parameters for the KMH v3 system are enhanced: L3 cache size is now explicitly set to 32MB in the ideal configuration, and BTBTAGE branch predictor table geometry is tuned by increasing table T2–T6 capacities to 8192 entries and raising table T2 associativity to 4 ways. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| updateOnRead = Param.Bool(True, "Enable update on read, no need to save tage meta in FTQ") | ||
| numPredictors = Param.Unsigned(8, "Number of TAGE predictors") | ||
| tableSizes = VectorParam.Unsigned([2048, 2048, 2048, 2048, 2048, 2048, 2048,2048],"the TAGE T0~Tn length") | ||
| tableSizes = VectorParam.Unsigned([2048, 2048, 8192, 8192, 8192, 8192, 8192, 2048], "the TAGE T0~Tn length") |
There was a problem hiding this comment.
不要在这里修改tage 配置,应该在idealkmhv3.py 中修改,这里修改会让kmhv3.py 配置也被拉大了
🚀 Coremark Smoke Test Results
✅ Difftest smoke test passed! |
tage way tunning and enlarge L3 size to 32MB
Summary by CodeRabbit