Quote from: peoples on August 28, 2016, 08:40:12 PM

Quote from: peoples on August 24, 2016, 11:41:45 PM

Quote from: wpsd on August 24, 2016, 02:49:42 AM

Quote from: peoples on August 23, 2016, 10:48:55 PM

Quote from: Claymore on August 23, 2016, 09:32:03 AM

About ADL in GPU-PRO drivers, just found a beautiful answer:

https://bugs.freedesktop.org/show_bug.cgi?id=96786

...
ADL is not supported in the amdgpu-pro driver. The same functionality can be accessed via standard Linux interfaces (e.g., hwmon for temperature and fan control, sysfs for clock control, etc..
...

So AMD decided not to worry about ADL support in new drivers for Linux. Excellent solution.

Hey, thanks for 6.3 release. Linux version looks smooth so far, no more VM fault errors.
One of your corefeatures in my opinion is the selectiv shutdown of cards in case of overheat, maybe you can implement this also with the new AMDGPU driver ?

You can query the temperature of the cards from: /sys/class/hwmon/hwmonX/temp1_input
To identify the cards that are useing amdgpu driver you can query the /sys/class/hwmon/hwmonX/name class ?

BTW: Overclocking is done by the keys /sys/class/hwmon/hwmon1/pp_mclk_od (Memory Clock) and /sys/class/hwmon/hwmon1/pp_sclk_od GPU Clock. +1 changes value by 20.

Just to clarify
For temperature just get it from /sys/class/hwmon/hwmon/hwmonX/temp1_unit, the result must divide by 1000 to get the correct celcius
and we can check whether the hardware is amdgpu or not by checking /sys/class/hwmon/hwmonX/name , If it is amdgpu then it's the right one.

Above file can be access without root.

For overclocking and setup fan speed currently the user has to be root, even sudo is not working. Maybe the user need to be included in specific group first. I leave this to the linux user, since we only need to set it once. Note that when using root user the path is no longer /sys/class/hwmon but change to /sys/class/drm/cardX/device/hwmon

For memory +1 increase 20Mhz, for core clock +1 increase 12MHz. To get the highest one use -1 instead.

Maybe for now just include the temperature so the watchdog can protect the miner in case something happen.

@claymore
what do you think about this ?

@claymore any plans for temperature under linux ?

Third try.

@Claymore
Please update us on this one.