Skip to content

Use fdinfo's engine count when refreshing utilisation rate#387

Merged
Syllo merged 1 commit intoSyllo:masterfrom
larunbe:utilisation-rate-use-engine-count
Jun 28, 2025
Merged

Use fdinfo's engine count when refreshing utilisation rate#387
Syllo merged 1 commit intoSyllo:masterfrom
larunbe:utilisation-rate-use-engine-count

Conversation

@larunbe
Copy link
Contributor

@larunbe larunbe commented May 23, 2025

I originally moved gpuinfo_refresh_utilisation_rate() from Mali code into src/extract_gpuinfo.c when I realised utilisation rate could be calculated in a device-independent way simply by following the percentage utilisation guidelines given in
Documentation/gpu/drm-usage-stats.rst

However, I forgot to replace the magic number '2' which stood for the engine count in Mali GPUs with a value that make sense for different devices.

Source the engine count from gpu_info's static information values.

I originally moved gpuinfo_refresh_utilisation_rate() from Mali code
into src/extract_gpuinfo.c when I realised utilisation rate could be
calculated in a device-independent way simply by following the
percentage utilisation guidelines given in
Documentation/gpu/drm-usage-stats.rst

However, I forgot to replace the magic number '2' which stood for the
engine count in Mali GPUs with a value that make sense for different
devices.

Source the engine count from gpu_info's static information values.

Signed-off-by: Adrián Larumbe <adrian.larumbe@collabora.com>
@chealy
Copy link

chealy commented Jun 5, 2025

+1

@Syllo Syllo merged commit a48ce79 into Syllo:master Jun 28, 2025
3 checks passed
@Syllo
Copy link
Owner

Syllo commented Jun 28, 2025

Thanks for the patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants