Skip to content
This repository was archived by the owner on Apr 24, 2022. It is now read-only.

Hardware Monitoring for NVML, ADL and AMDGPU SysFS #319

Merged
merged 8 commits into from
Nov 27, 2017

Conversation

smurfy
Copy link
Collaborator

@smurfy smurfy commented Sep 15, 2017

This PR adds Hardware Monitoring to ethminer.
It (optionally) displays GPU Temp and Fanspeed percent to console and is available via API.

  • Support for NVML (tested on Windows + Linux)
  • Support for ADL (tested on Windows, should work on Linux)
  • Support for AMDGPU Linux SysFS (tested on Linux)
  • Added option -HWMON to enable cli display
  • NOT Supported by osx. If we find the nvml/adl .so on osx it could be possible to support

NVML and ADL on linux uses HARDCODED Paths to the .so files. This works on my archlinux but not on ubuntu. We need some testing what happens if only the .so file without the path is set.

@Blacksuu
Copy link

Blacksuu commented Sep 15, 2017

-HWMON on windows 7 work fine & nice add !!!
https://i.imgur.com/FMSU2LW.png
cheers

@amirulali
Copy link

Couldn't get it to run on Linux (Ubuntu 16.04.3, Nvidia 384.69, Cuda 8.0.61-1)

$ git clone [email protected]:smurfy/ethminer.git smurfy
$ cd smurfy
$ git checkout hwmon
$ mkdir build; cd build
$ cmake .. -DETHASHCUDA=ON
$ cmake --build .
$ build/ethminer/ethminer -HWMON
Invalid argument: -HWMON
$ grep HWMON ../ethminer/MinerAux.h
else if ((arg == "-HWMON") && i + 1 < argc)
<< " -HWMON Displays gpu temp and fan percent." << endl
$

@smurfy
Copy link
Collaborator Author

smurfy commented Nov 24, 2017

change output to:

gpu/0 25.64 53C 78% gpu/1 25.64 53C 78%

So combine output of hashrate and hwmon

@smurfy smurfy force-pushed the hwmon branch 2 times, most recently from 9887174 to 6591662 Compare November 24, 2017 23:33
@smurfy
Copy link
Collaborator Author

smurfy commented Nov 25, 2017

Well i think this one is ready to be merged

@smurfy smurfy merged commit 6f28ec5 into ethereum-mining:master Nov 27, 2017
@chfast
Copy link
Contributor

chfast commented Nov 29, 2017

@smurfy it would have been nice to get an approval from someone before merging it.

@smurfy
Copy link
Collaborator Author

smurfy commented Nov 29, 2017

@chfast @ddobreff gave it via gitter, sorry
i can revert it if you have reservations about merging it.

@legotheboss
Copy link

Is this temperature data working with the Claymore-compatibility JSON-API?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants