[PATCH 6/6] trace: add PMU
Tomasz Duszynski
tduszynski at marvell.com
Mon Jun 16 11:49:03 CEST 2025
>16/06/2025 08:53, Tomasz Duszynski:
>> @@ -86,6 +86,7 @@ always_enable = [
>> 'ring',
>> 'stack',
>> 'telemetry',
>> + 'pmu',
>> ]
>
>This list is alphabetically sorted.
Right, missed that.
>
>What is the justification for always enabling PMU lib?
>Is it a good idea to always enable an experimental library?
Well, since on Linux eal depends on pmu, disabling pmu ends up disabling
eal, which breaks minimal build.
As for the second question - no. I think ideally lib should remain
optional. However, since trace is part of eal, that indirectly forces
pmu to be built.
Alternatively, I guess we could make the library explicitly optional by
extra meson option. But given that there are already options for
controlling libraries that approach is not perfect either.
More information about the dev
mailing list