On Tue, Dec 16, 2025 at 01:00:32PM -0800, Ian Rogers wrote:

On Fri, Dec 12, 2025 at 7:32 AM James Clark [off-list ref] wrote:

quoted

The specific config field that an event format attribute is in is
consistently hard coded, even though the API is supposed to be that the
driver publishes the config field name. To stop this pattern from being
copy pasted and causing problems in the future, replace them all with
calls to a new helper that returns the value that a user set.

This reveals some issues in evsel__set_config_if_unset(). It doesn't
work with sparse bitfields, which are an unused but documented feature.
And it also only writes to the attr.config field. To fix it we need to
start tracking user changes for all config fields and then use existing
helper functions that support sparse bitfields. Some other refactoring
was also required and a test was added.

Signed-off-by: James Clark <james.clark@linaro.org>

Outside of some nits, for the series:
Reviewed-by: Ian Rogers <irogers@google.com>

Are you ok with v4?

- Arnaldo