Set size of perf_attr_struct prior to getting pfm encoding #88
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.
Pull Request Description
Simple fix to ensure correct size of struct gets mem copied in
pfm_lib_perf_event_encode
in libpfm4.Events from pfm4 which contain values in the config2 field of the
perf_event_attr
struct seem to not get copied inpfm_lib_perf_event_encode
when opening up a PAPI event.The general trace of this issue is:
PERF_ATTR_SIZE_VER0
being used as the default size used to memcpy theperf_event_attr
which the user gets back from the function here.