Project

General

Profile

Bug #3102

sys/dev/drm/radeon/radeon_uvd.c:897: possible cut'n'paste issue ?

Added by dcb 30 days ago. Updated 30 days ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Kernel
Target version:
-
Start date:
11/15/2017
Due date:
% Done:

0%


Description

dragonfly/sys/dev/drm/radeon/radeon_uvd.c:897] -> [dragonfly/sys/dev/drm/radeon/radeon_uvd.c:903]: (warning) Identical condition 'vclk_div>pd_max', second condition is always false

Source code is

if (vclk_div > pd_max)
break; /* vco is too big, it has to stop */

/* calc dclk divider with current vco freq */
dclk_div = radeon_uvd_calc_upll_post_div(vco_freq, dclk,
pd_min, pd_even);
if (vclk_div > pd_max)
break; /* vco is too big, it has to stop */

Maybe the 2nd if would be better if it sanity checked dclk_div, like

if (dclk_div > pd_max)
break; /* vco is too big, it has to stop */

History

#1 Updated by zrj 30 days ago

I agree on possible cut'n'paste issue. It does look like dclk_div should have been used.
Still present in radeon upstream version:
https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/radeon/radeon_uvd.c#L998

Could you report it there too?

#2 Updated by dcb 30 days ago

>Could you report it there too?

I had a look around for a bug button and couldn't see one, so no.

I am happy for someone who knows their way around github better than me
to have a go.

Also available in: Atom PDF