Add first simple test #360
Draft
Conversation
This file contains 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
Add a function for printing the perf_event_attr of Event's. This is incredibly useful for debugging. Printing out the perf_event_attr is triggered on event open and is only printed when the log level trace is specified Move pre-defined event table from global to function scope, so that the events in it do not get initialized before we set up logging.
Somehow, base_path does not get initialized correctly before read_proc is called, so move it all into read_proc
Also move use-case specific tracepoint code out of TracepointEvent
- Make Event clean base class. Use SimpleEvent when the event configuration needs to be specified manually. - Move almost all logic outside of the Event class. Event Configuration should happen fully from the outside - fix boolean operators - move sysfs specific code from Event to SysfsEvent
print [CPUS 0-0] as [CPU 0]
Designwise, the EventConfig class should handle all the ugly details of configuring the Events from the given Config(), such as setting specific bits. The sample::Reader for example asks the EventConfig for a sample event. The EventConfig then selects the correct sampling event using the EventProvider and the Config, configures all the options, such as sample_period and then gives it back to the sample::Reader. The sample::Reader does not need to change anything about the event. The EventProvider should only provide events, reading them from different sources, such as pre-defined, PFM, PMU events. The CounterProvider did not do anything special enough to warrant its continuing existence, merge it with EventConfig Smaller fixes - delete needless getter in CounterCollection - provide default values for bools in Config() - check exclude_kernel once in EventConfig() instead of for every event
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.
This is a proposal for a first simple test, testing if sampling works.
Sadly, I think I can throw the GitHub Action Idea into the bin because PMUs are not passed to the VM, and working around that (by only testing perf software events) seems to be like inviting a whole lot of weirdness into the test process that is not worth the hassle.