You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@gerddie raised this issue with the current INIT API that once we support a shared library C API it will not work well, because only the first call to init will get to set the config options.
Design alternatives:
Same as perfetto: passed in parameters to percetto_init.
Config file loaded during percetto_init -- if it is found it is used as defaults.
Build option to specify default values.
Update perfetto to handle dynamic changes to these settings to compromise between different options for different libraries. ie: if mesa wants shmem_size_hint_kb of 400 and virglrenderer wants 200, perfetto could dynamically update it to the greater of the two options -- 400.
A combination of the above.
The text was updated successfully, but these errors were encountered:
Note that with the current static library design of percetto, this isn't a problem. Mesa and virglrenderer, for example, should have their own percetto instance with their own config. That actually works well because those projects can choose perfetto config options based on their own tracing usage patterns.
Allow overriding perfetto config options via env vars.
With the current perfetto SDK design, another option similar to a config file is environment variables that override config options. For an application like crosvm that links against multiple libraries that use perfetto, the system would run it with environment variables that override each necessary perfetto option. This is slightly cleaner than a config file on systems where there are no commonly accessible file paths for all the security contexts where perfetto runs.
@gerddie raised this issue with the current INIT API that once we support a shared library C API it will not work well, because only the first call to init will get to set the config options.
Design alternatives:
The text was updated successfully, but these errors were encountered: