(improv) Report rules under thier respective modules#293
Merged
chetan-rathore merged 1 commit intoARM-software:mainfrom Mar 23, 2026
Merged
(improv) Report rules under thier respective modules#293chetan-rathore merged 1 commit intoARM-software:mainfrom
chetan-rathore merged 1 commit intoARM-software:mainfrom
Conversation
- When the first rule of a module is not implemented, it is getting reported as part of previous module in logs. - Due to this even in systemready log parser the rule is getting reported as part of different modules - Corrected this by reordering the enum to have first executable rule of that module at start - Defined GPU module to report rules under the SBSA l8 gpu module Other improv - When BSA/SBSA is run at highest verbosity, couple of PCIe prints uses multiple colons which breaks log parser - The colons are not bringing any extra readability, having a space is good enough - TODO in future - Modules which doesn't have even a single executable rule like TME are reported under PCIe right now - This will be handled in future as it needs update to RBX f/w Change-Id: I90520763a8ac9fe5b5daf0475d81d5200a7fa9bd
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
1 participant
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.
Other improv
When BSA/SBSA is run at highest verbosity, couple of PCIe prints uses multiple colons which breaks log parser
The colons are not bringing any extra readability, having a space is good enough
TODO in future
Change-Id: I90520763a8ac9fe5b5daf0475d81d5200a7fa9bd