search

sibradzic / amdgpu-clocks

Simple script to control power states of amdgpu driven GPUs
GNU General Public License v2.0
393 stars 44 forks source link

Script fails on 5700xt using Kernel 5.5.5 #9

Closed Keridos closed 4 years ago

Keridos commented 4 years ago

First of all the voltages in VDDC Curve are not correctly read since the kernel interface has an "@" added between clcok and voltage. For that to display correctly the function that shows that just needs to parse the 4th word on the line, not the third. Also applies to the veryfying user state values. it ommits the voltage completely from there.

This is how the pp_od_clk_voltage looks like on my card:

OD_SCLK:
0: 800Mhz
1: 1800Mhz
OD_MCLK:
1: 875MHz
OD_VDDC_CURVE:
0: 800MHz @ 709mV
1: 1350MHz @ 790mV
2: 1800MHz @ 1100mV
OD_RANGE:
SCLK:     800Mhz       2150Mhz
MCLK:     625Mhz        950Mhz
VDDC_CURVE_SCLK[0]:     800Mhz       2150Mhz
VDDC_CURVE_VOLT[0]:     750mV        1200mV
VDDC_CURVE_SCLK[1]:     800Mhz       2150Mhz
VDDC_CURVE_VOLT[1]:     750mV        1200mV
VDDC_CURVE_SCLK[2]:     800Mhz       2150Mhz
VDDC_CURVE_VOLT[2]:     750mV        1200mV
sibradzic commented 4 years ago

Thanks for report. I've just booted 5.6 RC3 and I can finally see the similar output on my 5700. I guess the pp_od_clk_voltage kernel interface went through some transitions lately, I'll try to adjust the parsing to match accordingly as soon as I get some free time (feel free to submit a PR in the mean time)...

sibradzic commented 4 years ago

@Keridos fixed, can you please confirm?

Keridos commented 4 years ago

Everything works now, thanks! And sorry for the late reply.