labels: Add ASRockRack ROMED8-2T memory topology#232
Closed
csmarshall wants to merge 4 commits intomchehab:masterfrom
Closed
labels: Add ASRockRack ROMED8-2T memory topology#232csmarshall wants to merge 4 commits intomchehab:masterfrom
csmarshall wants to merge 4 commits intomchehab:masterfrom
Conversation
Add memory slot labels for ASRockRack ROMED8-2T motherboard. This board has 8 memory slots (A1-H1) supporting DDR4 ECC memory. Optimal 4-DIMM configuration uses slots A1, B1, G1, H1. Each 64GB DIMM spans two memory controller rows (csrow0/csrow1) requiring dual coordinate mapping per physical slot. Tested on system with 4x 64GB DDR4-3200 ECC modules. Signed-off-by: Charles Marshall <charles@wozi.com>
Contributor
Author
|
Here's the layout: And a dmidecode (with some stuff commented out obviously): |
When using --error-count, consolidate entries that share the same DIMM label and sum their CE/UE counts. This provides cleaner output when physical DIMMs span multiple ranks, showing one line per physical DIMM slot instead of duplicate labels. Add --per-rank option to show each rank separately with location info (csrow/channel) for cases where detailed per-rank error tracking is needed. Signed-off-by: Charles Marshall <charles@wozi.com>
Pad labels to consistent width so CE/UE columns align properly. Account for location suffix length when using --per-rank. Signed-off-by: Charles Marshall <charles@wozi.com>
Contributor
Author
|
On systems where each physical DIMM spans multiple ranks (like the ROMED8-2T with 64GB DIMMs using 2 csrows each), the --error-count output showed each rank separately with the same label - resulting in confusing duplicate entries: Since what you actually care about is "which physical DIMM has errors", consolidating by label and summing the counts gives cleaner, actionable output: The --per-rank option preserves the detailed view (with location info) for cases where you need to see which specific rank within a DIMM is reporting errors. |
Owner
|
Merged, thanks! |
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
2 participants
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.
Add memory slot labels for ASRockRack ROMED8-2T motherboard. This board has 8 memory slots (A1-H1) supporting DDR4 ECC memory. Optimal 4-DIMM configuration uses slots A1, B1, G1, H1.
Each 64GB DIMM spans two memory controller rows (csrow0/csrow1) requiring dual coordinate mapping per physical slot.
Tested on system with 4x 64GB DDR4-3200 ECC modules.