Push 2025 10 31 #979
Merged
Push 2025 10 31 #979
Conversation
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
Change-Id: If5930428706b7e11b2a3edc522f212ef6c13e99f
Co-authored-by: Copilot <[email protected]>
Change-Id: I81517f0145c04cf511c20c15c08a6d6c028b3dcb
Change-Id: Ifce75a0aa806f6ebc1f40cf7cf45485c31115951
Change-Id: If448cb04e501d5b5b072a4175ec32eea8ecec241
Change-Id: I19b97b736df11e0c2137b11e96e31db5730df43e
- Implement PcmIioOutputBuilder class hierarchy for flexible output formatting: * Add abstract base class PcmIioOutputBuilder * Create concrete implementations PcmIioCsvBuilder and PcmIioDisplayBuilder * Move display logic from main code into specialized builder classes - Replace legacy PCIeEventNameMap_t (std::map-based) with modern PCIeEventNameMap (std::unordered_map-based) throughout the codebase - Update all function signatures, implementations, and tests to use the new type system - Extract display formatting logic from main application flow, improving separation of concerns and making the code more maintainable - Apply consistent naming conventions across type aliases (EventName, CounterName, CounterValueMap, EventDefinition) - Improve performance through targeted use of unordered containers where appropriate - Simplify future output format additions via the new builder interface
Regroup the code to avoid creation maps etc by default and decrease memory consumption
Co-authored-by: rdementi <[email protected]>
Co-authored-by: rdementi <[email protected]>
Change-Id: I0aa944142eed2731cd5e6834c3c218058809ece8
This reverts commit d8cd1bf5b06d218f019f5f2d91f5d2a6b1596a45.
Implemented by duplicating all ARL patterns:
- Added PTL enum definition (PCM_CPU_FAMILY_MODEL(6, 204))
- Added 8 case PTL statements across cpucounters.h and cpucounters.cpp
- Added 7 conditional checks for PTL in both files
- Added PTL to CPU name mapping ("Panther Lake")
Co-authored-by: rdementi <[email protected]>
Co-authored-by: rdementi <[email protected]>
Co-authored-by: rdementi <[email protected]>
Change-Id: I784a9795d919f64661ee2a4b5238d254aef8ba48
Change-Id: I6885e0a19d532db44734c862b5f952b56946b590
Change-Id: I0141b6e3efae8796e598a9187e2ac63982380dd5
Change-Id: Ic55c850906d195d43a677d341ed05198cad40152
Change-Id: Id2f57310e094759f41487cb9defaa0e19ac9f400
opcm
approved these changes
Oct 31, 2025
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.
No description provided.