diff options
| author | David Francis <David.Francis@amd.com> | 2026-08-05 09:51:35 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2026-09-17 11:38:28 -0400 |
| commit | 4f18c56630383c14bfc6b2d65f88f2f895d2121a (patch) | |
| tree | 3598324075f02594ad9f4680e59dd1af71b6f6ec /tools/perf/scripts/python/stackcollapse.py | |
| parent | 7ea6a47224e2c6e89a3a682d7fbaace4817a55aa (diff) | |
| download | linux-4f18c56630383c14bfc6b2d65f88f2f895d2121a.tar.gz linux-4f18c56630383c14bfc6b2d65f88f2f895d2121a.zip | |
drm/amdkfd: Avoid integer underflow with ffs in EOP ring size calc
The low 6 bits of cp_hqd_eop_control store the base-2 logarithm
of the EOP ring size. This was calculated as
ffs(q->eop_ring_buffer_size / sizeof(unsigned int)) - 1 - 1
But ffs can in theory return 1 or 0, so this could underflow
(although in practice the ring buffer size cannot be less than 4096).
Change this to
ffs(q->eop_ring_buffer_size / sizeof(unsigned int) / 4)
using properties of logarithms.
Reviewed-by: Kent Russell <kent.russell@amd.com>
Signed-off-by: David Francis <David.Francis@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions
