summaryrefslogtreecommitdiff
path: root/tools/perf/scripts/python/bin/stackcollapse-report
diff options
context:
space:
mode:
authorJames Lin <PingLei.Lin@amd.com>2026-08-13 16:50:28 +0800
committerAlex Deucher <alexander.deucher@amd.com>2026-09-17 11:38:27 -0400
commit0575f53925e832fba376583cd18667de8f31b13b (patch)
tree3d93e5dccb0abe222fe64d4946627404d31e32c0 /tools/perf/scripts/python/bin/stackcollapse-report
parentd4e5df4d7ec5cf163077f389e85d454d996c7805 (diff)
downloadlinux-0575f53925e832fba376583cd18667de8f31b13b.tar.gz
linux-0575f53925e832fba376583cd18667de8f31b13b.zip
drm/amd/display: Cast DP DTO pixel clock math to avoid overflow and narrowing
[Why] Commit 0421fc6ab3a8 ("drm/amd/display: fix __udivdi3 link error") switched get_dp_dto_frequency_100hz() and dcn401_get_dp_dto_frequency_100hz() to div_u64() but dropped the original explicit casts, leaving two issues: - In get_dp_dto_frequency_100hz(), "clock_hz * dp_dto_ref_khz * 10" is computed in 32-bit unsigned arithmetic (both operands are unsigned int) before being stored into the u64 temp, so the product can overflow before it is widened (flagged by Coverity OVERFLOW_BEFORE_WIDEN). - div_u64() returns a u64 that is assigned directly to the unsigned int *pixel_clk_100hz, an implicit narrowing conversion that trips -Wconversion (possible loss of data) on stricter builds. [How] Cast clock_hz to unsigned long long so the multiplication is performed in 64-bit, and make the u64 -> unsigned int narrowing explicit with an (unsigned int) cast on the div_u64() results in both functions. The computed values are unchanged. Fixes: 0421fc6ab3a8 ("drm/amd/display: fix __udivdi3 link error") Reviewed-by: Wayne Lin <Wayne.Lin@amd.com> Signed-off-by: James Lin <PingLei.Lin@amd.com> Signed-off-by: Chenyu Chen <chen-yu.chen@amd.com> Tested-by: Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-report')
0 files changed, 0 insertions, 0 deletions