Repository navigation
fix(data/cpu): correct the Xeon X5698's boost clock and release date - #380
Merged
Merged
Conversation
boost_clock_ghz was 1.0, below its 4.4 GHz base: Tier 0's only hard violation (!boost_ge_base). Wikipedia's Westmere-EP table gives multiplier 33 and turbo bins 0/1, so the top turbo is 34 x 133.33 MHz = 4.53 GHz, computed the same way as its siblings (X5690 3.73, X5687 3.86). release_date was 2013-06-01; the same table dates the part Q1 2011, so it is now 2011-01-01 and the file moves from 2013/ to 2011/ as the path rule requires. Refs #296
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
This fixes
cpu/intel/…/intel-xeon-x5698.json, the only record that breaks a hard Tier 0 rule (!boost_ge_base, forced red).boost_clock_ghzrelease_date2013/enterprise/2011/enterprise/(the year folder must matchrelease_date)0/1. The top turbo is therefore 34 × 133.33 MHz ≈ 4.53 GHz. That is the same calculation behind its siblings' stored values: X5690 at 3.73 and X5687 at 3.86.Closes #296
Source
List of Intel Xeon processors (Nehalem-based), "Westmere-EP", Dual Core:
model=Xeon X5698 | cores=2 | l3=12 | mult=33 | turbo=0/1 | tdp=130 | date=Q1, 2011. The record already cites this page. Intel never gave this off-roadmap part an ARK page.Checks
--changed-since develop.integrity_check.py --strictreports no hard anomalies.Checklist
develop(mainonly moves through release PRs)python -m app.validatepasses locallydata/<category>/<...>/<slug>.jsonpathsource_urlscites at least one canonical referenceprojectscope, and Copilot is out of quota, so it was reviewed by hand.