diff options
| author | David Francis <David.Francis@amd.com> | 2026-08-05 09:16:51 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2026-09-17 11:56:45 -0400 |
| commit | 8ee521b8b189799e361d4233c5180ba56656d4d4 (patch) | |
| tree | c1a06c284971ad572cbbb0e3fc5c25cf738bfbd6 /tools/perf/scripts/python | |
| parent | c883d0a132d430ef7ebb23fd94323be94d0fbdb8 (diff) | |
| download | linux-next-8ee521b8b189799e361d4233c5180ba56656d4d4.tar.gz linux-next-8ee521b8b189799e361d4233c5180ba56656d4d4.zip | |
drm/amdkfd: Avoid integer underflow in EOP ring size calculation.
The low 6 bits of cp_hqd_eop_control store the base-2 logarithm
of the EOP ring size. This was calculated as
order_base_2(q->eop_ring_buffer_size / 4) - 1
But order_base_2 can in theory return 0, so this could underflow
(although in practice the ring buffer size cannot be less than 4096).
Change this to
order_base_2(q->eop_ring_buffer_size / 8)
using properties of logarithms.
Also add to the above comment to make the mathematics more clear.
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>
(cherry picked from commit f0f43fcf8b2b3a924cad9444340921c96ed5f634)
Cc: stable@vger.kernel.org
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
